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

[Khalid B]

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().

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?