You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@wookie.apache.org by ps...@apache.org on 2010/04/09 17:27:38 UTC

svn commit: r932463 - /incubator/wookie/trunk/WebContent/shared/js/wookie-wrapper.js

Author: psharples
Date: Fri Apr  9 15:27:37 2010
New Revision: 932463

URL: http://svn.apache.org/viewvc?rev=932463&view=rev
Log:
See WOOKIE-44. Refactored to use widget.preferences.setItem() in the case of IE 8.

Modified:
    incubator/wookie/trunk/WebContent/shared/js/wookie-wrapper.js

Modified: incubator/wookie/trunk/WebContent/shared/js/wookie-wrapper.js
URL: http://svn.apache.org/viewvc/incubator/wookie/trunk/WebContent/shared/js/wookie-wrapper.js?rev=932463&r1=932462&r2=932463&view=diff
==============================================================================
--- incubator/wookie/trunk/WebContent/shared/js/wookie-wrapper.js (original)
+++ incubator/wookie/trunk/WebContent/shared/js/wookie-wrapper.js Fri Apr  9 15:27:37 2010
@@ -169,8 +169,8 @@ var Widget = {	
             }
             // Catch IE 8 error. See WOOKIE-44
             catch(err){
-            	eval("Widget.preferences." + key + "='" + obj["dvalue"] + "'");
-            	eval("this.preferences.prefs."+key+"=obj");
+            	eval("Widget.preferences.setItem('" + key + "','" + obj["dvalue"] + "')");
+            	eval("Widget.preferences.getItem('" + key + "') == '" + obj["dvalue"] + "'");
             }
 		}
 		this.preferences.calcLength();