Merge pull request #36 from larriereguichet/scrutinizer-patch-1

Scrutinizer Auto-Fixes
larriereguichet · Jun 13, 2016 · ceb2acc · ceb2acc
2 parents 01610a3 + 857b505
commit ceb2acc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/Action/Configuration/ActionConfiguration.php b/Action/Configuration/ActionConfiguration.php
@@ -109,11 +109,11 @@ public function configureOptions(OptionsResolver $resolver)
         $resolver
             ->setDefault('route', '')
             ->setAllowedTypes('route', 'string')
-            ->setNormalizer('route', function (Options $options, $value) {
+            ->setNormalizer('route', function(Options $options, $value) {
                 if (!$value) {
                     // if no route was provided, it should be linked to an Admin
                     if (!$this->admin) {
-                        throw new InvalidOptionsException('No route was provided for action : ' . $this->actionName);
+                        throw new InvalidOptionsException('No route was provided for action : '.$this->actionName);
                     }
 
                     // generate default route from admin
@@ -142,7 +142,7 @@ public function configureOptions(OptionsResolver $resolver)
             ->addAllowedValues('load_strategy', AdminInterface::LOAD_STRATEGY_UNIQUE)
             ->addAllowedValues('load_strategy', AdminInterface::LOAD_STRATEGY_MULTIPLE)
             ->addAllowedValues('load_strategy', null)
-            ->setNormalizer('load_strategy', function (Options $options, $value) {
+            ->setNormalizer('load_strategy', function(Options $options, $value) {
 
                 if (!$value) {
                     if ($this->actionName == 'create') {
@@ -167,7 +167,7 @@ public function configureOptions(OptionsResolver $resolver)
         // criteria used to find entity in the data provider
         $resolver
             ->setDefault('criteria', [])
-            ->setNormalizer('criteria', function (Options $options, $value) {
+            ->setNormalizer('criteria', function(Options $options, $value) {
 
                 if (!$value) {
                     $idActions = [
@@ -192,7 +192,7 @@ public function configureOptions(OptionsResolver $resolver)
         // menus
         $resolver
             ->setDefault('menus', [])
-            ->setNormalizer('menus', function (Options $options, $menus) {
+            ->setNormalizer('menus', function(Options $options, $menus) {
                 // set default to an array
                 if ($menus === false) {
                     $menus = [];

diff --git a/Controller/CRUDController.php b/Controller/CRUDController.php
@@ -110,7 +110,7 @@ public function batchAction(Request $request)
                 $admin->remove();
             }
         } else {
-            throw new NotFoundHttpException('Invalid batch parameters : ' . $form->getErrors(true, false));
+            throw new NotFoundHttpException('Invalid batch parameters : '.$form->getErrors(true, false));
         }
         // redirect to list view
         return $this->redirectToRoute($admin->generateRouteName('list'));