| Summary | Missing argument 7 for Whups_Driver::mail() |
| Queue | Whups |
| Queue Version | 1.0 |
| Type | Bug |
| State | Resolved |
| Priority | 1. Low |
| Owners | chuck (at) horde (dot) org |
| Requester | stevens (at) stevens (dot) it |
| Created | 11/12/2008 (6082 days ago) |
| Due | |
| Updated | 11/12/2008 (6082 days ago) |
| Assigned | |
| Resolved | 11/12/2008 (6082 days ago) |
| Github Issue Link | |
| Github Pull Request | |
| Milestone | |
| Patch | No |
Assigned to Chuck Hagenbuch
State ⇒ Resolved
http://cvs.horde.org/diff.php/whups/lib/Driver.php?r1=1.201&r2=1.202&ty=u
Priority ⇒ 1. Low
Patch ⇒ No
Milestone ⇒
Queue ⇒ Whups
Summary ⇒ Missing argument 7 for Whups_Driver::mail()
Type ⇒ Bug
State ⇒ Unconfirmed
"PHP Warning: Missing argument 7 for Whups_Driver::mail(), called in
/var/www/horde-3.3/whups-h3-1.0/lib/Whups.php on line 751 and defined
in /var/www/horde-3.3/whups-h3-1.0/lib/Driver.php on line 335"
lib/Whups.php:751
$whups_driver->mail(null, $user, $subject, $email, $user, true);
it could be ok to put a default for "$queue_id" in Whups_Driver::mail(); e.g.
function mail($ticket_id, $recipients, $subject, $message, $from,
$reminder, $queue_id = null, $is_new = false)
later in the function, we can check the "$queue_id" value instead of
using it directly; e.g.:
if (!is_null($queue_id)) {
$queue = $this->getQueue($queue_id);
} else {
...