In theme.inc string 134 has item that is not wrapped in t()

    return '<span class="topic-pager">(Page: ' . implode(", ", $topic_pager->initial_pages) . $last_page . ')</span>';

I changed it to

    return '<span class="topic-pager">('.t('Page').': ' . implode(", ", $topic_pager->initial_pages) . $last_page . ')</span>';

and it can be translated now.

CommentFileSizeAuthor
#2 fix_string_not_wrapped_in_t-1131900.patch554 bytesmcdruid
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sandino’s picture

sandino CreditAttribution: sandino commented
Title: String not translated » String is not wrapped with t()
mcdruid’s picture

mcdruid
Location 🇬🇧🇪🇺
CreditAttribution: mcdruid commented
Status: Active » Closed (fixed)
FileSize
fix_string_not_wrapped_in_t-1131900.patch554 bytes

Thanks sandino.

For future reference, it's better to attach a patch to the issue with any code fixes.

This is a very minor thing, but Drupal coding standards have a space on both sides of the dot when concatenating strings: http://drupal.org/coding-standards#concat

Committed to 6.x-2.x branch.

mcdruid’s picture

mcdruid
Location 🇬🇧🇪🇺
CreditAttribution: mcdruid commented
Status: Closed (fixed) » Fixed

oops - realised I should have been setting issues to "fixed" rather than "closed (fixed)"

Status: Fixed » Closed (fixed)

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