Download & Extend
Legal » Issues

Fix string localization in legal module

Posted by hailu on September 23, 2008 at 3:00am
Project:Legal
Version:master
Component:User interface
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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.

AttachmentSize
legal_install_i18n.patch1.98 KB
legal_module_i18n.patch20.45 KB
Login or register to post comments

Comments

#1

Posted by stella on October 6, 2008 at 3:36pm

Re-rolled the patches with a couple of tweaks.

Cheers,
Stella

AttachmentSize
legal_install_i18n.patch 1.01 KB
legal_module_i18n.patch 2.38 KB

#2

Posted by catch on October 6, 2008 at 8:06pm
Status:needs review» reviewed & tested by the community

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

#3

Posted by alexandreracine on March 6, 2009 at 2:55am

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

#4

Posted by xibun on June 7, 2009 at 9:05pm

+1

#5

Posted by pban02 on June 17, 2009 at 12:16pm

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

#6

Posted by recidive on September 25, 2010 at 4:21pm
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!

#7

Posted by System Message on October 9, 2010 at 4:30pm
Status:fixed» closed (fixed)

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