mediawiki-extensions-Visual.../modules
Catrope 04516bb02e Whitespace preservation was broken after the first run
The first run of getDomFromData() would preserve whitespace just fine,
but it blanked out the .veInternal.whitespace[1] element in certain
cases, contaminating the linear model and making the whitespace data
inconsistent. Subsequent runs of getDomFromData() would then refuse to
serialize that whitespace because the information about it was
inconsistent.

In getDomFromData(), we sometimes unset .veInternal.whitespace[1] (i.e.
set it to undefined) to prevent double processing. Because we're
potentially going to modify .veInternal, don't assign it by reference,
but copy the object.

Added tests asserting that the linear model is unchanged after calling
getDomFromData(), because that function should never modify
linear model data. This test failed in 4 cases (all whitespace-related)
before I added the copyObject() call.

Bug: 43543
Change-Id: Ic4c93510518163894201a693ab50331413715967
2013-04-17 11:28:05 +00:00
..
jquery Removed code for and links to unused libraries 2013-01-10 13:25:33 -08:00
qunit Upstream: Update QUnit from v1.10.0 to v1.11.0 2013-02-19 19:50:02 +01:00
rangy
unicodejs Implement next/prevBreakOffset and word skipping 2013-04-09 21:55:57 +00:00
ve Whitespace preservation was broken after the first run 2013-04-17 11:28:05 +00:00