Adamw has submitted this change and it was merged.
Change subject: Do not set a null checked variable to false
......................................................................
Do not set a null checked variable to false
Change-Id: I3350e8dd2bcaf9a73da0268d945a9dd90be4abe1
---
M sites/all/modules/wmf_common/Queue.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Adamw: Verified; Looks good to me, approved
diff --git a/sites/all/modules/wmf_common/Queue.php
b/sites/all/modules/wmf_common/Queue.php
index 90ad3a7..c91eed6 100644
--- a/sites/all/modules/wmf_common/Queue.php
+++ b/sites/all/modules/wmf_common/Queue.php
@@ -120,7 +120,7 @@
$this->connection->connect();
}
catch ( Stomp_Exception $e ) {
- $this->connection = false;
+ $this->connection = null;
watchdog( 'wmf_common', "Queue connection failure #$attempt: "
. $e->getMessage(), array(), WATCHDOG_ERROR );
}
}
--
To view, visit https://gerrit.wikimedia.org/r/86195
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3350e8dd2bcaf9a73da0268d945a9dd90be4abe1
Gerrit-PatchSet: 2
Gerrit-Project: wikimedia/fundraising/crm
Gerrit-Branch: master
Gerrit-Owner: Mwalker <[email protected]>
Gerrit-Reviewer: Adamw <[email protected]>
Gerrit-Reviewer: Katie Horn <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
