larriereguichet / AdminBundle

src/Field/ActionCollectionField.php

Spacing +4 added lines, -4 removed lines

@@ -39,7 +39,7 @@ 
                 'template' => '@LAGAdmin/Field/actionCollection.html.twig',
                 'actions' => [],
             ])
-            ->setNormalizer('actions', function (Options $options, $value) use ($actionConfiguration, $defaultActions) {
+            ->setNormalizer('actions', function(Options $options, $value) use ($actionConfiguration, $defaultActions) {
                 if (!is_array($value) || 0 === count($value)) {
                     $value = [
                         'edit' => [],
@@ -138,7 +138,7 @@ 
                 '_self',
                 '_blank',
             ])
-            ->setNormalizer('route', function (Options $options, $value) use ($actionConfiguration) {
+            ->setNormalizer('route', function(Options $options, $value) use ($actionConfiguration) {
                 // route or url should be defined
                 if (!$value && !$options->offsetGet('url') && !$options->offsetGet('admin')) {
                     throw new InvalidOptionsException(
@@ -156,7 +156,7 @@ 
 
                 return $value;
             })
-            ->setNormalizer('admin', function (Options $options, $value) {
+            ->setNormalizer('admin', function(Options $options, $value) {
                 // if a Admin is defined, an Action should be defined too
                 if ($value && !$options->offsetGet('action')) {
                     throw new InvalidOptionsException(
@@ -166,7 +166,7 @@ 
 
                 return $value;
             })
-            ->setNormalizer('parameters', function (Options $options, $values) {
+            ->setNormalizer('parameters', function(Options $options, $values) {
                 $cleanedValues = [];
 
                 foreach ($values as $name => $method) {

src/Factory/MenuFactory.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     public function __construct(
         RequestStack $requestStack,
         ApplicationConfigurationStorage $applicationConfigurationStorage
-    ){
+    ) {
         $this->requestStack = $requestStack;
         $this->applicationConfiguration = $applicationConfigurationStorage->getConfiguration();
     }

src/Event/Subscriber/ExtraConfigurationSubscriber.php

Doc Comments +8 added lines, -1 removed lines

@@ -138,6 +138,10 @@ 
         ];
     }
 
+    /**
+     * @param string $entityClass
+     * @param string $adminName
+     */
     private function addDefaultFields(array &$configuration, $entityClass, $adminName)
     {
         $fieldsMapping = [
@@ -343,7 +347,7 @@ 
     /**
      * Return the default action field if found.
      *
-     * @param array $fields
+     * @param string[] $fields
      *
      * @return string|null
      */
@@ -420,6 +424,9 @@ 
         return $metadata;
     }
 
+    /**
+     * @param string $type
+     */
     private function getOperatorFromFieldType($type)
     {
         $mapping = [

src/Factory/FieldFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -172,7 +172,7 @@
     /**
      * @param array               $configuration
      * @param ActionConfiguration $actionConfiguration
-     * @return array
+     * @return string
      * @throws Exception
      */
     private function resolveTopLevelConfiguration(array $configuration, ActionConfiguration $actionConfiguration)

tests/AdminBundle/Bridge/Doctrine/ORM/DataProvider/ORMDataProviderTest.php

Spacing +3 added lines, -3 removed lines

@@ -89,7 +89,7 @@ 
         $eventDispatcher
             ->expects($this->once())
             ->method('dispatch')
-            ->willReturnCallback(function ($eventName, $event) use ($queryBuilder, $admin) {
+            ->willReturnCallback(function($eventName, $event) use ($queryBuilder, $admin) {
                 $this->assertEquals(AdminEvents::DOCTRINE_ORM_FILTER, $eventName);
                 /** @var DoctrineOrmFilterEvent $event */
                 $this->assertInstanceOf(DoctrineOrmFilterEvent::class, $event);
@@ -217,7 +217,7 @@ 
             $requestStack
         );
 
-        $this->assertExceptionRaised(Exception::class, function () use ($provider, $admin) {
+        $this->assertExceptionRaised(Exception::class, function() use ($provider, $admin) {
             $provider->get($admin, 42);
         });
     }
@@ -230,7 +230,7 @@ 
         /** @var RequestStack|MockObject $requestStack */
         $requestStack = $this->createMock(RequestStack::class);
 
-        $entity =new EntityFixture(42);
+        $entity = new EntityFixture(42);
 
         /** @var EntityManagerInterface|MockObject $entityManager */
         $entityManager = $this->createMock(EntityManagerInterface::class);

tests/AdminBundle/Event/Subscriber/AdminSubscriberTest.php

Spacing +7 added lines, -7 removed lines

@@ -96,7 +96,7 @@ 
         $admin = $this->createMock(AdminInterface::class);
         $event = new AdminEvent($admin, $request);
 
-        $this->assertExceptionRaised(Exception::class, function () use ($subscriber, $event) {
+        $this->assertExceptionRaised(Exception::class, function() use ($subscriber, $event) {
             $subscriber->handleRequest($event);
         });
     }
@@ -106,7 +106,7 @@ 
      */
     public function testCreateView()
     {
-        list($subscriber, , $viewFactory) = $this->createAdminSubscriberMock();
+        list($subscriber,, $viewFactory) = $this->createAdminSubscriberMock();
         $view = $this->createMock(ViewInterface::class);
 
         $actionConfiguration = $this->createMock(ActionConfiguration::class);
@@ -244,7 +244,7 @@ 
      */
     public function testLoadEntitiesWithMultipleStrategy()
     {
-        list($subscriber, , , $dataProviderFactory, , , ,) = $this->createAdminSubscriberMock();
+        list($subscriber,,, $dataProviderFactory,,,,) = $this->createAdminSubscriberMock();
 
         $actionConfiguration = $this->createMock(ActionConfiguration::class);
         $actionConfiguration
@@ -314,7 +314,7 @@ 
      */
     public function testLoadEntitiesWithUniqueStrategy()
     {
-        list($subscriber, , , $dataProviderFactory) = $this->createAdminSubscriberMock();
+        list($subscriber,,, $dataProviderFactory) = $this->createAdminSubscriberMock();
 
         $actionConfiguration = $this->createMock(ActionConfiguration::class);
         $actionConfiguration
@@ -387,7 +387,7 @@ 
      */
     public function testLoadEntitiesWithoutIdentifier()
     {
-        list($subscriber, , , $dataProviderFactory) = $this->createAdminSubscriberMock();
+        list($subscriber,,, $dataProviderFactory) = $this->createAdminSubscriberMock();
 
         $actionConfiguration = $this->createMock(ActionConfiguration::class);
         $actionConfiguration
@@ -444,7 +444,7 @@ 
         $request = new Request();
         $event = new EntityEvent($admin, $request);
 
-        $this->assertExceptionRaised(Exception::class, function () use ($subscriber, $event) {
+        $this->assertExceptionRaised(Exception::class, function() use ($subscriber, $event) {
             $subscriber->loadEntities($event);
         });
     }
@@ -454,7 +454,7 @@ 
      */
     public function testSaveEntity()
     {
-        list($subscriber, , , $dataProviderFactory, , $session, $translator) = $this->createAdminSubscriberMock();
+        list($subscriber,,, $dataProviderFactory,, $session, $translator) = $this->createAdminSubscriberMock();
 
         $adminConfiguration = $this->createMock(AdminConfiguration::class);
         $adminConfiguration