larriereguichet / AdminBundle

src/Factory/FieldFactory.php

Unused Use Statements -2 removed lines

@@ -3,13 +3,11 @@
 namespace LAG\AdminBundle\Factory;
 
 use Exception;
-use LAG\AdminBundle\Admin\Exception\AdminException;
 use LAG\AdminBundle\Configuration\ActionConfiguration;
 use LAG\AdminBundle\Configuration\ApplicationConfiguration;
 use LAG\AdminBundle\Configuration\ApplicationConfigurationStorage;
 use LAG\AdminBundle\Field\FieldInterface;
 use LAG\AdminBundle\Field\TwigAwareFieldInterface;
-use LAG\AdminBundle\Field\TwigAwareInterface;
 use LAG\AdminBundle\Field\TranslatorAwareFieldInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
 use Symfony\Component\Translation\TranslatorInterface;

src/Configuration/ApplicationConfiguration.php

Spacing +2 added lines, -2 removed lines

@@ -79,7 +79,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 (strstr($value, '{admin}') === false) {
                     throw new InvalidOptionsException(
                         'Admin routing configuration pattern name should contains the {admin} placeholder'
@@ -93,7 +93,7 @@ 
 
                 return $value;
             })
-            ->setNormalizer('routing_url_pattern', function (Options $options, $value) {
+            ->setNormalizer('routing_url_pattern', function(Options $options, $value) {
                 if (strstr($value, '{admin}') === false) {
                     throw new InvalidOptionsException('Admin routing configuration url pattern should contains {admin} placeholder');
                 }

src/Configuration/AdminConfiguration.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
                 null,
                 'pagerfanta',
             ])
-            ->setNormalizer('actions', function (Options $options, $actions) {
+            ->setNormalizer('actions', function(Options $options, $actions) {
                 $normalizedActions = [];
                 $addBatchAction = false;
 

src/Field/LinkField.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,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(
@@ -64,7 +64,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(
@@ -74,7 +74,7 @@ 
 
                 return $value;
             })
-            ->setNormalizer('parameters', function (Options $options, $values) {
+            ->setNormalizer('parameters', function(Options $options, $values) {
                 $cleanedValues = [];
 
                 foreach ($values as $name => $method) {

src/Field/MappedField.php

Spacing +1 added lines, -1 removed lines

@@ -26,6 +26,6 @@
             $value = $this->options['map'][$value];
         }
 
-        return (string)$value;
+        return (string) $value;
     }
 }

sam.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
     'copy' => [],
 ];
 $eventDispatcher = new EventDispatcher();
-$eventDispatcher->addListener(NotificationEvent::NAME, function (NotificationEvent $event) {
+$eventDispatcher->addListener(NotificationEvent::NAME, function(NotificationEvent $event) {
     echo $event->getMessage()."\n";
 });
 
@@ -68,7 +68,7 @@ 
 $builder = new TaskBuilder();
 $tasks = $builder->build($tasks);
 
-$normalizer = new Normalizer(realpath(__DIR__ . '/AdminBundle'));
+$normalizer = new Normalizer(realpath(__DIR__.'/AdminBundle'));
 $locator = new Locator($normalizer);
 
 $runner = new TaskRunner(

tests/AdminBundle/DependencyInjection/CompilerPass/ResourceCompilerPassTest.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
         $resourceCollection
             ->expects($this->once())
             ->method('addMethodCall')
-            ->willReturnCallback(function ($name, $parameters) {
+            ->willReturnCallback(function($name, $parameters) {
                 $this->assertEquals('add', $name);
                 $this->assertCount(1, $parameters);
                 $this->assertInstanceOf(Reference::class, $parameters[0]);

tests/AdminBundle/Admin/AdminTest.php

Spacing +5 added lines, -5 removed lines

@@ -47,7 +47,7 @@ 
         $eventDispatcher
             ->expects($this->exactly(5))
             ->method('dispatch')
-            ->willReturnCallback(function ($eventName, $event) use ($action, $form) {
+            ->willReturnCallback(function($eventName, $event) use ($action, $form) {
                 if (AdminEvents::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(AdminEvents::VIEW, $eventName);
                 /** @var ViewEvent $event */
                 $this->assertInstanceOf(ViewEvent::class, $event);
@@ -178,7 +178,7 @@ 
         $eventDispatcher
             ->expects($this->exactly(4))
             ->method('dispatch')
-            ->willReturnCallback(function ($eventName, $event) use ($action, $form) {
+            ->willReturnCallback(function($eventName, $event) use ($action, $form) {
                 if (AdminEvents::HANDLE_REQUEST === $eventName) {
                     /** @var AdminEvent $event */
                     $this->assertInstanceOf(AdminEvent::class, $event);
@@ -208,7 +208,7 @@ 
         $eventDispatcher
             ->expects($this->exactly(4))
             ->method('dispatch')
-            ->willReturnCallback(function ($eventName, $event) use ($action, $form) {
+            ->willReturnCallback(function($eventName, $event) use ($action, $form) {
                 if (AdminEvents::HANDLE_REQUEST === $eventName) {
                     /** @var AdminEvent $event */
                     $this->assertInstanceOf(AdminEvent::class, $event);

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(AdminEvents::MENU, $eventName);
                 $this->assertInstanceOf(MenuEvent::class, $event);
             })