Tickets :: [#7427] Can't do address book searches that include spaces

Summary	Can't do address book searches that include spaces
Queue	IMP
Queue Version	HEAD
Type	Bug
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	chuck (at) horde (dot) org
Created	10/02/2008 (6122 days ago)
Due
Updated	10/02/2008 (6122 days ago)
Assigned
Resolved	10/02/2008 (6122 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

10/02/2008 05:56:02 PM	Michael Slusarz	Comment #3 State ⇒ Resolved	Reply to this comment
I'm not sure it's relevant to the original bug, though. Is it safe to remove it? Yeah, I think I was being overly paranoid here. But no need to sanity check addresses here - that is done later. Removing this line makes the AJAX autocomplete identically to the in-browser completion (when using < 200 addresses) so that's what we want.

10/02/2008 05:51:52 PM	CVS Commit	Comment #2	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/imp/lib/Compose.php?r1=1.389&r2=1.390&ty=u

10/02/2008 05:14:38 PM	Chuck Hagenbuch	Comment #1 Priority ⇒ 1. Low Patch ⇒ No Milestone ⇒ Assigned to Michael Slusarz Queue ⇒ IMP Summary ⇒ Can't do address book searches that include spaces Type ⇒ Bug State ⇒ Assigned	Reply to this comment
When entering a name in the an address field (either DIMP or IMP, any of the 3 expanded fields), if you include a space - for instance, starting to type the last name to disambiguate - the search returns nothing. This is because of the MIME::encodeAddress() check on line 2724 of IMP/lib/Compose.php. The check was added in rev. 1.376 (http://cvs.horde.org/co.php/imp/lib/Compose.php?r=1.376) for bug 7110. I'm not sure it's relevant to the original bug, though. Is it safe to remove it?