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 = [])

Adaptors/Symfony/MappedEntityFormFactory.php

Doc Comments +3 added lines

@@ -24,6 +24,9 @@
         $this->mapper  = $mapper;
     }
 
+    /**
+     * @param string $className
+     */
     public function createFormForClass($className)
     {
         $form = $this->factory->create();

DependencyInjection/CrudsEntitiesConfigurator.php

Doc Comments +3 added lines

@@ -191,6 +191,9 @@
         return $this->container->getDefinition('cruds.api.router_loader');
     }
 
+    /**
+     * @param string $name
+     */
     private function normalize($name)
     {
         return str_replace('-', '_', $name);

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 use ScayTrase\Api\Cruds\Controller\ReadController;
 use ScayTrase\Api\Cruds\Controller\SearchController;
 use ScayTrase\Api\Cruds\Controller\UpdateController;
-use ScayTrase\Api\Cruds\PropertyAccessProcessor;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Definition;
 use Symfony\Component\DependencyInjection\DefinitionDecorator;

Spacing +17 added lines, -17 removed lines

@@ -54,9 +54,9 @@ 
             $actionConfig['name']       = $name;
             $actionConfig['class']      = $class;
             $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) {
@@ -96,15 +96,15 @@ 
         );
         $definition->setPublic(true);
 
-        $controllerId = $this->normalize('cruds.api.generated.' . $name . '.create_controller');
+        $controllerId = $this->normalize('cruds.api.generated.'.$name.'.create_controller');
         $this->container->setDefinition($controllerId, $definition);
 
         $this->getLoaderDefinition()->addMethodCall(
             'addRoute',
             [
-                $this->normalize('cruds_api_' . $name . '_create'),
+                $this->normalize('cruds_api_'.$name.'_create'),
                 $path,
-                $controllerId . ':' . CreateController::ACTION,
+                $controllerId.':'.CreateController::ACTION,
                 ['POST'],
             ]
         );
@@ -120,15 +120,15 @@ 
             ]
         );
 
-        $controllerId = $this->normalize('cruds_api_' . $name . '_read_controller');
+        $controllerId = $this->normalize('cruds_api_'.$name.'_read_controller');
         $this->container->setDefinition($controllerId, $definition);
 
         $this->getLoaderDefinition()->addMethodCall(
             'addRoute',
             [
-                $this->normalize('cruds_api_' . $name . '_read'),
+                $this->normalize('cruds_api_'.$name.'_read'),
                 $path,
-                $controllerId . ':' . ReadController::ACTION,
+                $controllerId.':'.ReadController::ACTION,
                 ['GET'],
             ]
         );
@@ -152,15 +152,15 @@ 
             ]
         );
 
-        $controllerId = $this->normalize('cruds_api_' . $name . '_update_controller');
+        $controllerId = $this->normalize('cruds_api_'.$name.'_update_controller');
         $this->container->setDefinition($controllerId, $definition);
 
         $this->getLoaderDefinition()->addMethodCall(
             'addRoute',
             [
-                $this->normalize('cruds_api_' . $name . '_update'),
+                $this->normalize('cruds_api_'.$name.'_update'),
                 $path,
-                $controllerId . ':' . UpdateController::ACTION,
+                $controllerId.':'.UpdateController::ACTION,
                 ['POST'],
             ]
         );
@@ -177,15 +177,15 @@ 
             ]
         );
 
-        $controllerId = $this->normalize('cruds_api_' . $name . '_delete_controller');
+        $controllerId = $this->normalize('cruds_api_'.$name.'_delete_controller');
         $this->container->setDefinition($controllerId, $definition);
 
         $this->getLoaderDefinition()->addMethodCall(
             'addRoute',
             [
-                $this->normalize('cruds_api_' . $name . '_delete'),
+                $this->normalize('cruds_api_'.$name.'_delete'),
                 $path,
-                $controllerId . ':' . DeleteController::ACTION,
+                $controllerId.':'.DeleteController::ACTION,
                 ['POST'],
             ]
         );
