Download & Extend

currency_api.module should not use t() in hook_install()

Project:Currency Exchange
Version:6.x-1.x-dev
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

Title:currency_api.module should not use t() in hook_insrall()» currency_api.module should not use t() in hook_install()
Status:active» needs review

Attached patch removes t() from currency_api_schema.

AttachmentSize
484482-remove_t_function_from_hook_install.patch 2.08 KB

#2

Status:needs review» fixed

Fixed with http://drupal.org/cvs?commit=467052

#3

Status:fixed» closed (fixed)

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