I could not find a way to translate Coupon type label in checkout. Here is a patch that wrapps coupon type labels with a t(). Don't know if this it the correct way to do this but it works.

CommentFileSizeAuthor
#4 commerce_coupon-1780072-4.patch795 bytesCristian.Andrei
#2 1780072-2.patch649 byteskillua99
commerce_coupon_translate_coupontype_label.patch936 bytesfreakalis
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pcambra’s picture

pcambra
he/him
Primary language Spanish
Location Asturies
CreditAttribution: pcambra commented
Status: Needs review » Needs work

That's not how a label is translated, I'm afraid, you need to use i18n_field module for this.

killua99’s picture

killua99 CreditAttribution: killua99 commented
FileSize
1780072-2.patch649 bytes

Also I have an issue with the hook_commerce_price_component_type_info the $name of the label is unstranslated. This is the same issue with the i18n_field?

killua99’s picture

killua99 CreditAttribution: killua99 commented
Issue summary: View changes

Added more explenation

RiBu-Nezz’s picture

RiBu-Nezz CreditAttribution: RiBu-Nezz commented

Hi I have the same problem, my coupon type name/label doesn't be translated even I use i18n_field module. When the coupon line item is added at the checkout page, the coupon type name is still be in English. How can I change it to my language? Any advice will be appreciated!

Cristian.Andrei’s picture

Cristian.Andrei CreditAttribution: Cristian.Andrei commented
Issue summary: View changes
FileSize
commerce_coupon-1780072-4.patch795 bytes

Here'a patch based on killua99's solution with an added context. Works just fine.

Cristian.Andrei’s picture

Cristian.Andrei CreditAttribution: Cristian.Andrei commented
Status: Needs work » Needs review

changed status

pcambra’s picture

pcambra
he/him
Primary language Spanish
Location Asturies
CreditAttribution: pcambra commented
Status: Needs review » Needs work

As I said in #1, using t() for variables is not the right solution, i18n_fields and i18n_strings should be used instead

killua99’s picture

killua99 CreditAttribution: killua99 commented

Surprisingly I couldn't never put it work with i18n_fields and i18n_strings, maybe some miss configuration.

Thanks Cristian.Andrei I'm gonna update my patch url in my make file.

killua99’s picture

killua99 CreditAttribution: killua99 commented
Status: Needs work » Reviewed & tested by the community

This patch work just great, none issue with the $variables use in t function.

It's clear some never get work the i18n_field correctly so, this solution just works.

pcambra’s picture

pcambra
he/him
Primary language Spanish
Location Asturies
CreditAttribution: pcambra commented
Status: Reviewed & tested by the community » Needs work

Please don't RTBC this, as I've mentioned in the above comments, t() shouldn't not be used for variables like this, the solution is using the other suggested methods.