Hello,

I am getting the following error (see screenshot).

I think its just a missing variable as in the patch I am attaching.

From :

$add_license_setting = isset($aggregate_settings['variables']['advagg_js_compress_add_license']) ? ['variables']['advagg_js_compress_add_license'] : variable_get('advagg_js_compress_add_license', ADVAGG_JS_COMPRESS_ADD_LICENSE);

to :

$add_license_setting = isset($aggregate_settings['variables']['advagg_js_compress_add_license']) ? $aggregate_settings['variables']['advagg_js_compress_add_license'] : variable_get('advagg_js_compress_add_license', ADVAGG_JS_COMPRESS_ADD_LICENSE);

Best regards

CommentFileSizeAuthor
#6 advagg-2682463-5-fix-syntax.patch1.27 KBmikeytown2
#2 2682463.jpg137.25 KBjoaogarin
#2 advagg-undefined-advagg_js_compress_add_license-2682463-2.patch2.24 KBjoaogarin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joaogarin created an issue. See original summary.

joaogarin’s picture

joaogarin CreditAttribution: joaogarin as a volunteer commented
FileSize
advagg-undefined-advagg_js_compress_add_license-2682463-2.patch2.24 KB
2682463.jpg137.25 KB

And the patch and screenshot

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Needs review » Fixed

Thanks for the patch! #2 has been committed.

  • mikeytown2 committed edd6337 on 7.x-2.x 
    Issue #2682463 by mikeytown2: Fix php syntax.
mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
FileSize
advagg-2682463-5-fix-syntax.patch1.27 KB

Missed one; this has been committed

Status: Fixed » Closed (fixed)

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