dojox.mvc.Repeat problem on IE with the dojox.mobile.parser

[Ed Chatelain]

There is a dojox.mvc.Repeat problem on IE with the dojox.mobile.parser,

This test is failing on IE, dojox/mvc/tests/doh_mvc_mobile-demo.html It is failing on the Repeat Data Binding Example page because the textboxes are not being set correctly from the model.

This test works with the dojo/parser but not with the dojox/mobile/parser.

[Ed Chatelain]

Akira Sudoh reported the following "I found that there is a case where the check in dojox.mobile.parser to skip DOM nodes for stopParser (if(n._skip) fails, though I reproduce that in one web server and don’t in another web server, using the exact same code and exact same browser, which is very weird. When such symptom happens, my IE seems to treat property assignment to DOM node as assigning a value to DOM attribute. It’ll cause dojox.mvc.Repeat.postscript() eventually pick up skip=“undefined” attribute as the template HTML, and eventually set the template HTML back to DOM, and when the parser runs against template HTML, n._skip there becomes string “undefined”, which is not expected by the if block in question.

[Akira Sudoh]

Patch to dojox/mobile/parser.js to work around a problem with to-string-conversion of DOMNode._skip

[Ed Chatelain]

A simplier testcase to see the problem, place this in dojox/mvc/tests I modified this test to use the dojox.mobile.parser.

[ykami]

The patch looks ok. I'll apply it to dojox.mobile.parser.

[ykami]

In [28208]:

Fixes #15028 !strict. Avoid error on IE by changing the value type for node._skip to string. Thanks asudoh for the patch.