Ejegg has submitted this change and it was merged.
Change subject: servers don't have to be an array
......................................................................
servers don't have to be an array
In fact, we can't use transactions if connecting to an "aggregate connection"
anyway.
Change-Id: I70c94ee82c09dd7cbbd3bab3ab76c3684e1c3317
---
M src/PHPQueue/Backend/Predis.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Ejegg: Verified; Looks good to me, approved
diff --git a/src/PHPQueue/Backend/Predis.php b/src/PHPQueue/Backend/Predis.php
index ef35438..1778f31 100644
--- a/src/PHPQueue/Backend/Predis.php
+++ b/src/PHPQueue/Backend/Predis.php
@@ -46,7 +46,7 @@
public function __construct($options=array())
{
parent::__construct();
- if (!empty($options['servers']) && is_array($options['servers'])) {
+ if (!empty($options['servers'])) {
$this->servers = $options['servers'];
}
if (!empty($options['redis_options']) &&
is_array($options['redis_options'])) {
--
To view, visit https://gerrit.wikimedia.org/r/216340
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I70c94ee82c09dd7cbbd3bab3ab76c3684e1c3317
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/php-queue
Gerrit-Branch: master
Gerrit-Owner: Awight <[email protected]>
Gerrit-Reviewer: Ejegg <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
