The current warning says:

"Parameters of watchdog() changed, you no longer need to use t (Drupal Docs)"

However, the fact is that if you watchdog() *with* t() then for each log entry, on admin/reports/dblog you get:

# warning: Invalid argument supplied for foreach() in /Users/webchick/Sites/6x/includes/common.inc on line 779.
# warning: strtr() [function.strtr]: The second argument is not an array. in /Users/webchick/Sites/6x/includes/common.inc on line 796.

... just by viewing the page. Not good.

This patch just re-words the error to be a bit stronger:

"Parameters of !watchdog() changed, you must remove calls to !t()"

CommentFileSizeAuthor
#1 coder-watchdog-t-222405-1.patch816 byteswebchick

Comments

webchick’s picture

Status: Active » Needs review
StatusFileSize
new816 bytes
douggreen’s picture

Status: Needs review » Fixed

Committed, thanks!

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.