Ricordisamoa has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/341985 )
Change subject: Fix some typos ...................................................................... Fix some typos The names of two private methods have been changed. Change-Id: I966930bdf44190c3b5262ddaf13b9ed46a478735 --- M LoginNotify.hooks.php M LoginNotify_body.php M i18n/qqq.json 3 files changed, 12 insertions(+), 12 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/LoginNotify refs/changes/85/341985/1 diff --git a/LoginNotify.hooks.php b/LoginNotify.hooks.php index f560f0e..dff1ffb 100644 --- a/LoginNotify.hooks.php +++ b/LoginNotify.hooks.php @@ -98,7 +98,7 @@ /** * Hook for login auditing post 1.27 * - * @param $ret AuthenticationResponse Is login succesful? + * @param $ret AuthenticationResponse Is login successful? * @param $user User|null User object on successful auth * @param $username String Username for failed attempts. */ @@ -171,7 +171,7 @@ * Hook for loading options. * * This is a bit hacky. Used to be able to set a different - * default for admins then other users + * default for admins than other users * * @param $user User * @param &$options array @@ -183,12 +183,12 @@ return true; } - if ( !self::isUserOptionOverriden( $user ) ) { + if ( !self::isUserOptionOverridden( $user ) ) { return true; } $defaultOpts = User::getDefaultOptions(); - $optionsToCheck = self::getOverridenOptions(); + $optionsToCheck = self::getOverriddenOptions(); foreach ( $optionsToCheck as $opt ) { if ( $options[$opt] === self::OPTIONS_FAKE_FALSE ) { @@ -208,7 +208,7 @@ * Hook for saving options. * * This is a bit hacky. Used to be able to set a different - * default for admins then other users. Since admins are higher value + * default for admins than other users. Since admins are higher value * targets, it may make sense to have notices enabled by default for * them, but disabled for normal users. * @@ -220,9 +220,9 @@ * @return bool */ public static function onUserSaveOptions( User $user, array &$options ) { - $optionsToCheck = self::getOverridenOptions(); + $optionsToCheck = self::getOverriddenOptions(); $defaultOpts = User::getDefaultOptions(); - if ( !self::isUserOptionOverriden( $user ) ) { + if ( !self::isUserOptionOverridden( $user ) ) { return true; } foreach ( $optionsToCheck as $opt ) { @@ -252,7 +252,7 @@ * * @return array Which option keys to check */ - private static function getOverridenOptions() { + private static function getOverriddenOptions() { // For login-success, it makes most sense to email // people about it, but auto-subscribing people to email // is a bit icky as nobody likes to be spammed. @@ -262,7 +262,7 @@ ]; } - private static function isUserOptionOverriden( User $user ) { + private static function isUserOptionOverridden( User $user ) { global $wgLoginNotifyEnableForPriv; // Note: isAllowedAny calls into session for per-session restrictions, // which we do not want to take into account, and more importantly diff --git a/LoginNotify_body.php b/LoginNotify_body.php index 8abe5f2..21e46e0 100644 --- a/LoginNotify_body.php +++ b/LoginNotify_body.php @@ -245,7 +245,7 @@ * Actually do the query of the check user table. * * @note This catches and ignores database errors. - * @param $userId int User id number (Not neccesarily for the local wiki) + * @param $userId int User id number (Not necessarily for the local wiki) * @param $ipFragment string Prefix to match against cuc_ip (from $this->getIPNetwork()) * @param $dbr Database A database connection (possibly foreign) * @return boolean If $ipFragment is in check user db @@ -607,7 +607,7 @@ /** * Clear attempt counter for user. * - * When a user succesfully logs in, we start back from 0, as + * When a user successfully logs in, we start back from 0, as * otherwise a mistake here and there will trigger the warning. * * @param $user User diff --git a/i18n/qqq.json b/i18n/qqq.json index b6fe6f9..d606ee2 100644 --- a/i18n/qqq.json +++ b/i18n/qqq.json @@ -17,5 +17,5 @@ "notification-header-login-success": "Text of notification for when someone successfully logs in as the current account from a computer that has not been previously used by that account. This is shown in the user's echo notifications, and is possibly emailed to the user.\n* $1 = Username for account, formatted for display\n* $2 = Username for account, unformatted for use in GENDER", "notification-loginnotify-login-fail-new-emailbatch": "Body of email notification that someone from a computer not previously used by the user has attempted and failed to log into the user's account. Subject of Message is {{msg-mw|notification-loginnotify-login-fail-email-subject}}. $1 is account name. $2 is the number of attempts", "notification-loginnotify-login-fail-known-emailbatch": "Body of email notification that someone from a computer which has been previously used by the user has attempted and failed to log into the user's account. Subject of Message is {{msg-mw|notification-loginnotify-login-fail-email-subject}}. $1 is account name. $2 is the number of attempts", - "notification-loginnotify-login-success-emailbatch": "Body of email notification that someone from a computer not previously used by the user has succesfully logged into the user's account. Subject of Message is {{msg-mw|notification-loginnotify-login-success-email-subject}}. $1 is account name." + "notification-loginnotify-login-success-emailbatch": "Body of email notification that someone from a computer not previously used by the user has successfully logged into the user's account. Subject of Message is {{msg-mw|notification-loginnotify-login-success-email-subject}}. $1 is account name." } -- To view, visit https://gerrit.wikimedia.org/r/341985 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I966930bdf44190c3b5262ddaf13b9ed46a478735 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/LoginNotify Gerrit-Branch: master Gerrit-Owner: Ricordisamoa <ricordisa...@openmailbox.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits