[development] Video module getting ready for 4.7 release: need help debugging

Khalid B kb at 2bits.com
Mon Jun 19 00:53:11 UTC 2006

On 6/18/06, Steven Wittens <steven at 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().


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

Or did I miss something?

More information about the development mailing list