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,
CommentFileSizeAuthor
#3 239008_3_watchdog_translatable.patch1.36 KBgreggles

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

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.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

I actually don't see a problem with putting the link into the title. Can you clarify?

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Title: Watchdog message can't have link in title. » Watchdog message not translatable
Status: Active » Needs review
StatusFileSize
new 239008_3_watchdog_translatable.patch1.36 KB

Here is a patch to fix the translatability of the message...

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Needs review » Fixed

and fixed.

If you can clarify the problem with putting a link in the message I'd appreciate it.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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