Hey,
When I installed the module and tried to browse to admin/messaging/notifications/subscriptions, I got this error.
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | notifications-array-combine-workaround-1021146-10.patch | 1.1 KB | scottrigby |
Hey,
When I installed the module and tried to browse to admin/messaging/notifications/subscriptions, I got this error.
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | notifications-array-combine-workaround-1021146-10.patch | 1.1 KB | scottrigby |
Comments
Comment #1
kartagisThe very same thing happens for 4.0-beta7 too
Comment #2
Michsk commentedyeh same here.
Comment #3
miro_dietikerCould you please inspect the code line and check the variables using e.g. var_dump() to display the data...
I wonder which one is non-array in your case...
Did you install notifications previously and there might be some old config data around somehow?
Comment #4
Michsk commentedi removed notifications.
Comment #5
ryan_courtnage commented@miro_dietiker
Offending line is:
$types += array_combine($type_keys, $type_keys);$types is an empty array. $type_keys is an empty array.
Comment #6
robotjox commentedsubscribing
Comment #7
danny englanderSubscribing, same issue.
Comment #8
udvranto commentedInstalling content notification solved the problem.
Comment #9
askibinski commentedI can confirm enabling content notification solves the problem. Should be a dependency maybe?
Comment #10
scottrigbyThis seems to be from a php bug in array_combine().
@see http://bugs.php.net/bug.php?id=34857
array_combine(array(), array()) will return an error instead of an empty array.
Comment #11
scottrigbyComment #12
neilnz commentedConfirm this works as advertised and fixes the fatal error for me. I'm not using content_notifications, only notifications_custom
Comment #13
ggevalt commentedThis discussion was made a year ago. I just ran into the same problem with the 6.x-4.0-beta7 version.
What was the outcome? Was this reviewed? It does not appear to have been fixed... Status?
thanks.
Here's the error we get: array_combine(): Both parameters should have at least 1 element in … modules/notifications/notifications.module on line 1096
Comment #14
scottrigby@ggevalt did you still get the error after applying the patch in #10?
Comment #15
ggevalt commentedHere's the quick answer:
Yes.
But here's what happened to me... I installed Beta8 on dev site ... I was thrown a lot of errors and decided to go ahead and uninstall earlier version of Notifications, deleted folder, etc. and reinstalled modules. Again, did this on the dev. site. Everything seemed to go OK. UNTIL I hit the subscribe tab. Enabled content notifications and problem seemed to resolve itself.
However, production site a different story. Did the uninstall on the early version. Uploaded the new version. did update.php. Installed content notifications. Everything seemed OK.
Then enabled flags and set that up. Everthing OK until I enabled a token on flags to count number of flags per node. WSOD.
Unwound hours of work (all this was fine on the dev version, which remains a mystery), then used the patch. Still a problem. I kept unwinding and was able to pinpoint that the problem is occurring with content notifications module -- when it's enabled, it conflicts with Flags token to count flags... when it's not enable, everything fine, except, of course, the subscribe tab.
Just seems to me there is something bad in the fact that in this version, there seems to be a fight going on between two of the modules.
I am going to go backwards on versions and see how I fare.
Thanks again for your courtesy in responding.
geoff
Comment #16
rfay#10 works fine and does the job.
This is a critical, fatal failure. php 5.3.2 on Ubuntu Lucid. Visiting admin/messaging/notifications/subscriptions
Comment #17
miro_dietikerFixed. The bug seems not to be present in 7.x.1-x.
I think we should cover this by tests also...