I keep getting an error:
Failed to load resource: the server responded with a status of 404 (Not Found) /profiles/openatrium/modules/contrib/jquery_update/replace/ui/ui/minified/jquery.effects.core.min.js?v=1.10.2

this file is missing from the dev version of jquery_update

CommentFileSizeAuthor
#3 effects-core-filename-2040991-3.patch455 bytespeterpoe
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrfelton’s picture

This commit broke it:

commit d32f50d45b54b1b1f6430ea09b83f005032075f0
Author: Eric J. Duran
Date: Fri Mar 22 20:39:23 2013 -0400

Issue #1947438 by ericduran: Fixed Update jQuery UI to the latest version (1.10.2).

reverting to the commit prior to that (469fac05d630668d6f3d8e24fef60c35614ff0ca) resolves the issue as a temporary workaround.

tseven’s picture

Thank you mrfelton, this commit almost worked for me.

Commit 469fac05d630668d6f3d8e24fef60c35614ff0ca fixed the missing .js issue but caused a modal layout issues in Views along with breaking the ajax calls.

I did a git bisect using 469fac... as the good point and HEAD as the bad point and found that:

dd3d76ea2726471a489380fcc7a00a1d65f9be16 is the first bad commit
commit dd3d76ea2726471a489380fcc7a00a1d65f9be16
Author: Eric J. Duran
Date: Wed May 1 17:10:48 2013 -0400

Issue #1974974 by ericduran: Adding ui.tooltip.

Commit: 7b66e7375d05a0b3c81de9c21076693a8f6af695 appears to working fine on my setup.

peterpoe’s picture

Status: Active » Needs review
FileSize
455 bytes

The file has changed name from jquery.effects.core.min.js to jquery.ui.effect.min.js.

ericduran’s picture

Status: Needs review » Fixed

Great catch.

This is now fixed. Proper commit credit giving to @peterpoe. :)

--
http://drupalcode.org/project/jquery_update.git/commit/4043821

altrugon’s picture

OnkelTem’s picture

@altrugon

Sorry, this is not entirely correct. The patch which was finally applied I created 6 months ago: #1966750-1: 'effects' now point to 'ui.effect', no 'effects.core'
And the patch you created with reasonable enhancements 2 months ago still not applied: #1966750-4: 'effects' now point to 'ui.effect', no 'effects.core'

altrugon’s picture

Yeah I saw your patch, I even mentioned you in the comment. Anyway it is fixed :)

Status: Fixed » Closed (fixed)

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