Closed (fixed)
Project:
Storm
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
8 Nov 2008 at 17:10 UTC
Updated:
12 Mar 2009 at 14:00 UTC
getting this error :
Fatal error: Call to undefined function storm_t() in /home/xweb/public_html/sites/all/modules/storm/stormticket/stormticket.module on line 159
Comments
Comment #1
jorditr commentedIt's weird, any module should call that function except a storm module. It looks like the stormticket.module is still active, becuase it's that the one calling for storm_t(). Maybe you deactivated storm.module before deactivating stormticket.module.
Comment #2
najibx commentedhmm I don't think so as I can't deactivated storm.module since stormticket.module depended on it.
Anyway, I removed in modules in system, and removed the folder and tables manually
Comment #3
higherform commentedMay be related:
I was getting the following error in the Apache log:
Fatal error: Call to undefined function storm_t() in /*removed*/sites/all/modules/storm/stormattribute/stormattribute.module on line 31, referer: *removed*/admin/build/modules/uninstall/confirm"
when attempting to uninstall the storm attribute module.
Comment #4
Roberto Gerola commentedThe dependencies seem to be correctly set.
I'll test it, but I am afraid there is nothing I can do to fix this.
I'll see what is possible to do in any case.
Roberto
Comment #5
higherform commentedIf you re-install the main Storm module, uninstall of the sub modules proceeds correctly.
For someone uninstalling for the first time, uninstall all of the sub modules in one pass first, then disable/uninstall the Storm main module in a second pass.
@Roberto -
It appears dependencies do not apply to disabled modules for uninstall. Please be sure the uninstall routines have no reference to storm_t().
Comment #6
Roberto Gerola commented> Please be sure the uninstall routines have no reference to storm_t()
It is already so.
It seems that storm module has been removed prior to remove the dependent modules,
otherwise the only option is to remove the support for storm_t function and force the users
to use the replace strings module to achieve the same result.
Comment #7
clemens.tolboomThe reason the uninstall fails is the uninstall needs to know what menus to remove.
You are using storm_t in your hook_menu() which I think is conceptually wrong. If you use t() the problem would go away :-)
Comment #8
Roberto Gerola commented>You are using storm_t in your hook_menu() which I think is conceptually wrong.
storm_t permits to override strings before passing them to Drupal t function.
I don't know if someone is using this feature.
I'll remove it. The users interested in customizing the labels can use strings_override module.
Thanks, Roberto
Comment #9
Roberto Gerola commentedstorm_t has been removed now from anywhere.
User strings_override module if you need to change the default labels.
Thanks, Roberto
Comment #10
clemens.tolboomThanks.
Guess we can close the issue now. Looks better for your project ;)