[Tickets #2929] NEW: Reminders doesn't work because of call to implode function with string argument

[bugs@bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/?id=2929
-----------------------------------------------------------------------
 Ticket             | 2929
 Created By         | peter at md.kth.se
 Summary            | Reminders doesn't work because of call to implode function with string argument
 Queue              | Kronolith
 Version            | 2.0.4
 State              | Unconfirmed
 Priority           | 3. High
 Type               | Bug
 Owners             | 
-----------------------------------------------------------------------


peter at md.kth.se (2005-11-06 10:27) wrote:

Reminders doesn't work for me. Found a call to implode where the argument is
a string (not an array) in the file
kronolith-h3-2.0.4/lib/Scheduler/kronolith.php. Seems like PHP 4.4.1 doesn't
like that.

The fix for me was:

209,211c209,211
<                 Horde::logMessage(sprintf('Sending reminder for %s to %s',
$event->title, implode(', ', $tf_recipients)), __FILE__, __LINE__,
PEAR_LOG_DEBUG);
<                 $sent = $mime->send(implode(', ', $tf_recipients),
$msg_headers, $mail_driver, $mail_params);
<                 if (is_a($sent, 'PEAR_Error')) {
---
>                           Horde::logMessage(sprintf('Sending reminder for
%s to %s', $event->title, $tf_recipients), __FILE__, __LINE__,
PEAR_LOG_DEBUG);
>                 $sent = $mime->send($tf_recipients, $msg_headers,
$mail_driver, $mail_params);
>                                        if (is_a($sent, 'PEAR_Error')) {

And then everything works.

Hope this helps!

Best regards,

Peter