Siebrand has uploaded a new change for review.

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


Change subject: Update formatting
......................................................................

Update formatting

Change-Id: Ie273d5ffd4d9f4c99a43d9f9ee9664c00963cc1f
---
M SpecialNotifyTranslators.php
M resources/ext.translationnotifications.notifytranslators.js
M scripts/DigestEmailer.php
3 files changed, 8 insertions(+), 10 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/TranslationNotifications 
refs/changes/10/88710/1

diff --git a/SpecialNotifyTranslators.php b/SpecialNotifyTranslators.php
index 206c011..755128d 100644
--- a/SpecialNotifyTranslators.php
+++ b/SpecialNotifyTranslators.php
@@ -105,7 +105,7 @@
 
                $formFields = array();
 
-               $default = (int) $tpage !== 0 ? $tpage : 'unset';
+               $default = (int)$tpage !== 0 ? $tpage : 'unset';
                $formFields['TranslatablePage'] = array(
                        'type' => 'select',
                        'label-message' => array( 
'translationnotifications-translatablepage-title' ),
@@ -603,10 +603,10 @@
                        $notificationText
                )->numParams( count( $relevantLanguages ) )
                        ->inLanguage( $userFirstLanguage )
-                       ->text()
-                       // Bidi-isolation of site name from date
-                       . $userFirstLanguage->getDirMarkEntity()
-                       . ', ~~~~~'; // Date and time
+                       ->text();
+               // Bidi-isolation of site name from date
+               $text .= $userFirstLanguage->getDirMarkEntity() .
+                       ', ~~~~~'; // Date and time
 
                $editSummary = $this->msg(
                        'translationnotifications-edit-summary',
diff --git a/resources/ext.translationnotifications.notifytranslators.js 
b/resources/ext.translationnotifications.notifytranslators.js
index 1a1d0fa..20a8c8c 100644
--- a/resources/ext.translationnotifications.notifytranslators.js
+++ b/resources/ext.translationnotifications.notifytranslators.js
@@ -36,7 +36,7 @@
                } );
 
                // Attach the language autocomplete widget.
-               $( '#wpUserLanguage' ).multiselectautocomplete( { inputbox : 
'#mw-input-wpLanguagesToNotify' } );
+               $( '#wpUserLanguage' ).multiselectautocomplete( { inputbox: 
'#mw-input-wpLanguagesToNotify' } );
 
                /**
                 * Notification preview
@@ -100,6 +100,5 @@
                                .hide()
                                .attr( { id: previewId } )
                );
-
        } );
 }( jQuery, mediaWiki ) );
diff --git a/scripts/DigestEmailer.php b/scripts/DigestEmailer.php
index 9a7ad50..70bc8cc 100644
--- a/scripts/DigestEmailer.php
+++ b/scripts/DigestEmailer.php
@@ -64,9 +64,8 @@
                                }
                        }
                        $firstLangCode = $user->getOption( 
'translationnotifications-lang-1' );
-                       $firstLang = Language::factory(
-                               $firstLangCode )->fetchLanguageName( 
$signedUpLangCodes[0]
-                       );
+                       $firstLang = Language::factory( $firstLangCode )
+                               ->fetchLanguageName( $signedUpLangCodes[0] );
                        $this->output( "\tSigned up for: " . implode( ', ', 
$signedUpLangCodes ) . "\n" );
 
                        // Draft the mail for this user

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie273d5ffd4d9f4c99a43d9f9ee9664c00963cc1f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/TranslationNotifications
Gerrit-Branch: master
Gerrit-Owner: Siebrand <siebr...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to