Changeset 22907

Timestamp:

11/28/2012 10:27:13 PM (12 years ago)

Author:

ryan

Message:

Media modal: Don't fire two AJAX calls on settings update.

Props koopersmith
fixes #22611

File:

• trunk/wp-includes/js/media-views.js (modified) (1 diff)

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

@@ -2575 +2575 @@
 
         updateSetting: function( event ) {
-            var $setting = $( event.target ).closest('[data-setting]');
+            var $setting = $( event.target ).closest('[data-setting]'),
+                setting, value;
 
             if ( ! $setting.length )
                 return;
 
-            this.model.save( $setting.data('setting'), event.target.value );
+            setting = $setting.data('setting');
+            value   = event.target.value;
+
+            if ( this.model.get( setting ) !== value )
+                this.model.save( setting, value );
         },