Maslosoft / Whitelist

debug-test.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
  * Codeception CLI
  */
 
-require_once dirname(__FILE__).'/vendor/autoload.php';
+require_once dirname(__FILE__) . '/vendor/autoload.php';
 
 use Symfony\Component\Console\Application;
 

src/Whitelist.php

Spacing +8 added lines, -8 removed lines

@@ -98,9 +98,9 @@ 
 
 		// Instantiate checkers if not already provided as instances
 		$checkers = [];
-		foreach($this->checkers as $checker)
+		foreach ($this->checkers as $checker)
 		{
-			if(!$checker instanceof CheckerInterface)
+			if (!$checker instanceof CheckerInterface)
 			{
 				assert(is_string($checker) || is_array($checker));
 				$checkers[] = EmbeDi::fly()->apply($checker);
@@ -109,11 +109,11 @@ 
 		$this->checkers = $checkers;
 
 		// Ensure that all aspects have sane values
-		foreach(['whitelist', 'require'] as $name)
+		foreach (['whitelist', 'require'] as $name)
 		{
-			foreach($this->aspects as $aspect)
+			foreach ($this->aspects as $aspect)
 			{
-				if(empty($this->$name[$aspect]) || !is_array($this->$name[$aspect]))
+				if (empty($this->$name[$aspect]) || !is_array($this->$name[$aspect]))
 				{
 					$this->$name[$aspect] = [];
 				}
@@ -125,17 +125,17 @@ 
 	{
 		$tokenizer = new Tokenizer($code);
 
-		if(empty($ec))
+		if (empty($ec))
 		{
 			$ec = new ErrorCollector;
 		}
 
 		$results = [];
 
-		foreach($this->checkers as $checker)
+		foreach ($this->checkers as $checker)
 		{
 			assert($checker instanceof CheckerInterface);
-			if($checker instanceof LoggerAwareInterface)
+			if ($checker instanceof LoggerAwareInterface)
 			{
 				$checker->setLogger($this->logger);
 			}

Braces +1 added lines, -2 removed lines

@@ -90,8 +90,7 @@
 		if (is_string($config))
 		{
 			EmbeDi::fly($config)->configure($this);
-		}
-		else
+		} else
 		{
 			EmbeDi::fly()->apply($config, $this);
 		}

src/Helpers/ListNormalizer.php

Spacing +3 added lines, -3 removed lines

@@ -23,14 +23,14 @@
 	public static function normalize(array $list)
 	{
 		$proper = [];
-		foreach($list as $key => $value)
+		foreach ($list as $key => $value)
 		{
-			if(is_string($key) && is_bool($value))
+			if (is_string($key) && is_bool($value))
 			{
 				$key = str_replace(['(', ')'], '', $key);
 				$proper[$key] = $value;
 			}
-			elseif(is_numeric($key) && is_string($value))
+			elseif (is_numeric($key) && is_string($value))
 			{
 				$value = str_replace(['(', ')'], '', $value);
 				$proper[$value] = true;

Braces +2 added lines, -4 removed lines

@@ -29,13 +29,11 @@
 			{
 				$key = str_replace(['(', ')'], '', $key);
 				$proper[$key] = $value;
-			}
-			elseif(is_numeric($key) && is_string($value))
+			} elseif(is_numeric($key) && is_string($value))
 			{
 				$value = str_replace(['(', ')'], '', $value);
 				$proper[$value] = true;
-			}
-			else
+			} else
 			{
 				assert(false, "Unsupported list");
 			}

src/Tokenizer/Collectors/StaticMethodCallsCollector.php

Spacing +5 added lines, -5 removed lines

@@ -22,7 +22,7 @@ 
 		foreach ($tokens as $index => $token)
 		{
 			// Skip all except double colon
-			if($token->not(T_DOUBLE_COLON))
+			if ($token->not(T_DOUBLE_COLON))
 			{
 				continue;
 			}
@@ -30,13 +30,13 @@ 
 			$next = $token->next();
 
 			// Should start with class name
-			if($prev->not(T_STRING))
+			if ($prev->not(T_STRING))
 			{
 				continue;
 			}
 
 			// Should have method name
-			if($next->not(T_STRING))
+			if ($next->not(T_STRING))
 			{
 				continue;
 			}
@@ -44,7 +44,7 @@ 
 			$finishing = $next->next();
 
 			// Should have opening bracket after method name
-			if($finishing->value !== '(')
+			if ($finishing->value !== '(')
 			{
 				continue;
 			}
@@ -55,7 +55,7 @@ 
 			$methodName = $next->value;
 			$name = sprintf('%s::%s', $className, $methodName);
 
-			$result[] = new StaticMethod($name,$tokens, $index);
+			$result[] = new StaticMethod($name, $tokens, $index);
 		}
 		return $result;
 	}

src/Tokenizer/Collectors/FunctionCallsCollector.php

Spacing +4 added lines, -4 removed lines

@@ -53,9 +53,9 @@ 
 				T_DO
 			];
 
-			foreach($toCheck as $type)
+			foreach ($toCheck as $type)
 			{
-				if($token->is($type))
+				if ($token->is($type))
 				{
 					$result[] = $token;
 					continue 2;
@@ -83,13 +83,13 @@ 
 			$prev = $token->prev();
 
 			// Method call
-			if($prev->is(T_OBJECT_OPERATOR))
+			if ($prev->is(T_OBJECT_OPERATOR))
 			{
 				continue;
 			}
 
 			// Static method call
-			if($prev->is(T_DOUBLE_COLON))
+			if ($prev->is(T_DOUBLE_COLON))
 			{
 				continue;
 			}

src/Tokenizer/Collectors/MethodCallsCollector.php

Spacing +15 added lines, -15 removed lines

@@ -28,7 +28,7 @@ 
 		foreach ($tokens as $index => $token)
 		{
 			// Skip all except object operator
-			if($token->not(T_OBJECT_OPERATOR))
+			if ($token->not(T_OBJECT_OPERATOR))
 			{
 				continue;
 			}
@@ -40,13 +40,13 @@ 
 			// $object->call()
 			// (new Object)->call()
 			// $objects[1]->call()
-			if($prev->not(T_VARIABLE) xor $prev->value !== ')' xor $prev->value !== ']')
+			if ($prev->not(T_VARIABLE) xor $prev->value !== ')' xor $prev->value !== ']')
 			{
 				continue;
 			}
 
 			// Should have method name
-			if($next->not(T_STRING))
+			if ($next->not(T_STRING))
 			{
 				continue;
 			}
@@ -54,7 +54,7 @@ 
 			$finishing = $next->next();
 
 			// Should have opening bracket after method name
-			if($finishing->value !== '(')
+			if ($finishing->value !== '(')
 			{
 				continue;
 			}
@@ -73,18 +73,18 @@ 
 			// myFunc()->method()
 			// This does not (yet?) support nested calls like:
 			// EG.: Yii::app()->getClientScript()->registerScript()
-			if(implode('', $className) === '()')
+			if (implode('', $className) === '()')
 			{
 				$function = $prev->prev()->prev();
 				array_unshift($className, $function->value);
 				$calledFrom = $function->prev();
-				if($calledFrom->is(T_DOUBLE_COLON) || $calledFrom->is(T_OBJECT_OPERATOR))
+				if ($calledFrom->is(T_DOUBLE_COLON) || $calledFrom->is(T_OBJECT_OPERATOR))
 				{
 					array_unshift($className, $calledFrom->value);
 					$parent = $calledFrom->prev();
-					if($parent->is(T_STRING) || $parent->is(T_VARIABLE))
+					if ($parent->is(T_STRING) || $parent->is(T_VARIABLE))
 					{
-						array_unshift($className,$parent->value);
+						array_unshift($className, $parent->value);
 					}
 				}
 			}
@@ -92,7 +92,7 @@ 
 			$methodName = $next->value;
 			$name = sprintf('%s->%s', implode($className), $methodName);
 
-			$result[] = new StaticMethod($name,$tokens, $index);
+			$result[] = new StaticMethod($name, $tokens, $index);
 		}
 		return $result;
 	}
@@ -100,33 +100,33 @@ 
 	private function resolve(&$className, TokenInterface $prev, $opened, $until)
 	{
 		$opening = null;
-		if($prev->value === $until)
+		if ($prev->value === $until)
 		{
 			$part = $prev->value;
 			$opening = $prev;
-			while($part !== $opened)
+			while ($part !== $opened)
 			{
 				$opening = $opening->prev();
-				if(empty($opening))
+				if (empty($opening))
 				{
 					break;
 				}
 				$part = $opening->value;
 
 				// Ensure space after `new`
-				if($opening->is(T_NEW))
+				if ($opening->is(T_NEW))
 				{
 					$part = "$part ";
 				}
 				array_unshift($className, $part);
 			}
 		}
-		if(empty($opening))
+		if (empty($opening))
 		{
 			return;
 		}
 		$preOpening = $opening->prev();
-		if($preOpening->is(T_VARIABLE))
+		if ($preOpening->is(T_VARIABLE))
 		{
 			array_unshift($className, $preOpening->value);
 		}

src/Tokenizer/Tokenizer.php

Braces +1 added lines, -2 removed lines

@@ -40,8 +40,7 @@
 			if (is_array($token))
 			{
 				$this->tokens[$index] = new Token($token, $this->tokens, $index);
-			}
-			else
+			} else
 			{
 				$this->tokens[$index] = new SimpleToken($token, $this->tokens, $index);
 			}