Run only once

Author: aleksip

src/aleksip/DataTransformPlugin/DataTransformer.php

@@ -10,27 +10,32 @@ class DataTransformer
 {
     protected static $processed = array();
 
-    protected $env;
     protected $reservedKeys;
     protected $patternDataStore;
+    protected $env;
+    protected $hasRun;
 
-    public function __construct(\Twig_Environment $env)
+    public function __construct()
     {
-        $this->env = $env;
         // TODO: Add an accessor function for $reservedKeys to the Data class?
         $this->reservedKeys = array("cacheBuster","link","patternSpecific","patternLabHead","patternLabFoot");
         $this->patternDataStore = PatternData::get();
     }
 
-    public function run()
+    public function run(\Twig_Environment $env)
     {
+        $this->env = $env;
+        if ($this->hasRun) {
+            return;
+        }
         // Process global data.
         $dataStore = $this->processData(Data::get());
         Data::replaceStore($dataStore);
         // Process pattern specific data.
         foreach (array_keys($this->patternDataStore) as $pattern) {
             $this->processPattern($pattern);
         }
+        $this->hasRun = true;
     }
 
     protected function isProcessed($pattern)

src/aleksip/DataTransformPlugin/PatternLabListener.php

@@ -8,6 +8,9 @@
 
 class PatternLabListener extends Listener
 {
+    protected $dt;
+    protected $nv;
+
     public function __construct()
     {
         $this->addListener(
@@ -18,10 +21,12 @@ public function __construct()
 
     public function twigPatternLoaderCustomize()
     {
+        $this->dt ?: $this->dt = new DataTransformer();
+        $this->nv ?: $this->nv = new PatternDataNodeVisitor($this->dt);
+
         $env = TwigUtil::getInstance();
-        $dt = new DataTransformer($env);
-        $env->addNodeVisitor(new PatternDataNodeVisitor($dt));
+        $env->addNodeVisitor($this->nv);
         TwigUtil::setInstance($env);
-        $dt->run();
+        $this->dt->run($env);
     }
 }