WordPress.org
Get WordPress

Make WordPress Core

Changeset 29490


Ignore:
Timestamp:
08/14/2014 06:30:49 PM (10 years ago)
Author:
wonderboymusic
Message:

Media Grid, support MEDIA_TRASH:

  • Add a setting to _wpMediaViewsL10n.settings: mediaTrash
  • In the attachment edit modal, properly toggle between Trash/Untrash
  • In media.view.Attachment, add a method for untrashAttachment
  • When creating the grid toolbar, switch the setting order of subviews so that media.view.DeleteSelectedButton can listen to the instance of media.view.AttachmentFilters.All to update the text in its UI.
  • Add a new filter to media.view.AttachmentFilters.All, trash, when settings.mediaTrash is true
  • Allow the cached queries in Query.get() to be flushed when race conditions exist and collections need to be refreshed. This is currently only being used when MEDIA_TRASH is set, to refresh the filtered/mirrored collections related to all, trash, and any already queried filter.
  • Cleanup the bootstrapping of media.view.MediaFrame.Manage
  • Allow wp_ajax_query_attachments() to return items from the trash when MEDIA_TRASH is true
  • Allow wp_ajax_save_attachment() to set post_status when MEDIA_TRASH is true. It allows wp_delete_post() to be called, which will trash the attachment instead of deleting when the flag is set.

Props koop for the knowledge sharing and thought partnership.
See #29145.

