Added caching of result.

Signed-off-by: Jan Pecha <janpecha@email.cz>

Author: janpecha

src/Dependency.php

@@ -16,6 +16,9 @@ class Dependency
 
 		/** @var  array  [(string) item => TRUE] */
 		private $cache = array();
+
+		/** @var  bool */
+		private $useCached = TRUE;
 
 
 
@@ -32,6 +35,7 @@ public function add($item, $depends = NULL)
 			}
 
 			$this->items[(string)$item] = $depends;
+			$this->useCached = FALSE;
 			return $this;
 		}
 
@@ -55,10 +59,16 @@ public function reset()
 		 */
 		public function getResolved()
 		{
+			if($this->useCached)
+			{
+				return $this->result;
+			}
+
 			$this->result = array();
 			$this->cache = array();
 
 			array_walk($this->items, array($this, 'applyWalk'));
+			$this->useCached = TRUE;
 			return $this->result;
 		}
 

tests/Dependency/basic.phpt

@@ -45,6 +45,8 @@ $resolver->add('a', array('b', 'c'));
 $resolver->add('b', 'd');
 $resolver->add('c', array('d'));
 
+Assert::same(array('d', 'b', 'c', 'a', 'x'), $resolver->getResolved());
+# cache test
 Assert::same(array('d', 'b', 'c', 'a', 'x'), $resolver->getResolved());
 $resolver->reset();
 
@@ -59,5 +61,3 @@ $result = $resolver->add('x', array('a', 'b'))
 	->getResolved();
 
 Assert::same(array('d', 'b', 'c', 'a', 'x'), $result);
-
-