custom_breadcrumbs.install should not use t() in hook_schema()

mrfelton - June 7, 2009 - 17:05
Project:Custom Breadcrumbs
Version:6.x-2.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

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.

#1

MGN - June 8, 2009 - 01:02

Thanks for catching this...I will change it in the next round of commits.

#2

MGN - June 9, 2009 - 23:08
Status:active» fixed

Committed: http://drupal.org/cvs?commit=223292

and http://drupal.org/cvs?commit=223334 for 6.x-1.x-dev.

#3

System Message - June 23, 2009 - 23:10
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.