Jdlrobson has uploaded a new change for review.
https://gerrit.wikimedia.org/r/172457
Change subject: Hygiene: Clearly identify when/how things get initialised.
......................................................................
Hygiene: Clearly identify when/how things get initialised.
Change-Id: I5dbf45481676aeb47dcff58bd570c9ab01fcadd9
---
M includes/Resources.php
R javascripts/modules/editor/init.js
R javascripts/modules/issues/init.js
R javascripts/modules/languages/init.js
R javascripts/modules/lastEdited/init.js
A javascripts/modules/preferredLanguages/init.js
D javascripts/modules/preferredLanguages/preferred.js
D javascripts/modules/preferredLanguages/profiler.js
R javascripts/modules/redlinks/init.js
R javascripts/modules/references/init.js
R javascripts/modules/search/init.js
R javascripts/modules/toc/init.js
R javascripts/modules/toggling/init.js
R javascripts/modules/tutorials/init.js
R javascripts/modules/wikigrok/init.js
15 files changed, 61 insertions(+), 69 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend
refs/changes/57/172457/1
diff --git a/includes/Resources.php b/includes/Resources.php
index 65f5c54..9c1115a 100644
--- a/includes/Resources.php
+++ b/includes/Resources.php
@@ -109,7 +109,7 @@
'mobile.toggling',
),
'scripts' => array(
- 'javascripts/modules/toc/toc.js',
+ 'javascripts/modules/toc/init.js',
),
'styles' => array(
'less/modules/toc/toc.less',
@@ -142,7 +142,7 @@
'javascripts/modes.js',
'javascripts/mainmenu.js',
'javascripts/modules/lastEdited/time.js',
- 'javascripts/modules/lastEdited/lastEdited.js',
+ 'javascripts/modules/lastEdited/init.js',
),
'messages' => array(
// lastEdited.js
@@ -203,7 +203,7 @@
'mediawiki.user',
),
'scripts' => array(
- 'javascripts/modules/redlinks/redlinks.js',
+ 'javascripts/modules/redlinks/init.js',
),
),
@@ -237,7 +237,7 @@
'mobile-frontend-editor-edit',
),
'scripts' => array(
- 'javascripts/modules/editor/editor.js',
+ 'javascripts/modules/editor/init.js',
),
),
@@ -454,8 +454,7 @@
'mobile.startup',
),
'scripts' => array(
- 'javascripts/modules/preferredLanguages/profiler.js',
- 'javascripts/modules/preferredLanguages/preferred.js',
+ 'javascripts/modules/preferredLanguages/init.js',
),
),
@@ -486,7 +485,7 @@
'scripts' => array(
'javascripts/modules/search/SearchApi.js',
'javascripts/modules/search/SearchOverlay.js',
- 'javascripts/modules/search/search.js',
+ 'javascripts/modules/search/init.js',
),
'templates' => array(
'SearchOverlay.hogan' =>
'templates/modules/search/SearchOverlay.hogan',
@@ -603,7 +602,7 @@
'mobile.loggingSchemas',
),
'scripts' => array(
- 'javascripts/modules/wikigrok/wikigrok.js',
+ 'javascripts/modules/wikigrok/init.js',
),
),
@@ -752,7 +751,7 @@
),
'scripts' => array(
'javascripts/modules/references/ReferencesDrawer.js',
- 'javascripts/modules/references/references.js',
+ 'javascripts/modules/references/init.js',
),
),
@@ -764,7 +763,7 @@
'less/modules/toggle.less',
),
'scripts' => array(
- 'javascripts/modules/toggling/toggle.js',
+ 'javascripts/modules/toggling/init.js',
),
),
@@ -796,7 +795,7 @@
'mobile.loggingSchemas',
),
'scripts' => array(
- 'javascripts/modules/tutorials/newbieEditor.js',
+ 'javascripts/modules/tutorials/init.js',
),
'messages' => array(
// newbieEditor.js
@@ -851,7 +850,7 @@
'javascripts/modules/uploads/init.js',
'javascripts/modules/mainmenutweaks.js',
'javascripts/modules/mediaViewer/init.js',
- 'javascripts/modules/languages/languages.js',
+ 'javascripts/modules/languages/init.js',
),
),
@@ -887,7 +886,7 @@
),
'scripts' => array(
'javascripts/modules/issues/CleanupOverlay.js',
- 'javascripts/modules/issues/issues.js',
+ 'javascripts/modules/issues/init.js',
),
'messages' => array(
// issues.js
diff --git a/javascripts/modules/editor/editor.js
b/javascripts/modules/editor/init.js
similarity index 100%
rename from javascripts/modules/editor/editor.js
rename to javascripts/modules/editor/init.js
diff --git a/javascripts/modules/issues/issues.js
b/javascripts/modules/issues/init.js
similarity index 100%
rename from javascripts/modules/issues/issues.js
rename to javascripts/modules/issues/init.js
diff --git a/javascripts/modules/languages/languages.js
b/javascripts/modules/languages/init.js
similarity index 100%
rename from javascripts/modules/languages/languages.js
rename to javascripts/modules/languages/init.js
diff --git a/javascripts/modules/lastEdited/lastEdited.js
b/javascripts/modules/lastEdited/init.js
similarity index 100%
rename from javascripts/modules/lastEdited/lastEdited.js
rename to javascripts/modules/lastEdited/init.js
diff --git a/javascripts/modules/preferredLanguages/init.js
b/javascripts/modules/preferredLanguages/init.js
new file mode 100644
index 0000000..48488e2
--- /dev/null
+++ b/javascripts/modules/preferredLanguages/init.js
@@ -0,0 +1,49 @@
+// When set orders LanguageOverlay list of languages by most frequently chosen
+( function ( M, $ ) {
+ var supported = M.supportsLocalStorage,
+ langMap = M.settings.getUserSetting( 'langMap' ),
+ curLanguage = mw.config.get( 'wgContentLanguage' );
+
+ langMap = langMap ? $.parseJSON( langMap ) : {};
+
+ function loadLanguageMap() {
+ langMap = M.settings.getUserSetting( 'langMap' );
+ langMap = langMap ? JSON.parse( langMap ) : {};
+ }
+
+ function profileLanguage( language ) {
+ // if not supported, don't do anything
+ if ( supported && langMap ) {
+ var count;
+ count = langMap[ language ] || 0;
+ count += 1;
+ // cap at 100 as this is enough data to work on
+ langMap[ language ] = count > 100 ? 100 : count;
+ M.settings.saveUserSetting( 'langMap', JSON.stringify(
langMap ) );
+ }
+ }
+
+ function initProfiler() {
+ if ( supported ) {
+ loadLanguageMap();
+ profileLanguage( curLanguage );
+ }
+ }
+
+ M.on( 'language-overlay-initialize', function ( options ) {
+ options.languages = options.languages.sort( function ( a, b ) {
+ var x = langMap[ a.lang ] || 0,
+ y = langMap[ b.lang ] || 0;
+ if ( x > 0 ) {
+ a.preferred = true;
+ }
+ if ( y > 0 ) {
+ b.preferred = true;
+ }
+ return x < y;
+ } );
+ } );
+ M.on( 'language-select', profileLanguage );
+ initProfiler();
+
+}( mw.mobileFrontend, jQuery ) );
diff --git a/javascripts/modules/preferredLanguages/preferred.js
b/javascripts/modules/preferredLanguages/preferred.js
deleted file mode 100644
index 04f2c1e..0000000
--- a/javascripts/modules/preferredLanguages/preferred.js
+++ /dev/null
@@ -1,20 +0,0 @@
-// When set orders LanguageOverlay list of languages by most frequently chosen
-( function ( M, $ ) {
- var langMap = M.settings.getUserSetting( 'langMap' );
- langMap = langMap ? $.parseJSON( langMap ) : {};
-
- M.on( 'language-overlay-initialize', function ( options ) {
- options.languages = options.languages.sort( function ( a, b ) {
- var x = langMap[ a.lang ] || 0,
- y = langMap[ b.lang ] || 0;
- if ( x > 0 ) {
- a.preferred = true;
- }
- if ( y > 0 ) {
- b.preferred = true;
- }
- return x < y;
- } );
- } );
-
-}( mw.mobileFrontend, jQuery ) );
diff --git a/javascripts/modules/preferredLanguages/profiler.js
b/javascripts/modules/preferredLanguages/profiler.js
deleted file mode 100644
index 519e7c8..0000000
--- a/javascripts/modules/preferredLanguages/profiler.js
+++ /dev/null
@@ -1,36 +0,0 @@
-// Watches users use of the language overlay and profiles the languages
-// that user clicks on. Stores this locally under the key langMap
-( function ( M ) {
-
- var supported = M.supportsLocalStorage,
- langMap,
- curLanguage = mw.config.get( 'wgContentLanguage' );
-
- function loadLanguageMap() {
- langMap = M.settings.getUserSetting( 'langMap' );
- langMap = langMap ? JSON.parse( langMap ) : {};
- }
-
- function profileLanguage( language ) {
- // if not supported, don't do anything
- if ( supported && langMap ) {
- var count;
- count = langMap[ language ] || 0;
- count += 1;
- // cap at 100 as this is enough data to work on
- langMap[ language ] = count > 100 ? 100 : count;
- M.settings.saveUserSetting( 'langMap', JSON.stringify(
langMap ) );
- }
- }
-
- function initProfiler() {
- if ( supported ) {
- loadLanguageMap();
- profileLanguage( curLanguage );
- }
- }
-
- M.on( 'language-select', profileLanguage );
- initProfiler();
-
-}( mw.mobileFrontend ) );
diff --git a/javascripts/modules/redlinks/redlinks.js
b/javascripts/modules/redlinks/init.js
similarity index 100%
rename from javascripts/modules/redlinks/redlinks.js
rename to javascripts/modules/redlinks/init.js
diff --git a/javascripts/modules/references/references.js
b/javascripts/modules/references/init.js
similarity index 100%
rename from javascripts/modules/references/references.js
rename to javascripts/modules/references/init.js
diff --git a/javascripts/modules/search/search.js
b/javascripts/modules/search/init.js
similarity index 100%
rename from javascripts/modules/search/search.js
rename to javascripts/modules/search/init.js
diff --git a/javascripts/modules/toc/toc.js b/javascripts/modules/toc/init.js
similarity index 100%
rename from javascripts/modules/toc/toc.js
rename to javascripts/modules/toc/init.js
diff --git a/javascripts/modules/toggling/toggle.js
b/javascripts/modules/toggling/init.js
similarity index 100%
rename from javascripts/modules/toggling/toggle.js
rename to javascripts/modules/toggling/init.js
diff --git a/javascripts/modules/tutorials/newbieEditor.js
b/javascripts/modules/tutorials/init.js
similarity index 100%
rename from javascripts/modules/tutorials/newbieEditor.js
rename to javascripts/modules/tutorials/init.js
diff --git a/javascripts/modules/wikigrok/wikigrok.js
b/javascripts/modules/wikigrok/init.js
similarity index 100%
rename from javascripts/modules/wikigrok/wikigrok.js
rename to javascripts/modules/wikigrok/init.js
--
To view, visit https://gerrit.wikimedia.org/r/172457
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5dbf45481676aeb47dcff58bd570c9ab01fcadd9
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
