[Tickets #2888] NEW: lib/Scheduler/kronolith.php Error in Line 210 / 211

[bugs@bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/?id=2888
-----------------------------------------------------------------------
 Ticket             | 2888
 Created By         | horde at public.linkpool.de
 Summary            | lib/Scheduler/kronolith.php Error in Line 210 / 211
 Queue              | Kronolith
 Version            | 2.0.4
 State              | Unconfirmed
 Priority           | 2. Medium
 Type               | Bug
 Owners             | 
+New Attachment     | kronolith.php
-----------------------------------------------------------------------


horde at public.linkpool.de (2005-10-30 14:11) wrote:

Error is:
======
PHP Warning:  implode(): Bad arguments. in 
/home/httpd/html/horde/kronolith/lib/Scheduler/kronolith.php on line 210
PHP Warning:  implode(): Bad arguments. in
/home/httpd/html/horde/kronolith/lib/Scheduler/kronolith.php on line 211

Reason:
=======
if $tf_recipients contains just one recipient - no array - the code won't
work correctly.

I've changed those lines:

if (is_array ($tf_recipients))
{
  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);
}
else
{
  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);
}

So $tf_recipients can contain an array of recipients or a single recipient