wikimedia/mediawiki-extensions-VisualEditor
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor synced 2024-11-15 02:23:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

Revert of all the modifications I made to es.TransactionProcessor - things were getting exponentially worse with each commit - I think I need some sleep

Browse source
This commit is contained in:
Trevor Parscal 2011-12-07 02:22:06 +00:00
parent 893d337c48
commit 0d78a63da0
1 changed files with 19 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

34
modules/es/es.TransactionProcessor.js
View file

@ -256,23 +256,20 @@ es.TransactionProcessor.prototype.retain = function( op ) {
};
es.TransactionProcessor.prototype.insert = function( op ) {
var node = this.model.getNodeFromOffset( this.cursor ),
var node,
index,
offset;
// Perform insert on linear data model
es.insertIntoArray( this.model.data, this.cursor, op.data );
this.applyAnnotations( this.cursor + op.data.length );
if (
es.DocumentModel.isStructuralOffset( this.model.data, this.cursor ) &&
( typeof op.data[0].type === 'string' && op.data[0].type.charAt( 0 ) !== '/' )
) {
offset = node === this.model ? 0 : this.model.getOffsetFromNode( node );
this.rebuildNodes(
this.model.data.slice( this.cursor, op.data.length ),
null,
node,
node.getIndexFromOffset( this.cursor - offset )
);
if ( es.DocumentModel.isStructuralOffset( this.model.data, this.cursor ) ) {
// FIXME: This fails when inserting something like </list><list> between 2 list items
// @see test #30 in es.TransactionProcessor.test.js
es.insertIntoArray( this.model.data, this.cursor, op.data );
this.applyAnnotations( this.cursor + op.data.length );
node = this.model.getNodeFromOffset( this.cursor );
offset = this.model.getOffsetFromNode( node );
index = node.getIndexFromOffset( this.cursor - offset );
this.rebuildNodes( op.data, null, node, index );
} else {
node = this.model.getNodeFromOffset( this.cursor );
if ( node.getParent() === this.model ) {
offset = this.model.getOffsetFromNode( node );
} else {
@ -280,6 +277,9 @@ es.TransactionProcessor.prototype.insert = function( op ) {
offset = this.model.getOffsetFromNode( node );
}
if ( es.DocumentModel.containsElementData( op.data ) ) {
// Perform insert on linear data model
es.insertIntoArray( this.model.data, this.cursor, op.data );
this.applyAnnotations( this.cursor + op.data.length );
// Synchronize model tree
if ( offset === -1 ) {
throw 'Invalid offset error. Node is not in model tree';
@ -289,6 +289,10 @@ es.TransactionProcessor.prototype.insert = function( op ) {
[node]
);
} else {
// Perform insert on linear data model
// TODO this is duplicated from above
es.insertIntoArray( this.model.data, this.cursor, op.data );
this.applyAnnotations( this.cursor + op.data.length );
// Update model tree
node.adjustContentLength( op.data.length, true );
node.emit( 'update', this.cursor - offset );