Technically speaking this not a bug in AdvAgg; but rather with the JSMin PHP extension. It is not a problem with the standalone JSMin either. Obviously it would be good to fix in the extension; however in the meantime and or for users with old versions.

CommentFileSizeAuthor
#2 jsmin_php_extension-2699169-2.patch966 bytesNickDickinsonWilde
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

NickWilde created an issue. See original summary.

NickDickinsonWilde’s picture

NickDickinsonWilde
He/Him/His
Primary language English
Location Victoria, BC (T'So-uke lands)
CreditAttribution: NickDickinsonWilde commented
Status: Active » Fixed
FileSize
jsmin_php_extension-2699169-2.patch966 bytes

  • NickWilde committed 689d466 on 8.x-2.x 
    Issue #2699169 by NickWilde: JSMin PHP Extension fails with mixed tabs/...

  • mikeytown2 committed 7980741 on 7.x-2.x authored by NickWilde 
    Issue #2699169 by NickWilde, mikeytown2: JSMin PHP Extension fails with...
mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
FileSize
advagg-2699995-2-fix-php-warning.patch1.04 KB

Committed this fix for the 7.x branch.

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
FileSize
advagg-2699169-4-strip-tabs-jsmin.patch1.73 KB

  • mikeytown2 committed 7d91643 on 7.x-2.x 
    Issue #2699169 by NickWilde, mikeytown2: JSMin PHP Extension fails with...

Status: Fixed » Closed (fixed)

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