I am trying to get the complete translation of the admin/commerce/config/shipping overview, but the string "Machine name: " is not exposed to the translation interface.

I spotted in the commerce_shipping_ui.admin.inc file (both in 7.x-2.0 and 7.x-2.x-dev files) that those strings are not wrapped into the t() function.

So where it is the implementation of the theme_shipping_service_admin_overview($variables), (approximately line 96), I changed the following line:

function theme_shipping_service_admin_overview($variables) {
...
  $output .= ' <small> (Machine name: ' . check_plain($shipping_service['name']) . ')</small>';
...

And added the t() function:

  $output .= t(' <small> (Machine name: ') . check_plain($shipping_service['name']) . ')</small>';

Note that the "small" tag is useful to distinguish that string from others (e.g. those that the flat rate module expose to the translation interface).

It works for me, but I would like to hear other people's opinion.

Cheers

Comments

googletorp’s picture

Status: Active » Fixed

I've comitted a fix for this in dev

Status: Fixed » Closed (fixed)

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