[ 
https://issues.apache.org/jira/browse/WOOKIE-205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13015824#comment-13015824
 ] 

Paul Sharples commented on WOOKIE-205:
--------------------------------------

OK, the patch as it stands will fail on IE8.  All but the first widget fails 
with the error "Object doesnt support this method or property".
As in other places in wookie-wrapper we need to catch it when 
Object.defineProperty fails. (the IE9 fix). (see patch2.txt above). With that 
in place the test instance gets further and only fails on the following one in 
IE8...

http://dev.w3.org/2006/waf/widgets-api/test-suite/test-cases/ta-ah/ax/ax.wgt

This is because of a problem in that widget which contains syntax which will 
not work in <IE9 (they need to update the widget to feature detect the browser 
-  line 44) and replace it with.

// prior to IE9 have to use attachEvent instead of addeventlistener for IE
if (window.addEventListener){
  window.addEventListener("Storage", function(){ fail("Test Failed because 
Storage event was fired") }, true);
} 
else if (window.attachEvent){
  window.attachEvent("Storage", function(){ fail("Test Failed because Storage 
event was fired") });
}

(BTW: If I do that manually in wookie, IE8 stops complaining)


Two points

(1) in all browsers I seem to get the following error in the eclipse console 
when i load the generated html text...
 [java] 10:41:53,179  WARN Runtime:89 - Detected possible orphaned reference to 
deleted object "[1183, 1184, 1185, 1186]" in 
"org.apache.wookie.beans.jpa.impl.WidgetInstanceImpl.preferences<element:class 
org.apache.wookie.beans.jpa.impl.PreferenceImpl>" of instance "1212".

(2) There are more tests working in Firefox than in IE8. This almost certainly 
because the fix for IE8 in patch2.txt (line 48) does not enforce the read only 
mechanism.


> Ensure that all metadata attributes of widgets are read-only
> ------------------------------------------------------------
>
>                 Key: WOOKIE-205
>                 URL: https://issues.apache.org/jira/browse/WOOKIE-205
>             Project: Wookie
>          Issue Type: Sub-task
>          Components: Server
>            Reporter: Scott Wilson
>             Fix For: 0.9.2
>
>         Attachments: patch.txt, patch2.txt
>
>
> Metadata attributes (name, version etc) should be read-only according to the 
> specification.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to