wikimedia/mediawiki-extensions-VisualEditor
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor synced 2024-09-26 19:56:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "Fix adjustment of newRange in TransactionProcessor"

Browse source
This commit is contained in:
Trevor Parscal 2012-06-21 06:38:42 +00:00 committed by Gerrit Code Review
parent df4e5f0200 0b9934b2cc
commit cb3eba185d
1 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
modules/ve/dm/ve.dm.TransactionProcessor.js
View file

@ -243,7 +243,8 @@ ve.dm.TransactionProcessor.processors.replace = function( op ) {
minInsertLevel = 0,
coveringRange,
scopeStart,
scopeEnd;
scopeEnd,
opAdjustment = 0;
while ( true ) {
if ( operation.type == 'replace' ) {
var opRemove = this.reversed ? operation.insert : operation.remove,
@ -323,6 +324,7 @@ ve.dm.TransactionProcessor.processors.replace = function( op ) {
}
// Update adjustment
this.adjustment += opInsert.length - opRemove.length;
opAdjustment += opInsert.length - opRemove.length;
} else {
// We know that other operations won't cause adjustments, so we
// don't have to update adjustment
@ -341,8 +343,12 @@ ve.dm.TransactionProcessor.processors.replace = function( op ) {
// From all the affected ranges we have gathered, compute a range that covers all
// of them, and rebuild that
coveringRange = ve.Range.newCoveringRange( affectedRanges );
this.synchronizer.pushRebuild( coveringRange, new ve.Range( coveringRange.start,
coveringRange.end + this.adjustment )
this.synchronizer.pushRebuild(
coveringRange,
new ve.Range(
coveringRange.start + this.adjustment - opAdjustment,
coveringRange.end + this.adjustment
)
);
}
};