keeko / tools

src/services/CodeGeneratorService.php

Switch Indentation +7 added lines, -7 removed lines

@@ -178,14 +178,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 'timestampable':
+				$fields[] = $behavior->getParameter('create_column');
+				$fields[] = $behavior->getParameter('update_column');
+				break;
 	
-				case 'aggregate_column':
-					$fields[] = $behavior->getParameter('name');
-					break;
+			case 'aggregate_column':
+				$fields[] = $behavior->getParameter('name');
+				break;
 			}
 		}
 	

src/command/GenerateSerializerCommand.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use keeko\tools\generator\serializer\base\ModelSerializerTraitGenerator;
 use keeko\tools\generator\serializer\SkeletonSerializerGenerator;
 use keeko\tools\helpers\QuestionHelperTrait;
-use phootwork\file\File;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;

Spacing +5 added lines, -5 removed lines

@@ -49,7 +49,7 @@ 
 	protected function initialize(InputInterface $input, OutputInterface $output) {
 		parent::initialize($input, $output);
 
-		$loader = new \Twig_Loader_Filesystem($this->service->getConfig()->getTemplateRoot() . '/serializer');
+		$loader = new \Twig_Loader_Filesystem($this->service->getConfig()->getTemplateRoot().'/serializer');
 		$this->twig = new \Twig_Environment($loader);
 	}
 
