php-rpc / cruds

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');

DependencyInjection/CrudsExtension.php

Spacing +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
     /** {@inheritdoc} */
     public function load(array $configs, ContainerBuilder $container)
     {
-        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
+        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('cruds.yml');
 
         $config = $this->processConfiguration($this->getConfiguration([], $container), $configs);
@@ -20,7 +20,7 @@ 
         $configurator = new CrudsEntitiesConfigurator($container);
         $container->addObjectResource($configurator);
         foreach ($config['entities'] as $name => $entityConfig) {
-            $entityConfig['prefix'] = $entityConfig['prefix'] ?: '/' . $name;
+            $entityConfig['prefix'] = $entityConfig['prefix'] ?: '/'.$name;
             $configurator->processEntityConfiguration($name, $entityConfig);
         }
 
@@ -29,7 +29,7 @@ 
                 continue;
             }
 
-            $container->removeDefinition('cruds.api.listener.' . $listener);
+            $container->removeDefinition('cruds.api.listener.'.$listener);
         }
     }
 }

DependencyInjection/Configuration.php

Spacing +1 added lines, -1 removed lines

@@ -91,7 +91,7 @@
             ->beforeNormalization()
             ->ifArray()
             ->then(
-                function (array $v) {
+                function(array $v) {
                     if (array_keys($v) !== range(0, count($v) - 1)) {
                         return $v;
                     }

Tests/Unit/Controller/CountControllerTest.php

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 use ScayTrase\Api\Cruds\PublicPropertyMapper;
 use ScayTrase\Api\Cruds\ReferenceProviderInterface;
 use ScayTrase\Api\Cruds\Tests\Fixtures\AbcClass;
-use ScayTrase\Api\Cruds\Tests\Unit\AbstractControllerTest;
 use Symfony\Component\EventDispatcher\Event;
 use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 

Doc Comments +4 added lines, -1 removed lines

@@ -49,6 +49,9 @@ 
         self::assertSame(3, $controller->countAction(['a' => [1, 2, 4]]));
     }
 
+    /**
+     * @param integer $a
+     */
     private function createFixture($a, $b, $c)
     {
         $entity    = new AbcClass();
@@ -60,7 +63,7 @@ 
     }
 
     /**
-     * @return ReferenceProviderInterface|\PHPUnit_Framework_MockObject_MockObject
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     private function getReferenceProvider()
     {

DependencyInjection/Compiler/SymfonyFormsCompilerPass.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
             return;
         }
 
-        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__ . '/../../Resources/config'));
+        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../../Resources/config'));
         $loader->load('forms.yml');
     }
 }

Tests/KernelProvider.php

Spacing +2 added lines, -2 removed lines

@@ -17,11 +17,11 @@
     public static function getClass()
     {
         if (false === ($alias = getenv(self::ENV_VAR))) {
-            throw new \LogicException('Cannot obtain kernel alias from "' . self::ENV_VAR . '"" ENV variable');
+            throw new \LogicException('Cannot obtain kernel alias from "'.self::ENV_VAR.'"" ENV variable');
         }
 
         if (!array_key_exists($alias, self::$map)) {
-            throw new \LogicException('Invalid kernel alias: ' . $alias);
+            throw new \LogicException('Invalid kernel alias: '.$alias);
         }
 
         return self::$map[$alias];

dump_reference.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 use ScayTrase\Api\Cruds\DependencyInjection\Configuration;
 use Symfony\Component\Config\Definition\Dumper\YamlReferenceDumper;
 
-require_once __DIR__ . '/bootstrap.php';
+require_once __DIR__.'/bootstrap.php';
 
 $config = new Configuration();
 $dumper = new YamlReferenceDumper();

Exception/EntityProcessingException.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
         }
 
         return new static(
-            'Invalid data submitted for entity: ' . PHP_EOL .
+            'Invalid data submitted for entity: '.PHP_EOL.
             Yaml::dump($errors)
         );
     }

DependencyInjection/CrudsEntitiesConfigurator.php

Doc Comments +1 added lines, -1 removed lines

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

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 use ScayTrase\Api\Cruds\Controller\SearchController;
 use ScayTrase\Api\Cruds\Controller\UpdateController;
 use ScayTrase\Api\Cruds\Criteria\NestedCriteriaConfigurator;
-use ScayTrase\Api\Cruds\ReflectionConstructorFactory;
 use Symfony\Component\DependencyInjection\ChildDefinition;
 use Symfony\Component\DependencyInjection\DefinitionDecorator;
 use Symfony\Component\DependencyInjection\ContainerBuilder;

Spacing +14 added lines, -14 removed lines

@@ -43,7 +43,7 @@ 
         $mount = $config['mount'];
 
         if (null === $manager) {
-            $manager = 'cruds.class_' . $class . '.object_manager';
+            $manager = 'cruds.class_'.$class.'.object_manager';
             $managerDef = new Definition(ObjectManager::class);
             $managerDef->setPublic(false);
             $managerDef->setFactory([new Reference('doctrine'), 'getManagerForClass']);
@@ -53,7 +53,7 @@ 
         $manager = new Reference($this->filterReference($manager));
 
         if (null === $repository) {
-            $repository = 'cruds.class_' . $class . '.entity_repository';
+            $repository = 'cruds.class_'.$class.'.entity_repository';
             $repositoryDef = new Definition(EntityRepository::class);
             $repositoryDef->setPublic(false);
             $repositoryDef->setFactory([$manager, 'getRepository']);
@@ -72,10 +72,10 @@ 
             $actionConfig['class'] = $class;
             $actionConfig['mount'] = $mount;
             $actionConfig['repository'] = $repository;
-            $actionConfig['path'] = $prefix . $actionConfig['path'];
+            $actionConfig['path'] = $prefix.$actionConfig['path'];
             $actionConfig['manager'] = $manager;
             $actionConfig['prefix'] = $prefix;
-            $function = new \ReflectionMethod($this, 'register' . ucfirst($action) . 'Action');
+            $function = new \ReflectionMethod($this, 'register'.ucfirst($action).'Action');
             $args = [];
 
             foreach ($function->getParameters() as $parameter) {
@@ -95,7 +95,7 @@ 
         $controllerId = $this->generateControllerId($name, $actionName);
 
         if (null === $factory) {
-            $factory = $controllerId . '.entity_factory';
+            $factory = $controllerId.'.entity_factory';
 
             if (class_exists(ChildDefinition::class)) {
                 $factoryDef = new ChildDefinition('cruds.factory.reflection');
@@ -131,7 +131,7 @@ 
 
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . CreateController::ACTION;
+        $action = $controllerId.':'.CreateController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
@@ -158,7 +158,7 @@ 
         $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . ReadController::ACTION;
+        $action = $controllerId.':'.ReadController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
@@ -193,7 +193,7 @@ 
         $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . UpdateController::ACTION;
+        $action = $controllerId.':'.UpdateController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
@@ -222,7 +222,7 @@ 
         $controllerId = $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . DeleteController::ACTION;
+        $action = $controllerId.':'.DeleteController::ACTION;
         $definition->setPublic(true);
         $this->registerRoute(
             $mount,
@@ -272,7 +272,7 @@ 
         $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . SearchController::ACTION;
+        $action = $controllerId.':'.SearchController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
@@ -298,12 +298,12 @@ 
         $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . CountController::ACTION;
+        $action = $controllerId.':'.CountController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
             $actionName,
-            $prefix . $count_path,
+            $prefix.$count_path,
             $action,
             ['GET', 'POST'],
             ['class' => $class, 'arguments' => ['criteria']]
@@ -325,7 +325,7 @@ 
 
     private function generateControllerId(string $name, string $actionName): string
     {
-        return $this->normalize('cruds.generated_controller.' . $name . '.' . $actionName);
+        return $this->normalize('cruds.generated_controller.'.$name.'.'.$actionName);
     }
 
     /**
@@ -363,7 +363,7 @@ 
             'addRoute',
             [
                 $mount,
-                $this->normalize('cruds.routing.' . $name . '.' . $actionName),
+                $this->normalize('cruds.routing.'.$name.'.'.$actionName),
                 $path,
                 $action,
                 $methods,