bug #35764 [Messenger] Add missing return in AmazonSqsReceiver::getMessageCount (mynameisbogdan)

fabpot · fabpot · commit baf42452032f · 2020-02-18T09:08:26.000+01:00
This PR was merged into the 5.1-dev branch. Discussion ---------- [Messenger] Add missing return in AmazonSqsReceiver::getMessageCount | Q | A | ------------- | --- | Branch? | master  | Bug fix? | yes | New feature? | no  | Deprecations? | no  | License | MIT Commits ------- 7042ff8 [Messenger] Add missing return in AmazonSqsReceiver::getMessageCount
diff --git a/src/Symfony/Component/Messenger/Bridge/AmazonSqs/Transport/AmazonSqsReceiver.php b/src/Symfony/Component/Messenger/Bridge/AmazonSqs/Transport/AmazonSqsReceiver.php
@@ -93,7 +93,7 @@ public function reject(Envelope $envelope): void
     public function getMessageCount(): int
     {
         try {
-            $this->connection->getMessageCount();
+            return $this->connection->getMessageCount();
         } catch (HttpExceptionInterface $e) {
             throw new TransportException($e->getMessage(), 0, $e);
         }

Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,7 @@ public function reject(Envelope $envelope): void`
`93`	`93`	`public function getMessageCount(): int`
`94`	`94`	`{`
`95`	`95`	`try {`
`96`		`- $this->connection->getMessageCount();`
	`96`	`+ return $this->connection->getMessageCount();`
`97`	`97`	`} catch (HttpExceptionInterface $e) {`
`98`	`98`	`throw new TransportException($e->getMessage(), 0, $e);`
`99`	`99`	`}`