Fixing bug with defining queue

ragboyjr · ragboyjr · commit b549b275fcea · 2018-11-06T07:36:44.000-08:00
Signed-off-by: RJ Garcia &lt;rj@bighead.net&gt;
diff --git a/src/RedisConnection.php b/src/RedisConnection.php
@@ -28,7 +28,7 @@ public static function fromDsn(string $dsn, array $options = []): self {
 
         \parse_str($parsedUrl['query'] ?? '', $query);
 
-        if ($options['queue'] ?? $query['queue'] ?? null) {
+        if (($options['queue'] ?? $query['queue'] ?? null) === null) {
             throw new \InvalidArgumentException(sprintf('The queue option must be included in the query parameters or configuration options.'));
         }
 

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ public static function fromDsn(string $dsn, array $options = []): self {`
`28`	`28`
`29`	`29`	`\parse_str($parsedUrl['query'] ?? '', $query);`
`30`	`30`
`31`		`- if ($options['queue'] ?? $query['queue'] ?? null) {`
	`31`	`+ if (($options['queue'] ?? $query['queue'] ?? null) === null) {`
`32`	`32`	`throw new \InvalidArgumentException(sprintf('The queue option must be included in the query parameters or configuration options.'));`
`33`	`33`	`}`
`34`	`34`