kalaomer / knot

src/Knot/Dict/AbstractDictBody.php

Unused Use Statements -1 removed lines

@@ -1,6 +1,5 @@
 <?php namespace Knot\Dict;
 
-use ArrayAccess;
 use Countable;
 use Illuminate\Contracts\Support\Arrayable;
 use IteratorAggregate;

Spacing +11 added lines, -11 removed lines

@@ -48,7 +48,7 @@ 
 	 */
 	public function __construct(array &$data, AbstractDictBody $parent = null, $path = '')
 	{
-		$this->data        =& $data;
+		$this->data        = & $data;
 		$this->path        = $path;
 		$this->parentArray = $parent;
 	}
@@ -72,16 +72,16 @@ 
 	 */
 	public function &__get($key)
 	{
-		if ( array_key_exists($key, $this->data) )
+		if (array_key_exists($key, $this->data))
 		{
-			$target =& $this->data[$key];
+			$target = & $this->data[$key];
 		}
 		else
 		{
 			throw new WrongArrayPathException($key);
 		}
 
-		if ( is_array($target) )
+		if (is_array($target))
 		{
 			$r = new ChildDict($target, $this->childParent(), $this->path($key));
 
@@ -101,18 +101,18 @@ 
 	 * @return mixed
 	 * @throws \Exception
 	 */
-	public function call($methodPath, array $arguments = [ ])
+	public function call($methodPath, array $arguments = [])
 	{
 		$function = $this->get($methodPath, false);
 
-		if ( ! $function || ! is_callable($function) )
+		if ( ! $function || ! is_callable($function))
 		{
 			throw new WrongFunctionException("Wrong function or not callable key!");
 		}
 
 		try
 		{
-			$arguments = array_merge([ &$this->data ], $arguments);
+			$arguments = array_merge([&$this->data], $arguments);
 
 			return call_user_func_array($function, $arguments);
 		}
@@ -132,7 +132,7 @@ 
 	 * @return $this|mixed
 	 * @throws \Exception|WrongFunctionException
 	 */
-	public function __call($method, $arguments = [ ])
+	public function __call($method, $arguments = [])
 	{
 		try
 		{
@@ -152,7 +152,7 @@ 
 	 */
 	public function __isset($key)
 	{
-		return isset( $this->data[$key] );
+		return isset($this->data[$key]);
 	}
 
 
@@ -161,7 +161,7 @@ 
 	 */
 	public function __unset($key)
 	{
-		unset( $this->data[$key] );
+		unset($this->data[$key]);
 	}
 
 
@@ -187,7 +187,7 @@ 
 	 */
 	public function keyExists($key)
 	{
-		return isset( $this->data[$key] );
+		return isset($this->data[$key]);
 	}
 
 

Braces +3 added lines, -6 removed lines

@@ -75,8 +75,7 @@ 
 		if ( array_key_exists($key, $this->data) )
 		{
 			$target =& $this->data[$key];
-		}
-		else
+		} else
 		{
 			throw new WrongArrayPathException($key);
 		}
@@ -115,8 +114,7 @@ 
 			$arguments = array_merge([ &$this->data ], $arguments);
 
 			return call_user_func_array($function, $arguments);
-		}
-		catch (\Exception $e)
+		} catch (\Exception $e)
 		{
 			throw new FunctionExecuteException($methodPath);
 		}
@@ -137,8 +135,7 @@ 
 		try
 		{
 			return $this->getHelperManager()->execute($method, $arguments, $this);
-		}
-		catch (\Exception $e)
+		} catch (\Exception $e)
 		{
 			throw $e;
 		}

src/Knot/Dict/PathOperationsTrait.php

Doc Comments +5 added lines, -2 removed lines

@@ -15,6 +15,9 @@ 
 	protected $path = '';
 
 
+	/**
+	 * @return null|AbstractDictBody
+	 */
 	abstract public function childParent();
 
 
@@ -74,7 +77,7 @@ 
 	/**
 	 * @param $path
 	 *
-	 * @return array|ChildDict|Mixed
+	 * @return callable
 	 * @throws WrongArrayPathException
 	 */
 	public function get($path)
@@ -184,7 +187,7 @@ 
 
 
 	/**
-	 * @param $rawPath
+	 * @param null|string $rawPath
 	 *
 	 * @return $this
 	 */

Spacing +21 added lines, -21 removed lines

@@ -25,7 +25,7 @@ 
 	 */
 	public function path($add = null)
 	{
-		return $add ? $this->path != null ? $this->path . self::$ARRAY_PATH_DELIMITER . $add : $add : $this->path;
+		return $add ? $this->path != null ? $this->path.self::$ARRAY_PATH_DELIMITER.$add : $add : $this->path;
 	}
 
 
@@ -81,20 +81,20 @@ 
 	{
 		$arguments = func_get_args();
 
-		if ( isset( $arguments[1] ) )
+		if (isset($arguments[1]))
 		{
 			$default_return = $arguments[1];
 		}
 
-		$target_data =& $this->data;
+		$target_data = & $this->data;
 
 		foreach (static::pathParser($path) as $way)
 		{
 
-			if ( ! isset( $target_data[$way] ) )
+			if ( ! isset($target_data[$way]))
 			{
 
-				if ( isset( $default_return ) )
+				if (isset($default_return))
 				{
 					$r = $this->set($path, $default_return);
 
@@ -107,7 +107,7 @@ 
 			$target_data = &$target_data[$way];
 		}
 
-		if ( is_array($target_data) )
+		if (is_array($target_data))
 		{
 			return new ChildDict($target_data, $this->childParent(), $path);
 		}
@@ -128,7 +128,7 @@ 
 	{
 		$arguments = func_get_args();
 
-		if ( isset( $arguments[1] ) )
+		if (isset($arguments[1]))
 		{
 			$value = $arguments[1];
 		}
@@ -139,9 +139,9 @@ 
 		}
 		catch (WrongArrayPathException $e)
 		{
-			if ( isset( $value ) )
+			if (isset($value))
 			{
-				return $this->value($value, [ $path ]);
+				return $this->value($value, [$path]);
 			}
 
 			throw $e;
@@ -157,24 +157,24 @@ 
 	 */
 	public function set($rawPath, $value)
 	{
-		$target_data =& $this->data;
+		$target_data = & $this->data;
 
 		foreach (static::pathParser($rawPath) as $path)
 		{
 			// If there is no way to go or this is not an array!
-			if ( ! isset( $target_data[$path] ) || ! is_array($target_data[$path]) )
+			if ( ! isset($target_data[$path]) || ! is_array($target_data[$path]))
 			{
-				$target_data[$path] = [ ];
+				$target_data[$path] = [];
 			}
 
-			$target_data =& $target_data[$path];
+			$target_data = & $target_data[$path];
 		}
 
-		$value = $this->value($value, [ $rawPath ]);
+		$value = $this->value($value, [$rawPath]);
 
 		$target_data = $value;
 
-		if ( is_array($target_data) )
+		if (is_array($target_data))
 		{
 			return new ChildDict($target_data, $this->childParent(), $this->path());
 		}
@@ -190,7 +190,7 @@ 
 	 */
 	public function del($rawPath)
 	{
-		$target_data =& $this->data;
+		$target_data = & $this->data;
 
 		$paths = static::pathParser($rawPath);
 
@@ -199,24 +199,24 @@ 
 		foreach ($paths as $path)
 		{
 			// If there is no way to go or this is not an array!
-			if ( ! isset( $target_data[$path] ) || ! is_array($target_data[$path]) )
+			if ( ! isset($target_data[$path]) || ! is_array($target_data[$path]))
 			{
 				return $this;
 			}
 
-			$target_data =& $target_data[$path];
+			$target_data = & $target_data[$path];
 		}
 
-		if ( isset( $target_data[$target_key] ) )
+		if (isset($target_data[$target_key]))
 		{
-			unset( $target_data[$target_key] );
+			unset($target_data[$target_key]);
 		}
 
 		return $this;
 	}
 
 
-	protected function value($value, array $arguments = [ ])
+	protected function value($value, array $arguments = [])
 	{
 		return $value instanceof \Closure ? call_user_func_array($value, $arguments) : $value;
 	}

Braces +2 added lines, -4 removed lines

@@ -63,8 +63,7 @@ 
 			$this->get($path);
 
 			return true;
-		}
-		catch (WrongArrayPathException $e)
+		} catch (WrongArrayPathException $e)
 		{
 			return false;
 		}
@@ -136,8 +135,7 @@ 
 		try
 		{
 			return $this->get($path);
-		}
-		catch (WrongArrayPathException $e)
+		} catch (WrongArrayPathException $e)
 		{
 			if ( isset( $value ) )
 			{

src/Knot/Exceptions/WrongArrayPathException.php

Spacing +1 added lines, -1 removed lines

@@ -4,6 +4,6 @@
 
 	public function __toString()
 	{
-		return __CLASS__ . ":[" . $this->code . "]: Wrong Path. Path: " . $this->message . '\n';
+		return __CLASS__.":[".$this->code."]: Wrong Path. Path: ".$this->message.'\n';
 	}
 }
\ No newline at end of file

src/Knot/Exceptions/FunctionExecuteException.php

Spacing +1 added lines, -1 removed lines

@@ -6,6 +6,6 @@
 
 	public function __toString()
 	{
-		return __CLASS__ . ":[" . $this->code . "]: Function execute error. Function: " . $this->message . '\n';
+		return __CLASS__.":[".$this->code."]: Function execute error. Function: ".$this->message.'\n';
 	}
 }
\ No newline at end of file

src/Knot/Dict/Helpers/UnderscoreHelper.php

Spacing +4 added lines, -4 removed lines

@@ -81,7 +81,7 @@ 
 
 	public function __construct()
 	{
-		if ( class_exists("__") )
+		if (class_exists("__"))
 		{
 			$this->ready = true;
 		}
@@ -97,11 +97,11 @@ 
 	public function addRoutes(HelperManager $helperManager)
 	{
 		// If underscore is not exists, don't add any routes to helper.
-		if ( $this->ready === true )
+		if ($this->ready === true)
 		{
 			foreach ($this->functions as $functionName)
 			{
-				$helperManager->addRoute($functionName, [ __CLASS__, "execute" ]);
+				$helperManager->addRoute($functionName, [__CLASS__, "execute"]);
 			}
 		}
 	}
@@ -112,7 +112,7 @@ 
 		$data = $knot->toArray();
 
 		$underscoreObject = \__($data);
-		$targetFunction   = [ $underscoreObject, $functionName ];
+		$targetFunction   = [$underscoreObject, $functionName];
 
 		return call_user_func_array($targetFunction, $arguments);
 	}

src/Knot/Dict/Helpers/PHPArrayEqualizerHelper.php

Spacing +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
 		foreach ($this->functions as $functionName)
 		{
 			$route = $this->convertPHPFunctionToRoute($functionName);
-			$helperManager->addRoute($route, [ __CLASS__, "execute" ]);
+			$helperManager->addRoute($route, [__CLASS__, "execute"]);
 		}
 	}
 
@@ -66,7 +66,7 @@ 
 	public static function execute(AbstractDictBody $knot, $arguments, $methodName)
 	{
 		$methodName = self::convertRouteToPHPFunction($methodName);
-		$data       =& $knot->toArray();
+		$data       = & $knot->toArray();
 
 		array_unshift($arguments, $data);
 		$data = call_user_func_array($methodName, $arguments);

src/Knot/Dict/Helpers/AbstractPHPArrayHelper.php

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Created by PhpStorm.
- * User: kalaomer
- * Date: 31.05.2015
- * Time: 01:16
- */
+	 * Created by PhpStorm.
+	 * User: kalaomer
+	 * Date: 31.05.2015
+	 * Time: 01:16
+	 */
 
 namespace Knot\Dict\Helpers;
 

Spacing +4 added lines, -4 removed lines

@@ -14,7 +14,7 @@ 
 	{
 		$route = ltrim($phpFunctionName, "array_");
 
-		return preg_replace_callback('/\_([a-z])/', function ($matches)
+		return preg_replace_callback('/\_([a-z])/', function($matches)
 		{
 			return strtoupper($matches[1]);
 		}, $route);
@@ -23,12 +23,12 @@ 
 
 	protected static function convertRouteToPHPFunction($route)
 	{
-		$right_side = preg_replace_callback('/([A-Z])/', function ($matches)
+		$right_side = preg_replace_callback('/([A-Z])/', function($matches)
 		{
-			return '_' . strtolower($matches[1]);
+			return '_'.strtolower($matches[1]);
 		}, $route);
 
-		return 'array_' . $right_side;
+		return 'array_'.$right_side;
 	}
 
 }
\ No newline at end of file

src/Knot/Dict/Helpers/PHPArrayChangerHelper.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
 		foreach ($this->functions as $functionName)
 		{
 			$route = $this->convertPHPFunctionToRoute($functionName);
-			$helperManager->addRoute($route, [ __CLASS__, "execute" ]);
+			$helperManager->addRoute($route, [__CLASS__, "execute"]);
 		}
 	}
 
@@ -44,8 +44,8 @@ 
 	public static function execute(AbstractDictBody $knot, $arguments, $methodName)
 	{
 		$methodName = self::convertRouteToPHPFunction($methodName);
-		$data       =& $knot->toArray();
+		$data       = & $knot->toArray();
 
-		return call_user_func_array($methodName, array_merge([ &$data ], $arguments));
+		return call_user_func_array($methodName, array_merge([&$data], $arguments));
 	}
 }
\ No newline at end of file

src/Knot/Dict/ArrayAccessTrait.php

Spacing +3 added lines, -3 removed lines

@@ -29,9 +29,9 @@ 
 	 */
 	public function &offsetGet($offset = null)
 	{
-		if ( count(func_get_args()) == 0 or is_null($offset) )
+		if (count(func_get_args()) == 0 or is_null($offset))
 		{
-			$this->data[] = [ ];
+			$this->data[] = [];
 
 			return $this->data[$this->lastKey()];
 		}
@@ -48,7 +48,7 @@ 
 	 */
 	public function offsetSet($offset, $value)
 	{
-		if ( is_null($offset) )
+		if (is_null($offset))
 		{
 			$this->data[] = $value;
 		}

Braces +1 added lines, -2 removed lines

@@ -51,8 +51,7 @@
 		if ( is_null($offset) )
 		{
 			$this->data[] = $value;
-		}
-		else
+		} else
 		{
 			$this->data[$offset] = $value;
 		}