Ticket #22637: 22637.diff

wp-includes/css/media-views.css

@@ -267 +267 @@
         *max-width: 55%; /* IE7 */
 }
 
+.media-sidebar .setting input[type="checkbox"] {
+        margin-top: 10px;
+}
+
 .media-sidebar .setting span,
 .compat-item label span {
         float: left;

wp-includes/js/media-editor.js

@@ -189 +189 @@
                                 args.type    = 'image';
                                 args.perPage = -1;
 
+                                // Mark the `orderby` override attribute.
+                                if ( 'rand' === attrs.orderby )
+                                        attrs._orderbyRandom = true;
+
                                 // Map the `orderby` attribute to the corresponding model property.
                                 if ( ! attrs.orderby || /^menu_order(?: ID)?$/i.test( attrs.orderby ) )
                                         args.orderby = 'menuOrder';
@@ -232 +236 @@
                                 if ( props.uploadedTo )
                                         attrs.id = props.uploadedTo;
 
+                                // Check if the gallery is randomly ordered.
+                                if ( attrs._orderbyRandom )
+                                        attrs.orderby = 'rand';
+                                delete attrs._orderbyRandom;
+
                                 // If the `ids` attribute is set and `orderby` attribute
                                 // is the default value, clear it for cleaner output.
                                 if ( attrs.ids && 'post__in' === attrs.orderby )

wp-includes/js/media-views.js

@@ -530 +530 @@
                 },
 
                 gallerySettings: function() {
-                        var library = this.get('library');
+                        var library = this.get('library'),
+                                browser;
 
                         if ( ! library )
                                 return;
 
                         library.gallery = library.gallery || new Backbone.Model();
 
-                        this.frame.content.view().sidebar.set({
+                        browser = this.frame.content.view();
+
+                        browser.sidebar.set({
                                 gallery: new media.view.Settings.Gallery({
                                         controller: this,
                                         model:      library.gallery,
                                         priority:   40
                                 })
                         });
+
+                        browser.toolbar.set( 'reverse', {
+                                text:     l10n.reverseOrder,
+                                priority: 80,
+
+                                click: function() {
+                                        library.reset( library.toArray().reverse() );
+                                }
+                        });
                 }
         });
 
@@ -3369 +3381 @@
                         } else if ( $setting.is('input[type="text"], textarea') ) {
                                 if ( ! $setting.is(':focus') )
                                         $setting.val( value );
+
+                        // Handle checkboxes.
+                        } else if ( $setting.is('input[type="checkbox"]') ) {
+                                $setting.attr( 'checked', !! value );
                         }
                 },
 
@@ -3382 +3398 @@
                         if ( ! $setting.length )
                                 return;
 
+                        // Use the correct value for checkboxes.
+                        if ( $setting.is('input[type="checkbox"]') )
+                                value = $setting[0].checked;
+
+                        // Update the corresponding setting.
                         this.model.set( $setting.data('setting'), value );
 
                         // If the setting has a corresponding user setting,
@@ -3451 +3472 @@
 
                         $input.show();
 
-                        if ( 'post' == linkTo ) {
+                        if ( 'post' === linkTo ) {
                                 $input.val( attachment.get('link') );
-                        } else if ( 'file' == linkTo ) {
+                        } else if ( 'file' === linkTo ) {
                                 $input.val( attachment.get('url') );
                         } else if ( ! this.model.get('linkUrl') ) {
                                 $input.val('http://');
                         }
 
-                        $input.prop('readonly', 'custom' !== linkTo);
+                        $input.prop( 'readonly', 'custom' !== linkTo );
 
                         // If the input is visible, focus and select its contents.
                         if ( $input.is(':visible') )

wp-includes/media.php

@@ -670 +670 @@
         $instance++;
 
         if ( ! empty( $attr['ids'] ) ) {
-                // 'ids' is explicitly ordered
-                $attr['orderby'] = 'post__in';
+                // 'ids' is explicitly ordered, unless you specify otherwise.
+                if ( empty( $attr['orderby'] ) )
+                        $attr['orderby'] = 'post__in';
                 $attr['include'] = $attr['ids'];
         }
 
@@ -1474 +1475 @@
                 'updateGallery'      => __( 'Update gallery' ),
                 'continueEditing'    => __( 'Continue editing' ),
                 'addToGallery'       => __( 'Add to gallery' ),
+                'reverseOrder'       => __( 'Reverse order' ),
         );
 
         $settings = apply_filters( 'media_view_settings', $settings, $post );
@@ -1847 +1849 @@
                                 <?php endfor; ?>
                         </select>
                 </label>
+
+                <label class="setting">
+                        <span><?php _e('Random'); ?></span>
+                        <input type="checkbox" data-setting="_orderbyRandom" />
+                </label>
         </script>
 
         <script type="text/html" id="tmpl-embed-link-settings">