bug #36747 Queue name is a required parameter (theravel)

nicolas-grekas · nicolas-grekas · commit e75c227d7919 · 2020-05-08T12:02:38.000+02:00
This PR was submitted for the 5.0 branch but it was merged into the 4.4 branch instead. Discussion ---------- Queue name is a required parameter | Q | A | ------------- | --- | Branch? | 5.0 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - Queue name is required parameter, which means this block will just fail Commits ------- 9d48eed Queue name is a required parameter
diff --git a/src/Symfony/Component/Messenger/Transport/AmqpExt/Connection.php b/src/Symfony/Component/Messenger/Transport/AmqpExt/Connection.php
@@ -326,7 +326,7 @@ public function get(string $queueName): ?\AMQPEnvelope
                 // If we get a 404 for the queue, it means we need to set up the exchange & queue.
                 $this->setupExchangeAndQueues();
 
-                return $this->get();
+                return $this->get($queueName);
             }
 
             throw $e;

Original file line number	Diff line number	Diff line change
`@@ -326,7 +326,7 @@ public function get(string $queueName): ?\AMQPEnvelope`
`326`	`326`	`// If we get a 404 for the queue, it means we need to set up the exchange & queue.`
`327`	`327`	`$this->setupExchangeAndQueues();`
`328`	`328`
`329`		`- return $this->get();`
	`329`	`+ return $this->get($queueName);`
`330`	`330`	`}`
`331`	`331`
`332`	`332`	`throw $e;`