I get this error when displaying nodes and the drop-down subscription form does not render correctly. checkboxes are rendered but they are displayed without the drop-down.

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

The "full node links" option seems to work fine.

CommentFileSizeAuthor
#2 828556.patch724 bytesroderik
#1 notifications-bug.jpg12.93 KBstg11

Comments

stg11’s picture

stg11 commented
Category: support » bug
StatusFileSize
new notifications-bug.jpg12.93 KB

I did more testing. The collapsible form works in edit mode, but not in view mode. In addition to error message mentioned above, see attached for a sample of the incomplete rendering of the form.

roderik’s picture

roderik
Primary language Dutch
Location Amsterdam,NL / Budapest,HU
commented
Status: Active » Needs review
StatusFileSize
new 828556.patch724 bytes

notifications_ui/notifications_ui.module:function notifications_ui_nodeapi() has the following line:

    $node->body .= drupal_get_form('notifications_ui_options_form_'.$form_instance_id, $subscriptions, TRUE, TRUE);

But the form definition is

function notifications_subscriptions_options_form($form_state, $subscriptions, $account = NULL, $destination = NULL, $length = 40) {

That can't be; the passed function argument for $account is inconsistent with the definition.

I don't know the code well, but I figured it best to put the 'account' argument in the drupal_get_form call to NULL, instead of TRUE.

roderik’s picture

roderik
Primary language Dutch
Location Amsterdam,NL / Budapest,HU
commented
Status: Needs review » Closed (fixed)

Fix incorporated 2010-07-24, issue not closed yet
http://drupalcode.org/project/notifications.git/commitdiff/ba77ce62b96b1...

roderik’s picture

roderik
Primary language Dutch
Location Amsterdam,NL / Budapest,HU
commented
Status: Closed (fixed) » Needs review

Hm, sorry. Wrong issue. Fix not incorporated.