Re-applying transactions to internal nodes can throw exceptions
Open, LowPublic8 Story Points

Description

Create the following WT document:

<ref>'''B'''</ref>
<references/>

Run the following commands to delete the 'B' from the reference body:

var sm = ve.init.target.surface.model;
sm.change( ve.dm.Transaction.newFromRemoval( sm.documentModel, new ve.Range( 9, 10 ) ) );

Turn on 'pause on uncaught exceptions' then press undo, then redo. Observe that an exception is thrown because the ContentBranchNode has no root.

Esanders created this task.May 12 2016, 1:12 PM
Restricted Application added subscribers: Zppix, Aklapper. · View Herald TranscriptMay 12 2016, 1:12 PM
Jdforrester-WMF triaged this task as High priority.May 12 2016, 5:50 PM
Jdforrester-WMF set the point value for this task to 8.
Elitre added a subscriber: Elitre.May 12 2016, 5:52 PM

Change 289831 had a related patch set uploaded (by Esanders):
Workaround for T135127: Generate replace transactions in MWTransclusionModel

https://gerrit.wikimedia.org/r/289831

Change 289831 merged by jenkins-bot:
Workaround for T135127: Generate replace transactions in MWTransclusionModel

https://gerrit.wikimedia.org/r/289831

Deskana lowered the priority of this task from High to Low.Sep 20 2017, 12:29 PM
Deskana added a subscriber: Deskana.

Lowering in priority since there is a workaround.

Deskana moved this task from To Triage to Backlog on the VisualEditor board.Sep 20 2017, 12:40 PM