In the .install, strings were passed through t(), however localization services appear to not be available during installation processes, so we use st() instead.

In the .module, t()'s were added in a few places.

CommentFileSizeAuthor
#1 legal_install_i18n.patch1.01 KBstella
#1 legal_module_i18n.patch2.38 KBstella
legal_module_i18n.patch20.45 KBhailu
legal_install_i18n.patch1.98 KBhailu
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

stella’s picture

stella CreditAttribution: stella commented
FileSize
legal_module_i18n.patch2.38 KB
legal_install_i18n.patch1.01 KB

Re-rolled the patches with a couple of tweaks.

Cheers,
Stella

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Needs review » Reviewed & tested by the community

Both patches look great, and apply cleanly, so RTBC.

alexandreracine’s picture

alexandreracine CreditAttribution: alexandreracine commented

Does this work? Should the patch be commited? Is it already commited?

xibun’s picture

xibun CreditAttribution: xibun commented

+1

prodosh’s picture

prodosh CreditAttribution: prodosh commented

Can't translate the text in the "terms and conditions" box. Using Patch #1.
Am using Beta4. Should I be using HEAD?

recidive’s picture

recidive CreditAttribution: recidive commented
Status: Reviewed & tested by the community » Fixed

I've commited legal_module_i18n.patch in #1 and removed the messages output in hook_install() so no need for the legal_install_i18n.patch.

Thanks!

Status: Fixed » Closed (fixed)

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