php-rpc / cruds

Adaptors/Symfony/FormProcessor.php

Doc Comments +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
      * FormProcessor constructor.
      *
      * @param FormFactoryInterface     $factory
-     * @param string|FormTypeInterface $form
+     * @param \Symfony\Component\Form\FormInterface $form
      * @param array                    $options
      */
     public function __construct(FormFactoryInterface $factory, $form, array $options = [])

Tests/AbstractCrudsWebTest.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
             implode(
                 "\n\n",
                 array_map(
-                    function ($l) {
+                    function($l) {
                         return implode("\n\n", $l);
                     },
                     $errors

DependencyInjection/Compiler/DoctrineOrmCompilerPass.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
             return;
         }
 
-        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__ . '/../../Resources/config'));
+        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../../Resources/config'));
         $loader->load('doctrine.yml');
 
         $factory = $container->getDefinition('cruds.factory.reflection');

Adaptors/JmsSerializer/JmsValidatorSubscriber.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
             throw new EntityProcessingException(
                 'Data for the entity is not valid',
                 array_map(
-                    function (ConstraintViolationInterface $violation) {
+                    function(ConstraintViolationInterface $violation) {
                         return $violation->getMessage();
                     },
                     iterator_to_array($list)

Adaptors/Symfony/MappedEntityFormFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
     }
 
     /**
-     * @param $className
+     * @param string $className
      *
      * @return FormInterface
      *

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace ScayTrase\Api\Cruds\Adaptors\Symfony;
 
 use ScayTrase\Api\Cruds\Exception\EntityProcessingException;
-use ScayTrase\Api\Cruds\Exception\MapperException;
 use ScayTrase\Api\Cruds\PropertyMapperInterface;
 use Symfony\Component\Form\FormFactoryInterface;
 use Symfony\Component\Form\FormInterface;

DependencyInjection/CrudsEntitiesConfigurator.php

Doc Comments +1 added lines, -1 removed lines

@@ -227,7 +227,7 @@
      * @param string $actionName
      * @param string $path
      * @param string $action
-     * @param array  $methods
+     * @param string[]  $methods
      * @param array  $options
      *
      * @return Definition

Spacing +9 added lines, -9 removed lines

@@ -59,9 +59,9 @@ 
             $actionConfig['class']      = $class;
             $actionConfig['mount']      = $mount;
             $actionConfig['repository'] = $repositoryDefinition;
-            $actionConfig['path']       = $prefix . $actionConfig['path'];
+            $actionConfig['path']       = $prefix.$actionConfig['path'];
             $actionConfig['manager']    = $manager;
-            $function                   = new \ReflectionMethod($this, 'register' . ucfirst($action) . 'Action');
+            $function                   = new \ReflectionMethod($this, 'register'.ucfirst($action).'Action');
             $args                       = [];
 
             foreach ($function->getParameters() as $parameter) {
@@ -105,7 +105,7 @@ 
         $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . CreateController::ACTION;
+        $action = $controllerId.':'.CreateController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
@@ -131,7 +131,7 @@ 
         $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . ReadController::ACTION;
+        $action = $controllerId.':'.ReadController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
@@ -165,7 +165,7 @@ 
         $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . UpdateController::ACTION;
+        $action = $controllerId.':'.UpdateController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
@@ -192,7 +192,7 @@ 
         $controllerId = $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . DeleteController::ACTION;
+        $action = $controllerId.':'.DeleteController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
@@ -232,7 +232,7 @@ 
         $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . SearchController::ACTION;
+        $action = $controllerId.':'.SearchController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
@@ -287,7 +287,7 @@ 
             'addRoute',
             [
                 $mount,
-                $this->normalize('cruds.routing.' . $name . '.' . $actionName),
+                $this->normalize('cruds.routing.'.$name.'.'.$actionName),
                 $path,
                 $action,
                 $methods,
@@ -310,7 +310,7 @@ 
      */
     private function generateControllerId($name, $actionName)
     {
-        return $this->normalize('cruds.generated_controller.' . $name . '.' . $actionName);
+        return $this->normalize('cruds.generated_controller.'.$name.'.'.$actionName);
     }
 
     /**

Routing/EntityRouteLoader.php

Doc Comments +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
     }
 
     /**
-     * @return string[]
+     * @return integer[]
      */
     public function getMounts()
     {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace ScayTrase\Api\Cruds\Routing;
 
 use Symfony\Component\Config\Loader\Loader;
-use Symfony\Component\Routing\Route;
 use Symfony\Component\Routing\RouteCollection;
 
 class EntityRouteLoader extends Loader

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
      */
     public function getRoutes($mount)
     {
-        if (!array_key_exists($mount, $this->routes)){
+        if (!array_key_exists($mount, $this->routes)) {
             throw new \OutOfBoundsException('Mount does not exist');
         }
 

Tests/Fixtures/Common/CrudsTestKernel.php

Spacing +5 added lines, -5 removed lines

@@ -24,26 +24,26 @@
 
     public function getCacheDir()
     {
-        return __DIR__ . '/../../../build/' . $this->getClassName() . '/cache';
+        return __DIR__.'/../../../build/'.$this->getClassName().'/cache';
     }
 
     public function getLogDir()
     {
-        return __DIR__ . '/../../../build/' . $this->getClassName() . '/logs';
+        return __DIR__.'/../../../build/'.$this->getClassName().'/logs';
     }
 
     /** {@inheritdoc} */
     public function registerContainerConfiguration(LoaderInterface $loader)
     {
-        return $loader->load(__DIR__ . '/config.yml');
+        return $loader->load(__DIR__.'/config.yml');
     }
 
     /** {@inheritdoc} */
     protected function buildContainer()
     {
         $container = parent::buildContainer();
-        $container->addResource(new FileResource(__DIR__ . '/config.yml'));
-        $container->addResource(new FileResource(__DIR__ . '/routing.yml'));
+        $container->addResource(new FileResource(__DIR__.'/config.yml'));
+        $container->addResource(new FileResource(__DIR__.'/routing.yml'));
 
         return $container;
     }

DependencyInjection/Configuration.php

Spacing +12 added lines, -12 removed lines

@@ -53,7 +53,7 @@ 
             ->defaultValue('api')
             ->cannotBeEmpty()
             ->info(
-                'Route mount. You can create different entries ' .
+                'Route mount. You can create different entries '.
                 'with different mounts. You can use this value when loading routes'
             )
             ->example('my-mount-name');
@@ -73,7 +73,7 @@ 
             ->beforeNormalization()
             ->ifArray()
             ->then(
-                function (array $v) {
+                function(array $v) {
                     if (array_keys($v) !== range(0, count($v) - 1)) {
                         return $v;
                     }
@@ -108,9 +108,9 @@ 
             ->defaultNull()
             ->example('@my_entity.factory')
             ->info(
-                'Service ID implementing ' . PHP_EOL .
-                EntityFactoryInterface::class . PHP_EOL .
-                'Defaults to ' . ReflectionConstructorFactory::class
+                'Service ID implementing '.PHP_EOL.
+                EntityFactoryInterface::class.PHP_EOL.
+                'Defaults to '.ReflectionConstructorFactory::class
             );
 
         $create
@@ -119,9 +119,9 @@ 
             ->defaultNull()
             ->example('@my_entity.factory')
             ->info(
-                'Service ID implementing ' . PHP_EOL .
-                EntityFactoryInterface::class . PHP_EOL .
-                'Defaults to ' . ReflectionConstructorFactory::class
+                'Service ID implementing '.PHP_EOL.
+                EntityFactoryInterface::class.PHP_EOL.
+                'Defaults to '.ReflectionConstructorFactory::class
             );
 
         $this->configureActionNode($create, 'create');
@@ -161,9 +161,9 @@ 
             ->defaultNull()
             ->example('@my_entity.processor')
             ->info(
-                'Service ID implementing ' . PHP_EOL .
-                EntityProcessorInterface::class . PHP_EOL .
-                'Defaults to ' . PropertyAccessProcessor::class
+                'Service ID implementing '.PHP_EOL.
+                EntityProcessorInterface::class.PHP_EOL.
+                'Defaults to '.PropertyAccessProcessor::class
             );
 
         $this->configureActionNode($update, 'update');
@@ -185,6 +185,6 @@ 
             ->children()
             ->scalarNode('path')
             ->info('Action path (will be prefixed with entity prefix)')
-            ->defaultValue('/' . $action);
+            ->defaultValue('/'.$action);
     }
 }