Merge "Revert "DesktopArticleTarget: Don't abort in SET mode if veswitched is set""

This commit is contained in:
jenkins-bot 2015-12-14 20:53:28 +00:00 committed by Gerrit Code Review
commit 33cbf199f1
2 changed files with 3 additions and 3 deletions

View file

@ -75,7 +75,7 @@ class VisualEditorHooks {
$user->getOption( 'visualeditor-autodisable' ) || $user->getOption( 'visualeditor-autodisable' ) ||
$user->getOption( 'visualeditor-tabs' ) === 'prefer-wt' || $user->getOption( 'visualeditor-tabs' ) === 'prefer-wt' ||
( $veConfig->get( 'VisualEditorDisableForAnons' ) && $user->isAnon() ) || ( $veConfig->get( 'VisualEditorDisableForAnons' ) && $user->isAnon() ) ||
false // TODO: Detect incompatibility - T121298, P2373 false // TODO: Detect incompatibility - P2373
) { ) {
return true; return true;
} }
@ -102,13 +102,13 @@ class VisualEditorHooks {
$title->getContentModel() !== CONTENT_MODEL_WIKITEXT || $title->getContentModel() !== CONTENT_MODEL_WIKITEXT ||
// check for parameters that VE does not handle // check for parameters that VE does not handle
// TODO: other params too? See identical list in ve.init.mw.DesktopArticleTarget.init.js // TODO: other params too? See identical list in ve.init.mw.DesktopArticleTarget.init.js
in_array( 'veswitched', $params ) ||
in_array( 'undo', $params ) || in_array( 'undo', $params ) ||
in_array( 'undoafter', $params ) || in_array( 'undoafter', $params ) ||
in_array( 'editintro', $params ) || in_array( 'editintro', $params ) ||
in_array( 'preload', $params ) || in_array( 'preload', $params ) ||
in_array( 'preloadtitle', $params ) || in_array( 'preloadtitle', $params ) ||
in_array( 'preloadparams', $params ); in_array( 'preloadparams', $params );
// Known-good parameters: edit, veaction, section, vesection, veswitched
} }
/** /**

View file

@ -713,13 +713,13 @@
) )
) )
) && ) &&
uri.query.veswitched === undefined &&
uri.query.undo === undefined && uri.query.undo === undefined &&
uri.query.undoafter === undefined && uri.query.undoafter === undefined &&
uri.query.editintro === undefined && uri.query.editintro === undefined &&
uri.query.preload === undefined && uri.query.preload === undefined &&
uri.query.preloadtitle === undefined && uri.query.preloadtitle === undefined &&
uri.query.preloadparams === undefined uri.query.preloadparams === undefined
// Known-good parameters: edit, veaction, section, vesection, veswitched
// TODO: other params too? See identical list in VisualEditor.hooks.php // TODO: other params too? See identical list in VisualEditor.hooks.php
) { ) {
trackActivateStart( { trackActivateStart( {