Closed (fixed)
Project:
Notifications
Version:
5.x-1.0-alpha1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Mar 2008 at 00:28 UTC
Updated:
14 Apr 2008 at 16:31 UTC
Jump to comment: Most recent file
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.