19 Jun
2006
19 Jun
'06
2:53 a.m.
On 6/18/06, Steven Wittens <steven@acko.net> wrote:
t('play %link', array('%link' => $node->title))
this is used as title attribute for a link... maybe check_plain() should be used insted of theme('placeholder') as suggested.
Neither check_plain() or theme('placeholder') are necessary or even make sense here. Title attributes cannot contain HTML, so their content is passed as plain-text to l(). The attribute content is escaped right before outputting in drupal_attributes().
Steven l() does call check_plain() if the last argument ($html) is true. But Fabio and Dries are talking about t(), not l(), which does not call check_plain(). Or did I miss something?