Andreas Dick wrote:
If I remember gmail has such option, but I don't know if this works on addcontact action. If user prefers e.g. Surname,Forename order it doesn't means sender will prefer it too. What if user has a two (fore)names? What if it is a nick (like mine) or company name? Too much guessing and possible mistakes.
In Roundcube the name field is for display name and it should be not changed in any way (a part of your patch). When name is not specified in address string we're trying to build the (display) name. A possible solution would be to get rid of it and leave display-name empty (currently it's a required field).
Ok, I agree with your arguments! Another problem (and why I made the patch) is, that the name field is used as CN and thus as key identifier in the LDAP (at least in my setup, others can use the email). So it makes conflicts if it is empty at creating. For me, it would make more sense to try to set the "surname", "firstname" fields directly at this place, but I think this fields are not accessible, isnt it?
My question now: shall I investigate more? Because I can live with my patch for now, and if this topic is planed anyway to be redesigned in the future it makes maybe no sense.
The entire address book is subject to be completely re-written so we should postpone this discussion to that. I've re-opened the ticket and moved it to the according milestone. With more advanced features of the address book the name formatting question will arise anyway.
I think a configuration option for how to compose the "display name" from all the given fields (e.g. surname, middlename, family name, prefix, suffix) will be needed. Of course this doesn't automatically solve the parsing of a given name string into the right components. If you have more inputs or suggestions please let us know. However showing a dialog every time is the least preferred option for me and I don't know any major mail clients that do that.
~Thomas _______________________________________________ List info: http://lists.roundcube.net/dev/ BT/aba52c80