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

Merge "MWReferenceSearchWidget buildIndex optimisations"

Browse source
This commit is contained in:
jenkins-bot 2014-07-23 02:23:43 +00:00 committed by Gerrit Code Review
parent e9f78820c3 d4cd816b6f
commit 960922cdb0
2 changed files with 17 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
modules/ve-mw/dm/models/ve.dm.MWReferenceModel.js
View file

@ -23,6 +23,7 @@ ve.dm.MWReferenceModel = function VeDmMWReferenceModel() {
this.listIndex = null; this.listIndex = null;
this.group = ''; this.group = '';
this.doc = null; this.doc = null;
this.deferDoc = null;
}; };
/* Inheritance */ /* Inheritance */
@ -47,7 +48,10 @@ ve.dm.MWReferenceModel.static.newFromReferenceNode = function ( node ) {
ref.setListGroup( attr.listGroup ); ref.setListGroup( attr.listGroup );
ref.setListIndex( attr.listIndex ); ref.setListIndex( attr.listIndex );
ref.setGroup( attr.refGroup ); ref.setGroup( attr.refGroup );
ref.setDocument( doc.cloneFromRange( internalList.getItemNode( attr.listIndex ).getRange() ) ); ref.deferDoc = function () {
// cloneFromRange is very expensive, so lazy evaluate it
return doc.cloneFromRange( internalList.getItemNode( attr.listIndex ).getRange() );
};
return ref; return ref;
}; };
@ -149,7 +153,7 @@ ve.dm.MWReferenceModel.prototype.updateInternalItem = function ( surfaceModel )
itemNodeRange = internalList.getItemNode( this.listIndex ).getRange(); itemNodeRange = internalList.getItemNode( this.listIndex ).getRange();
surfaceModel.change( ve.dm.Transaction.newFromRemoval( doc, itemNodeRange, true ) ); surfaceModel.change( ve.dm.Transaction.newFromRemoval( doc, itemNodeRange, true ) );
surfaceModel.change( surfaceModel.change(
ve.dm.Transaction.newFromDocumentInsertion( doc, itemNodeRange.start, this.doc ) ve.dm.Transaction.newFromDocumentInsertion( doc, itemNodeRange.start, this.getDocument() )
); );
}; };
@ -219,12 +223,16 @@ ve.dm.MWReferenceModel.prototype.getGroup = function () {
*/ */
ve.dm.MWReferenceModel.prototype.getDocument = function () { ve.dm.MWReferenceModel.prototype.getDocument = function () {
if ( !this.doc ) { if ( !this.doc ) {
this.doc = new ve.dm.Document( [ if ( this.deferDoc ) {
{ 'type': 'paragraph', 'internal': { 'generated': 'wrapper' } }, this.doc = this.deferDoc();
{ 'type': '/paragraph' }, } else {
{ 'type': 'internalList' }, this.doc = new ve.dm.Document( [
{ 'type': '/internalList' } { 'type': 'paragraph', 'internal': { 'generated': 'wrapper' } },
] ); { 'type': '/paragraph' },
{ 'type': 'internalList' },
{ 'type': '/internalList' }
] );
}
} }
return this.doc; return this.doc;
}; };

2
modules/ve-mw/ui/widgets/ve.ui.MWReferenceSearchWidget.js
View file

@ -131,7 +131,7 @@ ve.ui.MWReferenceSearchWidget.prototype.buildIndex = function ( internalList ) {
view.$element.find( 'a[href]' ).each( extractAttrs ); view.$element.find( 'a[href]' ).each( extractAttrs );
this.index.push( { this.index.push( {
'$element': view.$element.clone().show(), '$element': view.$element.clone(),
'text': text, 'text': text,
'reference': ref, 'reference': ref,
'citation': citation, 'citation': citation,