Edgewall Software
Modify

Opened 15 years ago

Closed 11 years ago

#2178 closed enhancement (duplicate)

Show users' name instead of username (login) in ticket's owner field

Reported by: dserodio@… Owned by: Jonas Borgström
Priority: normal Milestone:
Component: general Version: devel
Severity: normal Keywords:
Cc: vyt@…, dserodio@…, mat@…, yotta@…, trac@…, a.a.vykhodtsev@… Branch:
Release Notes:
API Changes:
Internal Changes:

Description

My users asked me if it was possible to show the user's name (real name) instead of the username (login) in the 'Assign to' drop-down, so I implemented this feature.

It works like this:

  • If the value of the restrict_owner setting in TracIni is true or username, it shows the username, which is the current behaviour;
  • If the value of the restrict_owner setting in TracIni is name and the user's name is known, the user's name is shown.

Attachments (3)

trac-owner-show-name.patch (1015 bytes ) - added by dserodio@… 15 years ago.
Patch against r2322
trac-r2353.fix-2178-and-2063.diff (7.7 KB ) - added by dserodio@… 15 years ago.
New patch, replaces the previous one
trac-r2353.fix-2178-and-2063-v2.diff (6.7 KB ) - added by dserodio@… 15 years ago.
Sorry, the previous patch had some "leftovers"

Download all attachments as: .zip

Change History (16)

by dserodio@…, 15 years ago

Attachment: trac-owner-show-name.patch added

Patch against r2322

comment:1 by vyt@…, 15 years ago

Cc: vyt@… added

IMHO, username also should be available in selection, for example as:

Name (username)

comment:2 by zgantar@…, 15 years ago

This patch breaks notification!!'''

Stil working on patch for this patch. If anybody finds solution for this problem sooner, please post it here!

comment:3 by anonymous, 15 years ago

I think I fixed it! At source:/trunk/trac/Notify.py@2333#L85 change username with name! It works for me, as far as I tested it!

comment:4 by dserodio@…, 15 years ago

Cc: dserodio@… added

Sorry for jumping the gun on this, I didn't test notification before submitting this patch.
Digging deeper, I found that the <select> fields didn't have a value attribute (#2063), so the "label" (show to the user) has to be the same as the "value" (foreign key to the selected entity).

So I implemented a fix for both this ticket and #2063, but it's a much more "intrusive" change, if anyone wants to help me test it, I'd be thankful.

I also implemented your sugestion, so if restrict_owners=name+username, User's Name (username) is shown.

PS: Should I accept this ticket, or should only "comitters" accept tickets?

by dserodio@…, 15 years ago

New patch, replaces the previous one

by dserodio@…, 15 years ago

Sorry, the previous patch had some "leftovers"

comment:5 by vyt@…, 15 years ago

Danial, in your patch trac-r2353.fix-2178-and-2063-v2.diff exist mistype, templates/ticket.cs (for line 197):

<?cs var option:value ?>

Now this patch is huge for me, since I have already some modifications of code for translation.

Please update patch against trac-0.9 release and I'll try it again.

comment:6 by vyt@…, 15 years ago

Sorry, s/Danial/Daniel/ :)

comment:7 by Christian Boos, 15 years ago

Milestone: 0.12

Might be considered for milestone:0.12

comment:8 by Emmanuel Blot, 15 years ago

I think it would be worth waiting for the IUserDirectory implementation.

comment:9 by anonymous, 14 years ago

Cc: mat@… added

comment:10 by anonymous, 14 years ago

Cc: yotta@… added

comment:11 by trac@…, 12 years ago

Cc: trac@… added

comment:12 by a.a.vykhodtsev@…, 12 years ago

Cc: a.a.vykhodtsev@… added

comment:13 by Christian Boos, 11 years ago

Milestone: 0.13
Resolution: duplicate
Status: newclosed

Superseded by #7339.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jonas Borgström.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jonas Borgström to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.