I am getting some warnings on latest dev of jQuery Update. Looks related to David Rothstein's recent patch for jquery.effects.

Warning: file_get_contents(profiles/wetkit/modules/contrib/jquery_update/replace/ui/ui/minified/jquery.effects.drop.min.js): failed to open stream: No such file or directory in _locale_parse_js_file() (line 1488 of /mnt/www/html/wet-boew-drupal/includes/locale.inc).

Looking at the ui/ui folder jquery.effects no longer exists it is referred to as jquery.ui.effects.drop.min.js

Comments

sylus’s picture

StatusFileSize
new1.1 KB

Attaching patch...

sylus’s picture

Status: Active » Needs review
rkallensee’s picture

Issue summary: View changes
StatusFileSize
new1.49 KB

I experienced the same issue - I wasn't able to load jQuery UI effect JS files e.g. via drupal_add_library('system','effects.slide'); because the filenames changed in jQuery UI 1.10.x.

But the previous patch didn't work for me because I think it's important to preserve the keys as specified in system_libraries() while loading the JS files from the new location.

I added another patch where the path segments for the UI effect JS files are specified explicitly.

raphaelhuefner’s picture

StatusFileSize
new3.7 KB

I took a look where and how the file name mapping array which gets improved in the 2 previous patches gets used and discovered that it is not just a mapping from old filenames to new ones, but that there are up to 3 different strings involved:

  • library hook name ("namespaced" in system.module)
  • file name segment used in core (system.module)
  • file name segment used in jquery_update.module

I made a patch for this which is rolled against https://drupal.org/commitlog/commit/3224/65eecb0f1fc69cf6831a66440f72e33...

hefox’s picture

Fixed it, though looks like some commented out code in the patch?

havran’s picture

I have this problem with autocomplete_deluxe (jquery_update-7.x.-2.x from 2013-Oct-14, + autocomplete_deluxe-7.x-2.x-dev from 2013-Nov-26). I use autocomplete_deluxe for freetaging taxonomy field in my content type.

Error in Google Chrome console:

Failed to load resource: the server responded with a status of 404 (Not Found) http://admin.hnonline.dev/sites/all/modules/contrib/jquery_update/replac...
Uncaught TypeError: Cannot set property '_resizeMenu' of undefined

I try use patch from #4 but without success.

raphaelhuefner’s picture

re #5 @hefox Are you referring to the patch from #4?

Please bear with me on this one, it's kinda hard to explain:

It only looks like commented code on first glance. That is an "optical illusion" produced by the way how git diff is producing patches: It does not consider logical code units like foreach loops, function declarations etc. , just individual lines. And the starting lines of the doc comments of the inserted function and of the immediately following function just look the same to git diff, so it applies it's "changed line" logic right smack in the middle of the doc comments. Hence the optical illusion.

Once this patch actually gets git applyed, it will become more apparent what I was trying to explain here. Sorry again!

-----

re #6 @havran How exactly did the #4 patch fail for you?

Please provide more details, like for instance the (error) output of patch -p1 < jquery_effects_missing-2123973-3_0.patch or git apply --verbose jquery_effects_missing-2123973-3_0.patch.

Did you clear caches after applying the patch?

... Etc.

Without sufficient details:
A) Your problem can not be fully understood (and hopefully solved!) by other people looking at this issue.
B) It can not reliably be determined whether in your observation #6 the patch is at fault or not.

hefox’s picture

Status: Needs review » Reviewed & tested by the community

I'm going ahead and RTBC this since encountered it again on a different project.

I really don't see the line I thought was commented out code now (I look at a lot of patch files).

acbramley’s picture

I can't review the logic in this patch but it did fix my issues where a jquery 1.10.x version of an effects file was trying to be included when I had set jquery 1.7 up for admin pages.

ericduran’s picture

I think the file name is just wrong?

I think it might be easier to do that then the patch. Thoughts?

  • Commit d84d48b on 7.x-2.x by ericduran:
    Issue #2123973 by raphaelhuefner, rkallensee, sylus: Failed to open:...
ericduran’s picture

Status: Reviewed & tested by the community » Fixed

This is now Fixed.

Change set should show up soon.

Thanks.

Status: Fixed » Closed (fixed)

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

sylus’s picture

StatusFileSize
new4.1 KB

I had to attach the commit that was made as a patch here so I could leverage it in my drush make while keeping jquery_update to stable.

sokrplare’s picture

Ran into this issue in Hierarchical Select (#873170: HS + jQuery UI 1.8.2 compatibility) - any chance a 2.5 release will come out so less people have contrib module compatibility problems? Looking at that thread a decent amount of time was spent trying to track down just because people were running 7.x-2.4. Thanks for getting a patch on this too, btw!

othermachines’s picture

Thanks for the patch, @sylus.

lklimek’s picture

I had some issues applying patch #14 so I've re-rolled it against 7.x-2.4.

damienmckenna’s picture

avig’s picture

On the jQuery Update configuration page I changed the "Alternate jQuery version for administrative pages " from Default jQuery version to Default provided by Drupal.
This removed the error I had.