Closed (fixed)
Project:
Memcache API and Integration
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
13 Jul 2010 at 14:14 UTC
Updated:
3 Jan 2014 at 01:42 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
cardioid1 commentedI get the same error while trying to run an external script to node_save data.
EDIT: memcached PECL module wasn't enabled...
Comment #2
otsuarez commentedi had the same error, i'd installed php pecl memcache, installed and started memcached daemon, but forgot to restart apache. Doing that solved the issue and remove the error.
Comment #3
Daniel Norton commentedThis patch first checks to see if t() is defined and skips the translation step if it's not. (Perhaps it never is at this stage?)
Comment #4
Daniel Norton commentedSubmit patch to bot
Comment #5
killes@www.drop.org commentedI got this too, the patch helps.
Comment #6
mrfelton commentedI think this should be committed - without it, you get a whitescreen of death if the PECL extension is not installed.
Comment #7
cyberwolf commentedSubscribing.
Comment #8
robertdouglass commentedIn my testing t() just doesn't exist at this point in execution. I'm therefore removing it altogether. If anyone has suggestions about how it can be made localizable, let me know. Attached patch is being committed.
Comment #9
robertdouglass commented#852794 by Daniel Norton, robertDouglass | cardioid1: Fixed Call to undefined function t() in ....../dmemcache.inc on line 236.
Comment #10
jeremy commentedPerhaps we should be adding a comment here in the code noting why we're not translating, so later on we don't come along and add the t() back...
Comment #11
robertdouglass commented@jeremy - good idea. I'll do that, and then port to D7 as well.
Comment #12
Daniel Norton commentedFYI:
#322549: t() should work regardless of the bootstrap phase
Comment #13
robertdouglass commentedA possibly better solution? #921506: Missing include
Comment #14
robertdouglass commentedHere's the docs I added.
Comment #15
robertdouglass commentedIt's now a question whether this applies to D7.
Comment #16
robertdouglass commentedComment #17
_-. commentednon-issue; ignore
Comment #18
catcht()'s now defined in bootstrap.inc so this shouldn't be a problem.