You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by jo...@apache.org on 2009/04/17 03:44:57 UTC

svn commit: r765827 - /incubator/shindig/trunk/features/src/main/javascript/features/core/prefs.js

Author: johnh
Date: Fri Apr 17 01:44:57 2009
New Revision: 765827

URL: http://svn.apache.org/viewvc?rev=765827&view=rev
Log:
No longer overwrite empty-string userprefs with default pref values.

Resolves SHINDIG-1016.


Modified:
    incubator/shindig/trunk/features/src/main/javascript/features/core/prefs.js

Modified: incubator/shindig/trunk/features/src/main/javascript/features/core/prefs.js
URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/features/src/main/javascript/features/core/prefs.js?rev=765827&r1=765826&r2=765827&view=diff
==============================================================================
--- incubator/shindig/trunk/features/src/main/javascript/features/core/prefs.js (original)
+++ incubator/shindig/trunk/features/src/main/javascript/features/core/prefs.js Fri Apr 17 01:44:57 2009
@@ -80,7 +80,7 @@
  */
 function mergeDefaults() {
   for (var name in defaultPrefs) {
-    if (!prefs[name]) {
+    if (typeof prefs[name] === 'undefined') {
       prefs[name] = defaultPrefs[name];
     }
   }