Guys,

the check that detects if the translation ends with period when the original strings have is inverted. Also, it doesnt consider Unicode strings, which might lead to wrong checks.

The fix to be placed on coder_i18n.inc, around line 127, replace:

     // Check the translation trailing periods.
    if (substr($msgid, -1, 1) == '.' && !substr($msgstr, -1, 1) != '.') {
      $rule = array(
        '#warning' => 'The translation text should end in a period when the same original text also ends in a period.',
      );
      _coder_error($results, $rule, $severity_name, $lineno, $msgstr);
    }

to

    // Check the translation trailing periods.
    if (drupal_substr($msgid, -1, 1) == '.' && drupal_substr($msgstr, -1, 1) != '.') {
      $rule = array(
        '#warning' => 'The translation text should end in a period when the same original text also ends in a period.',
      );
      _coder_error($results, $rule, $severity_name, $lineno, $msgstr);
    }

regards,

massa

Comments

douggreen’s picture

Version: 6.x-1.0 » 7.x-2.x-dev
Assigned: Unassigned » douggreen
douggreen’s picture

Status: Needs review » Closed (fixed)

Thanks. I apologize that no one looked at this earlier, but as this is a 4 year old ticket, it looks like the fix has already been applied.