As a follow-up of #1559054: I18n varriable for the AUTHOR_MAILTEXT and DEFAULT_MAILTEXT, here's a patch to add i18n_variable integration. This adds the two text fields to the variables list, so that the text can be translated in each language.

CommentFileSizeAuthor
#1 2331697-1-comment-notify-i18n_variables-integration.patch1.34 KBBarisW
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

BarisW’s picture

BarisW CreditAttribution: BarisW commented
Status: Active » Needs review
FileSize
2331697-1-comment-notify-i18n_variables-integration.patch1.34 KB
rfay’s picture

rfay
Primary language English
Location Palisade, CO, USA
CreditAttribution: rfay commented

There's more to it than just making the variables translatable though - You also have to make it choose the correct language for the outgoing language. It has always notified the recipient in the *commenter's* language, rather than the recipient language. So this would require explicit comment_notify support to work correctly, not just i18n variables.