Closed (fixed)
Project:
Buy Me a Beer
Version:
6.x-1.0
Component:
Code
Priority:
Critical
Category:
Task
Assigned:
Reporter:
Created:
29 Apr 2009 at 07:12 UTC
Updated:
13 May 2009 at 10:10 UTC
You should never never use define together with the t function:
see http://drupal.org/node/317001
this is a performance disaster, the language cache is rebuild every time, so here is a patch
thx :)
perhaps the t function should be left of variable_get
| Comment | File | Size | Author |
|---|---|---|---|
| buymeabeer-not-definie-t.patch | 1.56 KB | dawehner |
Comments
Comment #1
yaph commentedThanks for the patch. I removed the calls to t() within define calls and moved it to variable_get. Is there any evidence that t() should not be used in variable_get as well?
Comment #2
yaph commentedComment #3
avpadernoComment #4
yaph commentedFixed in release buymeabeer 6.x-1.1-beta3 http://drupal.org/node/448228