Majr has uploaded a new change for review.
https://gerrit.wikimedia.org/r/247849
Change subject: Use the new 'difftotextpst' option of prop=revisions API in
live preview script
......................................................................
Use the new 'difftotextpst' option of prop=revisions API in live preview script
This should improve the diff performance on large pages by a fair amount,
especially on slow connections, as you now just upload the page content once
and then download the diff, rather than having to upload the page content,
download the PST'd page content, re-upload the PST'd page content, and then
finally download the diff.
If there is a summary, the parse request is still performed with just the
summary
text, but it is done in parallel to the diff request, and will likely finish
before the diff request anyway, so won't impact the diff performance.
A further improvement could be potentially made, by not sending a summary parse
request when the summary is plain text. This would be fairly easy to detect
(assuming extensions can't modify what is parsed in a summary), just
look for "[[" and "/*".
Bug: T116178
Change-Id: Ic81bad25ccc8b96c862feb31b7dedd80d1507eeb
---
M resources/src/mediawiki.action/mediawiki.action.edit.preview.js
1 file changed, 45 insertions(+), 44 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/49/247849/1
diff --git a/resources/src/mediawiki.action/mediawiki.action.edit.preview.js
b/resources/src/mediawiki.action/mediawiki.action.edit.preview.js
index 6d0175e..8d148f0 100644
--- a/resources/src/mediawiki.action/mediawiki.action.edit.preview.js
+++ b/resources/src/mediawiki.action/mediawiki.action.edit.preview.js
@@ -8,7 +8,7 @@
* @param {jQuery.Event} e
*/
function doLivePreview( e ) {
- var isDiff, api, request, postData, copySelectors, section,
+ var isDiff, api, parseRequest, diffRequest, postData,
copySelectors, section,
$wikiPreview, $wikiDiff, $editform, $textbox, $summary,
$copyElements, $spinner, $errorBox;
isDiff = ( e.target.name === 'wpDiff' );
@@ -75,55 +75,57 @@
api = new mw.Api();
postData = {
action: 'parse',
- uselang: mw.config.get( 'wgUserLanguage' ),
title: mw.config.get( 'wgPageName' ),
- text: $textbox.textSelection( 'getContents' ),
summary: $summary.textSelection( 'getContents' ),
- sectionpreview: section !== ''
+ prop: ''
};
-
- if ( section === 'new' ) {
- postData.section = 'new';
- postData.sectiontitle = postData.summary;
- }
if ( isDiff ) {
$wikiPreview.hide();
- // First PST the input, then diff it
- postData.onlypst = true;
- request = api.post( postData );
- request.done( function ( response ) {
- api.post( {
- action: 'query',
- indexpageids: true,
- prop: 'revisions',
- titles: mw.config.get( 'wgPageName' ),
- rvdifftotext: response.parse.text[ '*'
],
- rvprop: [],
- rvsection: section === '' ? undefined :
section
- } ).done( function ( response ) {
- try {
- var diffHtml =
response.query.pages[ response.query.pageids[ 0 ] ]
- .revisions[ 0 ].diff[
'*' ];
- $wikiDiff.find( 'table.diff
tbody' ).html( diffHtml );
- } catch ( e ) {
- // "result.blah is undefined"
error, ignore
- mw.log.warn( e );
- }
- $wikiDiff.show();
- } );
+ if ( postData.summary ) {
+ parseRequest = api.post( postData );
+ }
+
+ diffRequest = api.post( {
+ action: 'query',
+ indexpageids: true,
+ prop: 'revisions',
+ titles: mw.config.get( 'wgPageName' ),
+ rvdifftotext: $textbox.textSelection(
'getContents' ),
+ rvdifftotextpst: true,
+ rvprop: '',
+ rvsection: section === '' ? undefined : section
+ } ).done( function ( response ) {
+ try {
+ var diffHtml = response.query.pages[
response.query.pageids[ 0 ] ]
+ .revisions[ 0 ].diff[ '*' ];
+ $wikiDiff.find( 'table.diff tbody'
).html( diffHtml );
+ } catch ( e ) {
+ // "result.blah is undefined" error,
ignore
+ mw.log.warn( e );
+ }
+ $wikiDiff.show();
} );
} else {
$wikiDiff.hide();
+
$.extend( postData, {
+ prop:
'text|displaytitle|modules|jsconfigvars|categorieshtml|templates|langlinks|limitreporthtml',
+ text: $textbox.textSelection( 'getContents' ),
pst: true,
preview: true,
- prop:
'text|displaytitle|modules|jsconfigvars|categorieshtml|templates|langlinks|limitreporthtml',
- disableeditsection: true
+ sectionpreview: section !== '',
+ disableeditsection: true,
+ uselang: mw.config.get( 'wgUserLanguage' )
} );
- request = api.post( postData );
- request.done( function ( response ) {
+ if ( section === 'new' ) {
+ postData.section = 'new';
+ postData.sectiontitle = postData.summary;
+ }
+
+ parseRequest = api.post( postData );
+ parseRequest.done( function ( response ) {
var li, newList, $displaytitle, $content,
$parent, $list;
if ( response.parse.jsconfigvars ) {
mw.config.set(
response.parse.jsconfigvars );
@@ -203,34 +205,33 @@
$wikiPreview.append( $content );
$wikiPreview.show();
-
}
} );
}
- request.done( function ( response ) {
- var isSubject = ( section === 'new' ),
+ $.when( parseRequest, diffRequest ).done( function ( parseResp
) {
+ var parse = parseResp && parseResp[ 0 ].parse,
+ isSubject = ( section === 'new' ),
summaryMsg = isSubject ? 'subject-preview' :
'summary-preview',
$summaryPreview = $editform.find(
'.mw-summary-preview' ).empty();
- if ( response.parse.parsedsummary &&
response.parse.parsedsummary[ '*' ] !== '' ) {
+ if ( parse && parse.parsedsummary &&
parse.parsedsummary[ '*' ] !== '' ) {
$summaryPreview.append(
mw.message( summaryMsg ).parse(),
' ',
$( '<span>' ).addClass( 'comment'
).html(
// There is no equivalent to
rawParams
mw.message( 'parentheses'
).escaped()
- .replace( '$1',
response.parse.parsedsummary[ '*' ] )
+ .replace( '$1',
parse.parsedsummary[ '*' ] )
)
);
}
mw.hook( 'wikipage.editform' ).fire( $editform );
- } );
- request.always( function () {
+ } ).always( function () {
$spinner.hide();
$copyElements.animate( {
opacity: 1
}, 'fast' );
} );
- request.fail( function ( code, result ) {
+ $.when( parseRequest ).fail( function ( code, result ) {
var errorMsg = 'API error: ' + code;
if ( code === 'http' ) {
errorMsg = 'HTTP error: ';
--
To view, visit https://gerrit.wikimedia.org/r/247849
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic81bad25ccc8b96c862feb31b7dedd80d1507eeb
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Majr <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
