Context Navigation

← Previous Ticket
Next Ticket →

#27836 closed defect (bug) (fixed)

Uncaught TypeError in media-editor.js on edit.php screen

Reported by:	helgatheviking	Owned by:	markjaquith
Priority:	normal	Milestone:	3.9
Component:	Posts, Post Types	Version:	3.9
Severity:	normal	Keywords:
Cc:		Focuses:	javascript

Description

I'm seeing the following error in my console on the edit.php screen for all post types.

Uncaught TypeError: Cannot read property 'id' of undefined
media-editor.js?ver=3.9-RC2-src:538

Does this script even need to be loaded on this page?

Attachments (1)

27836.diff (451 bytes ) - added by nacin 12 years ago.

Download all attachments as: .zip

Change History (11)

#1 @nacin
12 years ago

This script isn't loaded on this page. Plugin?

@nacin
12 years ago

Attachment 27836.diff added

#2 @nacin
12 years ago

Milestone Awaiting Review → 3.9

Confirmed that

#3 @nacin
12 years ago

Confirmed that if the script is loaded on this page, it produces a type error that was not present in 3.8.

#4 @helgatheviking
12 years ago

Yes, my mistake, a plugin is loading the script. I don't think the plugin needs to be loading the media-editor script on this page, so I have reported the bug to the plugin developer.

Last edited 12 years ago by helgatheviking (previous) (diff)

#5 @nacin
12 years ago

Which plugin?

This ticket was mentioned in IRC in #wordpress-dev by nacin. View the logs.

12 years ago

#7 @helgatheviking
12 years ago

Projects by Woothemes

#8 @markjaquith
12 years ago

Owner set to markjaquith
Resolution → fixed
Status new → closed

In 28149:

Ensure that wp.media.view.settings.post is set before accessing the post's id

Avoids JS error in cases where media-editor is loaded individually
instead of with the whole wp_enqueue_media() process.

fixes #27836. props nacin

#9 @markjaquith
12 years ago

@helgatheviking — Thanks for catching this. Just in the nick of time for 3.9!

#10 @helgatheviking
12 years ago

Whoa! That seems like the fastest fix ever! Props.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#27836 closed defect (bug) (fixed)

Uncaught TypeError in media-editor.js on edit.php screen

Description

Attachments (1)

Change History (11)

#1 @nacin 12 years ago

@nacin 12 years ago

#2 @nacin 12 years ago

#3 @nacin 12 years ago

#4 @helgatheviking 12 years ago

#5 @nacin 12 years ago

This ticket was mentioned in IRC in #wordpress-dev by nacin. ​View the logs. 12 years ago

#7 @helgatheviking 12 years ago

#8 @markjaquith 12 years ago

#9 @markjaquith 12 years ago

#10 @helgatheviking 12 years ago