It looks like for minified libraries, you can provide a map to developer tools like chrome to help demystify the minified variables.

The jquery version in core doesn't have this, but 1.10.2 at least does. Works fine out of box, but using core aggegated setting moves the js to the files directory, but not the map, so it looks in the wrong place for the map.

I don't really know how this would be fixed other than moving the map to the js files directory on aggegation of the js, if that is at all possible.

http://stackoverflow.com/questions/18365315/jquerys-jquery-1-10-2-min-ma... for more information about this file.

CommentFileSizeAuthor
#5 2217263-jquery-min-map.patch153.6 KBjcisio
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MXT’s picture

MXT
Primary language Italian
Location Milan
CreditAttribution: MXT commented

I'm experiencing this issue too.

Thank you very much for resolving this

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Title: Missing file sites/default/files/js/jquery-1.10.2.min.map in chrome developer's tools when process enabled » Add jquery-1.10.2.min.map

Closed a duplicate: #2331141: Is it possible to cache jquery-1.10.2.min.map?

jcisio’s picture

jcisio CreditAttribution: jcisio commented
Title: Add jquery-1.10.2.min.map » Safari: jquery-1.10.2.min.map not found in sites/default/files/js
Category: Feature request » Bug report
jcisio’s picture

jcisio CreditAttribution: jcisio commented

Since jQuery 1.11.0 there is no source map indication in the minified file (http://blog.jquery.com/2014/01/24/jquery-1-11-and-2-1-released/). I think we can simply remove this line in 1.10.2.

jcisio’s picture

jcisio CreditAttribution: jcisio commented
Status: Active » Needs review
FileSize
2217263-jquery-min-map.patch153.6 KB

I don't know how if there is any other way to use the shipped sourcemaps file so I left it as is.

echoz’s picture

echoz CreditAttribution: echoz commented

Deleting this line got rid of the reproducible not found warning, and jquery is working fine. I don't know enough about the best fix, so leaving this as Needs review.

Tess Bakker’s picture

Tess Bakker
they/them
Primary language Dutch
CreditAttribution: Tess Bakker at ezCompany commented
Status: Needs review » Reviewed & tested by the community

Tested the patch with an active site: Perfect result, no watchdog messages any more and everything (JS) still working
Tested the patch with current 7.x-2.x branch: without any issue

Conclusion: RTBC

jenlampton’s picture

jenlampton
she / her
CreditAttribution: jenlampton at Jeneration Web Development for Metropolitan Transportation Commission commented

Confirming RTBC status. Patch works perfectly, thanks!

helmo’s picture

helmo CreditAttribution: helmo as a volunteer and at Initfour websolutions for Aegir Cooperative commented
Version: 7.x-2.x-dev » 7.x-3.x-dev

Another +1 for RTBC

It also applies to 7.x-3.x.

amme’s picture

amme CreditAttribution: amme commented
Parent issue: » #2852350: [jquery_update] 7.x-3.0 stable release

  • markcarver committed 27fb37d on 7.x-3.x authored by jcisio 
    Issue #2217263 by jcisio, hefox, Tessa Bakker: Safari: jquery-1.10.2.min...
markhalliwell’s picture

markhalliwell
Primary language English
CreditAttribution: markhalliwell at Tag1 Consulting commented
Status: Reviewed & tested by the community » Fixed
markhalliwell’s picture

markhalliwell
Primary language English
CreditAttribution: markhalliwell at Tag1 Consulting commented
markhalliwell’s picture

markhalliwell
Primary language English
CreditAttribution: markhalliwell at Tag1 Consulting commented

Status: Fixed » Closed (fixed)

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