Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I am posting it here because I found the issue while using Openatrium but I don't think it's an Openatrium bug per se.
backbone library has backbone-min.map
at the end of backbone-min.js there is this line:
/*
//@ sourceMappingURL=backbone-min.map
*/
When aggregation is turned on, backbone-min.map is expected to be found in the same directory as the backbone-min.js file which after aggregation is files/js.
I had to change the line to
/*
//@ sourceMappingURL=/profiles/openatrium/libraries/backbone-min.map
*/
Is this line necessary anyway?
This is a library issue so I am just noting it here. I am not quite sure what you could do other than adding it to the Readme file.
Comment | File | Size | Author |
---|---|---|---|
#13 | 2235299-12.patch | 212 bytes | xtfer |
#11 | 2235299-11.patch | 202 bytes | xtfer |
Comments
Comment #1
JKingsnorth CreditAttribution: JKingsnorth commentedConfirmed this bug.
Comment #2
mpotter CreditAttribution: mpotter commentedYou should post this in the issue queue for the Navbar module since that is where the backbone.js dependency is being added. If there is a patch for this, let me know, or let Panopoly know, and we can include it. But this isn't something I can fix directly in OA2.
Comment #3
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedchanging to this issue queue to make you aware too
Comment #4
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedComment #5
SweetTomato CreditAttribution: SweetTomato commentedWhat do we lose by losing access to that file? It's missing in mine as well with advagg, but other than performance, I'm not aware of how it hurts our application.
Comment #6
bkildow CreditAttribution: bkildow commentedI believe this is only seen with Chrome dev-tools. This has some information on how to avoid or fix this: http://stackoverflow.com/questions/18365315/jquerys-jquery-1-10-2-min-ma....
Comment #7
jasonlttl CreditAttribution: jasonlttl commentedI'm sure this is a dumb solution but....
The following is some sample code to register a menu entry for the js aggregation based backbone-min.map path. If the libraries module is installed and the backbone library is found, it will redirect the incorrect backbone path to the correct one and you'll stop getting 404's for this in watchdog.
Comment #8
xtfer CreditAttribution: xtfer commentedThis is causing issues with the Panels IPE.
The following patch can be applied in a .make file to remove the code from the backbone library.
If a .make file was provided for navbar, it could be resolved there.
Comment #10
xtfer CreditAttribution: xtfer commentedTagged release and archive not in the same format, for some reason.
Comment #11
xtfer CreditAttribution: xtfer commentedAnd again, since the previous one wouldn't apply anyway.
Comment #13
xtfer CreditAttribution: xtfer commentedAnd another for Underscore, which suffers from the same issue.
Comment #14
xtfer CreditAttribution: xtfer commentedComment #16
Marko B CreditAttribution: Marko B commentedI have the same problem when I turn on JS aggregation.
Remote Address:127.0.0.1:80
Request URL:http://site.dd/sites/default/files/js/backbone-min.map;
Request Method:GET
Status Code:404 Not Found
What I find strange is that Request URL:http://site.dd/sites/default/files/js/backbone-min.map; ends with ;
Comment #17
hass CreditAttribution: hass commentedI think we need to have a core aggregation case to remove all occurences of sourceMappingURL and sourceURL including //@ and //# syntax.
Comment #18
hass CreditAttribution: hass commentedComment #19
hass CreditAttribution: hass commentedPlease join #2400287: Remove all occurences of sourceMappingURL and sourceURL when JS files are aggregated and test the patches to get them committed to all core versions.
I'm postponing here in hope more people see it and help testing. Normally we mark such cases as duplicate at later stage. We may need to add a known issue information somewhere to the module.
Comment #20
hass CreditAttribution: hass commentedSince this is a core issue I'm closing now. Please help getting it into core.
Comment #21
Marko B CreditAttribution: Marko B commentedFor my case scenario the solution was this.
http://stackoverflow.com/questions/18365315/jquerys-jquery-1-10-2-min-ma...
Comment #22
Marko B CreditAttribution: Marko B commentedFor my case scenario the solution was this.
http://stackoverflow.com/questions/18365315/jquerys-jquery-1-10-2-min-ma...