Catrope has submitted this change and it was merged. Change subject: (bug 42219) Fix for document sync issue ......................................................................
(bug 42219) Fix for document sync issue When working with a document containing only a slug in an empty paragraph, tree synchronization would break because it was trying to rebuild a non-existent text node. This change makes the rebuild always occur on the outer range, rather than the inner range, which prevents absent text nodes from being asked to be rebuilt. Thank you to Roan for debugging this for like 20 min. Change-Id: I8c3dad921ace395f0694f77cec44305a680657fe --- M modules/ve/dm/ve.dm.TransactionProcessor.js 1 file changed, 4 insertions(+), 3 deletions(-) Approvals: Catrope: Verified; Looks good to me, approved -- To view, visit https://gerrit.wikimedia.org/r/34597 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8c3dad921ace395f0694f77cec44305a680657fe Gerrit-PatchSet: 2 Gerrit-Project: mediawiki/extensions/VisualEditor Gerrit-Branch: master Gerrit-Owner: Trevor Parscal <[email protected]> Gerrit-Reviewer: Catrope <[email protected]> Gerrit-Reviewer: Trevor Parscal <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
