[Tickets #5434] compose.php breaks when "Expand Names" finds no results

[bugs at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=5434
-----------------------------------------------------------------------
 Ticket             | 5434
 Created By         | pdmckone at engmail.uwaterloo.ca
 Summary            | compose.php breaks when "Expand Names" finds no results
 Queue              | IMP
 Version            | 4.1.3
 Type               | Bug
 State              | Unconfirmed
 Priority           | 1. Low
 Owners             | 
-----------------------------------------------------------------------


pdmckone at engmail.uwaterloo.ca (2007-06-04 10:37) wrote:

When composing a message, if "Expand Names" doesn't find any results
(throwing a PEAR_Error), the compose window reloads with only the header
"Message Composition", followed by a line break.

It appeared to be the IMP::status call in templates/compose/compose.inc
that was failing.  I think it's because the line in compose.php's
_expandAddresses function:
$GLOBALS['notification']->push($error, 'horde.warning');
should be:
$GLOBALS['notification']->push($error->getMessage(), 'horde.warning');
so that it's pushing just the error message rather than the whole
PEAR_Error object.

I've edited my compose.php, and resolved the problem on my setup.

PDM