WordPress.org

Make WordPress Core

Ticket #24716: 24716.42.diff

File 24716.42.diff, 3.8 KB (added by ericlewis, 4 years ago)
  • src/wp-includes/js/media-grid.js

    diff --git a/src/wp-includes/js/media-grid.js b/src/wp-includes/js/media-grid.js
    index b0541a6..5798b6a 100644
    a b  
    1717         * @augments wp.media.controller.State
    1818         * @augments Backbone.Model
    1919         */
    20         media.controller.EditImageNoFrame = media.controller._State.extend({
     20        media.controller.EditImageNoFrame = media.controller.State.extend({
    2121                defaults: {
    2222                        id:      'edit-attachment',
    2323                        title:   l10n.editImage,
     
    2929                        url:     ''
    3030                },
    3131
    32                 initialize: function() {
    33                         media.controller._State.prototype.initialize.apply( this, arguments );
    34                 },
     32                _ready: function() {},
    3533
     34                /**
     35                 * Override media.controller.State._postActivate, since this state doesn't
     36                 * include the regions expected there.
     37                 */
    3638                _postActivate: function() {
    3739                        this._content();
    3840                        this._router();
    3941                },
    4042
    41                 deactivate: function() {
    42                         this.stopListening( this.frame );
    43                 },
    44 
    4543                /**
    4644                 * @access private
    4745                 */
  • src/wp-includes/js/media-views.js

    diff --git a/src/wp-includes/js/media-views.js b/src/wp-includes/js/media-views.js
    index 328f59a..b199c1b 100644
    a b  
    337337        });
    338338
    339339        /**
    340          * A more abstracted state, because media.controller.State expects
    341          * specific regions (menu, title, etc.) to exist on the frame, which do not
    342          * exist in media.view.Frame.EditAttachment.
     340         * wp.media.controller.State
     341         *
     342         * A state is a step in a workflow that when set will trigger the controllers
     343         * for the regions to be updated as specified in the frame. This is the base
     344         * class that the various states used in wp.media extend.
     345         *
     346         * @constructor
     347         * @augments Backbone.Model
    343348         */
    344         media.controller._State = Backbone.Model.extend({
     349        media.controller.State = Backbone.Model.extend({
    345350                constructor: function() {
    346351                        this.on( 'activate', this._preActivate, this );
    347352                        this.on( 'activate', this.activate, this );
     
    349354                        this.on( 'deactivate', this._deactivate, this );
    350355                        this.on( 'deactivate', this.deactivate, this );
    351356                        this.on( 'reset', this.reset, this );
     357                        this.on( 'ready', this._ready, this );
    352358                        this.on( 'ready', this.ready, this );
    353359                        /**
    354360                         * Call parent constructor with passed arguments
    355361                         */
    356362                        Backbone.Model.apply( this, arguments );
     363                        this.on( 'change:menu', this._updateMenu, this );
    357364                },
    358 
    359365                /**
    360366                 * @abstract
    361367                 */
     
    375381                /**
    376382                 * @access private
    377383                 */
    378                 _preActivate: function() {
    379                         this.active = true;
    380                 },
    381                 /**
    382                  * @access private
    383                  */
    384                 _postActivate: function() {},
    385                 /**
    386                  * @access private
    387                  */
    388                 _deactivate: function() {
    389                         this.active = false;
    390                 }
    391         });
    392 
    393         /**
    394          * wp.media.controller.State
    395          *
    396          * A state is a step in a workflow that when set will trigger the controllers
    397          * for the regions to be updated as specified in the frame. This is the base
    398          * class that the various states used in wp.media extend.
    399          *
    400          * @constructor
    401          * @augments Backbone.Model
    402          */
    403         media.controller.State = media.controller._State.extend({
    404                 constructor: function() {
    405                         this.on( 'activate', this._preActivate, this );
    406                         this.on( 'activate', this.activate, this );
    407                         this.on( 'activate', this._postActivate, this );
    408                         this.on( 'deactivate', this._deactivate, this );
    409                         this.on( 'deactivate', this.deactivate, this );
    410                         this.on( 'reset', this.reset, this );
    411                         this.on( 'ready', this._ready, this );
    412                         this.on( 'ready', this.ready, this );
    413                         /**
    414                          * Call parent constructor with passed arguments
    415                          */
    416                         Backbone.Model.apply( this, arguments );
    417                         this.on( 'change:menu', this._updateMenu, this );
     384                _ready: function() {
     385                        this._updateMenu();
    418386                },
    419 
    420387                /**
    421388                 * @access private
    422                  */
    423                 _ready: function() {
    424                         this._updateMenu();
     389                */
     390                _preActivate: function() {
     391                        this.active = true;
    425392                },
    426 
    427393                /**
    428394                 * @access private
    429395                 */