Nikerabbit has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/109450


Change subject: Fix ULS integration
......................................................................

Fix ULS integration

Update module dependencies and fixed a bug which caused signup
code to be loaded for logged in users.

Change-Id: I96f1b8d780d5dd95cddac6752a98b709a85d0f44
---
M Resources.php
M resources/js/ext.translate.mainpage.js
M resources/js/ext.translate.mainpage.signup.js
3 files changed, 7 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/TwnMainPage 
refs/changes/50/109450/1

diff --git a/Resources.php b/Resources.php
index 955f9d8..d3762c7 100644
--- a/Resources.php
+++ b/Resources.php
@@ -17,10 +17,10 @@
                'ext.translate.groupselector',
                'ext.translate.mainpage.icons',
                'ext.translate.statsbar',
-               'ext.uls.mediawiki',
                'ext.uls.interface',
-               'mediawiki.language',
+               'ext.uls.pt',
                'mediawiki.Uri',
+               'mediawiki.language',
                'mediawiki.util',
        ),
        'messages' => array(
@@ -34,7 +34,6 @@
        'scripts' => 'resources/js/ext.translate.mainpage.signup.js',
        'dependencies' => array(
                'ext.uls.mediawiki',
-               'ext.uls.interface',
                'mediawiki.api',
                'mediawiki.api.login',
                'jquery.json',
diff --git a/resources/js/ext.translate.mainpage.js 
b/resources/js/ext.translate.mainpage.js
index 46f03c8..e4e5ff2 100644
--- a/resources/js/ext.translate.mainpage.js
+++ b/resources/js/ext.translate.mainpage.js
@@ -176,10 +176,5 @@
                $tiles.eq( maxProjectTiles - 1 ).replaceWith( $selector );
        }
 
-       $( document ).ready( function () {
-               setupProjectTiles();
-               mw.loader.using( 'ext.translate.mainpage.signup', function () {
-                       mw.translate.setupSignupForm( $( '.login-widget' ) );
-               } );
-       } );
+       $( document ).ready( setupProjectTiles );
 }( jQuery, mediaWiki ) );
diff --git a/resources/js/ext.translate.mainpage.signup.js 
b/resources/js/ext.translate.mainpage.signup.js
index 2537850..932b39c 100644
--- a/resources/js/ext.translate.mainpage.signup.js
+++ b/resources/js/ext.translate.mainpage.signup.js
@@ -192,4 +192,8 @@
 
                $form.on( 'submit', handleSubmit );
        };
+
+       $( document ).ready( function () {
+               mw.translate.setupSignupForm( $( '.login-widget' ) );
+       } );
 }( jQuery, mediaWiki ) );

-- 
To view, visit https://gerrit.wikimedia.org/r/109450
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I96f1b8d780d5dd95cddac6752a98b709a85d0f44
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/TwnMainPage
Gerrit-Branch: master
Gerrit-Owner: Nikerabbit <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to