WordPress.org

Make WordPress Core

Ticket #24716: 24716.42.diff

File 24716.42.diff, 3.8 KB (added by ericlewis, 3 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                 */