genesis-php / genesis

commands/filesystem/Filesystem.php

Braces +2 added lines, -2 removed lines

@@ -137,10 +137,10 @@
 				if(is_file($destination)){
 					if($options['onDuplicate'] == self::ERROR){
 						$this->error("File '$destination' already exists.");
-					}elseif($options['onDuplicate'] == self::SKIP){
+					} elseif($options['onDuplicate'] == self::SKIP){
 						$this->log("File '$destination' already exists, skipping ...");
 						continue;
-					}elseif($options['onDuplicate'] == self::REWRITE){
+					} elseif($options['onDuplicate'] == self::REWRITE){
 						@unlink($destination);
 					}
 				}

Spacing +5 added lines, -5 removed lines

@@ -130,17 +130,17 @@
 				$this->log("Directory '$directory' cleaned.");
 			}
 		}
-		if(count($this->filesToCopy) > 0) {
+		if (count($this->filesToCopy) > 0) {
 			$this->log(Cli::getColoredString('Copying files', 'light_blue'));
 			$command = new File();
 			foreach ($this->filesToCopy as $destination => $options) {
-				if(is_file($destination)){
-					if($options['onDuplicate'] == self::ERROR){
+				if (is_file($destination)) {
+					if ($options['onDuplicate'] == self::ERROR) {
 						$this->error("File '$destination' already exists.");
-					}elseif($options['onDuplicate'] == self::SKIP){
+					}elseif ($options['onDuplicate'] == self::SKIP) {
 						$this->log("File '$destination' already exists, skipping ...");
 						continue;
-					}elseif($options['onDuplicate'] == self::REWRITE){
+					}elseif ($options['onDuplicate'] == self::REWRITE) {
 						$command->delete($destination);
 					}
 				}

commands/filesystem/File.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 			return;
 		}
 		$result = @unlink($file);
-		if(!$result){
+		if (!$result) {
 			$this->error("Cannot delete file '$file'.");
 		}
 	}

Bootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -151,7 +151,7 @@
 	protected function autowire(IBuild $build, Container $container)
 	{
 		foreach ($this->getAutowiredProperties($build) as $property => $service) {
-			if(!$container->hasService($service)){
+			if (!$container->hasService($service)) {
 				throw new Exception("Cannot found service '$service' to inject into " . get_class($build) . "::$property.");
 			}
 			$build->$property = $container->getService($service);

Braces +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 			$container = require_once $bootstrapFile;
 			if ($container === 1 || $container === TRUE) { // 1 = success, TRUE = already required
 				$container = NULL;
-			}elseif (!($container instanceof Container)) {
+			} elseif (!($container instanceof Container)) {
 				$this->log("Returned value from bootstrap.php must be instance of 'Genesis\\Container\\Container' or nothing (NULL).", 'red');
 				exit(255);
 			}

Doc Comments +15 added lines

@@ -99,12 +99,18 @@ 
 	}
 
 
+	/**
+	 * @param integer $code
+	 */
 	protected function terminate($code)
 	{
 		throw new TerminateException(NULL, $code);
 	}
 
 
+	/**
+	 * @param string $workingDir
+	 */
 	protected function detectBootstrapFilename($workingDir)
 	{
 		return $workingDir . DIRECTORY_SEPARATOR . 'bootstrap.php';
@@ -112,6 +118,8 @@ 
 
 
 	/**
+	 * @param string $workingDir
+	 * @param string $configFile
 	 * @return Container
 	 */
 	protected function createContainer($workingDir, $configFile, Container $bootstrapContainer = NULL)
@@ -165,6 +173,9 @@ 
 	}
 
 
+	/**
+	 * @param IBuild $class
+	 */
 	protected function getAutowiredProperties($class)
 	{
 		$return = [];
@@ -180,6 +191,10 @@ 
 	}
 
 
+	/**
+	 * @param string $color
+	 * @param string $backgroundColor
+	 */
 	protected function log($message, $color = NULL, $backgroundColor = NULL)
 	{
 		echo Cli::getColoredString($message . PHP_EOL, $color, $backgroundColor);

config/ContainerFactory.php

Spacing +18 added lines, -18 removed lines

@@ -89,34 +89,34 @@ 
 
 		// BC break check
 		$mainSections = ['includes', 'class', 'parameters', 'services'];
-		foreach($config as $key => $val){
-			if(!in_array($key, $mainSections)) {
+		foreach ($config as $key => $val) {
+			if (!in_array($key, $mainSections)) {
 				throw new NotSupportedException("Since version 2.0 are supported main only these sections: " . implode(", ", $mainSections) . ". Section '$key' found. Move your variables into parameters section.");
 			}
 		}
 
 		$container = new Container();
 		$container->setClass($config['class']);
-		if(isset($config['parameters'])){
+		if (isset($config['parameters'])) {
 			$container->setParameters($config['parameters']);
 		}
-		if(isset($config['services'])){
-			foreach($config['services'] as $name => $config){
-				if(!is_array($config)){
+		if (isset($config['services'])) {
+			foreach ($config['services'] as $name => $config) {
+				if (!is_array($config)) {
 					$container->addService($name, $config); // is directly service object from merged container
 					continue;
 				}
 				$class = $config['class'];
 				$arguments = [];
-				if($config['class'] instanceof \Nette\Neon\Entity){
+				if ($config['class'] instanceof \Nette\Neon\Entity) {
 					$class = $config['class']->value;
 					$arguments = $config['class']->attributes;
 				}
 				$reflectionClass = new \ReflectionClass($class);
 				$service = $reflectionClass->newInstanceArgs($arguments);
-				if(isset($config['setup'])){
-					foreach($config['setup'] as $neonEntity){
-						if(!method_exists($service, $neonEntity->value)){
+				if (isset($config['setup'])) {
+					foreach ($config['setup'] as $neonEntity) {
+						if (!method_exists($service, $neonEntity->value)) {
 							throw new ContainerFactoryException("Class $class does not have method $neonEntity->value().");
 						}
 						call_user_func_array(array($service, $neonEntity->value), $neonEntity->attributes);
@@ -177,18 +177,18 @@ 
 	{
 		$config = $this->resolveUnmergables($config);
 		foreach ($config as $key => $value) {
-			if($value instanceof \Nette\Neon\Entity){
+			if ($value instanceof \Nette\Neon\Entity) {
 				$value->value = $this->parseValue($value->value, $allConfig);
-				foreach($value->attributes as $k => $v){
-					if(is_array($v)){
+				foreach ($value->attributes as $k => $v) {
+					if (is_array($v)) {
 						$value->attributes[$k] = $this->parseValues($v, $allConfig, array_merge($keysPath, [$key]));
-					} else{
+					} else {
 						$value->attributes[$k] = $this->parseValue($v, $allConfig);
 					}
 				}
 			} elseif (is_array($value)) {
 				$value = $this->parseValues($value, $allConfig, array_merge($keysPath, [$key]));
-			} elseif(!is_object($value)) {
+			} elseif (!is_object($value)) {
 				$value = $this->parseValue($value, $allConfig);
 			}
 
@@ -215,14 +215,14 @@ 
 		if (preg_match_all('#%([^%]+)%#', $value, $matches)) {
 			foreach ($matches[1] as $match) {
 				$parameter = $config['parameters'];
-				foreach(explode(".", $match) as $m){
+				foreach (explode(".", $match) as $m) {
 					if (!array_key_exists($m, $parameter)) {
 						throw new ContainerFactoryException("Cannot find variable '$match', part '$m'.");
 					}
 					$parameter = $parameter[$m];
 				}
-				if(is_array($parameter)){
-					if("%$match%" !== $value){ // if is variable value an array, must not be part of a string
+				if (is_array($parameter)) {
+					if ("%$match%" !== $value) { // if is variable value an array, must not be part of a string
 						throw new ContainerFactoryException("Array value cannot be part of a string.");
 					}
 					return $parameter;

commands/Help.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
 	 */
 	public function setSectionTasks($section, array $tasks)
 	{
-		if(!isset($this->sections[$section])){
+		if (!isset($this->sections[$section])) {
 			throw new InvalidArgumentException("Section '$section' not found.");
 		}
 		$this->sections[$section]['tasks'] = $tasks;

commands/Git.php

Doc Comments +4 added lines, -2 removed lines

@@ -62,7 +62,6 @@ 
 
 	/**
 	 * Sets path to working directory
-	 * @param string $workingTree
 	 */
 	public function setWorkingDirectory($workingDirectory)
 	{
@@ -100,7 +99,7 @@ 
 
 	/**
 	 * Sets git command (to be executed later)
-	 * @param mixed $command
+	 * @param string $command
 	 */
 	public function setCommand($command)
 	{
@@ -139,6 +138,9 @@ 
 	}
 
 
+	/**
+	 * @param string $command
+	 */
 	private function exec($command)
 	{
 		$exec = new Exec();

Spacing +2 added lines, -2 removed lines

@@ -127,12 +127,12 @@
 	public function execute()
 	{
 		$command = '';
-		if($this->workingDirectory !== NULL){
+		if ($this->workingDirectory !== NULL) {
 			$command .= 'cd ' . escapeshellarg($this->workingDirectory) . ' && ';
 		}
 		$command .= escapeshellarg($this->gitExecutable);
 		$execCommand = $command . ' ' . $this->command;
-		if($this->redirectStderrToStdout){
+		if ($this->redirectStderrToStdout) {
 			$execCommand .= ' 2>&1';
 		}
 		return $this->exec($execCommand);