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 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 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 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 commentedThank you, miro_dietiker - I needed this too.
Comment #8
FranciscoLuz commented#4 fixes it.
Comment #9
pvdjay commentedI removed superfluous path information from #4 to allow installation via drush make. Otherwise, the patch bombs out.
Comment #10
steven jones commentedThis is a lovely patch, thanks!
Comment #11
ecopeter commented#3 Solved the problem with e-commerce extra panes.