I suggest we change this in user_resources.inc:

theme('placeholder', $user->name)

to: 

$user->name

alternatively (minor issue with this approach is that in the watchdog, html is ignored).

drupal_placeholder($user->name)

Comments

kylebrowning’s picture

kylebrowning commented
Category: bug » feature

Why?

marcingy’s picture

marcingy commented
Status: Active » Postponed (maintainer needs more info)
marcingy’s picture

marcingy commented
Category: feature » bug
Priority: Minor » Normal
Status: Postponed (maintainer needs more info) » Active

theme placeholder no longer exists so we need to swap it out.

kylebrowning’s picture

kylebrowning commented
Status: Active » Closed (fixed)
magnusk’s picture

magnusk commented
Version: 7.x-3.0-rc3 » 7.x-3.x-dev
Status: Closed (fixed) » Active

In _user_resource_logout()(), the use of drupal_placeholder() in the watchdog call is redundant with the use of %, so the site log displays a literal <em> tag in italics:

Session closed for <em class="placeholder">USERNAME</em>.

The watchdog call should be

watchdog('user', 'Session closed for %name.', array('%name' => $user->name));

marcingy’s picture

marcingy commented
Status: Active » Closed (fixed)

Please open a new issue rather reopening an issue that is 6 months old and unrelated.