Page MenuHomePhabricator

VisualEditor: Transaction with replaceMetadata then replace at the same offset doesn't rollback cleanly
Open, LowPublic


This is kind of a ridiculous case, but it cost me an hour of debugging time, so documenting it here for posterity. It won't be a practical issue until we start composing transactions together.


var data =,

oldDoc = new ve.copyArray( data ) ),
newDoc = new ve.copyArray( data ) ),
tx = new;

tx.pushRetain( 11 );
tx.pushReplaceMetadata( [], [ { metadata element here } ] );
tx.pushReplace( [] , [ 'a', 'b', 'c' ] ); newDoc, tx ); newDoc, tx );

After running this, you'd expect oldDoc.metadata and newDoc.metadata to be equal, but they're not:





When the transaction is rolled back, the metadata element that was added was never removed. I think this is because the content insertion pushes the metadata back, and then when the rollback tries to remove the metadata again, it can't find it. When the metadata replacement and content replacement are swapped, it works fine, so that's my workaround for now.

Version: unspecified
Severity: enhancement



Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 1:18 AM
bzimport added a project: Technical-Debt.
bzimport set Reference to bz46138.
Catrope created this task.Mar 14 2013, 10:35 PM