reflection for ease and confusion

src/DeferredInterface.php

@@ -14,14 +14,14 @@ interface DeferredInterface extends RegistryInterface {
 	 * @param string $key The key of the value to retrieve
 	 * @return mixed
 	 */
-	function get($key);
+	public function get($key);
 
 	/**
 	 * Method to retrieve the value stored at key via another invocation.
 	 * @param string $key The key of the value to retrieve
 	 * @param array $args The args to pass the callable
 	 * @return mixed
 	 */
-	function getNew($key, array $args = []);
+	public function getNew($key, array $args = []);
 
-}
+}

src/Filter.php

@@ -6,32 +6,32 @@ class Filter {
 
 	protected $map = [];
 
-	function __construct(array $map){
+	public function __construct(array $map){
 		foreach($map as $k => $v){
 			$this->map[$k] = $v;
 		}
 	}
 
-	function get($name){
+	public function get($name){
 		if(!isset($this->map[$name])){
 			return null;
 		}
 		return $this->map[$name];
 	}
 
-	function getFiltered($name, callable $callback = null){
+	public function getFiltered($name, callable $callback = null){
 		$value = $this->get($name);
 		if($value != null && $callback){
 			return call_user_func($callback, $value, $name);
 		}
 		return $value;
 	}
 
-	function getAll(){
+	public function getAll(){
 		return $this->map;
 	}
 
-	function getAllFiltered(callable $callback = null){
+	public function getAllFiltered(callable $callback = null){
 		$values = $this->getAll();
 		if($callback){
 			array_walk_recursive($values, $callback);

src/Reference.php

@@ -20,7 +20,7 @@ class Reference extends Registry implements RegistryInterface {
 	 * @param array &$map The array to reference
 	 * @return
 	 */
-	function __construct( &$map = null ) {
+	public function __construct( &$map = null ) {
 		if( $map === null ) {
 			$map = array();
 		}

src/RegistryInterface.php

@@ -14,13 +14,13 @@ interface RegistryInterface {
 	 * @param mixed $value The value to store
 	 * @return
 	 */
-	function set($key, $value);
+	public function set($key, $value);
 
 	/**
 	 * Method to retrieve the value stored at key
 	 * @param scalar $key The key of the value to retrieve
 	 * @return mixed
 	 */
-	function get($key);
+	public function get($key);
 
-}
+}

src/ServiceLoader.php

@@ -21,14 +21,14 @@
  */
 class ServiceLoader {
 
-	function loadServices($path){
+	public function loadServices($path){
 
 		return (new ObjectLoader)->loadObject(new Di, $path);
 
 	}
 
-	function loadDi($path){
+	public function loadDi($path){
 		return $this->loadServices($path);
 	}
 
-}
+}

src/Set.php

@@ -18,15 +18,15 @@ class Set implements SetInterface, \Countable, \IteratorAggregate {
 	 * unshift values to our list
 	 * @param mixed $value
 	 */
-	function lpush($value){
+	public function lpush($value){
 		array_unshift($this->map, $value);
 	}
 
 	/**
 	 * push values to our list
 	 * @param mixed $value
 	 */
-	function rpush($value){
+	public function rpush($value){
 		// array_push
 		$this->map[] = $value;
 	}
@@ -35,7 +35,7 @@ function rpush($value){
 	 * shift values off our list
 	 * @return mixed
 	 */
-	function lpop(){
+	public function lpop(){
 		if($this->count() <= 0) return null;
 		return array_shift($this->map);
 	}
@@ -44,7 +44,7 @@ function lpop(){
 	 * pop values off our array
 	 * @return mixed
 	 */
-	function rpop(){
+	public function rpop(){
 		if($this->count() <= 0) return null;
 		return array_pop($this->map);
 	}
@@ -53,15 +53,15 @@ function rpop(){
 	 * \Countable -- how big is our list
 	 * @return int
 	 */
-	function count(){
+	public function count(){
 		return count($this->map);
 	}
 
 	/**
 	 * spin off an iterator for our list
 	 * @return \ArrayIterator
 	 */
-	function getIterator(){
+	public function getIterator(){
 		return new \ArrayIterator($this->map);
 	}
 

src/SetInterface.php

@@ -12,30 +12,30 @@ interface SetInterface {
 	 * unshift values to our list
 	 * @param mixed $value
 	 */
-	function lpush($value);
+	public function lpush($value);
 
 	/**
 	 * push values to our list
 	 * @param mixed $value
 	 */
-	function rpush($value);
+	public function rpush($value);
 
 	/**
 	 * shift values off our list
 	 * @return mixed
 	 */
-	function lpop();
+	public function lpop();
 
 	/**
 	 * pop values off our array
 	 * @return mixed
 	 */
-	function rpop();
+	public function rpop();
 
 	/**
 	 * \Countable -- how big is our list
 	 * @return int
 	 */
-	function count();
+	public function count();
 
-}
+}

tests/PHPUnit/ServiceLoaderTest.php

@@ -58,14 +58,4 @@ function test_demoClass1(){
 		$this->assertEquals(1, $O->var);
 	}
 
-	/**
-	 * @expectedException \Exception
-	 */
-	// function test_demoClass2(){
-	// 	$path = __DIR__ . "/nothelpers";
-	// 	$di = (new \Chevron\Containers\ServiceLoader)->loadDi($path);
-
-	// 	$di->set("error", 404);
-	// }
-
 }