Location:
trunk/src
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-admin/includes/ajax-actions.php

    r29454 r29490  
    21622162
    21632163    $query['post_type'] = 'attachment';
    2164     $query['post_status'] = 'inherit';
     2164    if ( MEDIA_TRASH
     2165        && ! empty( $_REQUEST['query']['post_status'] )
     2166        && 'trash' === $_REQUEST['query']['post_status'] ) {
     2167        $query['post_status'] = 'trash';
     2168    } else {
     2169        $query['post_status'] = 'inherit';
     2170    }
     2171
    21652172    if ( current_user_can( get_post_type_object( 'attachment' )->cap->read_private_posts ) )
    21662173        $query['post_status'] .= ',private';
    … …  
    22162223    if ( isset( $changes['description'] ) )
    22172224        $post['post_content'] = $changes['description'];
     2225
     2226    if ( MEDIA_TRASH && isset( $changes['status'] ) )
     2227        $post['post_status'] = $changes['status'];
    22182228
    22192229    if ( isset( $changes['alt'] ) ) {
    … …  
    22442254    }
    22452255
    2246     wp_update_post( $post );
     2256    if ( MEDIA_TRASH && isset( $changes['status'] ) && 'trash' === $changes['status'] ) {
     2257        wp_delete_post( $id );
     2258    } else {
     2259        wp_update_post( $post );
     2260    }
     2261
    22472262    wp_send_json_success();
    22482263}

  • trunk/src/wp-includes/css/media-views.css

    r29489 r29490  
    16041604.attachment-info .refresh-attachment,
    16051605.attachment-info .delete-attachment,
    1606 .attachment-info .trash-attachment {
     1606.attachment-info .trash-attachment,
     1607.attachment-info .untrash-attachment {
    16071608    display: block;
    16081609    text-decoration: none;
    … …  
    16211622
    16221623.media-modal .delete-attachment,
    1623 .media-modal .trash-attachment {
     1624.media-modal .trash-attachment,
     1625.media-modal .untrash-attachment {
    16241626    color: #bc0b0b;
    16251627}
    16261628
    16271629.media-modal .delete-attachment:hover,
    1628 .media-modal .trash-attachment:hover {
     1630.media-modal .trash-attachment:hover,
     1631.media-modal .untrash-attachment:hover {
    16291632    color: red;
    16301633}
    … …  
    27442747}
    27452748
    2746 .edit-attachment-frame .delete-attachment {
     2749.edit-attachment-frame .delete-attachment,
     2750.edit-attachment-frame .trash-attachment,
     2751.edit-attachment-frame .untrash-attachment {
    27472752    float: right;
    27482753    margin-top: 7px;

  • trunk/src/wp-includes/js/media-grid.js

    r29489 r29490  
    184184            wp.media( {
    185185                frame:       'edit-attachments',
    186                 gridRouter:  this.gridRouter,
     186                controller:  this,
    187187                library:     this.state().get('library'),
    188188                model:       model
    … …  
    231231
    232232        bindDeferred: function() {
     233            if ( ! this.browserView.dfd ) {
     234                return;
     235            }
    233236            this.browserView.dfd.done( _.bind( this.startHistory, this ) );
    234237        },
    … …  
    353356
    354357        events: {
    355             'click':                    'collapse',
    356             'click .delete-media-item': 'deleteMediaItem',
    357358            'click .left':              'previousMediaItem',
    358359            'click .right':             'nextMediaItem'
    … …  
    360361
    361362        initialize: function() {
    362             var self = this;
    363 
    364363            media.view.Frame.prototype.initialize.apply( this, arguments );
    365364
    … …  
    369368            });
    370369
    371             this.gridRouter = this.options.gridRouter;
    372 
     370            this.controller = this.options.controller;
     371            this.gridRouter = this.controller.gridRouter;
    373372            this.library = this.options.library;
    374373
    … …  
    376375                this.model = this.options.model;
    377376            } else {
     377                // this is a hack
    378378                this.model = this.library.at( 0 );
    379379            }
    380380
     381            this.bindHandlers();
     382            this.createStates();
     383            this.createModal();
     384
     385            this.title.mode( 'default' );
     386
     387            this.options.hasPrevious = this.hasPrevious();
     388            this.options.hasNext = this.hasNext();
     389        },
     390
     391        bindHandlers: function() {
     392            // Bind default title creation.
     393            this.on( 'title:create:default', this.createTitle, this );
     394
    381395            // Close the modal if the attachment is deleted.
    382             this.listenTo( this.model, 'destroy', this.close, this );
    383 
    384             this.createStates();
     396            this.listenTo( this.model, 'change:status destroy', this.close, this );
    385397
    386398            this.on( 'content:create:edit-metadata', this.editMetadataMode, this );
    … …  
    388400            this.on( 'content:render:edit-image', this.editImageModeRender, this );
    389401            this.on( 'close', this.detach );
    390 
    391             // Bind default title creation.
    392             this.on( 'title:create:default', this.createTitle, this );
    393             this.title.mode( 'default' );
    394 
    395             this.options.hasPrevious = this.hasPrevious();
    396             this.options.hasNext = this.hasNext();
     402        },
     403
     404        createModal: function() {
     405            var self = this;
    397406
    398407            // Initialize modal container view.
    … …  
    610619        initialize: function() {
    611620            media.view.Button.prototype.initialize.apply( this, arguments );
     621            if ( this.options.filters ) {
     622                this.listenTo( this.options.filters.model, 'change', this.filterChange );
     623            }
    612624            this.listenTo( this.controller, 'selection:toggle', this.toggleDisabled );
    613625        },
    614626
     627        filterChange: function( model ) {
     628            if ( 'trash' === model.get( 'status' ) ) {
     629                this.model.set( 'text', l10n.untrashSelected );
     630            } else if ( media.view.settings.mediaTrash ) {
     631                this.model.set( 'text', l10n.trashSelected );
     632            } else {
     633                this.model.set( 'text', l10n.deleteSelected );
     634            }
     635        },
     636
    615637        toggleDisabled: function() {
    616             this.$el.attr( 'disabled', ! this.controller.state().get( 'selection' ).length );
     638            this.model.set( 'disabled', ! this.controller.state().get( 'selection' ).length );
    617639        },
    618640
    619641        render: function() {
    620642            media.view.Button.prototype.render.apply( this, arguments );
    621             this.$el.addClass( 'delete-selected-button hidden' );
     643            if ( this.controller.isModeActive( 'select' ) ) {
     644                this.$el.addClass( 'delete-selected-button' );
     645            } else {
     646                this.$el.addClass( 'delete-selected-button hidden' );
     647            }
    622648            return this;
    623649        }

  • trunk/src/wp-includes/js/media-models.js

    r29076 r29490  
    825825         * @access private
    826826         */
    827         _requery: function() {
     827        _requery: function( cache ) {
     828            var props;
    828829            if ( this.props.get('query') ) {
    829                 this.mirror( Query.get( this.props.toJSON() ) );
     830                props = this.props.toJSON();
     831                props.cache = ( true !== cache );
     832                this.mirror( Query.get( props ) );
    830833            }
    831834        },
    … …  
    948951
    949952                return uploadedTo === attachment.get('uploadedTo');
     953            },
     954            /**
     955             * @static
     956             * @param {wp.media.model.Attachment} attachment
     957             *
     958             * @this wp.media.model.Attachments
     959             *
     960             * @returns {Boolean}
     961             */
     962            status: function( attachment ) {
     963                var status = this.props.get('status');
     964                if ( _.isUndefined( status ) ) {
     965                    return true;
     966                }
     967
     968                return status === attachment.get('status');
    950969            }
    951970        }
    … …  
    11451164            'perPage':   'posts_per_page',
    11461165            'menuOrder': 'menu_order',
    1147             'uploadedTo': 'post_parent'
     1166            'uploadedTo': 'post_parent',
     1167            'status':     'post_status'
    11481168        },
    11491169        /**
    … …  
    11701190                    orderby  = Query.orderby,
    11711191                    defaults = Query.defaultProps,
    1172                     query;
     1192                    query,
     1193                    cache    = !! props.cache;
    11731194
    11741195                // Remove the `query` property. This isn't linked to a query,
    11751196                // this *is* the query.
    11761197                delete props.query;
     1198                delete props.cache;
    11771199
    11781200                // Fill default args.
    … …  
    12081230
    12091231                // Search the query cache for matches.
    1210                 query = _.find( queries, function( query ) {
    1211                     return _.isEqual( query.args, args );
    1212                 });
     1232                if ( cache ) {
     1233                    query = _.find( queries, function( query ) {
     1234                        return _.isEqual( query.args, args );
     1235                    });
     1236                } else {
     1237                    queries = [];
     1238                }
    12131239
    12141240                // Otherwise, create a new query and add it to the cache.

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

    r29484 r29490  
    56725672                    text: text,
    56735673                    props: {
     5674                        status:  null,
    56745675                        type:    key,
    56755676                        uploadedTo: null,
    … …  
    56835684                text:  l10n.allMediaItems,
    56845685                props: {
     5686                    status:  null,
    56855687                    type:    null,
    56865688                    uploadedTo: null,
    … …  
    56955697                    text:  l10n.uploadedToThisPost,
    56965698                    props: {
     5699                        status:  null,
    56975700                        type:    null,
    56985701                        uploadedTo: media.view.settings.post.id,
    … …  
    57075710                text:  l10n.unattached,
    57085711                props: {
     5712                    status:     null,
    57095713                    uploadedTo: 0,
    57105714                    type:       null,
    … …  
    57145718                priority: 50
    57155719            };
     5720
     5721            if ( media.view.settings.mediaTrash ) {
     5722                filters.trash = {
     5723                    text:  l10n.trash,
     5724                    props: {
     5725                        uploadedTo: null,
     5726                        status:     'trash',
     5727                        type:       null,
     5728                        orderby:    'date',
     5729                        order:      'DESC'
     5730                    },
     5731                    priority: 50
     5732                };
     5733            }
    57165734
    57175735            this.filters = filters;
    … …  
    57665784
    57675785        createToolbar: function() {
    5768             var filters,
    5769                 LibraryViewSwitcher,
    5770                 FiltersConstructor;
     5786            var LibraryViewSwitcher, Filters;
    57715787
    57725788            /**
    … …  
    57785794
    57795795            this.views.add( this.toolbar );
     5796
     5797            this.toolbar.set( 'spinner', new media.view.Spinner({
     5798                priority: -60
     5799            }) );
     5800
     5801            if ( -1 !== $.inArray( this.options.filters, [ 'uploaded', 'all' ] ) ) {
     5802                // "Filters" will return a <select>, need to render
     5803                // screen reader text before
     5804                this.toolbar.set( 'filtersLabel', new media.view.Label({
     5805                    value: l10n.filterByType,
     5806                    attributes: {
     5807                        'for':  'media-attachment-filters'
     5808                    },
     5809                    priority:   -80
     5810                }).render() );
     5811
     5812                if ( 'uploaded' === this.options.filters ) {
     5813                    this.toolbar.set( 'filters', new media.view.AttachmentFilters.Uploaded({
     5814                        controller: this.controller,
     5815                        model:      this.collection.props,
     5816                        priority:   -80
     5817                    }).render() );
     5818                } else {
     5819                    Filters = new media.view.AttachmentFilters.All({
     5820                        controller: this.controller,
     5821                        model:      this.collection.props,
     5822                        priority:   -80
     5823                    });
     5824
     5825                    this.toolbar.set( 'filters', Filters.render() );
     5826                }
     5827            }
    57805828
    57815829            // Feels odd to bring the global media library switcher into the Attachment
    … …  
    58155863
    58165864                this.toolbar.set( 'deleteSelectedButton', new media.view.DeleteSelectedButton({
     5865                    filters: Filters,
    58175866                    style: 'primary',
    58185867                    disabled: true,
    5819                     text: l10n.deleteSelected,
     5868                    text: media.view.settings.mediaTrash ? l10n.trashSelected : l10n.deleteSelected,
    58205869                    controller: this.controller,
    58215870                    priority: -60,
    58225871                    click: function() {
    5823                         while ( this.controller.state().get( 'selection' ).length > 0 ) {
    5824                             this.controller.state().get( 'selection' ).at( 0 ).destroy();
     5872                        var model, changed = [],
     5873                            selection = this.controller.state().get( 'selection' ),
     5874                            library = this.controller.state().get( 'library' );
     5875
     5876                        while ( selection.length > 0 ) {
     5877                            model = selection.at( 0 );
     5878                            if ( media.view.settings.mediaTrash && 'trash' === model.get( 'status' ) ) {
     5879                                model.set( 'status', 'inherit' );
     5880                                changed.push( model.save() );
     5881                                selection.remove( model );
     5882                            } else if ( media.view.settings.mediaTrash ) {
     5883                                model.set( 'status', 'trash' );
     5884                                changed.push( model.save() );
     5885                                selection.remove( model );
     5886                            } else {
     5887                                model.destroy();
     5888                            }
     5889                        }
     5890
     5891                        if ( changed.length ) {
     5892                            $.when( changed ).then( function() {
     5893                                library._requery( true );
     5894                            } );
    58255895                        }
    58265896                    }
    5827                 }).render() );
    5828             }
    5829 
    5830             this.toolbar.set( 'spinner', new media.view.Spinner({
    5831                 priority: -60
    5832             }) );
    5833 
    5834             filters = this.options.filters;
    5835             if ( 'uploaded' === filters ) {
    5836                 FiltersConstructor = media.view.AttachmentFilters.Uploaded;
    5837             } else if ( 'all' === filters ) {
    5838                 FiltersConstructor = media.view.AttachmentFilters.All;
    5839             }
    5840 
    5841             if ( FiltersConstructor ) {
    5842                 // "FiltersConstructor" will return a <select>, need to render
    5843                 // screen reader text before
    5844                 this.toolbar.set( 'filtersLabel', new media.view.Label({
    5845                     value: l10n.filterByType,
    5846                     attributes: {
    5847                         'for':  'media-attachment-filters'
    5848                     },
    5849                     priority:   -80
    5850                 }).render() );
    5851                 this.toolbar.set( 'filters', new FiltersConstructor({
    5852                     controller: this.controller,
    5853                     model:      this.collection.props,
    5854                     priority:   -80
    58555897                }).render() );
    58565898            }
    … …  
    64216463            'click .delete-attachment':       'deleteAttachment',
    64226464            'click .trash-attachment':        'trashAttachment',
     6465            'click .untrash-attachment':      'untrashAttachment',
    64236466            'click .edit-attachment':         'editAttachment',
    64246467            'click .refresh-attachment':      'refreshAttachment',
    … …  
    64546497         */
    64556498        trashAttachment: function( event ) {
     6499            var library = this.controller.library;
    64566500            event.preventDefault();
    64576501
    6458             this.model.destroy();
     6502            if ( media.view.settings.mediaTrash ) {
     6503                this.model.set( 'status', 'trash' );
     6504                this.model.save().done( function() {
     6505                    library._requery( true );
     6506                } );
     6507            }  else {
     6508                this.model.destroy();
     6509            }
     6510        },
     6511        /**
     6512         * @param {Object} event
     6513         */
     6514        untrashAttachment: function( event ) {
     6515            var library = this.controller.library;
     6516            event.preventDefault();
     6517
     6518            this.model.set( 'status', 'inherit' );
     6519            this.model.save().done( function() {
     6520                library._requery( true );
     6521            } );
    64596522        },
    64606523        /**

  • trunk/src/wp-includes/media-template.php

    r29457 r29490  
    317317                    <# if ( ! data.uploading && data.can.remove ) { #>
    318318                        <?php if ( MEDIA_TRASH ): ?>
     319                        <# if ( 'trash' === data.status ) { #>
     320                            <a class="untrash-attachment" href="#"><?php _e( 'Untrash' ); ?></a>
     321                        <# } else { #>
    319322                            <a class="trash-attachment" href="#"><?php _e( 'Trash' ); ?></a>
     323                        <# } #>
    320324                        <?php else: ?>
    321325                            <a class="delete-attachment" href="#"><?php _e( 'Delete Permanently' ); ?></a>

  • trunk/src/wp-includes/media.php

    r29486 r29490  
    28702870        'contentWidth' => $content_width,
    28712871        'months'       => $months,
     2872        'mediaTrash'   => MEDIA_TRASH ? 1 : 0
    28722873    );
    28732874
    … …  
    29322933        'insertIntoPost'         => $hier ? __( 'Insert into page' ) : __( 'Insert into post' ),
    29332934        'unattached'             => __( 'Unattached' ),
     2935        'trash'                  => __( 'Trash' ),
    29342936        'uploadedToThisPost'     => $hier ? __( 'Uploaded to this page' ) : __( 'Uploaded to this post' ),
    29352937        'warnDelete'             => __( "You are about to permanently delete this item.\n  'Cancel' to stop, 'OK' to delete." ),
    … …  
    29372939        'bulkSelect'             => __( 'Bulk Select' ),
    29382940        'cancelSelection'        => __( 'Cancel Selection' ),
     2941        'trashSelected'          => __( 'Trash Selected' ),
     2942        'untrashSelected'        => __( 'Untrash Selected' ),
    29392943        'deleteSelected'         => __( 'Delete Selected' ),
    29402944        'deletePermanently'      => __( 'Delete Permanently' ),
Note: See TracChangeset for help on using the changeset viewer.