Throw LogicException when rendering block without template

nucleos · Mar 9, 2021 · f8d8b90 · f8d8b90
1 parent 445af59
commit f8d8b90
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 0 deletions.
diff --git a/src/Block/Service/ShariffShareBlockService.php b/src/Block/Service/ShariffShareBlockService.php
@@ -11,6 +11,7 @@
 
 namespace Nucleos\ShariffBundle\Block\Service;
 
+use LogicException;
 use Sonata\BlockBundle\Block\BlockContextInterface;
 use Sonata\BlockBundle\Block\Service\AbstractBlockService;
 use Sonata\BlockBundle\Block\Service\EditableBlockService;
@@ -35,6 +36,10 @@ public function execute(BlockContextInterface $blockContext, ?Response $response
             'block'    => $blockContext->getBlock(),
         ];
 
+        if (!\is_string($blockContext->getTemplate())) {
+            throw new LogicException('Cannot render block without template');
+        }
+
         return $this->renderResponse($blockContext->getTemplate(), $parameters, $response);
     }
 

diff --git a/tests/Block/Service/ShariffShareBlockServiceTest.php b/tests/Block/Service/ShariffShareBlockServiceTest.php
@@ -11,6 +11,7 @@
 
 namespace Nucleos\ShariffBundle\Tests\Block\Service;
 
+use LogicException;
 use Nucleos\ShariffBundle\Block\Service\ShariffShareBlockService;
 use Sonata\BlockBundle\Block\BlockContext;
 use Sonata\BlockBundle\Form\Mapper\FormMapper;
@@ -71,6 +72,24 @@ public function testExecute(): void
         static::assertSame('TWIG_CONTENT', $response->getContent());
     }
 
+    public function testExecuteWithNullTemplate(): void
+    {
+        $this->expectException(LogicException::class);
+        $this->expectExceptionMessage('Cannot render block without template');
+
+        $block = new Block();
+
+        $blockContext = new BlockContext($block, [
+            'template'    => null,
+        ]);
+
+        $response = new Response();
+
+        $blockService = new ShariffShareBlockService($this->twig);
+
+        static::assertSame($response, $blockService->execute($blockContext, $response));
+    }
+
     public function testGetMetadata(): void
     {
         $blockService = new ShariffShareBlockService($this->twig);