Tickets :: [#12312] Add address from address book broke addresses with "given name, name" <user1@example.com>

6.0.0-beta1

7/4/25

Summary	Add address from address book broke addresses with "given name, name" <user1@example.com>
Queue	IMP
Queue Version	6.1.0
Type	Bug
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	samuel.wolf (at) wolf-maschinenbau (dot) de
Created	06/06/2013 (4411 days ago)
Due
Updated	06/07/2013 (4410 days ago)
Assigned
Resolved	06/07/2013 (4410 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

06/07/2013 10:43:20 PM	Michael Slusarz	Comment #8 Assigned to Michael Slusarz State ⇒ Resolved	Reply to this comment
IMP 6.1.1.

06/07/2013 10:43:06 PM	Git Commit	Comment #7	Reply to this comment
Changes have been made in Git (master): commit 8d99d7f68ac4bc6573fa8fe170237616c5dd4432 Author: Michael M Slusarz <slusarz@horde.org> Date: Fri Jun 7 16:41:19 2013 -0600 [mms] Fix adding addresses from the contacts popup in certain situations (~~Bug #12312~~). Situation = last address does not have a trailing comma and contains a comma in the personal part imp/docs/CHANGES \| 2 ++ imp/js/compose-base.js \| 14 +++++++------- imp/js/compose-dimp.js \| 2 +- imp/js/compose.js \| 7 +------ imp/package.xml \| 2 ++ 5 files changed, 13 insertions(+), 14 deletions(-) http://git.horde.org/horde-git/-/commit/8d99d7f68ac4bc6573fa8fe170237616c5dd4432

06/07/2013 08:19:50 PM	samuel (dot) wolf (at) wolf-maschinenbau (dot) de	Comment #6 New Attachment: horde_bug-12312_example.png	Reply to this comment
I type in the field so the following happens: Do not type in the field, use the address book, see the example.

06/07/2013 08:00:44 PM	Michael Slusarz	Comment #5	Reply to this comment
Can't reproduce. Given this address (in either To, CC, or BCC): "foo, bar" <foo@example.com> I type in the field so the following happens: "foo, bar" <foo@example.com>, baz I am given autocomplete options for baz. Clicking on the desired entry gives me: "foo, bar" <foo@example.com>, "baz, foo2" <baz@example.com>, which is the expected output.

06/07/2013 07:47:25 PM	Michael Slusarz	Priority ⇒ 1. Low

06/07/2013 10:39:40 AM	samuel (dot) wolf (at) wolf-maschinenbau (dot) de	Comment #4	Reply to this comment
Add address from the address book to Cc: or Bcc: broke (if there is a comma in the name) or replace the last address complete. Add into the To: field work fine.

06/06/2013 01:55:01 PM	samuel (dot) wolf (at) wolf-maschinenbau (dot) de	Comment #3	Reply to this comment
The problem is the missing comma "," in the last address. does not work: "fist name.last name" <user2@example.com> work: "fist name.last name" <user2@example.com>, Sorry wrong example: does not work: "last name, fist name" <user2@example.com> work: "last name, fist name" <user2@example.com>,

06/06/2013 01:37:37 PM	samuel (dot) wolf (at) wolf-maschinenbau (dot) de	Comment #2	Reply to this comment
The problem is the missing comma "," in the last address. does not work: "fist name.last name" <user2@example.com> work: "fist name.last name" <user2@example.com>,

06/06/2013 01:28:39 PM	samuel (dot) wolf (at) wolf-maschinenbau (dot) de	Comment #1 Priority ⇒ 3. High Type ⇒ Bug Summary ⇒ Add address from address book broke addresses with "given name, name" <user1@example.com> Queue ⇒ IMP Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
Reply to a mail which have a address in cc: like this "given name, name" <user1@example.com> Now click on the "Cc:" link and add a second address for Cc: like .. "fist name.last name" <user2@example.com> .. from the address book broke the first address complete: "lastname, "fistname.lastname" <user2@example.com>,