Merge pull request #1051 from plentymarkets/hotfix/catch_context_errors

disable content cache if context initialization failed
plentymarkets · Nov 15, 2021 · 5d44fa4 · 5d44fa4
2 parents 32d53e3 + 49eff5e
commit 5d44fa4
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/Extensions/TwigTemplateContextExtension.php b/src/Extensions/TwigTemplateContextExtension.php
@@ -7,6 +7,7 @@
 use IO\Helper\EventDispatcher;
 use IO\Helper\TemplateContainer;
 use IO\Services\TemplateService;
+use Plenty\Modules\ContentCache\Contracts\ContentCacheRepositoryContract;
 use Plenty\Plugin\Templates\Extensions\Twig_Extension;
 
 class TwigTemplateContextExtension extends Twig_Extension
@@ -64,7 +65,14 @@ public function getGlobals(): array
                 $context->init(TemplateService::$currentTemplateData);
             }
 
-            return ArrayHelper::toArray($context);
+            $contextArray = ArrayHelper::toArray($context);
+            if(is_null($contextArray)) {
+                /** @var ContentCacheRepositoryContract $contentCacheRepository */
+                $contentCacheRepository = pluginApp(ContentCacheRepositoryContract::class);
+                $contentCacheRepository->disableCacheForResponse('Twig context could not be initialized');
+                return [];
+            }
+            return $contextArray;
         }
 
         return [];