@@ -129,7 +129,7 @@ 
 	}
 
 	private function generateModel($modelName) {
-		$this->logger->info('Generate Serializer from Model: ' . $modelName);
+		$this->logger->info('Generate Serializer from Model: '.$modelName);
 		$model = $this->modelService->getModel($modelName);
 		
 		// generate class
@@ -143,7 +143,7 @@ 
 		$this->codegenService->dumpStruct($trait, true);
 		
 		// add serializer + ApiModelInterface on the model
-		$class = new PhpClass(str_replace('\\\\', '\\', $model->getNamespace() . '\\' . $model->getPhpName()));
+		$class = new PhpClass(str_replace('\\\\', '\\', $model->getNamespace().'\\'.$model->getPhpName()));
 		$file = $this->codegenService->getFile($class);
 		if ($file->exists()) {
 			$class = PhpClass::fromFile($this->codegenService->getFilename($class));
@@ -173,10 +173,10 @@ 
 	 * @param string $name
 	 */
 	private function generateSkeleton($name) {
-		$this->logger->info('Generate Skeleton Serializer: ' . $name);
+		$this->logger->info('Generate Skeleton Serializer: '.$name);
 		$input = $this->io->getInput();
 
-		$className = NamespaceResolver::getNamespace('src/serializer', $this->package) . '\\' . $name;
+		$className = NamespaceResolver::getNamespace('src/serializer', $this->package).'\\'.$name;
 		
 		if (!Text::create($className)->endsWith('Serializer')) {
 			$className .= 'Serializer';

src/generator/domain/SkeletonDomainGenerator.php

Doc Comments +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 	/**
 	 * Add default blank methods
 	 * 
-	 * @param string $classname
+	 * @param string $className
 	 */
 	public function generate($className) {
 		$class = $this->generateClass($className);

src/generator/responder/ApiJsonResponderGenerator.php

Doc Comments +3 added lines

@@ -8,6 +8,9 @@
 
 	protected $serializer;
 
+	/**
+	 * @param string $serializer
+	 */
 	public function setSerializer($serializer) {
 		$this->serializer = $serializer;
 	}

src/generator/serializer/ModelSerializerGenerator.php

Spacing +5 added lines, -5 removed lines

@@ -27,13 +27,13 @@
 	}
 	
 	protected function getClassName(Table $model) {
-		return str_replace('model', 'serializer', $model->getNamespace()) . 
-			'\\' . $model->getPhpName() . 'Serializer';
+		return str_replace('model', 'serializer', $model->getNamespace()). 
+			'\\'.$model->getPhpName().'Serializer';
 	}
 
 	protected function ensureDomainTrait(PhpClass $class, Table $model) {
-		$class->addUseStatement(str_replace('model', 'serializer\\base', $model->getNamespace()) . 
-			'\\' . $model->getPhpName() . 'SerializerTrait');
-		$class->addTrait($model->getPhpName() . 'SerializerTrait');
+		$class->addUseStatement(str_replace('model', 'serializer\\base', $model->getNamespace()). 
+			'\\'.$model->getPhpName().'SerializerTrait');
+		$class->addTrait($model->getPhpName().'SerializerTrait');
 	}
 }
\ No newline at end of file

src/command/GenerateActionCommand.php

Spacing +10 added lines, -10 removed lines

@@ -190,7 +190,7 @@ 
 	}
 
 	private function generateModel($modelName) {
-		$this->logger->info('Generate Action from Model: ' . $modelName);
+		$this->logger->info('Generate Action from Model: '.$modelName);
 		$input = $this->io->getInput();
 		$model = $this->modelService->getModel($modelName);
 
@@ -209,7 +209,7 @@ 
 		foreach ($types as $type) {
 			$input->setOption('acl', ['admin']);
 			$input->setOption('type', $type);
-			$actionName = $modelName . '-' . $type;
+			$actionName = $modelName.'-'.$type;
 			
 			if ($model->isReadOnly() && in_array($type, ['create', 'update', 'delete'])) {
 				$this->logger->info(sprintf('Skip generate Action (%s), because Model (%s) is read-only', $actionName, $modelName));
@@ -252,13 +252,13 @@ 
 		$name = NameUtils::dasherize($modelName);
 		switch ($type) {
 			case 'list':
-				return 'List all ' . NameUtils::pluralize($name);
+				return 'List all '.NameUtils::pluralize($name);
 
 			case 'create':
 			case 'read':
 			case 'update':
 			case 'delete':
-				return ucfirst($type) . 's ' . (in_array($name[0], ['a', 'e', 'i', 'o', 'u']) ? 'an' : 'a') . ' ' . $name;
+				return ucfirst($type).'s '.(in_array($name[0], ['a', 'e', 'i', 'o', 'u']) ? 'an' : 'a').' '.$name;
 		}
 	}
 
@@ -290,7 +290,7 @@ 
 	 * @param string $actionName
 	 */
 	private function generateSkeleton($actionName) {
-		$this->logger->info('Generate Skeleton Action: ' . $actionName);
+		$this->logger->info('Generate Skeleton Action: '.$actionName);
 		$input = $this->io->getInput();
 		
 		// generate action
@@ -349,7 +349,7 @@ 
 	
 	private function guessClassname($name) {
 		$namespace = NamespaceResolver::getNamespace('src/action', $this->package);
-		return $namespace . '\\' . NameUtils::toStudlyCase($name) . 'Action';
+		return $namespace.'\\'.NameUtils::toStudlyCase($name).'Action';
 	}
 	
 	/**
@@ -413,10 +413,10 @@ 
 		foreach (array_keys($generators) as $type) {
 			// generate fqcn
 			$className = sprintf('%s%s%sAction', $model->getPhpName(), $fkModelName, ucfirst($type));
-			$fqcn = $this->packageService->getNamespace() . '\\action\\' . $className;
+			$fqcn = $this->packageService->getNamespace().'\\action\\'.$className;
 
 			// generate action
-			$action = new ActionSchema($actionNamePrefix . '-' . $type);
+			$action = new ActionSchema($actionNamePrefix.'-'.$type);
 			$action->addAcl('admin');
 			$action->setClass($fqcn);
 			$action->setTitle(str_replace(
@@ -454,10 +454,10 @@ 
 		foreach (array_keys($generators) as $type) {
 			// generate fqcn
 			$className = sprintf('%s%s%sAction', $model->getPhpName(), $fkModelName, ucfirst($type));
-			$fqcn = $this->packageService->getNamespace() . '\\action\\' . $className;
+			$fqcn = $this->packageService->getNamespace().'\\action\\'.$className;
 	
 			// generate action
-			$action = new ActionSchema($actionNamePrefix . '-' . $type);
+			$action = new ActionSchema($actionNamePrefix.'-'.$type);
 			$action->addAcl('admin');
 			$action->setClass($fqcn);
 			$action->setTitle(str_replace(

src/command/GenerateResponseCommand.php

Spacing +13 added lines, -13 removed lines

@@ -149,7 +149,7 @@ 
 				
 				// aks for serializer
 				if ($format == 'json' && $template == 'api') {
-					$guessedSerializer = NameUtils::toStudlyCase($name) . 'Serializer';
+					$guessedSerializer = NameUtils::toStudlyCase($name).'Serializer';
 					$serializerQuestion = new Question('Which format', $guessedSerializer);
 					$serializer = $this->askQuestion($serializerQuestion);
 					$input->setOption('serializer', $serializer);
@@ -190,7 +190,7 @@ 
 	
 		// generate responders for crud actions
 		foreach ($types as $type) {
-			$actionName = $modelName . '-' . $type;
+			$actionName = $modelName.'-'.$type;
 	
 			$this->generateResponder($actionName);
 		}
@@ -206,14 +206,14 @@ 
 				$fk = $relationship['fk'];
 				$foreignName = $fk->getForeignTable()->getOriginCommonName();
 				foreach ($types[$relationship['type']] as $type) {
-					$this->generateResponder($modelName . '-to-' . $foreignName . '-relationship-' . $type);
+					$this->generateResponder($modelName.'-to-'.$foreignName.'-relationship-'.$type);
 				}
 			}
 		}
 	}
 	
 	protected function generateResponder($actionName) {
-		$this->logger->info('Generate Responder for: ' . $actionName);
+		$this->logger->info('Generate Responder for: '.$actionName);
 		$module = $this->packageService->getModule();
 		
 		if (!$module->hasAction($actionName)) {
@@ -234,7 +234,7 @@ 
 
 		if (!$action->hasResponse($format)) {
 			$className = str_replace('action', 'responder', $action->getClass());
-			$className = preg_replace('/Action$/', ucwords($format) . 'Responder', $className);
+			$className = preg_replace('/Action$/', ucwords($format).'Responder', $className);
 			$action->setResponse($format, $className);
 		}
 
@@ -301,13 +301,13 @@ 
 			$foreign->getPhpName()
 		);
 		
-		$many = $module->hasAction($prefix . '-read')
-			&& $module->hasAction($prefix . '-update')
-			&& $module->hasAction($prefix . '-add')
-			&& $module->hasAction($prefix . '-remove')
+		$many = $module->hasAction($prefix.'-read')
+			&& $module->hasAction($prefix.'-update')
+			&& $module->hasAction($prefix.'-add')
+			&& $module->hasAction($prefix.'-remove')
 		;
-		$single = $module->hasAction($prefix . '-read')
-			&& $module->hasAction($prefix . '-update')
+		$single = $module->hasAction($prefix.'-read')
+			&& $module->hasAction($prefix.'-update')
 			&& !$many
 		;
 		
@@ -336,8 +336,8 @@ 
 		// check fqcn
 		$class = PhpClass::create($serializer);
 		if ($class->getQualifiedName() == $serializer) {
-			$class->setQualifiedName(NamespaceResolver::getNamespace('src/serializer', $this->package) . 
-				'\\' . $serializer);
+			$class->setQualifiedName(NamespaceResolver::getNamespace('src/serializer', $this->package). 
+				'\\'.$serializer);
 		}
 		
 		// check serializer exists

src/command/GenerateDomainCommand.php

Spacing +4 added lines, -4 removed lines

@@ -46,7 +46,7 @@ 
 	protected function initialize(InputInterface $input, OutputInterface $output) {
 		parent::initialize($input, $output);
 
-		$loader = new \Twig_Loader_Filesystem($this->service->getConfig()->getTemplateRoot() . '/domain');
+		$loader = new \Twig_Loader_Filesystem($this->service->getConfig()->getTemplateRoot().'/domain');
 		$this->twig = new \Twig_Environment($loader);
 	}
 
@@ -129,7 +129,7 @@ 
 	}
 
 	private function generateModel($modelName) {
-		$this->logger->info('Generate Domain from Model: ' . $modelName);
+		$this->logger->info('Generate Domain from Model: '.$modelName);
 		$model = $this->modelService->getModel($modelName);
 		
 		// generate class
@@ -146,10 +146,10 @@ 
 	}
 	
 	private function generateSkeleton($name) {
-		$this->logger->info('Generate Skeleton Domain: ' . $name);
+		$this->logger->info('Generate Skeleton Domain: '.$name);
 		$input = $this->io->getInput();
 
-		$className = NamespaceResolver::getNamespace('src/domain', $this->package) . '\\' . $name;
+		$className = NamespaceResolver::getNamespace('src/domain', $this->package).'\\'.$name;
 		
 		if (!Text::create($className)->endsWith('Domain')) {
 			$className .= 'Domain';