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

kscheirer’s picture

Status: Needs review » Active
cor3huis’s picture

Assigned: kscheirer » Unassigned

Set this not assigned to yourself otherwise no one will start looking into the issue.

cor3huis’s picture

Status: Active » Closed (works as designed)

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);
      }
    }
  }