Related to #1335752 Aggregate JavaScript files.

User had an unrelated problem but the PHP notice mentioned in the post wasn't resolved. It just popped up for me on a fresh install.

Location

admin/structure/nodequeue

Action

Click "Add content" before the field is populated. (Yep, first time users like me will actually do that.)

Messages

Notice: Undefined offset: 1 in nodequeue_arrange_subqueue_form_add_submit() (line 1033 of /home/alonk/public_html/modules/nodequeue/includes/nodequeue.admin.inc).

Please enter a valid node title.

Patch

Simple isset() check added.

Cheers -

CommentFileSizeAuthor
nodequeue-undefined_offset.patch950 bytesothermachines
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

delphian’s picture

delphian CreditAttribution: delphian commented
Status: Active » Needs review
kurtzhong’s picture

kurtzhong CreditAttribution: kurtzhong commented

The patch works, thanks.

eugene.ilyin’s picture

eugene.ilyin CreditAttribution: eugene.ilyin commented

Please commit this patch

bessone’s picture

bessone CreditAttribution: bessone commented

Thanks!

othermachines’s picture

othermachines CreditAttribution: othermachines commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Patch still applies against -dev. Commit please?

  • fizk committed 81f7657 on 7.x-2.x authored by othermachines 
    Issue #1797280 by othermachines: Undefined offset: 1 in...
fizk’s picture

fizk CreditAttribution: fizk commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks!

Status: Fixed » Closed (fixed)

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