GWicke has uploaded a new change for review. Change subject: Make sure the serializer initialState remains unchanged ......................................................................
Make sure the serializer initialState remains unchanged extendProps is just a shallow assignment of passed-in values, so for anything that is not assigned by value (objects for example) it would just assign a reference. These objects are later modified and will thus modify the initial serializer state for later serializer runs. This patch clones both options and initialState to avoid this issue. My hope is that this will fix the random list issue we see in production. Change-Id: Ic71e384b89241cf96bbff2186d2ea19945ed9c12 --- M js/lib/mediawiki.WikitextSerializer.js 1 file changed, 6 insertions(+), 1 deletion(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Parsoid refs/changes/66/34666/1 -- To view, visit https://gerrit.wikimedia.org/r/34666 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic71e384b89241cf96bbff2186d2ea19945ed9c12 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Parsoid Gerrit-Branch: master Gerrit-Owner: GWicke <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
