Do not call saveSettings() on anon users

Also actually check wfReadOnly().

Change-Id: I83e78532c66039e9764e70246bd6597a2bf4e60f
This commit is contained in:
Aaron Schulz 2016-04-21 07:33:24 -07:00
parent ad436e29ff
commit 2c877414c3

View file

@ -132,9 +132,11 @@ class VisualEditorHooks {
if ( isset( $params['venoscript'] ) ) {
$req->response()->setCookie( 'VEE', 'wikitext', 0, [ 'prefix' => '' ] );
$user->setOption( 'visualeditor-editor', 'wikitext' );
DeferredUpdates::addCallableUpdate( function () use ( $user ) {
$user->saveSettings();
} );
if ( !wfReadOnly() && !$user->isAnon() ) {
DeferredUpdates::addCallableUpdate( function () use ( $user ) {
$user->saveSettings();
} );
}
return true;
}