Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#18001 closed defect (bug) (wontfix)

tiny_mce_preload_dialogs hook just disappeared

Reported by: dvankooten's profile dvankooten Owned by: azaozz's profile azaozz
Milestone: Priority: normal
Severity: normal Version: 3.2
Component: Editor Keywords: 2nd-opinion dev-feedback
Focuses: Cc:

Description

Apparently, wp_tiny_mce_preload_dialogs() got removed and has changed into wp_preload_dialogs(). The tiny_mce_preload_dialogs hook just disappeared with this, breaking a lot of plugins.

wp-admin/includes/post.php differences (3.1.4 > 3.2)

Change History (6)

#1 @kawauso
13 years ago

Looks like it was removed in [17785], part of Distraction Free Writing.

https://core.trac.wordpress.org/changeset/17785/trunk/wp-admin/includes/post.php

See also: #17136

Last edited 13 years ago by kawauso (previous) (diff)

#2 @nacin
13 years ago

  • Milestone changed from Awaiting Review to 3.2.1

#3 @nacin
13 years ago

  • Owner set to azaozz
  • Status changed from new to reviewing

#4 @azaozz
13 years ago

This was an internal hook for the wp-links dialog and was replaced by two new hooks when DFW was implemented. At the time no plugin was using it, so didn't see the need to keep it.

There are some planned enhancements to the_editor() and wp_tiny_mce() for 3.3 that will change how plugins use both editors anyways. Also the new hooks, before_wp_tiny_mce and after_wp_tiny_mce will stay for sure :)

#5 @azaozz
13 years ago

  • Component changed from TinyMCE to Editor
  • Milestone 3.2.1 deleted
  • Resolution set to wontfix
  • Status changed from reviewing to closed

Closing as wontfix, if there are still plugins that try to use the old hook, it would be best to update them. An update for them would probably be needed for 3.3 in any case.

#6 @downloadbook
12 years ago

Nothing.

Last edited 12 years ago by SergeyBiryukov (previous) (diff)
Note: See TracTickets for help on using tickets.