Merge pull request #92 from elgentos/hotfix-return-type-webhook-contr…

…ollers optional return type for webhook controllers
elgentos · Jan 17, 2023 · 3163f92 · 3163f92
2 parents fa306ab + 4119a9c
commit 3163f92
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/Controller/Webhook/Cache.php b/Controller/Webhook/Cache.php
@@ -46,10 +46,10 @@ public function __construct(
         $this->typeList = $typeList;
     }
 
-    public function execute(): ResultInterface
+    public function execute(): ?ResultInterface
     {
         if (!$this->protectRoute()) {
-            return;
+            return null;
         }
 
         $result = $this->resultFactory->create(ResultFactory::TYPE_JSON);

diff --git a/Controller/Webhook/Url.php b/Controller/Webhook/Url.php
@@ -70,10 +70,10 @@ public function __construct(
         $this->urlRewriteResource = $urlRewriteResource;
     }
 
-    public function execute(): ResultInterface
+    public function execute(): ?ResultInterface
     {
         if (!$this->protectRoute()) {
-            return;
+            return null;
         }
 
         $result = $this->resultFactory->create(ResultFactory::TYPE_JSON);