WordPress.org

Make WordPress Core

Opened 8 months ago

Closed 8 months ago

Last modified 8 months ago

#24994 closed defect (bug) (fixed)

jquery-1.10.2.min.map 404 error

Reported by: mindctrl Owned by: nacin
Milestone: 3.6.1 Priority: normal
Severity: normal Version: 3.6
Component: External Libraries Keywords: has-patch
Focuses: Cc:

Description

Starting with 3.6 (I believe), many sites are producing a 404 error looking for /wp-includes/js/jquery/jquery-1.10.2.min.map. This can be seen in a default WP 3.6 install with no active plugins and with the default theme activated. This can also be seen on make.wordpress.org. A variant is happening on several wordpress.com sites I checked. There it's throwing an error 400.

Attachments (1)

24994.patch (91.3 KB) - added by Jayjdk 8 months ago.

Download all attachments as: .zip

Change History (11)

comment:1 helen8 months ago

Related: #24879

comment:3 doodlebee8 months ago

I can verify that this is an issue - on all sites I've worked on that have upgraded to 3.6, this has shown up on every one of them - even brand new clean installs. I just tried the fix for line 2 on jquery.js, and the issue still persists. However, if I completely *delete* line 2, it fixes it. (apparently commenting the line out doesn't fix the problem - which is really weird.)

comment:4 nacin8 months ago

  • Component changed from General to External Libraries
  • Milestone changed from Awaiting Review to 3.6.1

Yeah, we should be removing this line.

#24879 would be the enhancement here to making this line work in the future.

Jayjdk8 months ago

comment:5 Jayjdk8 months ago

  • Keywords has-patch added

comment:6 nacin8 months ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In 25072:

Remove sourceMappingURL from jquery.min.js.

props Jayjdk.
fixes #24994.

comment:7 follow-up: unsalkorkmaz8 months ago

http://wordpress.org/support/topic/get-wp-adminjquery-1102minmap-404-not-found
paul irish saying:
"the real fix is just to include the .map file in the place it belongs.
download http://code.jquery.com/jquery-1.10.2.min.map and toss it in!"

comment:8 in reply to: ↑ 7 nacin8 months ago

Replying to unsalkorkmaz:

http://wordpress.org/support/topic/get-wp-adminjquery-1102minmap-404-not-found
paul irish saying:
"the real fix is just to include the .map file in the place it belongs.
download http://code.jquery.com/jquery-1.10.2.min.map and toss it in!"

Indeed, but also: http://bugs.jquery.com/ticket/14225#comment:1. We currently have jquery.js, not jquery.min.js. This is something we can look at in #24879.

comment:9 nacin8 months ago

In 25073:

Remove sourceMappingURL from jquery.min.js.

Merges [25072] to the 3.6 branch.

props Jayjdk.
fixes #24994.

comment:10 cais8 months ago

  • Cc edward.caissie@… added
Note: See TracTickets for help on using tickets.