Posted by mrfelton on June 7, 2009 at 5:01pm
| Project: | Simplenews |
| Version: | 6.x-1.0-rc6 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
Modules should not be using t() in hook_schema. For more details, see these notes about When to use st(), t() and get_t() in install and update functions.
For a number of reasons (see http://drupal.org/node/332123), including website performance and the desire avoid encouraging translators to spend time on work of possibly limited value, Drupal 7 and also Drupal 6.9 and later no longer use t() on these strings. You should include schema table and field descriptions as verbatim strings without wrapping them in localization code.
Comments
#1
mrfelton, thanks for the tip. I was not aware of this change, but it a good one.
Attached patch committed to HEAD and 6.x-1.x-dev
#2
Automatically closed -- issue fixed for 2 weeks with no activity.