Notice: Undefined index: scope in advagg_mod_js_move_to_footer() (line 1175 of /sites/all/modules/contrib/advagg/advagg_mod/advagg_mod.module).
Notice: Undefined index: scope in advagg_mod_js_move_to_footer() (line 1179 of /sites/all/modules/contrib/advagg/advagg_mod/advagg_mod.module).
CommentFileSizeAuthor
#5 advagg-2790299-5-remove-non-scope-js.patch1.22 KBmikeytown2
#2 undefined_index_scope-2790299-2.patch803 bytesmarkdorison
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

markdorison created an issue. See original summary.

markdorison’s picture

markdorison
he/him
CreditAttribution: markdorison at Chromatic commented
Status: Active » Needs review
FileSize
undefined_index_scope-2790299-2.patch803 bytes
mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented

Does this happen to you with #2752147-9: Undefined index: weight/group/every_page in advagg_mod_sort_css_js/advagg_drupal_sort_css_js_stable, the latest dev in short?

markdorison’s picture

markdorison
he/him
CreditAttribution: markdorison at Chromatic commented

@markdorison: Yes, this occurs with the latest dev release.

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
FileSize
advagg-2790299-5-remove-non-scope-js.patch1.22 KB

This should fix the root issue; js files have to have scope otherwise they don't get added. Noted that this should fix the issue in here as well #2790301: Undefined index: scope in drupal_get_js()

I still think you have a js_alter hook that is doing something wrong.

  • mikeytown2 committed 2e61707 on 7.x-2.x 
    Issue #2790299 by markdorison, mikeytown2: Undefined index: scope in...
mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Needs review » Fixed

#5 has been committed.

Status: Fixed » Closed (fixed)

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