Hello
I am trying to install notifications in a drupal 5.7 site, and have the following error after enabling the module:
warning: Missing argument 1 for notifications_module_info() in /home/humanizasus/htdocs/modules/notifications/notifications.module on line 537.
Any clues?
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | notifications-232610.patch | 4.02 KB | jody lynn |
Comments
Comment #1
visionquest commentedI just had the same problem.
Line 537 is:
function notifications_module_info($op, $arg0 = NULL, $arg1 = NULL, $arg2 = NULL) {The error message says "Missing argument 1" so I changed it to this and it works fine:
function notifications_module_info($op = NULL, $arg0 = NULL, $arg1 = NULL, $arg2 = NULL) {However I am not a coder so if this change could be confirmed as correct I would appreciate it.
Cheers,
Noel
Comment #2
jose reyero commentedThe first parameter for this function is mandatory, it doesn't make sense otherwise.
I cannot reproduce it and I think with the clean up we've been doing this should be gone re open if persits with latest versions.
Comment #3
jody lynnThis is still an issue with latest version. To reproduce, go to admin/build/modules and hit submit.
Comment #4
jody lynnvisionquest, do you use workflow_ng? I think it is a conflict with that module.
Comment #5
jody lynnYeah, I think workflow_ng is using a hook_module_info which is calling notifications_module_info accidentally. I'll post the issue to that project.
Comment #6
jody lynnhttp://drupal.org/node/238641
Comment #7
jose reyero commentedWell, now you guys have found the cause of the bug, this should be easy to fix, I'll just change the notifications_module_info() function name...
(I've closed the issue for workflow-ng btw)
Comment #8
jody lynnThanks Jose. (And I think you are the Jose we smoked cigarettes with at Drupalcon?) I agree with your assessment on the other thread.
Comment #9
jody lynnJose, what would you like the function to be renamed as? I can make a patch for it (the error message is driving me nuts)
Comment #10
jody lynnComment #11
jose reyero commentedThanks a lot!
It seems there are other modules using that name too :-)
Comment #12
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.