Changeset 42739

Timestamp:

02/25/2018 04:40:33 PM (8 years ago)

Author:

adamsilverstein

Message:

Media: grid view - correct placeholder positioning during uploads.

Preventing contextually created attachments from being added to the grid collection in Attachments.validator. Remove the previous filtering introduced in [41937] which caused the placement issue.

Props lancewillett, Junaidkbr, designsimply, joemcgill.
Fixes #42968.

Location:

trunk/src/wp-includes/js

Files:

• media-models.js (modified) (1 diff)
• media-views.js (modified) (2 diffs)
• media/controllers/library.js (modified) (2 diffs)
• media/models/attachments.js (modified) (1 diff)

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

@@ -643 +643 @@
          */
         validator: function( attachment ) {
+
+                // Filter out contextually created attachments (e.g. headers, logos, etc.).
+                if (
+                        ! _.isUndefined( attachment.attributes.context ) &&
+                        '' !== attachment.attributes.context
+                ) {
+                        return false;
+                }
+
                 if ( ! this.validateDestroyed && attachment.destroyed ) {
                         return false;

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

@@ -954 +954 @@
                 }
 
-                this._filterContext();
-                this.get('library').on( 'add', this._filterContext, this );
-
                 this.resetDisplays();
         },
@@ -1158 +1155 @@
         },
 
-        /**
-         * Filter out contextually created attachments (e.g. headers, logos, etc.)
-         *
-         * @since 4.9.0
-         */
-        _filterContext: function() {
-                var library = this.get('library');
-
-                library.set( library.filter( function( item ) {
-                        return item.get('context') === '';
-                } ) );
-        }
 });
 

trunk/src/wp-includes/js/media/controllers/library.js

@@ -86 +86 @@
                 }
 
-                this._filterContext();
-                this.get('library').on( 'add', this._filterContext, this );
-
                 this.resetDisplays();
         },
@@ -290 +287 @@
         },
 
-        /**
-         * Filter out contextually created attachments (e.g. headers, logos, etc.)
-         *
-         * @since 4.9.0
-         */
-        _filterContext: function() {
-                var library = this.get('library');
-
-                library.set( library.filter( function( item ) {
-                        return item.get('context') === '';
-                } ) );
-        }
 });
 

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

@@ -152 +152 @@
          */
         validator: function( attachment ) {
+
+                // Filter out contextually created attachments (e.g. headers, logos, etc.).
+                if (
+                        ! _.isUndefined( attachment.attributes.context ) &&
+                        '' !== attachment.attributes.context
+                ) {
+                        return false;
+                }
+
                 if ( ! this.validateDestroyed && attachment.destroyed ) {
                         return false;