ULS Frontend: Save preferences only for named users

Add a new method isNamed in ext.uls.common to use the isNamed
method if available else use the isAnon method as usual.

mw.user.isNamed was added in MW 1.40

Bug: T337780
Change-Id: I747c042a95e8edc793a2265a15ed6ba8ae9f1997

resources/js/ext.uls.common.js

@@ -63,7 +63,7 @@
 		// Track if event logging is enabled
 		mw.hook( 'mw.uls.interface.language.change' ).fire( language );
 
-		if ( mw.user.isAnon() ) {
+		if ( !mw.uls.isNamed() ) {
 			return changeLanguageAnon();
 		}
 
@@ -239,4 +239,13 @@
 		return ret;
 	};
 
+	/**
+	 * Determine if a user is named. Wrapper method is needed since mw.user.isNamed() was added in MW 1.40
+	 * For MW < 1.40
+	 * @returns {boolean}
+	 */
+	mw.uls.isNamed = function () {
+		return typeof mw.user.isNamed === 'function' ? mw.user.isNamed() : !mw.user.isAnon();
+	};
+
 }() );

resources/js/ext.uls.displaysettings.js

@@ -188,7 +188,7 @@
 				lang, i, language, $button, autonym;
 
 			// Don't let anonymous users change interface language
-			if ( !anonsAllowed && mw.user.isAnon() ) {
+			if ( !anonsAllowed && !mw.uls.isNamed() ) {
 				$loginCta = $( '<p>' )
 					.attr( 'id', 'uls-display-settings-anon-log-in-cta' );
 				autonym = $.uls.data.getAutonym( this.contentLanguage );

resources/js/ext.uls.interface.js

@@ -194,7 +194,7 @@
 	}
 
 	function userCanChangeLanguage() {
-		return mw.config.get( 'wgULSAnonCanChangeLanguage' ) || !mw.user.isAnon();
+		return mw.config.get( 'wgULSAnonCanChangeLanguage' ) || mw.uls.isNamed();
 	}
 
 	/**

resources/js/ext.uls.preferences.js

@@ -28,20 +28,22 @@
 		// https://www.mediawiki.org/wiki/Manual:Coding_conventions/JavaScript#Keys
 		this.preferenceName = 'uls-preferences';
 		this.username = mw.user.getName();
-		this.isAnon = mw.user.isAnon();
+		// For MW < 1.40. ext.uls.isNamed is inlined here to avoid dependency on ext.uls.common.
+		this.isNamed = typeof mw.user.isNamed === 'function' ? mw.user.isNamed() : !mw.user.isAnon();
 		this.preferences = null;
 		this.init();
 	};
 
 	ULSPreferences.prototype = {
 		init: function () {
-			if ( this.isAnon ) {
+			if ( this.isNamed ) {
-				this.preferences = mw.storage.getObject( this.preferenceName );
-			} else {
 				try {
 					this.preferences = JSON.parse( mw.user.options.get( this.preferenceName ) );
 				} catch ( e ) {
 				}
+
+			} else {
+				this.preferences = mw.storage.getObject( this.preferenceName );
 			}
 
 			if ( !$.isPlainObject( this.preferences ) ) {
@@ -78,12 +80,8 @@
 			var self = this;
 
 			callback = callback || function () {};
-			if ( this.isAnon ) {
+			if ( this.isNamed ) {
-				// Anonymous user. Save preferences in local storage
+				// Registered user. Use MW APIs to change preferences
-				mw.storage.setObject( this.preferenceName, this.preferences );
-				callback.call( this, true );
-			} else {
-				// Logged in user. Use MW APIs to change preferences
 				new mw.Api().saveOption(
 					this.preferenceName,
 					JSON.stringify( this.preferences )
@@ -92,6 +90,10 @@
 				} ).fail( function () {
 					callback.call( self, false );
 				} );
+			} else {
+				// Anonymous user. Save preferences in local storage
+				mw.storage.setObject( this.preferenceName, this.preferences );
+				callback.call( this, true );
 			}
 		}
 	};