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

Steven Wittens steven at acko.net
Sun Jun 18 23:42:28 UTC 2006


>
>>   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 Wittens


More information about the development mailing list