larriereguichet / AdminBundle

tests/AdminBundle/Event/Subscriber/MenuSubscriberTest.php

Spacing +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
 
     public function testBuildMenuWithoutConfiguration()
     {
-        list($subscriber, , $menuFactory, $eventDispatcher) = $this->createSubscriber();
+        list($subscriber,, $menuFactory, $eventDispatcher) = $this->createSubscriber();
         $event = new MenuEvent();
 
         $menuFactory

tests/AdminBundle/Event/Subscriber/SecuritySubscriberTest.php

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
 
     public function testHandleRequest()
     {
-        list($subscriber, , $tokenStorage, $authorizationChecker) = $this->createSubscriber([]);
+        list($subscriber,, $tokenStorage, $authorizationChecker) = $this->createSubscriber([]);
 
         $user = $this->createMock(UserInterface::class);
         $user
@@ -83,7 +83,7 @@ 
 
     public function testHandleRequestWithSecurityDisabled()
     {
-        list($subscriber, , $tokenStorage) = $this->createSubscriber([
+        list($subscriber,, $tokenStorage) = $this->createSubscriber([
             'enable_security' => false,
         ]);
 

tests/AdminBundle/Controller/HomeActionTest.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
         $eventDispatcher
             ->expects($this->once())
             ->method('dispatch')
-            ->willReturnCallback(function ($eventName, $event) {
+            ->willReturnCallback(function($eventName, $event) {
                 $this->assertEquals(Events::MENU, $eventName);
                 $this->assertInstanceOf(MenuEvent::class, $event);
             })

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(Events::DOCTRINE_ORM_FILTER, $eventName);
                 /** @var ORMFilterEvent $event */
                 $this->assertInstanceOf(ORMFilterEvent::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 FakeEntity(42);
+        $entity = new FakeEntity(42);
 
         /** @var EntityManagerInterface|MockObject $entityManager */
         $entityManager = $this->createMock(EntityManagerInterface::class);

src/Field/ActionCollectionField.php

Spacing +4 added lines, -4 removed lines

@@ -25,7 +25,7 @@ 
                 'template' => '@LAGAdmin/Field/actionCollection.html.twig',
                 'actions' => [],
             ])
-            ->setNormalizer('actions', function (Options $options, $value) use ($actionConfiguration) {
+            ->setNormalizer('actions', function(Options $options, $value) use ($actionConfiguration) {
                 if (!is_array($value) || 0 === count($value)) {
                     $value = [
                         'edit' => [],
@@ -124,7 +124,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(
@@ -142,7 +142,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(
@@ -152,7 +152,7 @@ 
 
                 return $value;
             })
-            ->setNormalizer('parameters', function (Options $options, $values) {
+            ->setNormalizer('parameters', function(Options $options, $values) {
                 $cleanedValues = [];
 
                 foreach ($values as $name => $method) {

tests/AdminBundle/Event/Subscriber/AdminSubscriberTest.php

Spacing +8 added lines, -8 removed lines

@@ -103,7 +103,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);
         });
     }
@@ -115,7 +115,7 @@ 
      */
     public function testCreateView(string $actionName)
     {
-        list($subscriber, , $viewFactory) = $this->createSubscriber();
+        list($subscriber,, $viewFactory) = $this->createSubscriber();
         $view = $this->createMock(ViewInterface::class);
 
         $actionConfiguration = $this->createMock(ActionConfiguration::class);
@@ -208,7 +208,7 @@ 
     public function testCreateViewWithRedirection()
     {
         $actionName = 'edit';
-        list($subscriber, , $viewFactory) = $this->createSubscriber();
+        list($subscriber,, $viewFactory) = $this->createSubscriber();
         $view = $this->createMock(ViewInterface::class);
 
         $actionConfiguration = $this->createMock(ActionConfiguration::class);
@@ -358,7 +358,7 @@ 
      */
     public function testLoadEntitiesWithMultipleStrategy()
     {
-        list($subscriber, , , $dataProviderFactory, , , ,) = $this->createSubscriber();
+        list($subscriber,,, $dataProviderFactory,,,,) = $this->createSubscriber();
 
         $actionConfiguration = $this->createMock(ActionConfiguration::class);
         $actionConfiguration
@@ -428,7 +428,7 @@ 
      */
     public function testLoadEntitiesWithUniqueStrategy()
     {
-        list($subscriber, , , $dataProviderFactory) = $this->createSubscriber();
+        list($subscriber,,, $dataProviderFactory) = $this->createSubscriber();
 
         $actionConfiguration = $this->createMock(ActionConfiguration::class);
         $actionConfiguration
@@ -501,7 +501,7 @@ 
      */
     public function testLoadEntitiesWithoutIdentifier()
     {
-        list($subscriber, , , $dataProviderFactory) = $this->createSubscriber();
+        list($subscriber,,, $dataProviderFactory) = $this->createSubscriber();
 
         $actionConfiguration = $this->createMock(ActionConfiguration::class);
         $actionConfiguration
@@ -558,7 +558,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);
         });
     }
@@ -568,7 +568,7 @@ 
      */
     public function testSaveEntity()
     {
-        list($subscriber, , , $dataProviderFactory, , $session, $translator) = $this->createSubscriber();
+        list($subscriber,,, $dataProviderFactory,, $session, $translator) = $this->createSubscriber();
 
         $adminConfiguration = $this->createMock(AdminConfiguration::class);
         $adminConfiguration