Watchdog message not translatable [#239008]

The watchdog notice is getting garbled.

Instead of this:

      if ($alert->uid != 0) {
        $mylink = l($alert->mail , 'user/'. $alert->uid .'/edit');
      }
      else {
        $mylink = $alert->mail;
      }

      // Add an entry to the watchdog log.
      watchdog('comment_notify', t('Notify ') . $mylink,

It should be this:

      // Add an entry to the watchdog log.
      watchdog('comment_notify', t('Notified: ') . $alert->mail,

Comment	File	Size	Author
#3	239008_3_watchdog_translatable.patch	1.36 KB	greggles
#3

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

greggles

he/him

English

Denver, Colorado, USA

CreditAttribution: greggles commented 28 March 2008 at 18:46

Yeah - even that is not the best way to include the $alert->mail because it assumes that the mail should come at the end of the line which may not be valid for all languages. Thanks for the report.

I won't be able to get to this for a month or two.