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

[dserodio@…]

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.

[dserodio@…]

Patch against r2322

[vyt@…]

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

Name (username)

[zgantar@…]

This patch breaks notification!!'''

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

[anonymous]

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!

[dserodio@…]

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?

[dserodio@…]

New patch, replaces the previous one

[dserodio@…]

Sorry, the previous patch had some "leftovers"

[vyt@…]

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.

[vyt@…]

Sorry, s/Danial/Daniel/ :)

[cboos]

Might be considered for milestone:0.12

[eblot]

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