6.0.0-beta1
7/22/25

[#2888] lib/Scheduler/kronolith.php Error in Line 210 / 211
Summary lib/Scheduler/kronolith.php Error in Line 210 / 211
Queue Kronolith
Queue Version 2.0.4
Type Bug
State Resolved
Priority 2. Medium
Owners
Requester horde (at) public (dot) linkpool (dot) de
Created 10/30/2005 (7205 days ago)
Due
Updated 10/30/2005 (7205 days ago)
Assigned
Resolved 10/30/2005 (7205 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
10/30/2005 11:08:51 PM Chuck Hagenbuch Comment #2
State ⇒ Resolved
Reply to this comment
This has already been fixed in CVS.
10/30/2005 10:11:16 PM horde (at) public (dot) linkpool (dot) de Comment #1
State ⇒ Unconfirmed
Priority ⇒ 2. Medium
Type ⇒ Bug
Summary ⇒ lib/Scheduler/kronolith.php Error in Line 210 / 211
Queue ⇒ Kronolith
New Attachment: kronolith.php Download
Reply to this comment
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

Saved Queries