Hello,
today i upgraded our multisite drupals and recognized a javascript error concerning jQuery and $-functions. The problems vanished after disabling "Advanced Aggregation".

So i investigated the source of this and found the problem within advagg_js_cdn.module on line 84:

if (isset($jquery_update_filepath) && ($path == $jquery_update_filepath . '/replace/jquery.min.js' || $path == $jquery_filepath . '/replace/jquery.js')) {

The last $jquery_filepath is wrong and should be $jquery_update_filepath:

if (isset($jquery_update_filepath) && ($path == $jquery_update_filepath . '/replace/jquery.min.js' || $path == $jquery_update_filepath . '/replace/jquery.js')) {

I created a patch for this which is attached to this posting. It needs to be placed within the modules folder.

Testing:
patch -p1 --dry-run < patch_advagg_jquery_google_cdn_fix.patch

Patching:
patch -p1 < patch_advagg_jquery_google_cdn_fix.patch

Regards
.nowrap

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rjbrown99’s picture

Status: Active » Needs review

Patch here.

mikeytown2’s picture

Status: Needs review » Fixed
FileSize
1.87 KB

This patch has been committed.
Works with the latest jquery update dev.

Status: Fixed » Closed (fixed)

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