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

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)

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 +11 added lines, -11 removed lines

@@ -65,10 +65,10 @@ 
             $actionConfig['class']      = $class;
             $actionConfig['mount']      = $mount;
             $actionConfig['repository'] = $repositoryDefinition;
-            $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) {
@@ -122,7 +122,7 @@ 
         $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . CreateController::ACTION;
+        $action = $controllerId.':'.CreateController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
@@ -150,7 +150,7 @@ 
      */
     private function generateControllerId($name, $actionName)
     {
-        return $this->normalize('cruds.generated_controller.' . $name . '.' . $actionName);
+        return $this->normalize('cruds.generated_controller.'.$name.'.'.$actionName);
     }
 
     /**
@@ -181,7 +181,7 @@ 
             'addRoute',
             [
                 $mount,
-                $this->normalize('cruds.routing.' . $name . '.' . $actionName),
+                $this->normalize('cruds.routing.'.$name.'.'.$actionName),
                 $path,
                 $action,
                 $methods,
@@ -215,7 +215,7 @@ 
         $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . ReadController::ACTION;
+        $action = $controllerId.':'.ReadController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
@@ -249,7 +249,7 @@ 
         $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . UpdateController::ACTION;
+        $action = $controllerId.':'.UpdateController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
@@ -276,7 +276,7 @@ 
         $controllerId = $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . DeleteController::ACTION;
+        $action = $controllerId.':'.DeleteController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
@@ -316,7 +316,7 @@ 
         $controllerId = $this->generateControllerId($name, $actionName);
         $this->container->setDefinition($controllerId, $definition);
 
-        $action = $controllerId . ':' . SearchController::ACTION;
+        $action = $controllerId.':'.SearchController::ACTION;
         $this->registerRoute(
             $mount,
             $name,
@@ -341,12 +341,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']]

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/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

Doc Comments +4 added lines, -1 removed lines

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

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;
 

Tests/Unit/Controller/SearchControllerTest.php

Doc Comments +4 added lines, -1 removed lines

@@ -95,6 +95,9 @@ 
         self::assertSame([$f1, $f2, $f3, $f4], $result->toArray());
     }
 
+    /**
+     * @param integer $a
+     */
     private function createFixture($a, $b, $c)
     {
         $entity    = new AbcClass();
@@ -106,7 +109,7 @@ 
     }
 
     /**
-     * @return ReferenceProviderInterface|\PHPUnit_Framework_MockObject_MockObject
+     * @return ReferenceProviderInterface
      */
     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');
     }
 }