Notice: Undefined index: panel in semantic_panels_onecol_settings_form() (line 56 of path/to/drupal/sites/all/modules/semantic_panels/plugins/layouts/semantic_panels_onecol/semantic_panels_onecol.inc).
Notice: Undefined index: panel in semantic_panels_onecol_settings_form() (line 66 of path/to/drupal/sites/all/modules/semantic_panels/plugins/layouts/semantic_panels_onecol/semantic_panels_onecol.inc).
Notice: Undefined index: regions in semantic_panels_onecol_settings_form() (line 102 of path/to/drupal/sites/all/modules/semantic_panels/plugins/layouts/semantic_panels_onecol/semantic_panels_onecol.inc).
Notice: Undefined index: regions in semantic_panels_onecol_settings_form() (line 112 of path/to/drupal/sites/all/modules/semantic_panels/plugins/layouts/semantic_panels_onecol/semantic_panels_onecol.inc).
Notice: Undefined index: panel in template_preprocess_semantic_panels_onecol() (line 130 of path/to/drupal/sites/all/modules/semantic_panels/plugins/layouts/semantic_panels_onecol/semantic_panels_onecol.inc).
Notice: Undefined index: regions in include() (line 21 of path/to/drupal/sites/all/modules/semantic_panels/plugins/layouts/semantic_panels_onecol/semantic-panels-onecol.tpl.php).
Warning: Invalid argument supplied for foreach() in include() (line 21 of path/to/drupal/sites/all/modules/semantic_panels/plugins/layouts/semantic_panels_onecol/semantic-panels-onecol.tpl.php).

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sillygwailo’s picture

Status: Active » Needs review
FileSize
3.18 KB

Patch attached. I went with sending an array with an empty 'type' attribute to _semantic_panels_prepare_element_settings() if the variable was empty to avoid a warning on the other end.

logaritmisk’s picture

Status: Needs review » Closed (fixed)

I couldn't recreate these warnings but I can see how this could be a problem. I tried your patch and it seems to work fine, thanks!