larriereguichet / AdminBundle

Tests/bootstrap.php

Spacing +3 added lines, -3 removed lines

@@ -4,13 +4,13 @@ 
 
 //use Symfony\Component\HttpFoundation\Request;
 
-if (!is_file($loaderFile = __DIR__.'/../vendor/autoload.php') && !is_file($loaderFile = __DIR__.'/../../../../../../vendor/autoload.php')) {
+if (!is_file($loaderFile = __DIR__ . '/../vendor/autoload.php') && !is_file($loaderFile = __DIR__ . '/../../../../../../vendor/autoload.php')) {
     throw new LogicException('Could not find autoload.php in vendor/. Did you run "composer install --dev"?');
 }
 
 /** @var \Composer\Autoload\ClassLoader $loader */
 $loader = require $loaderFile;
-$loader->add('', __DIR__.'/src/');
+$loader->add('', __DIR__ . '/src/');
 $loader->register();
 
 //require_once __DIR__ . '/src/Entity/TestEntity.php';
@@ -30,4 +30,4 @@ 
 //$loader = require_once __DIR__.'/app/bootstrap.php.cache';
 
 //require_once __DIR__ . '/src/Test/TestBundle/TestTestBundle.php';
-require_once __DIR__.'/app/AppKernel.php';
+require_once __DIR__ . '/app/AppKernel.php';

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

Doc Comments +1 added lines, -1 removed lines

@@ -139,7 +139,7 @@
     /**
      * @param AdminInterface $admin
      * @param array $configuration
-     * @param $adminName
+     * @param string $adminName
      */
     protected function doTestAdmin(AdminInterface $admin, array $configuration, $adminName)
     {

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();

Admin/Admin.php

Doc Comments +3 added lines, -3 removed lines

@@ -156,7 +156,7 @@ 
     /**
      * Check if user is allowed to be here
      *
-     * @param UserInterface|string $user
+     * @param null|\Symfony\Component\Security\Core\User\User $user
      * @throws Exception
      */
     public function checkPermissions($user)
@@ -325,7 +325,7 @@ 
     /**
      * Return loaded entities
      *
-     * @return mixed
+     * @return ArrayCollection
      */
     public function getEntities()
     {
@@ -401,7 +401,7 @@ 
     }
 
     /**
-     * @return array
+     * @return integer[]
      */
     public function getActionNames()
     {