Omega is generating the following error in the logs:

Notice: Undefined index: #id in omega_form_alter() (line 360 of omega/omega/template.php
CommentFileSizeAuthor
#2 omega-undefined_index-2391751-2.patch1.01 KBgrasmash
#1 omega-undefined_index-2391751-1.patch937 bytesgrasmash
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

grasmash’s picture

grasmash CreditAttribution: grasmash commented
Status: Active » Needs review
FileSize
omega-undefined_index-2391751-1.patch937 bytes
grasmash’s picture

grasmash CreditAttribution: grasmash at Acquia for U.S. Department of Justice commented
FileSize
omega-undefined_index-2391751-2.patch1.01 KB

A bit of a more robust solution.

das-peter’s picture

das-peter CreditAttribution: das-peter commented
Status: Needs review » Reviewed & tested by the community

A bit unfortunate that we've to do this that way but more defensive code is better :)
In my case it's Revision Scheduler that triggers the notice by invoking hook_form_alter() on its own.
Patch looks good, there's only one tiny nitpick:

+++ b/omega/template.php
@@ -357,7 +357,12 @@ function omega_form_alter(&$form, &$form_state, $form_id) {
+   $form['#attributes']['class'][] = drupal_clean_css_identifier($form_id);

One leading space missing.

But I'd say RTBC.

Edit: Typo fixed.

fubhy’s picture

fubhy CreditAttribution: fubhy commented
Status: Reviewed & tested by the community » Fixed

Thanks.

  • fubhy committed b1e0fcd on 7.x-4.x authored by grasmash 
    Issue #2391751 by grasmash: Notice: Undefined index: #id in...

Status: Fixed » Closed (fixed)

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