acelaya / website-expressive

src/Action/Factory/ActionAbstractFactory.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace Acelaya\Website\Action\Factory;
 
 use Acelaya\Website\Action\AbstractAction;
-use Doctrine\Common\Cache\Cache;
 use Zend\Expressive\Template\TemplateInterface;
 use Zend\ServiceManager\AbstractFactoryInterface;
 use Zend\ServiceManager\ServiceLocatorInterface;

src/Form/ContactFilter.php

Doc Comments +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
     }
 
     /**
-     * @param $name
+     * @param string $name
      * @param bool|true $required
      * @return Input
      */

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace Acelaya\Website\Form;
 
-use ReCaptcha\ReCaptcha;
 use Zend\Filter\StringTrim;
 use Zend\Filter\StripTags;
 use Zend\InputFilter\Input;

Indentation +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     {
         $input = new Input($name);
         $input->setRequired($required)
-              ->getFilterChain()->attach(new StripTags())
+                ->getFilterChain()->attach(new StripTags())
                                 ->attach(new StringTrim());
         return $input;
     }

src/Form/RecaptchaValidator.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace Acelaya\Website\Form;
 
-use ReCaptcha\ReCaptcha;
 use Zend\Validator\AbstractValidator;
 use Zend\Validator\Exception;
 

config/autoload/routes.global.php

Spacing +2 added lines, -2 removed lines

@@ -6,7 +6,7 @@ 
 $home = [
     'name' => 'home',
     'path' => '(/:lang)/',
-    'allowed_methods' => ['GET'],
+    'allowed_methods' => [ 'GET' ],
     'middleware' => Template::class,
     'options' => [
         'conditions' => [
@@ -43,7 +43,7 @@ 
         ArrayUtils::merge($home, [
             'name' => 'contact',
             'path' => '(/:lang)/contact/',
-            'allowed_methods' => ['POST'],
+            'allowed_methods' => [ 'POST' ],
             'middleware' => Contact::class,
             'options' => [
                 'defaults' => [

config/container.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 
 // Create a ServiceManager from service_manager config and register the merged config as a service
 $config = include __DIR__ . '/config.php';
-$sm = new ServiceManager(new Config(isset($config['service_manager']) ? $config['service_manager'] : []));
+$sm = new ServiceManager(new Config(isset($config[ 'service_manager' ]) ? $config[ 'service_manager' ] : [ ]));
 $sm->setService('config', $config);
 
 // Return the fully configured ServiceManager

public/index.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 if (class_exists(Dotenv::class)) {
     $dotenv = new Dotenv(__DIR__ . '/..');
     $dotenv->load();
-    $dotenv->required('APP_ENV')->allowedValues(['pro', 'dev']);
+    $dotenv->required('APP_ENV')->allowedValues([ 'pro', 'dev' ]);
 }
 
 // Set error reporting

src/Middleware/CacheMiddleware.php

Spacing +3 added lines, -3 removed lines

@@ -76,10 +76,10 @@
      * @param array $routeParams
      * @return bool
      */
-    protected function isResponseCacheable(Response $resp, array $routeParams = [])
+    protected function isResponseCacheable(Response $resp, array $routeParams = [ ])
     {
         return $resp->getStatusCode() === 200
-            && isset($routeParams['cacheable'])
-            && $routeParams['cacheable'] === true;
+            && isset($routeParams[ 'cacheable' ])
+            && $routeParams[ 'cacheable' ] === true;
     }
 }

src/Middleware/LanguageMiddleware.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
         $params = $matchedRoute->getMatchedParams();
 
         // Determine the language to use based on the lang parameter
-        $lang = isset($params['lang']) ? $params['lang'] : 'en';
+        $lang = isset($params[ 'lang' ]) ? $params[ 'lang' ] : 'en';
         $this->translator->setLocale($lang);
         return $out($request, $response);
     }

src/Action/Contact.php

Spacing +3 added lines, -3 removed lines

@@ -49,7 +49,7 @@ 
     public function dispatch(ServerRequestInterface $request, ResponseInterface $response, callable $next = null)
     {
         // On GET requests that are not comming from PRG, just return the template
-        if ($request->getMethod() === 'GET' && ! $this->container->offsetExists(self::PRG_DATA)) {
+        if ($request->getMethod() === 'GET' && !$this->container->offsetExists(self::PRG_DATA)) {
             return $this->createTemplateResponse($request);
         }
 
@@ -65,7 +65,7 @@ 
         $this->container->offsetUnset(self::PRG_DATA);
         $filter = $this->contactFilter;
         $filter->setData($params);
-        if (! $filter->isValid()) {
+        if (!$filter->isValid()) {
             return $this->createTemplateResponse($request, [
                 'errors' => $filter->getMessages(),
                 'currentData' => $params
@@ -74,6 +74,6 @@ 
 
         // If the form is valid, send the email
         $result = $this->contactService->send($filter->getValues());
-        return $this->createTemplateResponse($request, $result ? ['success' => true] : ['errors' => []]);
+        return $this->createTemplateResponse($request, $result ? [ 'success' => true ] : [ 'errors' => [ ] ]);
     }
 }