larriereguichet / AdminBundle

Controller/CRUDController.php

Spacing +2 added lines, -2 removed lines

@@ -249,7 +249,7 @@ 
         $hooks = [];
 
         foreach ($fields as $fieldName) {
-            $exporter->addHook(function ($fieldValue) {
+            $exporter->addHook(function($fieldValue) {
                 // if field is an array
                 if (is_array($fieldValue)) {
                     $value = recursiveImplode(', ', $fieldValue);
@@ -294,7 +294,7 @@ 
         $this->forward404Unless(
             $admin->isActionGranted($admin->getCurrentAction()->getName(), $roles),
             sprintf('User with roles %s not allowed for action "%s"',
-                implode(', ', array_map(function (Role $role) {
+                implode(', ', array_map(function(Role $role) {
                     return $role->getRole();
                 }, $roles)),
                 $admin->getCurrentAction()->getName()

Field/Field/ArrayField.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     public function render($value)
     {
         if (!is_array($value) && !($value instanceof Traversable)) {
-            throw new Exception('Value should be an array instead of '.gettype($value));
+            throw new Exception('Value should be an array instead of ' . gettype($value));
         }
         if ($value instanceof Collection) {
             $value = $value->toArray();

Field/Field/StringField.php

Spacing +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
      */
     public function setOptions(array $options)
     {
-        $this->length = (int)$options['length'];
+        $this->length = (int) $options['length'];
         $this->replace = $options['replace'];
         $this->translation = $options['translation'];
     }

Tests/AdminBundle/Field/ArrayFieldTest.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 {
     public function testRender()
     {
-        $arrayField = new ArrayField ();
+        $arrayField = new ArrayField();
         $arrayField->setOptions([
             'glue' => ', '
         ]);

Tests/AdminBundle/Admin/Factory/AdminFactoryTest.php

Spacing +3 added lines, -3 removed lines

@@ -81,7 +81,7 @@ 
         // test with no configuration
         $adminFactory = $this->mockAdminFactory();
         // unknow admin not exists, it should throw an exception
-        $this->assertExceptionRaised('Exception', function () use ($adminFactory) {
+        $this->assertExceptionRaised('Exception', function() use ($adminFactory) {
             $adminFactory->getAdmin('unknown_admin');
         });
         // test with configurations samples
@@ -105,7 +105,7 @@ 
         // test with no configuration
         $adminFactory = $this->mockAdminFactory();
         // unknow admin not exists, it should throw an exception
-        $this->assertExceptionRaised('Exception', function () use ($adminFactory) {
+        $this->assertExceptionRaised('Exception', function() use ($adminFactory) {
             $adminFactory->getAdmin('unknown_admin');
         });
         // test with configurations samples
@@ -129,7 +129,7 @@ 
         // test with no configuration
         $adminFactory = $this->mockAdminFactory();
         // unknow admin not exists, it should throw an exception
-        $this->assertExceptionRaised('Exception', function () use ($adminFactory) {
+        $this->assertExceptionRaised('Exception', function() use ($adminFactory) {
             $adminFactory->getAdmin('unknown_admin');
         });
         $dataProvider = $this->mockDataProvider();

Tests/AdminBundle/Admin/AdminTest.php

Spacing +5 added lines, -5 removed lines

@@ -40,13 +40,13 @@ 
 
 
             // with no action, handleRequest method SHOULD throw an exception
-            $this->assertExceptionRaised('Exception', function () use ($admin) {
+            $this->assertExceptionRaised('Exception', function() use ($admin) {
                 $request = new Request();
                 $admin->handleRequest($request);
             });
 
             // with a wrong action, handleRequest method SHOULD throw an exception
-            $this->assertExceptionRaised('Exception', function () use ($admin) {
+            $this->assertExceptionRaised('Exception', function() use ($admin) {
                 $request = new Request([], [], [
                     '_route_params' => [
                         '_action' => 'bad_action'
@@ -75,13 +75,13 @@ 
             $this->doTestAdmin($admin, $configuration, $adminName);
 
             // with a current action unset, checkPermissions method SHOULD throw an exception
-            $this->assertExceptionRaised('Exception', function () use ($admin) {
+            $this->assertExceptionRaised('Exception', function() use ($admin) {
                 $user = new User('JohnKrovitch', 'john1234');
                 $admin->checkPermissions($user);
             });
 
             // with the wrong roles, checkPermissions method SHOULD throw an exception
-            $this->assertExceptionRaised(NotFoundHttpException::class, function () use ($admin) {
+            $this->assertExceptionRaised(NotFoundHttpException::class, function() use ($admin) {
                 $request = new Request([], [], [
                     '_route_params' => [
                         '_action' => 'custom_list'
@@ -93,7 +93,7 @@ 
             });
 
             // with the wrong roles, checkPermissions method SHOULD throw an exception
-            $this->assertExceptionRaised(NotFoundHttpException::class, function () use ($admin) {
+            $this->assertExceptionRaised(NotFoundHttpException::class, function() use ($admin) {
                 $request = new Request([], [], [
                     '_route_params' => [
                         '_action' => 'custom_list'