From fc4ca9f823eeaca5211a3bc2468f315c538dc9dd Mon Sep 17 00:00:00 2001
From: Santhosh Thottingal <santhosh.thottingal@gmail.com>
Date: Mon, 29 Oct 2012 10:34:43 +0530
Subject: [PATCH] Callback for preference save action is optional

$.noop() was wrong. Changed it to $.noop as callback

Change-Id: Iff935e1a9c1b364efd947ce1bff785dacb81570e
---
 lib/jquery.ime/jquery.ime.js        | 6 ++----
 resources/js/ext.uls.preferences.js | 2 +-
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/lib/jquery.ime/jquery.ime.js b/lib/jquery.ime/jquery.ime.js
index ab08422d..60e14367 100644
--- a/lib/jquery.ime/jquery.ime.js
+++ b/lib/jquery.ime/jquery.ime.js
@@ -975,8 +975,7 @@
 				imeselector.position();
 
 				// save this preference
-				$.ime.preferences.save( function () {
-				} );
+				$.ime.preferences.save();
 			} );
 
 		},
@@ -995,8 +994,7 @@
 			this.$menu.removeClass( 'open' );
 			imeselector.position();
 			// save this preference
-			$.ime.preferences.save( function () {
-			} );
+			$.ime.preferences.save();
 		},
 
 		/**
diff --git a/resources/js/ext.uls.preferences.js b/resources/js/ext.uls.preferences.js
index e8fe88c1..36b59abd 100644
--- a/resources/js/ext.uls.preferences.js
+++ b/resources/js/ext.uls.preferences.js
@@ -70,7 +70,7 @@
 		 */
 		save: function ( callback ) {
 			var that = this, api;
-			callback = callback || $.noop();
+			callback = callback || $.noop;
 			if ( this.isAnon ) {
 				// Anonymous user- Save preferences in local storage
 				$.jStorage.set( this.preferenceName, this.preferences );