[wp-trac] [WordPress Trac] #31684: media.view.Settings.Gallery.extend doesn't work anymore

WordPress Trac noreply at wordpress.org
Tue Mar 31 02:02:38 UTC 2015


#31684: media.view.Settings.Gallery.extend doesn't work anymore
--------------------------+-----------------------------
 Reporter:  bduclos       |       Owner:  wonderboymusic
     Type:  defect (bug)  |      Status:  assigned
 Priority:  high          |   Milestone:  4.2
Component:  Media         |     Version:  trunk
 Severity:  normal        |  Resolution:
 Keywords:  dev-feedback  |     Focuses:  javascript
--------------------------+-----------------------------

Comment (by wonderboymusic):

 In [changeset:"31935"]:
 {{{
 #!CommitTicketReference repository="" revision="31935"
 Let us pray to the gods of backwards compatibility:

 * The way that the JS modules for media are currently set up turns the
 existing global `wp.media` namespace into a read-only API, this is bad.
 * For the existing module implementation to work with plugins, those
 looking to override or extend a class would have to modify their own
 plugin to use `browserify` - we can't expect this to happen
 * Because the general way that plugins override media classes is via
 machete (resetting them to something else), we cannot use `require(
 'module' )` in the internal code for media modules

 We CAN continue to use `require( 'fun/js' )` in the manifests for media.

 Future code/projects should carefully consider what is made to be public
 API. In 3.5, EVERYTHING was made public, so everything shall remain
 public.

 See #31684, #28510.
 }}}

--
Ticket URL: <https://core.trac.wordpress.org/ticket/31684#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list