When importing the fr.po translation for this module, I get:
2 translation strings were skipped because they contain disallowed HTML.

The issue seems similar to the one there:
http://drupal.org/node/362647

The two translatable strings used by fivestart that look like HTML tags are '' and ''. I *think* a good solution would be to use '<none>' instead, but maybe someone more knowledgeable on this will know better.

CommentFileSizeAuthor
#4 fivestar_t_brackets.patch1.81 KBquicksketch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

quicksketch’s picture

Looks like your text got parsed out by the HTML filter, wrap in <code> tags to keep them in place.

Quoted with code tags:

The two translatable strings used by fivestart that look like HTML tags are '<none>' and '<Hidden>'. I *think* a good solution would be to use '&lt;none&gt;' instead, but maybe someone more knowledgeable on this will know better.
quicksketch’s picture

Also, thanks for the report. :)

Perhaps what we should do is translate the inside of the the brackets?

$none = '<' . t('none') .'>';

We don't need to URL encode these strings since they are encoded for us when we put them into a select list. I've never heard of the error you described, but then again I don't work on many translated websites.

dbr’s picture

Yes, your solution does make sense. In all languages the < and > should be there, so only the text inside has to be translated.

quicksketch’s picture

Status: Active » Fixed
FileSize
1.81 KB

Fixed with the attached patch, which has been committed to CVS.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.