Hi there,

In our error logs we have the following whenever someone accesses the T&C page:

Notice: Undefined index: extra_pane__node__59 in commerce_extra_panes_termsofservice_form_alter() (line 80 of commerce_extra_panes/modules/commerce_extra_panes_termsofservice/commerce_extra_panes_termsofservice.module).

I've narrowed it down to the following code on line 80 of commerce_extra_panes_termsofservice.module but not sure what else to do?

Any help would be appreciated,

if ($enabled_panes[$pane_id]['page'] == $page_id) {

Thanks

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gc11’s picture

Assigned: gc11 » Unassigned
pcambra’s picture

Category: bug » support
Status: Active » Postponed (maintainer needs more info)

I can't reproduce this...

The page id as you can see is built by the chain "extra pane", + entity type + entity id

Did you delete the node by chance?, what do you have in node/59?

Stan Turyn’s picture

I had the exact same issue and tracked it down - the notice was being caused because I had an extra checkout pane containing a node (specified in that notice), which was disabled in the Checkout form (admin/commerce/config/checkout) but enabled in Checkout extra panes (admin/commerce/config/checkout/extra). I unpublished that node, which in turn disabled the pane in Checkout extra panes - that stopped the notice from being generated.

miro_dietiker’s picture

Category: support » bug
Status: Postponed (maintainer needs more info) » Needs review
FileSize
1.15 KB

I also run into this issue.

We needed adding an !empty check as provided in the patch.

Notice: Undefined index: extra_pane__node__215 in commerce_extra_panes_termsofservice_form_alter() (line 68 of /var/www/sites/all/modules/contrib/commerce_extra_panes/modules/commerce_extra_panes_termsofservice/commerce_extra_panes_termsofservice.module).

gc11’s picture

Issue summary: View changes
Status: Needs review » Closed (fixed)

Sorry it's taken me so long to get back to you. This patch works perfectly. thanks

miro_dietiker’s picture

Status: Closed (fixed) » Reviewed & tested by the community

This is not committed yet.

svouthi’s picture

Thank you, miro_dietiker - I needed this too.

FranciscoLuz’s picture

#4 fixes it.

pvdjay’s picture

I removed superfluous path information from #4 to allow installation via drush make. Otherwise, the patch bombs out.

Steven Jones’s picture

This is a lovely patch, thanks!

ecopeter’s picture

#3 Solved the problem with e-commerce extra panes.