Problem/Motivation

t() only has replacement vars.

+++ b/core/modules/simpletest/simpletest.pages.incundefined
@@ -149,11 +171,11 @@ function theme_simpletest_test_table($variables) {
+        'data' => '<div class="description">' . t('@description (@class)', array('@description' => $description, '@class' => $test_name)) . '</div>',

Seems strange because it's not really translatable, aside from change the order of description and class.

Proposed resolution

convert it to format_string.

Remaining tasks

  • Discuss if makes sense to leave as t() because of RTL LTR considerations.
  • (Novice) Implement.

User interface changes

No UI changes.

API changes

No API changes.

Original report by @tim.plunkett and @berdir

Follow up for #1919470-6: Add a search field to the test overview page and #7

+++ b/core/modules/simpletest/simpletest.pages.incundefined
@@ -149,11 +171,11 @@ function theme_simpletest_test_table($variables) {
+        'data' => '<div class="description">' . t('@description (@class)', array('@description' => $description, '@class' => $test_name)) . '</div>',

@tim.plunkett

This should be format_string, I'm pretty sure.

@Berdir

It is a weird thing to translate agreed, but what about rtl languages? What do you think about replacing the name with the classname? then this would no longer be necessary... We should probably discuss that in a separate issue, though.

Comments

yesct’s picture

s_leu’s picture

Status: Postponed » Closed (won't fix)

Since the current test guidelines do not require translations for assertion messages, we can drop the t() call here too i guess.