| 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/05 (7545 days ago) |
| Due | |
| Updated | 10/30/05 (7545 days ago) |
| Assigned | |
| Resolved | 10/30/05 (7545 days ago) |
| Github Issue Link | |
| Github Pull Request | |
| Milestone | |
| Patch | No |
State ⇒ Resolved
State ⇒ Unconfirmed
Priority ⇒ 2. Medium
Type ⇒ Bug
Summary ⇒ lib/Scheduler/kronolith.php Error in Line 210 / 211
Queue ⇒ Kronolith
New Attachment: kronolith.php
======
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