Translate "Terms & Conditions"
dicreat - April 8, 2009 - 09:58
| Project: | Legal |
| Version: | 6.x-2.2-beta4 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | reviewed & tested by the community |
Description
Hi.
I think, that link to "Terms & Conditions" must be translated. I've been change l('Terms & Conditions', 'legal') to l(t('Terms & Conditions'), 'legal') and it's work fine. Please review patch.
| Attachment | Size |
|---|---|
| legal.module.patch | 1.12 KB |

#1
Patch works fine.
#2
Generally speaking is a non sense to send the whole legal text throght the locale translation system. Locale should be used to translate arrays but not huge pieces of text as the legal page usually is. But, well, this is the solutions I've also used for the last two years.
When we'll have a really multilingual solution on such an important module as legal is? When a text box for every active language?
#3
This patch ist just about the link title, not the whole text.
Multilinguality of the legal text should be a different issue.
The attached patch replaces the tabs with spaces, everything else is well.
Please commit!