Download & Extend
Organic groups » Issues

Error during queue process / post submission: warning: array_key_exists()

Posted by AFowle on April 13, 2009 at 6:58pm
Project:Organic groups
Version:6.x-2.x-dev
Component:og_notifications
Category:bug report
Priority:normal
Assigned:Zen
Status:closed (fixed)

Issue Summary

This error has occurred several times when I have pressed the "Run Process" button in the queue manager.

warning: array_key_exists(): The first argument should be either a string or an integer in /host/sites/all/modules/notifications/notifications.module on line 671.

I have not managed to pin down the circumstances yet and I am sorry I cannot provide more details - I was actually trying to test OG 6.x-2.0-rc1. May be an interaction between the two.

Login or register to post comments

Comments

#1

Posted by AFowle on April 13, 2009 at 7:02pm

See also http://drupal.org/node/360291#comment-1470248 and wherever this comment gets reposted!

#2

Posted by michtoen on April 16, 2009 at 3:05pm

We got the same problem with OG 1.4:

warning: array_key_exists() [function.array-key-exists]: The first argument should be either a string or an integer in modules/notifications/notifications.module on line 671.

#3

Posted by TyraelTLK on April 20, 2009 at 11:40am

same problem with OG 1.4
subscribing

#4

Posted by Jose Reyero on April 22, 2009 at 10:30pm
Project:Notifications» Organic groups
Version:6.x-1.1» 6.x-1.1
Component:Code» og_notifications

og 1.4??

Added some notes about og/notifications compatibility in this project's front page.

Anyway I think this is an issue with notifications_og. The function _og_notification_check_message() just returns TRUE / FALSE so it is not used properly in og_notifications_message_alter().

Btw, couldn't we use a better method for this message template rewriting? which other hooks do we need?

#5

Posted by AFowle on April 24, 2009 at 11:29pm

D6.10
M&N 6.x-2.0-beta5
OG 6.x-2.x-dev (2009-Apr-24)

This combination is a huge improvement, for this error and others. Not quite perfect though. I think remaining errors are in messaging and OG notifications, so I'll post there.

#6

Posted by Jose Reyero on April 28, 2009 at 9:41am
Title:Error during queue process» Error during queue process / post submission: warning: array_key_exists()
Version:6.x-1.1» 6.x-2.x-dev
Priority:normal» critical

Making the title more explanatory. I keep getting bug reports in Notifications about this. We need to fix it before next release.

#7

Posted by 63reasons on May 5, 2009 at 4:11am
Assigned to:Anonymous» 63reasons

Hi,

I am fixing this issue, I using these modules:

- OG 2.x (today tarball)
- Notifications 2.0

Trying to get your bugs reported, but no luck. Bug fixed?

Thanks

#8

Posted by 63reasons on May 5, 2009 at 4:11am
Priority:critical» normal

#9

Posted by AFowle on May 5, 2009 at 6:21pm
Priority:normal» critical

I am sorry I do not understand your comment, but later versions do work better as you have discovered.

As for the "Priority" it should be "critical" according to the handbook (see link near "post new comment") and I don't think it was necessary for you to change it - Jose Reyero, main author of the module, chose critical himself.

#10

Posted by tsi on May 10, 2009 at 11:46am

Is this fixed if using the 2.x versions of both ?

#11

Posted by AFowle on May 10, 2009 at 8:28pm

Yes it seems to be. I'm currently using
Drupal 6.11
OG 6.x-2.x-dev (2009-May-02)
M & N 6.x-2.0

Trouble is there are so many variations to try with just these modules, then further interactions with views etc. So I can't say there are no problems, but this combination is a significant advance.

#12

Posted by Rosamunda on May 12, 2009 at 3:52pm

Subscribing.
Is this going to be solved at 6.1.x versions too?

#13

Posted by tsi on May 12, 2009 at 10:05pm
Version:6.x-2.x-dev» 6.x-2.0-rc1

I now work with :
Drupal 6.11
OG 6.x-2.0-rc1
M & N 6.x-2.0

and I got this error :
warning: array_key_exists() [function.array-key-exists]: The first argument should be either a string or an integer in .../modules/notifications/notifications.module on line 761

after commenting with immidiate notifications on.
So I guess this problem wasn't fixed, or should I try the 2.x-dev version of OG ?

#14

Posted by Rosamunda on May 12, 2009 at 11:27pm

It´s not fixed on 2.x version either (I didn´t tried the dev, only the stable one).

#15

Posted by pathfinderelite on May 13, 2009 at 1:52pm

Same issue using 6.x-1.3. subscribing.

#16

Posted by Zen on May 17, 2009 at 6:41am
Version:6.x-2.0-rc1» 6.x-2.x-dev
Priority:critical» normal
Assigned to:63reasons» Zen
Status:active» postponed (maintainer needs more info)

I tested this with the latest devs, delivery method = Web and send interval = 2 minutes with about 20 notifications queued. I could not reproduce the error. Please test again and provide detailed steps for reproduction of the issue, if need be.

This is non-critical.

-K
P.S. 63reasons: If you aren't working on a solution to an issue, please avoid setting yourself as the assignee.

#17

Posted by 63reasons on May 17, 2009 at 7:28am

We were still awaiting cooperation on the issue, but go ahead if you can get it before us.

#18

Posted by AFowle on May 17, 2009 at 8:54am

@ Zen

I consider this fixed (as the original poster). I haven't seen the problem for some time, in the rapidly evolving mix of OG and M&N. See post #11. I am now using

Drupal 6.12
Messaging 6.x-2.0
Notifications 6.x-2.0
Organic groups 6.x-2.x-dev (2009-May-14)

As others have said above, it would be more comfortable to have a released version of OG rather than the dev. I don't know what the practice is for closing an issue before the released version?

I am very grateful to the developers of both sets of modules.

#19

Posted by Zen on May 17, 2009 at 5:03pm
Status:postponed (maintainer needs more info)» fixed

Adrian, thanks for the confirmation. There will very likely be an RC out RSN.

-K

#20

Posted by System Message on May 31, 2009 at 5:10pm
Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

#21

Posted by ianchan on July 11, 2009 at 4:44pm

subscribe