Some strings contains HTML code that is marked as unsafe by locale_string_is_safe(), this includes the "tags" <none>, <hidden> and <br/> without an space.

Since the translators tend to mantain the original syntax, the process of add some language by import interface trigger the message "X translations string was skipped because it contains disallowed HTML".

The attached patch against HEAD replaces this strings with valid ones.

CommentFileSizeAuthor
#2 disallowed-html-994226-2.patch13.47 KBmagnus
disallowed-html-fix.patch3.92 KBdanillonunes

Comments

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Component: language system » locale.module

Changing system component and subscribing.

magnus’s picture

magnus commented
Version: 7.0-rc1 » 7.x-dev
StatusFileSize
new disallowed-html-994226-2.patch13.47 KB

Here is a new patch that removes the HTML entities from the strings. No need for translators to translate entities.
Also fixed some more <br/>-tags.

Status: Needs review » Needs work

The last submitted patch, disallowed-html-994226-2.patch, failed testing.

magnus’s picture

magnus commented
Status: Needs work » Needs review

#2: disallowed-html-994226-2.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, disallowed-html-994226-2.patch, failed testing.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Needs work » Closed (duplicate)

Please join forces with #1020842: Error messages when importing translated strings fails are not helpful at all, where the error message handling is also improved additionally to fixing untranslatable strings.