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

ryanboren · ryanboren · commit 4e51b43ad320 · 2012-11-28T22:27:13.000Z
Props koopersmith fixes #22611 git-svn-id: https://develop.svn.wordpress.org/trunk@22907 602fd350-edb4-49c9-b593-d223f7449a82
diff --git a/wp-includes/js/media-views.js b/wp-includes/js/media-views.js
@@ -2574,12 +2574,17 @@
 		},
 
 		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 );
 		},
 
 		updateAll: function() {
