Notice: Undefined index: module in views_plugin_list() (line 278 of /*/*/*/modules/contrib/views/views.module).

I got this php notice on the Feature recreate admin page when the feature contains at least one view.

CommentFileSizeAuthor
#5 panelizer-n2524258-5.patch536 bytesDamienMcKenna
#1 panelizer-php_notice-707484-1.patch736 byteskovacsaba
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kovacsaba’s picture

kovacsaba CreditAttribution: kovacsaba commented
FileSize
panelizer-php_notice-707484-1.patch736 bytes

Patch attached

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Version: 7.x-3.1 » 7.x-3.x-dev
Status: Active » Needs review

The code is still exactly the same in the latest -dev release, so lets test the patch.

BTW please remember to update the issue status to "needs review" when you upload a patch, that'll trigger the testbot and let others know that there's a patch to look at.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Parent issue: » #2155813: Plan for Panelizer 7.x-3.2 release

.

Status: Needs review » Needs work

The last submitted patch, 1: panelizer-php_notice-707484-1.patch, failed testing.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs work » Needs review
FileSize
panelizer-n2524258-5.patch536 bytes

Also, when you create a patch you should do it using a fresh copy of the module from git, rather than from your site's home directory, that way the directory paths line up.

I've rerolled the patch so it has the correct paths.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs review » Fixed

Thanks for the patch. Committed.

Status: Fixed » Closed (fixed)

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