Regression: Editor returns escaped HTML instead of text in IE8

[haysmark]

​http://archive.dojotoolkit.org/nightly/checkout/dojox/grid/tests/test_edit_dijit.html

In IE8, open an Editor cell and close it. The text is now surrounded by escaped paragraph tags where there were none before. Grid's Editor handler just passes through the value to and from Editor, so this is more likely a regression in Editor.

You see it first emerge going from 1.3 to 1.3.1:

​http://download.dojotoolkit.org/release-1.3.0/dojo-release-1.3.0/dojox/grid/tests/test_edit_dijit.html

​http://download.dojotoolkit.org/release-1.3.1/dojo-release-1.3.1/dojox/grid/tests/test_edit_dijit.html

[Jared Jurkiewicz]

This isn't editor. This is the escapeHTMLInData crap of grid.

[Jared Jurkiewicz]

Since grid-auto-escapes the HTML, this, of course, means Rich Text (HTML markup), isn't handled as HTML markup when viewed in grid by default.

The 'fix' for this is to disable the HTML escaping, of course. The grid folks may want to consider allowing HTML on a cell by cell basis, instead of a flat on or off for everything. Cell-by-cell makes sense esp when dealing with using Editor in the grid.

Attached is a patch that fixes the testcase by simply turning off HTML escaping. The grid folks may want to consider the alternate of allowing per-cell escaping as a better solution than this (which is why I didn't check it in).

[Jared Jurkiewicz]

Patch that turns off HTML escaping so that the output of RTE displays correctly.

[Nathan Toone]

Jared -

Feel free to check in this fix to the test case (I do not currently have my machine available, and cannot check in.) I will create a new bug with the enhancement request to support turning off html escaping per-cell.

Thanks.

[Adam Peller]

see #10086

[Nathan Toone]

(In [20557]) Fixes #10072 - prevent escaping of HTML