Merge pull request #794 from tentwofour/feature_seo_twig_extension_ca…

…che_3.3 [SeoBundle] Add cache to SeoTwigExtension
Kunstmaan · Oct 31, 2015 · 7a0a453 · 7a0a453
2 parents 0535fff + 93ad075
commit 7a0a453
Showing 1 changed file with 16 additions and 1 deletion.
diff --git a/src/Kunstmaan/SeoBundle/Twig/SeoTwigExtension.php b/src/Kunstmaan/SeoBundle/Twig/SeoTwigExtension.php
@@ -30,6 +30,13 @@ class SeoTwigExtension extends Twig_Extension
      */
     private $websiteTitle;
 
+    /**
+     * Saves querying the db multiple times, if you happen to use any of the defined
+     * functions more than once in your templates
+     * @var array
+     */
+    private $seoCache = [];
+
     /**
      * @param EntityManager $em
      */
@@ -61,7 +68,15 @@ public function getFunctions()
      */
     public function getSeoFor(AbstractPage $entity)
     {
-        return $this->em->getRepository('KunstmaanSeoBundle:Seo')->findOrCreateFor($entity);
+        $key = md5(get_class($entity).$entity->getId());
+
+        if (!array_key_exists($key, $this->seoCache))
+        {
+            $seo = $this->em->getRepository('KunstmaanSeoBundle:Seo')->findOrCreateFor($entity);
+            $this->seoCache[$key] = $seo;
+        }
+
+        return $this->seoCache[$key];
     }
 
     /**