WordPress.org

Make WordPress Core

Ticket #39824: 39824.diff

File 39824.diff, 3.0 KB (added by adamsilverstein, 5 years ago)
  • src/wp-includes/js/media/views/frame/post.js

     
    570570                                        style:    'primary',
    571571                                        text:     editing ? l10n.updateGallery : l10n.insertGallery,
    572572                                        priority: 80,
    573                                         requires: { library: true },
     573                                        requires: { library: true, uploadingComplete: true },
    574574
    575575                                        /**
    576576                                         * @fires wp.media.controller.State#update
  • src/wp-includes/js/media/views/toolbar.js

     
    138138                        }
    139139
    140140                        var requires = button.options.requires,
    141                                 disabled = false;
     141                                disabled = false,
     142                                modelsUploading = ! _.isEmpty( library.findWhere(  { 'uploading': true } ) );
    142143
    143144                        // Prevent insertion of attachments if any of them are still uploading
    144145                        disabled = _.some( selection.models, function( attachment ) {
     
    145146                                return attachment.get('uploading') === true;
    146147                        });
    147148
     149                        if ( requires.uploadingComplete && modelsUploading ) {
     150                                disabled = true;
     151                        }
     152
    148153                        if ( requires.selection && selection && ! selection.length ) {
    149154                                disabled = true;
    150155                        } else if ( requires.library && library && ! library.length ) {
  • src/wp-includes/js/media-views.js

     
    56845684                                        style:    'primary',
    56855685                                        text:     editing ? l10n.updateGallery : l10n.insertGallery,
    56865686                                        priority: 80,
    5687                                         requires: { library: true },
     5687                                        requires: { library: true, uploadingComplete: true },
    56885688
    56895689                                        /**
    56905690                                         * @fires wp.media.controller.State#update
     
    56915691                                         */
    56925692                                        click: function() {
    56935693                                                var controller = this.controller,
    5694                                                         state = controller.state();
     5694                                                        state = controller.state(),
     5695                                                        library = state.get('library');
    56955696
    56965697                                                controller.close();
    5697                                                 state.trigger( 'update', state.get('library') );
     5698                                                state.trigger( 'update', library );
    56985699
    56995700                                                // Restore and reset the default state.
    57005701                                                controller.setState( controller.options.state );
     
    77217722                        }
    77227723
    77237724                        var requires = button.options.requires,
    7724                                 disabled = false;
     7725                                disabled = false,
     7726                                modelsUploading = ! _.isEmpty( library.findWhere(  { 'uploading': true } ) );
    77257727
    77267728                        // Prevent insertion of attachments if any of them are still uploading
    77277729                        disabled = _.some( selection.models, function( attachment ) {
     
    77287730                                return attachment.get('uploading') === true;
    77297731                        });
    77307732
     7733                        if ( requires.uploadingComplete && modelsUploading ) {
     7734                                disabled = true;
     7735                        }
     7736
    77317737                        if ( requires.selection && selection && ! selection.length ) {
    77327738                                disabled = true;
    77337739                        } else if ( requires.library && library && ! library.length ) {