From c0fe2d1a54cd53ff0534b0e0d6752660a62bd0bd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Niklas=20Laxstr=C3=B6m?= Date: Tue, 12 Apr 2016 08:22:27 +0200 Subject: [PATCH] Fix language undo tooltip not showing Test plan: * Remove localstorage and cookies for a site * Go the the site * Change language and observe the undo tooltip Change-Id: Ibecb5ea1cd537b2f019e7a9aa67ffac1a4e0923d --- resources/js/ext.uls.interface.js | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/resources/js/ext.uls.interface.js b/resources/js/ext.uls.interface.js index 3c0a3712..bb6945f8 100644 --- a/resources/js/ext.uls.interface.js +++ b/resources/js/ext.uls.interface.js @@ -200,6 +200,14 @@ function userHasChangedLanguage() { var previousLang = mw.uls.getPreviousLanguages().slice( -1 )[ 0 ], currentLang = mw.config.get( 'wgUserLanguage' ); + + // Changed language is saved in showUndoTooltip, which is never + // called if previousLang is not defined, which will never be + // defined unless we do it now. + if ( previousLang === undefined ) { + mw.uls.setPreviousLanguages( [ currentLang ] ); + } + return previousLang && previousLang !== currentLang; }