Ticket #53171: 53171.diff

src/js/media/models/attachments.js

@@ -212 +212 @@
                 this.observers.push( attachments );
 
                 attachments.on( 'add change remove', this._validateHandler, this );
+                attachments.on( 'add', this._addToTotalAttachments, this );
+                attachments.on( 'remove', this._removeFromTotalAttachments, this );
                 attachments.on( 'reset', this._validateAllHandler, this );
                 this.validateAll( attachments );
                 return this;
@@ -237 +239 @@
                 return this;
         },
         /**
+         * Update total attachment count when items are added to a collection.
+         *
          * @access private
          *
+         * @since 5.8.0
+         *
          * @param {wp.media.model.Attachments} attachment
          * @param {wp.media.model.Attachments} attachments
          * @param {Object} options
@@ -245 +251 @@
          *
          * @return {wp.media.model.Attachments} Returns itself to allow chaining.
          */
+        _removeFromTotalAttachments: function( attachment, attachments, options ) {
+                this.mirroring.totalAttachments = this.mirroring.totalAttachments - 1;
+        },
+        /**
+         * Update total attachment count when items are added to a collection.
+         *
+         * @access private
+         *
+         * @since 5.8.0
+         *
+         * @param {wp.media.model.Attachments} attachment
+         * @param {wp.media.model.Attachments} attachments
+         * @param {Object} options
+         *
+         * @return {wp.media.model.Attachments} Returns itself to allow chaining.
+         */
+        _addToTotalAttachments: function( attachment, attachments, options ) {
+                if ( this.mirroring ) {
+                        this.mirroring.totalAttachments = this.mirroring.totalAttachments + 1;
+                }
+        },
+        /**
+         * @access private
+         *
+         * @param {wp.media.model.Attachments} attachment
+         * @param {wp.media.model.Attachments} attachments
+         * @param {Object} options
+         *
+         * @return {wp.media.model.Attachments} Returns itself to allow chaining.
+         */
         _validateHandler: function( attachment, attachments, options ) {
                 // If we're not mirroring this `attachments` collection,
                 // only retain the `silent` option.