keeko / tools

src/command/GenerateApiCommand.php

Unused Use Statements -6 removed lines

@@ -5,13 +5,7 @@
 use keeko\tools\utils\NameUtils;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Command\Command;
 use Propel\Generator\Model\Table;
-use Propel\Generator\Model\Database;
-use phootwork\json\Json;
-use phootwork\json\JsonException;
-use keeko\tools\exceptions\JsonEmptyException;
-use Symfony\Component\Filesystem\Filesystem;
 
 class GenerateApiCommand extends AbstractGenerateCommand {
 

Switch Indentation +52 added lines, -52 removed lines

@@ -158,20 +158,20 @@ 
 		
 		// find path branch
 		switch ($type) {
-			case 'list':
-			case 'create':
-				$endpoint = '/' . $modelPlural;
-				break;
+		case 'list':
+		case 'create':
+			$endpoint = '/' . $modelPlural;
+			break;
 
-			case 'read':
-			case 'update':
-			case 'delete':
-				$endpoint = '/' . $modelPlural . '/{id}';
-				break;
+		case 'read':
+		case 'update':
+		case 'delete':
+			$endpoint = '/' . $modelPlural . '/{id}';
+			break;
 
-			default:
-				throw new \RuntimeException('type (%s) not found, can\'t continue.');
-				break;
+		default:
+			throw new \RuntimeException('type (%s) not found, can\'t continue.');
+			break;
 		}
 
 		$path = isset($paths[$endpoint]) ? $paths[$endpoint] : [];
@@ -185,58 +185,58 @@ 
 		$responses = isset($operation['responses']) ? $operation['responses'] : [];
 		
 		switch ($type) {
-			case 'list':
-				$ok = isset($responses['200']) ? $responses['200'] : [];
-				$ok['description'] = sprintf('Array of %s', $modelPlural);
-				$ok['schema'] = ['$ref' => '#/definitions/' . 'Paged' . NameUtils::pluralize($modelObject)];
+		case 'list':
+			$ok = isset($responses['200']) ? $responses['200'] : [];
+			$ok['description'] = sprintf('Array of %s', $modelPlural);
+			$ok['schema'] = ['$ref' => '#/definitions/' . 'Paged' . NameUtils::pluralize($modelObject)];
 				
-				$responses['200'] = $ok;
-				break;
+			$responses['200'] = $ok;
+			break;
 				
-			case 'create':
-				// params
-				list($paramIndex, $param) = $this->findParam($params, 'body');
+		case 'create':
+			// params
+			list($paramIndex, $param) = $this->findParam($params, 'body');
 				
-				$param['name'] = 'body';
-				$param['in'] = 'body';
-				$param['description'] = sprintf('The new %s', $modelName);
-				$param['required'] = true;
-				$param['schema'] = ['$ref' => '#/definitions/Writable' . $modelObject];
+			$param['name'] = 'body';
+			$param['in'] = 'body';
+			$param['description'] = sprintf('The new %s', $modelName);
+			$param['required'] = true;
+			$param['schema'] = ['$ref' => '#/definitions/Writable' . $modelObject];
 				
-				$params = $this->updateArray($params, $paramIndex, $param);
+			$params = $this->updateArray($params, $paramIndex, $param);
 				
-				// response
-				$ok = isset($responses['201']) ? $responses['201'] : [];
-				$ok['description'] = sprintf('%s created', $modelName);
+			// response
+			$ok = isset($responses['201']) ? $responses['201'] : [];
+			$ok['description'] = sprintf('%s created', $modelName);
 				
-				$responses['201'] = $ok;
-				break;
+			$responses['201'] = $ok;
+			break;
 				
-			case 'read':
-				// response
-				$ok = isset($responses['200']) ? $responses['200'] : [];
-				$ok['description'] = sprintf('gets the %s', $modelName);
-				$ok['schema'] = ['$ref' => '#/definitions/' . $modelObject];
+		case 'read':
+			// response
+			$ok = isset($responses['200']) ? $responses['200'] : [];
+			$ok['description'] = sprintf('gets the %s', $modelName);
+			$ok['schema'] = ['$ref' => '#/definitions/' . $modelObject];
 				
-				$responses['200'] = $ok;
-				break;
+			$responses['200'] = $ok;
+			break;
 				
-			case 'update':
-				// response
-				$ok = isset($responses['200']) ? $responses['200'] : [];
-				$ok['description'] = sprintf('%s updated', $modelName);
-				$ok['schema'] = ['$ref' => '#/definitions/' . $modelObject];
+		case 'update':
+			// response
+			$ok = isset($responses['200']) ? $responses['200'] : [];
+			$ok['description'] = sprintf('%s updated', $modelName);
+			$ok['schema'] = ['$ref' => '#/definitions/' . $modelObject];
 				
-				$responses['200'] = $ok;
-				break;
+			$responses['200'] = $ok;
+			break;
 				
-			case 'delete':
-				// response
-				$ok = isset($responses['204']) ? $responses['204'] : [];
-				$ok['description'] = sprintf('%s deleted', $modelName);
+		case 'delete':
+			// response
+			$ok = isset($responses['204']) ? $responses['204'] : [];
+			$ok['description'] = sprintf('%s deleted', $modelName);
 				
-				$responses['204'] = $ok;
-				break;
+			$responses['204'] = $ok;
+			break;
 		}
 		
 		if ($type == 'read' || $type == 'update' || $type == 'delete') {

src/command/MagicCommand.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 use keeko\tools\command\AbstractGenerateCommand;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\ArrayInput;
 
 class MagicCommand extends AbstractGenerateCommand {

src/KeekoTools.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 		$this->setDispatcher($dispatcher);
 		$this->keekoDispatcher = $dispatcher;
 
-		$dispatcher->addListener(ConsoleEvents::TERMINATE, function (ConsoleTerminateEvent $event) {
+		$dispatcher->addListener(ConsoleEvents::TERMINATE, function(ConsoleTerminateEvent $event) {
 			$command = $event->getCommand();
 			$this->finishedCommands[] = $command->getName();
 		});

src/command/GenerateActionCommand.php

Switch Indentation +7 added lines, -7 removed lines

@@ -216,14 +216,14 @@
 
 	private function getActionTitle($model, $type) {
 		switch ($type) {
-			case 'list':
-				return 'List all ' . NameUtils::pluralize($model);
+		case 'list':
+			return 'List all ' . NameUtils::pluralize($model);
 
-			case 'create':
-			case 'read':
-			case 'update':
-			case 'delete':
-				return ucfirst($type) . 's ' . (in_array($model[0], ['a', 'e', 'i', 'o', 'u']) ? 'an' : 'a') . ' ' . $model;
+		case 'create':
+		case 'read':
+		case 'update':
+		case 'delete':
+			return ucfirst($type) . 's ' . (in_array($model[0], ['a', 'e', 'i', 'o', 'u']) ? 'an' : 'a') . ' ' . $model;
 		}
 	}
 

src/command/InitCommand.php

Switch Indentation +6 added lines, -6 removed lines

@@ -358,13 +358,13 @@
 		$type = $this->getPackageType();
 
 		switch ($type) {
-			case 'app':
-				$this->handleAppClass($class);
-				break;
+		case 'app':
+			$this->handleAppClass($class);
+			break;
 				
-			case 'module':
-				$this->handleModuleClass($class);
-				break;
+		case 'module':
+			$this->handleModuleClass($class);
+			break;
 		}
 
 		$this->codegenService->dumpStruct($class, true);

Spacing +2 added lines, -2 removed lines

@@ -174,7 +174,7 @@ 
 			$types = ['module', 'app'];
 			$question = new Question('Package type (module|app)', $type);
 			$question->setAutocompleterValues($types);
-			$question->setValidator(function ($answer) use ($types) {
+			$question->setValidator(function($answer) use ($types) {
 				if (!in_array($answer, $types)) {
 					throw new \RuntimeException('The name of the type should be one of: ' . 
 							implode(',', $types));
@@ -553,7 +553,7 @@ 
 	private function validateName($name) {
 		if (!preg_match('{^[a-z0-9_.-]+/[a-z0-9_.-]+$}', $name)) {
 			throw new \InvalidArgumentException(
-				'The package name '.$name.' is invalid, it should be lowercase and have a vendor name, a forward slash, and a package name, matching: [a-z0-9_.-]+/[a-z0-9_.-]+'
+				'The package name ' . $name . ' is invalid, it should be lowercase and have a vendor name, a forward slash, and a package name, matching: [a-z0-9_.-]+/[a-z0-9_.-]+'
 			);
 		}
 	}

src/generator/GeneratorFactory.php

Switch Indentation +20 added lines, -20 removed lines

@@ -13,20 +13,20 @@ 
 	 */
 	public static function createActionTraitGenerator($type, CommandService $service) {
 		switch ($type) {
-			case 'list':
-				return new ListActionTraitGenerator($service);
+		case 'list':
+			return new ListActionTraitGenerator($service);
 				
-			case 'create':
-				return new CreateActionTraitGenerator($service);
+		case 'create':
+			return new CreateActionTraitGenerator($service);
 				
-			case 'update':
-				return new UpdateActionTraitGenerator($service);
+		case 'update':
+			return new UpdateActionTraitGenerator($service);
 				
-			case 'read':
-				return new ReadActionTraitGenerator($service);
+		case 'read':
+			return new ReadActionTraitGenerator($service);
 				
-			case 'delete':
-				return new DeleteActionTraitGenerator($service);
+		case 'delete':
+			return new DeleteActionTraitGenerator($service);
 		}
 	}
 	
@@ -38,20 +38,20 @@ 
 	 */
 	public static function createJsonResponseGenerator($type, CommandService $service) {
 		switch ($type) {
-			case 'list':
-				return new ListJsonResponseGenerator($service);
+		case 'list':
+			return new ListJsonResponseGenerator($service);
 		
-			case 'create':
-				return new CreateJsonResponseGenerator($service);
+		case 'create':
+			return new CreateJsonResponseGenerator($service);
 		
-			case 'update':
-				return new UpdateJsonResponseGenerator($service);
+		case 'update':
+			return new UpdateJsonResponseGenerator($service);
 		
-			case 'read':
-				return new ReadJsonResponseGenerator($service);
+		case 'read':
+			return new ReadJsonResponseGenerator($service);
 		
-			case 'delete':
-				return new DeleteJsonResponseGenerator($service);
+		case 'delete':
+			return new DeleteJsonResponseGenerator($service);
 		}
 	}
 

src/helpers/QuestionHelperTrait.php

Spacing +1 added lines, -2 removed lines

@@ -24,8 +24,7 @@
 	 */
 	protected function getQuestion($question, $default = null, $sep = ':') {
 		return !empty($default) ?
-			sprintf('<info>%s</info> [<comment>%s</comment>]%s ', $question, $default, $sep) :
-			sprintf('<info>%s</info>%s ', $question, $sep);
+			sprintf('<info>%s</info> [<comment>%s</comment>]%s ', $question, $default, $sep) : sprintf('<info>%s</info>%s ', $question, $sep);
 	}
 	
 	/**

src/services/CodeGeneratorService.php

Switch Indentation +8 added lines, -8 removed lines

@@ -153,14 +153,14 @@
 		// iterate over behaviors to get their respective columns
 		foreach ($table->getBehaviors() as $behavior) {
 			switch ($behavior->getName()) {
-				case 'timestampable':
-					$fields[] = $behavior->getParameter('create_column');
-					$fields[] = $behavior->getParameter('update_column');
-					break;
-	
-				case 'aggregate_column':
-					$fields[] = $behavior->getParameter('name');
-					break;
+			case 'timestampable':
+				$fields[] = $behavior->getParameter('create_column');
+				$fields[] = $behavior->getParameter('update_column');
+				break;
+	
+			case 'aggregate_column':
+				$fields[] = $behavior->getParameter('name');
+				break;
 			}
 		}
 	

Spacing +1 added lines, -1 removed lines

@@ -105,7 +105,7 @@
 				$fields .= sprintf("'%s'", $prop);
 	
 				if (isset($conversions[$prop])) {
-					$fields .= ' => function($v) {'."\n\t".'return ' . $conversions[$prop] . ';'."\n".'}';
+					$fields .= ' => function($v) {' . "\n\t" . 'return ' . $conversions[$prop] . ';' . "\n" . '}';
 				}
 	
 				$fields .= ', ';

src/utils/NameUtils.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 	public static function toStudlyCase($input) {
 		$input = trim($input, '-_');
 		return ucfirst(preg_replace_callback('/([A-Z-_][a-z]+)/', function($matches) {
-			return ucfirst(str_replace(['-','_'], '',$matches[0]));
+			return ucfirst(str_replace(['-', '_'], '', $matches[0]));
 		}, $input));
 	}