@@ -208,15 +208,15 @@ 
             ]
         );
 
-        $controllerId = $this->normalize('cruds_api_' . $name . '_search_controller');
+        $controllerId = $this->normalize('cruds_api_'.$name.'_search_controller');
         $this->container->setDefinition($controllerId, $definition);
 
         $this->getLoaderDefinition()->addMethodCall(
             'addRoute',
             [
-                $this->normalize('cruds_api_' . $name . '_search'),
+                $this->normalize('cruds_api_'.$name.'_search'),
                 $path,
-                $controllerId . ':' . SearchController::ACTION,
+                $controllerId.':'.SearchController::ACTION,
                 ['GET', 'POST'],
             ]
         );

DependencyInjection/Configuration.php

Spacing +2 added lines, -2 removed lines

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

Tests/Fixtures/SymfonySerializer/SymfonyTestKernel.php

Spacing +2 added lines, -2 removed lines

@@ -11,13 +11,13 @@
     /** {@inheritdoc} */
     public function registerContainerConfiguration(LoaderInterface $loader)
     {
-        return $loader->load(__DIR__ . '/config.yml');
+        return $loader->load(__DIR__.'/config.yml');
     }
 
     protected function buildContainer()
     {
         $container = parent::buildContainer();
-        $container->addResource(new FileResource(__DIR__ . '/config.yml'));
+        $container->addResource(new FileResource(__DIR__.'/config.yml'));
 
         return $container;
     }

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

Tests/Fixtures/JmsSerializer/JmsTestKernel.php

Spacing +2 added lines, -2 removed lines

@@ -11,13 +11,13 @@
     /** {@inheritdoc} */
     public function registerContainerConfiguration(LoaderInterface $loader)
     {
-        return $loader->load(__DIR__ . '/config.yml');
+        return $loader->load(__DIR__.'/config.yml');
     }
 
     protected function buildContainer()
     {
         $container = parent::buildContainer();
-        $container->addResource(new FileResource(__DIR__ . '/config.yml'));
+        $container->addResource(new FileResource(__DIR__.'/config.yml'));
 
         return $container;
     }

Controller/SearchController.php

Doc Comments +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
      * @param int   $limit
      * @param int   $offset
      *
-     * @return object[]
+     * @return \Doctrine\Common\Collections\Collection
      * @throws CriteriaConfigurationException
      */
     public function findAction(array $criteria, array $order = [], $limit = 10, $offset = 0)

Adaptors/DoctrineOrm/DoctrineObjectNormalizer.php

Doc Comments +6 added lines

@@ -115,6 +115,9 @@ 
         return array_shift($identifier);
     }
 
+    /**
+     * @param string $attribute
+     */
     private function getObjectMetadata($object, $attribute)
     {
         $class = get_class($object);
@@ -132,6 +135,9 @@ 
         return $metadata;
     }
 
+    /**
+     * @param string $attribute
+     */
     private function getAssocMetadata(ClassMetadataInterface $metadata, $attribute)
     {
         $assoc        = $metadata->getAssociationTargetClass($attribute);

Indentation +3 added lines, -3 removed lines

@@ -139,9 +139,9 @@
 
         if (null === $assocManager) {
             throw new \LogicException($metadata->getName() .
-                                      '::$' .
-                                      $attribute .
-                                      ' references non-existent managed entity');
+                                        '::$' .
+                                        $attribute .
+                                        ' references non-existent managed entity');
         }
 
         return $assocManager->getClassMetadata($assoc);

Spacing +3 added lines, -3 removed lines

@@ -138,9 +138,9 @@
         $assocManager = $this->registry->getManagerForClass($assoc);
 
         if (null === $assocManager) {
-            throw new \LogicException($metadata->getName() .
-                                      '::$' .
-                                      $attribute .
+            throw new \LogicException($metadata->getName().
+                                      '::$'.
+                                      $attribute.
                                       ' references non-existent managed entity');
         }