Comment to follow from Roan.
Version: unspecified
Severity: critical
Comment to follow from Roan.
Version: unspecified
Severity: critical
Well, not *completely*, just mostly :)
Go to https://www.mediawiki.org/w/index.php?title=VisualEditor:TestReferenceContents&veaction=edit and select reference [3] by clicking on it, then press Ctrl+C. You'll get a JS error.
There are several things wrong here:
In findEndOfNode() inside of ve.dm.Converter.prototype.getDomSubtreeFromData:
In ve.ce.Surface.prototype.onCopy:
In ve.dm.Document.prototype.cloneSliceFromRange:
I forgot to include how these things lead to the actual failure:
Based on these findings, my expectation of what will and won't work is:
Probably broken:
Probably works:
Change 108692 had a related patch set uploaded by Esanders:
Fix exception thrown by findEndOfNode
Change 108720 had a related patch set uploaded by Esanders:
Fix up result of selectNodes to remove empty items
Change 109481 had a related patch set uploaded by Esanders:
Fix balancing of data in cloneSliceFromRange
The following JS error appears if I copy a reference note with some text following it.
Uncaught Error: Unbalanced input passed to document
Checked the cases mentioned above.All of them are working now on Betalabs.No JS error is appearing now after copy -paste of reference.