Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#24994 closed defect (bug) (fixed)

jquery-1.10.2.min.map 404 error

Reported by: mindctrl's profile mindctrl Owned by: nacin's profile 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 11 years ago.

Download all attachments as: .zip

Change History (11)

#1 @helen
11 years ago

Related: #24879

#3 @doodlebee
11 years 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.)

#4 @nacin
11 years 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.

@Jayjdk
11 years ago

#5 @Jayjdk
11 years ago

  • Keywords has-patch added

#6 @nacin
11 years 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.

#7 follow-up: @unsalkorkmaz
11 years 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!"

#8 in reply to: ↑ 7 @nacin
11 years 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.

#9 @nacin
11 years ago

In 25073:

Remove sourceMappingURL from jquery.min.js.

Merges [25072] to the 3.6 branch.

props Jayjdk.
fixes #24994.

#10 @cais
11 years ago

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