Posted by kscheirer on July 14, 2009 at 9:41pm
Jump to:
| Project: | Listhandler |
| Version: | master |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (works as designed) |
Issue Summary
in listhandler_send_mail()::line 308, $taxoarr is not defined. I'd assume this should be $taxoterm->tid instead, but considering that this bug appears to have been around since 2004 (r1.50) I'm not sure what the best way to handle this would be.
Comments
#1
#2
Set this not assigned to yourself otherwise no one will start looking into the issue.
#3
It is assigned and get to be the value of the object fetching action. The relevant code of the current 6.x -dev version is below as a reference to take a look how it is done:
...else {
$result = db_query("SELECT t.tid FROM {term_node} t WHERE t.nid = %d", $edit["nid"]);
while ($taxoarr = db_fetch_object($result)) {
// Now for the mailbox.
switch ($GLOBALS['db_type']) {
case 'mysql':
case 'mysqli':
$result2 = db_query("SELECT * FROM {mailhandler} m WHERE m.commands REGEXP '%s'", 'tid:[[:blank:]]*'. $taxoarr->tid .'[[:>:]]');
break;
case 'pgsql':
$result2 = db_query("SELECT * FROM {mailhandler} m WHERE m.commands ~ '%s'", 'tid:[[:blank:]]*'. $taxoarr->tid .'[[:>:]]');
}
while ($mbox = db_fetch_object($result2)) {
$mboxen[] = array($mbox->mid, $mbox->mailto, $taxoarr->tid);
}
}
}