Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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
Comment | File | Size | Author |
---|---|---|---|
#9 | commerce_extra_panes_termsofservice_notices_1922126_9.patch | 993 bytes | pvdjay |
#4 | commerce_extra_panes_termsofservice_1922126_notices.patch | 1.15 KB | miro_dietiker |
Comments
Comment #1
gc11 CreditAttribution: gc11 commentedComment #2
pcambraI 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?
Comment #3
Stan Turyn CreditAttribution: Stan Turyn commentedI 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.
Comment #4
miro_dietikerI 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).
Comment #5
gc11 CreditAttribution: gc11 commentedSorry it's taken me so long to get back to you. This patch works perfectly. thanks
Comment #6
miro_dietikerThis is not committed yet.
Comment #7
svouthi CreditAttribution: svouthi commentedThank you, miro_dietiker - I needed this too.
Comment #8
FranciscoLuz CreditAttribution: FranciscoLuz commented#4 fixes it.
Comment #9
pvdjay CreditAttribution: pvdjay commentedI removed superfluous path information from #4 to allow installation via drush make. Otherwise, the patch bombs out.
Comment #10
Steven Jones CreditAttribution: Steven Jones at ComputerMinds commentedThis is a lovely patch, thanks!
Comment #11
ecopeter CreditAttribution: ecopeter commented#3 Solved the problem with e-commerce extra panes.