Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I get the following errors when viewing log entries that have a comment_notify log entry in them:
* warning: Invalid argument supplied for foreach() in /home/.katona/davereid20/drupal-6/includes/common.inc on line 789.
* warning: strtr() [function.strtr]: The second argument is not an array in /home/.katona/davereid20/drupal-6/includes/common.inc on line 806.
The offending line of code:
watchdog('comment_notify', $watchdog_message,
WATCHDOG_NOTICE, l(t('source comment'), 'node/'. $nid, array('fragment' => 'comment-'. $alert->cid)));
Should be in the form of watchdog($type, $message, $variables = array(), $severity = WATCHDOG_NOTICE, $link = NULL) (D6 watchdog)
watchdog('comment_notify', $watchdog_message, array(), WATCHDOG_NOTICE, l(t('source comment'), 'node/'. $nid, array('fragment' => 'comment-'. $alert->cid)));
Comment | File | Size | Author |
---|---|---|---|
#2 | 306971_watchdog_parameters_translation_2.patch | 1.39 KB | greggles |
#1 | 306971_watchdog_parameters_translation.patch | 1.39 KB | greggles |
Comments
Comment #1
gregglesI think that fixes the error but not the underlying problem which was an old style watchdog message that wouldn't be translatable.
The attached patch fixes that (I believe).
Can you confirm that it fixes the errors you see?
Comment #2
gregglesWhoops that had a parse error.
Comment #3
Dave ReidChecking now.
Comment #4
Dave ReidFixed. Now I need to figure out why I'm also having this same issue: #299548: Line breaks not showing up in email.
Comment #5
gregglesGreat, fixed - http://drupal.org/cvs?commit=139464
Thanks, Dave Reid!
Comment #6
Dave ReidThanks very much greggles? Any word on #299548: Line breaks not showing up in email?
Edit: link fixed
Comment #7
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.