larriereguichet / AdminBundle

src/LAG/AdminBundle/Tests/AdminBundle/DependencyInjection/FieldCompilerPassTest.php

Spacing +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
 
         $this->assertExceptionRaised(
             InvalidConfigurationException::class,
-            function () use ($compilerPass, $containerBuilder) {
+            function() use ($compilerPass, $containerBuilder) {
                 // process the compiler pass
                 $compilerPass->process($containerBuilder);
             }

src/Configuration/MenuConfiguration.php

Spacing +5 added lines, -5 removed lines

@@ -53,7 +53,7 @@ 
                 'vertical',
                 null,
             ])
-            ->setNormalizer('position', function (Options $options, $value) {
+            ->setNormalizer('position', function(Options $options, $value) {
                 if ('top' === $this->menuName && null === $value) {
                     $value = 'horizontal';
                 }
@@ -64,7 +64,7 @@ 
 
                 return $value;
             })
-            ->setNormalizer('template', function (Options $options, $value) {
+            ->setNormalizer('template', function(Options $options, $value) {
                 // Define bootstrap navbar component template
                 if ('horizontal' === $options->offsetGet('position')) {
                     $value = '@LAGAdmin/Menu/menu.horizontal.html.twig';
@@ -77,7 +77,7 @@ 
 
                 return $value;
             })
-            ->setNormalizer('attr', function (Options $options, $value) {
+            ->setNormalizer('attr', function(Options $options, $value) {
                 $position = $options->offsetGet('position');
 
                 if (!key_exists('class', $value)) {
@@ -100,7 +100,7 @@ 
 
                 return $value;
             })
-            ->setNormalizer('item_css_class', function (Options $options, $value) {
+            ->setNormalizer('item_css_class', function(Options $options, $value) {
                 $position = $options->offsetGet('position');
 
                 if (!$value) {
@@ -117,7 +117,7 @@ 
 
                 return trim($value);
             })
-            ->setNormalizer('brand', function (Options $options, $value) {
+            ->setNormalizer('brand', function(Options $options, $value) {
                 if (null === $value && 'horizontal' === $options->offsetGet('position')) {
                     $value = $this->applicationName;
                 }

src/Configuration/ApplicationConfiguration.php

Spacing +5 added lines, -5 removed lines

@@ -86,7 +86,7 @@ 
             ->setAllowedTypes('routing_name_pattern', 'string')
             ->setAllowedTypes('routing_url_pattern', 'string')
             ->setAllowedTypes('page_parameter', 'string')
-            ->setNormalizer('routing_name_pattern', function (Options $options, $value) {
+            ->setNormalizer('routing_name_pattern', function(Options $options, $value) {
                 if (false === strstr($value, '{admin}')) {
                     throw new InvalidOptionsException(
                         'Admin routing configuration pattern name should contains the {admin} placeholder'
@@ -100,7 +100,7 @@ 
 
                 return $value;
             })
-            ->setNormalizer('routing_url_pattern', function (Options $options, $value) {
+            ->setNormalizer('routing_url_pattern', function(Options $options, $value) {
                 if (false === strstr($value, '{admin}')) {
                     throw new InvalidOptionsException('Admin routing configuration url pattern should contains {admin} placeholder');
                 }
@@ -126,7 +126,7 @@ 
             'pattern' => 'lag.admin.{key}',
         ]);
         $resolver->setAllowedTypes('translation', 'array');
-        $resolver->setNormalizer('translation', function (Options $options, $value) {
+        $resolver->setNormalizer('translation', function(Options $options, $value) {
             if (!array_key_exists('enabled', $value)) {
                 throw new InvalidOptionsException('Admin translation enabled parameter should be defined');
             }
@@ -168,7 +168,7 @@ 
 
         $resolver->setDefault('fields_mapping', $defaultMapping);
         $resolver->setAllowedTypes('fields_mapping', 'array');
-        $resolver->setNormalizer('fields_mapping', function (Options $options, $value) use ($defaultMapping) {
+        $resolver->setNormalizer('fields_mapping', function(Options $options, $value) use ($defaultMapping) {
             // Merge with default mapping to allow override
             $value = array_merge($defaultMapping, $value);
 
@@ -182,7 +182,7 @@ 
 
         $resolver->setDefault('fields_template_mapping', $defaultMapping);
         $resolver->setAllowedTypes('fields_template_mapping', 'array');
-        $resolver->setNormalizer('fields_template_mapping', function (Options $options, $value) use ($defaultMapping) {
+        $resolver->setNormalizer('fields_template_mapping', function(Options $options, $value) use ($defaultMapping) {
             // Merge with default mapping to allow override
             $value = array_merge($defaultMapping, $value);
 

src/Configuration/MenuItemConfiguration.php

Spacing +6 added lines, -6 removed lines

@@ -55,7 +55,7 @@ 
             ->setDefault('items', [])
             ->setDefault('icon', null)
             ->setDefault('link_css_class', 'nav-link')
-            ->setNormalizer('admin', function (Options $options, $adminName) {
+            ->setNormalizer('admin', function(Options $options, $adminName) {
                 // user has to defined either an admin name and an action name, or a route name with optional
                 // parameters, or an url
                 if (null === $adminName
@@ -70,7 +70,7 @@ 
                 return $adminName;
             })
             // if an admin name is set, an action name can provided. This action will be the menu link
-            ->setNormalizer('action', function (Options $options, $action) {
+            ->setNormalizer('action', function(Options $options, $action) {
                 // if an action name is provided, an admin name should be defined too
                 if (null !== $action && null === $options->offsetGet('admin')) {
                     throw new InvalidOptionsException(
@@ -86,7 +86,7 @@ 
                 return $action;
             })
             ->setAllowedTypes('parameters', 'array')
-            ->setNormalizer('attr', function (Options $options, $attr) {
+            ->setNormalizer('attr', function(Options $options, $attr) {
                 if (!is_array($attr)) {
                     $attr = [];
                 }
@@ -101,7 +101,7 @@ 
 
                 return $attr;
             })
-            ->setNormalizer('link_attr', function (Options $options, $value) {
+            ->setNormalizer('link_attr', function(Options $options, $value) {
                 if (!is_array($value)) {
                     $value = [];
                 }
@@ -112,7 +112,7 @@ 
 
                 return $value;
             })
-            ->setNormalizer('items', function (Options $options, $items) {
+            ->setNormalizer('items', function(Options $options, $items) {
                 if (!is_array($items)) {
                     $items = [];
                 }
@@ -129,7 +129,7 @@ 
 
                 return $resolvedItems;
             })
-            ->setNormalizer('text', function (Options $options, $text) {
+            ->setNormalizer('text', function(Options $options, $text) {
                 if (!$text) {
                     // TODO use translation pattern key instead
                     $text = ucfirst($this->name);

src/Configuration/ActionConfiguration.php

Spacing +8 added lines, -8 removed lines

@@ -158,7 +158,7 @@ 
      */
     private function getFieldsNormalizer()
     {
-        return function (Options $options, $fields) {
+        return function(Options $options, $fields) {
             $normalizedFields = [];
 
             foreach ($fields as $name => $field) {
@@ -180,7 +180,7 @@ 
      */
     private function getOrderNormalizer()
     {
-        return function (Options $options, $order) {
+        return function(Options $options, $order) {
             foreach ($order as $field => $sort) {
                 if (!is_string($sort) || !is_string($field) || !in_array(strtolower($sort), ['asc', 'desc'])) {
                     throw new Exception(
@@ -203,7 +203,7 @@ 
      */
     private function getLoadStrategyNormalizer()
     {
-        return function (Options $options, $value) {
+        return function(Options $options, $value) {
             if (!$value) {
                 if ('create' == $this->actionName) {
                     $value = LAGAdminBundle::LOAD_STRATEGY_NONE;
@@ -226,7 +226,7 @@ 
      */
     private function getMenuNormalizer()
     {
-        return function (Options $options, $menus) {
+        return function(Options $options, $menus) {
             // set default to an array
             if (false === $menus) {
                 $menus = [];
@@ -244,7 +244,7 @@ 
      */
     private function getCriteriaNormalizer()
     {
-        return function (Options $options, $value) {
+        return function(Options $options, $value) {
             if (!$value) {
                 $idActions = [
                     'edit',
@@ -293,7 +293,7 @@ 
      */
     private function getFiltersNormalizer(): Closure
     {
-        return function (Options $options, $data) {
+        return function(Options $options, $data) {
             $normalizedData = [];
 
             foreach ($data as $name => $field) {
@@ -319,7 +319,7 @@ 
 
     private function getRouteDefaultNormalizer(): Closure
     {
-        return function (Options $options, $value) {
+        return function(Options $options, $value) {
             if (!is_array($value)) {
                 $value = [];
             }
@@ -332,7 +332,7 @@ 
 
     private function getFormNormalizer(): Closure
     {
-        return function (Options $options, $value) {
+        return function(Options $options, $value) {
             if (null !== $value) {
                 return $value;
             }

tests/AdminBundle/Admin/AdminTest.php

Spacing +5 added lines, -5 removed lines

@@ -47,7 +47,7 @@ 
         $eventDispatcher
             ->expects($this->exactly(6))
             ->method('dispatch')
-            ->willReturnCallback(function ($eventName, $event) use ($action, $form) {
+            ->willReturnCallback(function($eventName, $event) use ($action, $form) {
                 if (Events::HANDLE_REQUEST === $eventName) {
                     /** @var AdminEvent $event */
                     $this->assertInstanceOf(AdminEvent::class, $event);
@@ -112,7 +112,7 @@ 
         );
         $request = new Request();
 
-        $this->assertExceptionRaised(Exception::class, function () use ($admin, $request) {
+        $this->assertExceptionRaised(Exception::class, function() use ($admin, $request) {
             $admin->handleRequest($request);
         });
     }
@@ -127,7 +127,7 @@ 
         $eventDispatcher
             ->expects($this->once())
             ->method('dispatch')
-            ->willReturnCallback(function ($eventName, $event) use ($view) {
+            ->willReturnCallback(function($eventName, $event) use ($view) {
                 $this->assertEquals(Events::VIEW, $eventName);
                 /** @var ViewEvent $event */
                 $this->assertInstanceOf(ViewEvent::class, $event);
@@ -178,7 +178,7 @@ 
         $eventDispatcher
             ->expects($this->exactly(5))
             ->method('dispatch')
-            ->willReturnCallback(function ($eventName, $event) use ($action, $form) {
+            ->willReturnCallback(function($eventName, $event) use ($action, $form) {
                 if (Events::HANDLE_REQUEST === $eventName) {
                     /** @var AdminEvent $event */
                     $this->assertInstanceOf(AdminEvent::class, $event);
@@ -208,7 +208,7 @@ 
         $eventDispatcher
             ->expects($this->exactly(5))
             ->method('dispatch')
-            ->willReturnCallback(function ($eventName, $event) use ($action, $form) {
+            ->willReturnCallback(function($eventName, $event) use ($action, $form) {
                 if (Events::HANDLE_REQUEST === $eventName) {
                     /** @var AdminEvent $event */
                     $this->assertInstanceOf(AdminEvent::class, $event);

tests/AdminBundle/Factory/MenuFactoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 
     public function testCreate()
     {
-        list($factory, , $requestStack,) = $this->createFactory();
+        list($factory,, $requestStack,) = $this->createFactory();
 
         $requestStack
             ->expects($this->atLeastOnce())

tests/AdminBundle/Event/Subscriber/AdminSubscriberTest.php

Spacing +7 added lines, -7 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);
         });
     }
@@ -113,7 +113,7 @@ 
      */
     public function testCreateView()
     {
-        list($subscriber, , $viewFactory) = $this->createSubscriber();
+        list($subscriber,, $viewFactory) = $this->createSubscriber();
         $view = $this->createMock(ViewInterface::class);
 
         $actionConfiguration = $this->createMock(ActionConfiguration::class);
@@ -251,7 +251,7 @@ 
      */
     public function testLoadEntitiesWithMultipleStrategy()
     {
-        list($subscriber, , , $dataProviderFactory, , , ,) = $this->createSubscriber();
+        list($subscriber,,, $dataProviderFactory,,,,) = $this->createSubscriber();
 
         $actionConfiguration = $this->createMock(ActionConfiguration::class);
         $actionConfiguration
@@ -321,7 +321,7 @@ 
      */
     public function testLoadEntitiesWithUniqueStrategy()
     {
-        list($subscriber, , , $dataProviderFactory) = $this->createSubscriber();
+        list($subscriber,,, $dataProviderFactory) = $this->createSubscriber();
 
         $actionConfiguration = $this->createMock(ActionConfiguration::class);
         $actionConfiguration
@@ -394,7 +394,7 @@ 
      */
     public function testLoadEntitiesWithoutIdentifier()
     {
-        list($subscriber, , , $dataProviderFactory) = $this->createSubscriber();
+        list($subscriber,,, $dataProviderFactory) = $this->createSubscriber();
 
         $actionConfiguration = $this->createMock(ActionConfiguration::class);
         $actionConfiguration
@@ -451,7 +451,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);
         });
     }
@@ -461,7 +461,7 @@ 
      */
     public function testSaveEntity()
     {
-        list($subscriber, , , $dataProviderFactory, , $session, $translator) = $this->createSubscriber();
+        list($subscriber,,, $dataProviderFactory,, $session, $translator) = $this->createSubscriber();
 
         $adminConfiguration = $this->createMock(AdminConfiguration::class);
         $adminConfiguration

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