jenkins-bot has submitted this change and it was merged.
Change subject: Handle StatusValue in authmanager events properly
......................................................................
Handle StatusValue in authmanager events properly
Change-Id: I6300095e49854c57f8abf76fe0e679eaab78cdb3
---
M includes/AuthManagerStatsdHandler.php
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Anomie: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/AuthManagerStatsdHandler.php
b/includes/AuthManagerStatsdHandler.php
index f8436da..02e1955 100644
--- a/includes/AuthManagerStatsdHandler.php
+++ b/includes/AuthManagerStatsdHandler.php
@@ -45,6 +45,7 @@
$successful = isset( $record['context']['successful'] ) ?
$record['context']['successful'] : null;
$error = null;
if ( $status instanceof Status || $status instanceof
StatusValue ) {
+ $status = Status::wrap( $status );
$successful = $status->isGood();
if ( !$successful ) {
$errorArray = $status->getErrorsArray() ?:
$status->getWarningsArray();
--
To view, visit https://gerrit.wikimedia.org/r/293242
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6300095e49854c57f8abf76fe0e679eaab78cdb3
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikimediaEvents
Gerrit-Branch: master
Gerrit-Owner: Gergő Tisza <gti...@wikimedia.org>
Gerrit-Reviewer: Anomie <bjor...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
