WordPress.org

Make WordPress Core

Ticket #25974: 25974.diff

File 25974.diff, 4.8 KB (added by kadamwhite, 7 years ago)

Patch to fix jshint errors in media-views.js

  • src/wp-includes/js/media-views.js

     
     1/* Token "_wpMediaViewsL10n" is used for localization */
     2/* global _wpMediaViewsL10n: false */
     3/* global confirm: false */
     4/* global getUserSetting: false */
     5/* global setUserSetting: false */
    16(function($){
    27        var media       = wp.media,
    38                Attachment  = media.model.Attachment,
    49                Attachments = media.model.Attachments,
    5                 Query       = media.model.Query,
    610                l10n;
    711
    812        // Link any localized strings.
     
    118122                },
    119123
    120124                trigger: function( event ) {
    121                         var base;
     125                        var base, args;
     126
    122127                        if ( ! this._mode )
    123128                                return;
    124129
    125                         var args = _.toArray( arguments );
     130                        args = _.toArray( arguments );
    126131                        base = this.id + ':' + event;
    127132
    128133                        // Trigger `region:action:mode` event.
     
    455460                },
    456461
    457462                defaultDisplaySettings: function( attachment ) {
    458                         settings = this._defaultDisplaySettings;
     463                        var settings = this._defaultDisplaySettings;
    459464                        if ( settings.canEmbed = this.canEmbed( attachment ) )
    460465                                settings.link = 'embed';
    461466                        return settings;
     
    495500
    496501                recordSelection: function() {
    497502                        var selection = this.get('selection'),
    498                                 manager = this.frame._selection,
    499                                 filtered;
     503                                manager = this.frame._selection;
    500504
    501505                        if ( ! this.get('syncSelection') || ! manager || ! selection )
    502506                                return;
     
    10591063
    10601064                        // Generate the tab states.
    10611065                        _.each( tabs, function( title, id ) {
    1062                                 var frame = this.state( 'iframe:' + id ).set( _.defaults({
     1066                                this.state( 'iframe:' + id ).set( _.defaults({
    10631067                                        tab:     id,
    10641068                                        src:     tabUrl + '&tab=' + id,
    10651069                                        title:   title,
     
    11231127
    11241128        // Map some of the modal's methods to the frame.
    11251129        _.each(['open','close','attach','detach','escape'], function( method ) {
    1126                 media.view.MediaFrame.prototype[ method ] = function( view ) {
     1130                media.view.MediaFrame.prototype[ method ] = function() {
    11271131                        if ( this.modal )
    11281132                                this.modal[ method ].apply( this.modal, arguments );
    11291133                        return this;
     
    11501154                },
    11511155
    11521156                createSelection: function() {
    1153                         var controller = this,
    1154                                 selection = this.options.selection;
     1157                        var selection = this.options.selection;
    11551158
    11561159                        if ( ! (selection instanceof media.model.Selection) ) {
    11571160                                this.options.selection = new media.model.Selection( selection, {
     
    19311934
    19321935                progress: function() {
    19331936                        var queue = this.queue,
    1934                                 $bar = this.$bar,
    1935                                 memo = 0;
     1937                                $bar = this.$bar;
    19361938
    19371939                        if ( ! $bar || ! queue.length )
    19381940                                return;
     
    21112113        // ----------------------------
    21122114        media.view.Toolbar.Select = media.view.Toolbar.extend({
    21132115                initialize: function() {
    2114                         var options = this.options,
    2115                                 controller = options.controller,
    2116                                 selection = controller.state().get('selection');
     2116                        var options = this.options;
    21172117
    21182118                        _.bindAll( this, 'clickSelect' );
    21192119
     
    26182618                                selection = this.options.selection,
    26192619                                model = this.model,
    26202620                                method = options && options.method,
    2621                                 single, between, models, singleIndex, modelIndex;
     2621                                single, models, singleIndex, modelIndex;
    26222622
    26232623                        if ( ! selection )
    26242624                                return;
     
    28992899
    29002900                        this._viewsByCid = {};
    29012901
    2902                         this.collection.on( 'add', function( attachment, attachments, options ) {
     2902                        this.collection.on( 'add', function( attachment ) {
    29032903                                this.views.add( this.createAttachmentView( attachment ), {
    29042904                                        at: this.collection.indexOf( attachment )
    29052905                                });
    29062906                        }, this );
    29072907
    2908                         this.collection.on( 'remove', function( attachment, attachments, options ) {
     2908                        this.collection.on( 'remove', function( attachment ) {
    29092909                                var view = this._viewsByCid[ attachment.cid ];
    29102910                                delete this._viewsByCid[ attachment.cid ];
    29112911
     
    30663066                        this.scroll();
    30673067                },
    30683068
    3069                 scroll: function( event ) {
     3069                scroll: function() {
    30703070                        // @todo: is this still necessary?
    30713071                        if ( ! this.$el.is(':visible') )
    30723072                                return;
     
    31493149                        this.filters = {};
    31503150                },
    31513151
    3152                 change: function( event ) {
     3152                change: function() {
    31533153                        var filter = this.filters[ this.el.value ];
    31543154
    31553155                        if ( filter )
     
    33963396
    33973397                createSingle: function() {
    33983398                        var sidebar = this.sidebar,
    3399                                 single = this.options.selection.single(),
    3400                                 views = {};
     3399                                single = this.options.selection.single();
    34013400
    34023401                        sidebar.set( 'details', new media.view.Attachment.Details({
    34033402                                controller: this.controller,
     
    36303629                                setUserSetting( userSetting, value );
    36313630                },
    36323631
    3633                 updateChanges: function( model, options ) {
     3632                updateChanges: function( model ) {
    36343633                        if ( model.hasChanged() )
    36353634                                _( model.changed ).chain().keys().each( this.update, this );
    36363635                }
     
    37563755                                this.model.destroy();
    37573756                },
    37583757
    3759                 editAttachment: function( event ) {
     3758                editAttachment: function() {
    37603759                        this.$el.addClass('needs-refresh');
    37613760                },
    37623761