larriereguichet / AdminBundle

Admin/Configuration/ApplicationConfiguration.php

Spacing +3 added lines, -3 removed lines

@@ -146,7 +146,7 @@ 
             'url_pattern',
             'name_pattern',
         ]);
-        $resolver->setNormalizer('url_pattern', function (Options $options, $value) {
+        $resolver->setNormalizer('url_pattern', function(Options $options, $value) {
             if (strstr($value, '{admin}') === false) {
                 throw new InvalidOptionsException('Admin routing configuration url pattern should contains {admin} placeholder');
             }
@@ -156,7 +156,7 @@ 
 
             return $value;
         });
-        $resolver->setNormalizer('name_pattern', function (Options $options, $value) {
+        $resolver->setNormalizer('name_pattern', function(Options $options, $value) {
             if (strstr($value, '{admin}') === false) {
                 throw new InvalidOptionsException('Admin routing configuration pattern name should contains {admin} placeholder');
             }
@@ -174,7 +174,7 @@ 
             ->clear()
             ->setDefault('enabled', true)
             ->setDefault('pattern', 'lag.admin.{key}');
-        $resolver->setNormalizer('pattern', function (Options $options, $value) {
+        $resolver->setNormalizer('pattern', function(Options $options, $value) {
             if (strstr($value, 'key') === false) {
                 throw new InvalidOptionsException('Admin translation configuration pattern should contains {key} placeholder');
             }

Doc Comments +6 added lines, -6 removed lines

@@ -223,7 +223,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getTitle()
     {
@@ -239,7 +239,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getLayout()
     {
@@ -255,7 +255,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getBlockTemplate()
     {
@@ -271,7 +271,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getDescription()
     {
@@ -303,7 +303,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getDateFormat()
     {
@@ -397,7 +397,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return integer
      */
     public function getStringLengthTruncate()
     {

Admin/Factory/ActionFactory.php

Spacing +3 added lines, -3 removed lines

@@ -157,7 +157,7 @@ 
                 Admin::LOAD_STRATEGY_UNIQUE,
                 Admin::LOAD_STRATEGY_MULTIPLE,
             ])
-            ->setNormalizer('route', function (Options $options, $value) use ($admin, $actionName) {
+            ->setNormalizer('route', function(Options $options, $value) use ($admin, $actionName) {
                 if (!$value) {
                     // if no route was provided, it should be linked to an Admin
                     if (!$admin) {
@@ -168,7 +168,7 @@ 
                 }
                 return $value;
             })
-            ->setNormalizer('title', function (Options $options, $value) use ($admin, $actionName) {
+            ->setNormalizer('title', function(Options $options, $value) use ($admin, $actionName) {
                 if (!$value) {
                     $adminKey = '';
                     // if an Admin is linked to this action, we use its name in translation key
@@ -179,7 +179,7 @@ 
                 }
                 return $value;
             })
-            ->setNormalizer('batch', function (Options $options, $value) use ($admin, $actionName) {
+            ->setNormalizer('batch', function(Options $options, $value) use ($admin, $actionName) {
                 if ($value) {
                     if (!is_array($value)) {
                         $value = [$value];

DependencyInjection/FieldCompilerPass.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
         // foreach tagged, with add this field type to the field factory
         foreach ($taggedServices as $id => $tags) {
             if (empty($tags[0]['type'])) {
-                throw new InvalidConfigurationException('You should defined a "type" attribute for field tag for service '.$id);
+                throw new InvalidConfigurationException('You should defined a "type" attribute for field tag for service ' . $id);
             }
             // add allowed field type to the field factory
             $definition->addMethodCall('addFieldMapping', [

Form/Handler/ListFormHandler.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
         }
         // check if they exists in entities displayed and if checkbox is checked
         foreach ($batchItems as $name => $batchItem) {
-            $batchId = (int)str_replace('batch_', '', $name);
+            $batchId = (int) str_replace('batch_', '', $name);
 
             if (array_key_exists($batchId, $cleanData['labels']) && $batchItem === true) {
                 $cleanData['ids'][] = $batchId;

Form/Type/AdminListType.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
                     ]
                 ])
             ;
-            $builder->addEventListener(FormEvents::PRE_SET_DATA, function (FormEvent $event) {
+            $builder->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event) {
                 $data = $event->getData();
                 $form = $event->getForm();
 

Routing/RoutingLoader.php

Spacing +2 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
         $path = str_replace('{action}', $action->getName(), $path);
         // by default, generic controller
         $defaults = [
-            '_controller' => $admin->getConfiguration()->getControllerName().':'.$action->getName(),
+            '_controller' => $admin->getConfiguration()->getControllerName() . ':' . $action->getName(),
             '_admin' => $admin->getName(),
             '_action' => $action->getName(),
         ];
@@ -135,7 +135,7 @@ 
     {
         $array = explode('\\', $namespace);
         $path = array_pop($array);
-        $path = strtolower(substr($path, 0, 1)).substr($path, 1);
+        $path = strtolower(substr($path, 0, 1)) . substr($path, 1);
 
         return $path;
     }

Admin/Admin.php

Doc Comments +1 added lines, -1 removed lines

@@ -358,7 +358,7 @@
     }
 
     /**
-     * @return array
+     * @return integer[]
      */
     public function getActionNames()
     {

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