Use t() or l() where appropriate

[Pierre Buyle]

I'm in the process to use this module for the Ubuntu-be.org website. Being Belgium we have four languages for our website. It would be nice to support the standard i18n for headers' text.

As a quick hack, I wrapped $rslt['centertext'] and $rslt['righttext'] into t() in udheader_build_block so the original text (presumably in English) can be translated with Drupal "Translate interface" admin page.

A better solution would be to have multiple text area (one for each language) when editing an header.

[Michael Lustfield (michaellustfield)]

This is more of a generic bug across all modules I've developed. I need to use t() and l() when I'm controlling data that will print to the screen. When I have some time I will run through everything and take care of it.

"A better solution would be to have multiple text area (one for each language) when editing an header."
 - This would be very sub-optimal. It is a better idea to leave the translations to drupal by using the correct functions.

[Michael Lustfield (michaellustfield)]

This has been taken care of in udheader r22.

[Michael Lustfield (michaellustfield)]

udcountdown r34

[Michael Lustfield (michaellustfield)]

udsidebar r9

[Michael Lustfield (michaellustfield)]

udtheme r148

[Michael Lustfield (michaellustfield)]

These fixes has been merged into drupal.org.