Aaron Schulz has submitted this change and it was merged.

Change subject: Fixed backwards logic for acr_id value.
......................................................................


Fixed backwards logic for acr_id value.

(cherry picked from commit d75ff74b7b011c6c35987f9ec89671ccdc57042d)

Change-Id: Ic92467924afcc147c512de43ea657c01c78c6764
---
M backend/UserAccountRequest.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Aaron Schulz: Verified; Looks good to me, approved



diff --git a/backend/UserAccountRequest.php b/backend/UserAccountRequest.php
index 7b0f19c..592fae3 100644
--- a/backend/UserAccountRequest.php
+++ b/backend/UserAccountRequest.php
@@ -324,7 +324,7 @@
        public function insertOn() {
                $dbw = wfGetDB( DB_MASTER );
                # Allow for some fields to be handled automatically...
-               $acr_id = is_null( $this->id )
+               $acr_id = !is_null( $this->id )
                        ? $this->id
                        : $dbw->nextSequenceValue( 
'account_requests_acr_id_seq' );
                # Insert into pending requests...

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic92467924afcc147c512de43ea657c01c78c6764
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ConfirmAccount
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Aaron Schulz <[email protected]>

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

Reply via email to