larriereguichet / AdminBundle

Admin/Action.php

Doc Comments +1 added lines, -1 removed lines

@@ -106,7 +106,7 @@
      * Return action configuration resolver
      *
      * @param OptionsResolver $resolver
-     * @param $actionName
+     * @param string $actionName
      * @param Admin|null $admin
      */
     protected function configureOptionsResolver(OptionsResolver $resolver, $actionName, Admin $admin = null)

Routing/RoutingLoader.php

Spacing +2 added lines, -2 removed lines

@@ -95,7 +95,7 @@ 
 
         // by default, generic controller
         $defaults = [
-            '_controller' => $admin->getConfiguration()->getParameter('controller') . ':' . $action->getName(),
+            '_controller' => $admin->getConfiguration()->getParameter('controller').':'.$action->getName(),
             '_admin' => $admin->getName(),
             '_action' => $action->getName(),
         ];
@@ -148,7 +148,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;
     }

Action/Configuration/ActionConfiguration.php

Spacing +4 added lines, -4 removed lines

@@ -82,11 +82,11 @@ 
         $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
@@ -114,7 +114,7 @@ 
             ->addAllowedValues('load_strategy', AdminInterface::LOAD_STRATEGY_NONE)
             ->addAllowedValues('load_strategy', AdminInterface::LOAD_STRATEGY_UNIQUE)
             ->addAllowedValues('load_strategy', AdminInterface::LOAD_STRATEGY_MULTIPLE)
-            ->setNormalizer('load_strategy', function (Options $options, $value) {
+            ->setNormalizer('load_strategy', function(Options $options, $value) {
 
                 if (!$value) {
                     if ($this->actionName == 'create') {
@@ -139,7 +139,7 @@ 
         // 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 = [

Application/Configuration/ApplicationConfiguration.php

Spacing +4 added lines, -4 removed lines

@@ -69,7 +69,7 @@ 
             'name_pattern' => 'lag.admin.{admin}',
         ]);
         $resolver->setAllowedTypes('routing', 'array');
-        $resolver->setNormalizer('routing', function (Options $options, $value) {
+        $resolver->setNormalizer('routing', function(Options $options, $value) {
 
             // url pattern should contain {admin} and {action} token
             $urlPattern = $value['url_pattern'];
@@ -97,7 +97,7 @@ 
             'pattern' => 'lag.admin.{key}'
         ]);
         $resolver->setAllowedTypes('translation', 'array');
-        $resolver->setNormalizer('translation', function (Options $options, $value) {
+        $resolver->setNormalizer('translation', function(Options $options, $value) {
 
             if (!is_bool($value['enabled'])) {
                 throw new InvalidOptionsException('Admin translation enabled parameter should be a boolean');
@@ -128,7 +128,7 @@ 
 
         $resolver->setDefault('fields_mapping', $defaultMapping);
         $resolver->setAllowedTypes('fields_mapping', 'array');
-        $resolver->setNormalizer('fields_mapping', function (Options $options, $value) use ($defaultMapping) {
+        $resolver->setNormalizer('fields_mapping', function(Options $options, $value) use ($defaultMapping) {
             // merge with default mapping to allow override
             $value = array_merge($defaultMapping, $value);
 
@@ -142,7 +142,7 @@ 
 
         $resolver->setDefault('fields_template_mapping', $defaultMapping);
         $resolver->setAllowedTypes('fields_template_mapping', 'array');
-        $resolver->setNormalizer('fields_template_mapping', function (Options $options, $value) use ($defaultMapping) {
+        $resolver->setNormalizer('fields_template_mapping', function(Options $options, $value) use ($defaultMapping) {
             // merge with default mapping to allow override
             $value = array_merge($defaultMapping, $value);
 

Configuration/Factory/ConfigurationFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -106,7 +106,7 @@
      * Return action configuration resolver
      *
      * @param OptionsResolver $resolver
-     * @param $actionName
+     * @param string $actionName
      * @param Admin|null $admin
      */
     protected function configureOptionsResolver(OptionsResolver $resolver, $actionName, Admin $admin = null)

Menu/Configuration/MenuItemConfiguration.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
         // user can defined an admin name
         $resolver
             ->setDefault('admin', null)
-            ->setNormalizer('admin', function (Options $options, $adminName) {
+            ->setNormalizer('admin', function(Options $options, $adminName) {
 
                 // user has to defined either an admin name and an action name, or a route name with optional
                 // parameters, or an url
@@ -43,7 +43,7 @@ 
                 // if an action name is provided, an admin name should be defined too
                 if ($action !== null && $options->offsetGet('admin') === null) {
                     throw new InvalidOptionsException(
-                        'You should provide an admin name for this action ' . $action
+                        'You should provide an admin name for this action '.$action
                     );
                 }