PHPCS #22

composer.json

@@ -12,7 +12,8 @@
         "cakephp/plugin-installer": "*",
         "holt59/cakephp3-bootstrap-helpers": "dev-master",
         "fzaninotto/Faker": "^1.6",
-        "cakephp/localized": "dev-master"
+        "cakephp/localized": "dev-master",
+        "cakephp/cakephp-codesniffer": "dev-master"
     },
     "require-dev": {
         "psy/psysh": "@stable",

config/Migrations/20160707205730_AlterAddUnqueAttributeForLoginEmail.php

@@ -12,4 +12,3 @@ public function down()
     {
     }
 }
-

config/Migrations/20160709184233_AddUserIdToPosts.php

@@ -19,7 +19,7 @@ public function change()
                   'null' => true
               ])
               ->addIndex(['user_id'], ['name' => 'fk_posts_users_idx'])
-              ->addForeignKey('user_id', 'users', 'id', ['delete'=> 'SET_NULL', 'update'=> 'NO_ACTION'])
+              ->addForeignKey('user_id', 'users', 'id', ['delete' => 'SET_NULL', 'update' => 'NO_ACTION'])
               ->save();
     }
 }

config/Migrations/20160709210604_AddDisplayNameColumn.php

@@ -24,4 +24,3 @@ public function down()
             ->update();
     }
 }
-

config/Migrations/20160709213450_ChangePasswordLength.php

@@ -26,4 +26,3 @@ public function down()
             ->update();
     }
 }
-

config/Migrations/20160710160333_AlterContentToText.php

@@ -7,7 +7,7 @@ class AlterContentToText extends AbstractMigration
     public function change()
     {
         $this->table('posts')
-        ->changeColumn('content', 'text',[
+        ->changeColumn('content', 'text', [
             'default' => null,
             'limit' => 4294967295,
             'null' => false

config/Migrations/20160714150701_AddStatusColumnInPosts.php

@@ -25,13 +25,13 @@ class AddStatusColumnInPosts extends AbstractMigration
      * Remember to call "create()" or "update()" and NOT "save()" when working
      * with the Table class.
      */
-     public function change()
-     {
-         $this->table('posts')
-         ->addColumn('state', 'enum', [
-             'default' => null,
-             'values' => ['published', 'pinned', 'draft'],
-             'null' => false,
-         ])->update();
-     }
+    public function change()
+    {
+        $this->table('posts')
+        ->addColumn('state', 'enum', [
+           'default' => null,
+           'values' => ['published', 'pinned', 'draft'],
+           'null' => false,
+        ])->update();
+    }
 }

config/Migrations/20160718160356_AddPostType.php

@@ -30,4 +30,3 @@ public function down()
             ->update();
     }
 }
-

config/Migrations/20160719203719_AlterAllowCommentToBeEmpty.php

@@ -26,4 +26,3 @@ public function down()
             ->update();
     }
 }
-

config/Seeds/PostsSeed.php

@@ -23,7 +23,7 @@ public function run()
         $faker = Faker\Factory::create();
         $data = [];
 
-        for ($i=0; $i < 100; $i++) {
+        for ($i = 0; $i < 100; $i++) {
             $data[] = [
                 'title' => $faker->sentence(5, true),
                 'content' => '<p>' . implode('</p><p>', $faker->paragraphs(rand(3, 20), false)) . '</p>',

config/Seeds/SignaturesSeed.php

@@ -22,7 +22,7 @@ public function run()
 
         $data = [];
 
-        for ($i=0; $i < 100; $i++) {
+        for ($i = 0; $i < 100; $i++) {
             $data[] = [
                 'first_name' => $faker->firstName,
                 'last_name' => $faker->lastName,

config/Seeds/UsersSeed.php

@@ -1,6 +1,7 @@
 <?php
 use Migrations\AbstractSeed;
 use Cake\Auth\DefaultPasswordHasher;
+
 /**
  * Users seed.
  */
@@ -19,7 +20,8 @@ class UsersSeed extends AbstractSeed
     public function run()
     {
         $hasher = new DefaultPasswordHasher();
-        $faker = $faker = Faker\Factory::create();;
+        $faker = $faker = Faker\Factory::create();
+        ;
         $data = [
             [
                 'login' => 'user1',
@@ -41,7 +43,7 @@ public function run()
             ]
         ];
 
-        for ($i=0; $i < 20; $i++) {
+        for ($i = 0; $i < 20; $i++) {
             $data[] = [
                 'login' => $faker->lastName . $faker->numberBetween(1, 5555),
                 'email' => $faker->safeEmail,

config/bootstrap.php

@@ -162,10 +162,14 @@
  */
 Request::addDetector('mobile', function ($request) {
     $detector = new \Detection\MobileDetect();
+
+
     return $detector->isMobile();
 });
 Request::addDetector('tablet', function ($request) {
     $detector = new \Detection\MobileDetect();
+
+
     return $detector->isTablet();
 });
 

config/routes.php

@@ -45,17 +45,17 @@
 
 Router::scope('/auth', function (RouteBuilder $routes) {
     $routes->connect('/', ['controller' => 'Auth', 'action' => 'login'], ['_name' => 'auth.login']);
-    $routes->connect('/logout', ['controller' => 'Auth', 'action' => 'logout',], [ '_name' => 'auth.logout']);
+    $routes->connect('/logout', ['controller' => 'Auth', 'action' => 'logout', ], [ '_name' => 'auth.logout']);
 });
 
 Router::scope('/petition', function (RouteBuilder $routes) {
     $routes->connect('/', ['controller' => 'Petition', 'action' => 'index'], ['_name' => 'petition.index']);
-    $routes->connect('/form', ['controller' => 'Petition', 'action' => 'add',], [ '_name' => 'petition.add']);
-    $routes->connect('/thank-you', ['controller' => 'Petition', 'action' => 'thank',], [ '_name' => 'petition.thank']);
+    $routes->connect('/form', ['controller' => 'Petition', 'action' => 'add', ], [ '_name' => 'petition.add']);
+    $routes->connect('/thank-you', ['controller' => 'Petition', 'action' => 'thank', ], [ '_name' => 'petition.thank']);
 });
 
 
-Router::scope('/', function (RouteBuilder $routes){
+Router::scope('/', function (RouteBuilder $routes) {
     $routes->connect('/', ['controller' => 'Blog', 'action' => 'index']);
     $routes->connect('/:id/*', ['controller' => 'Blog', 'action' => 'view'], ['pass' => ['id'], 'id' => '\d+']);
 

src/Console/Installer.php

@@ -179,12 +179,16 @@ public static function setSecuritySalt($dir, $io)
 
         if ($count == 0) {
             $io->write('No Security.salt placeholder to replace.');
+
+
             return;
         }
 
         $result = file_put_contents($config, $content);
         if ($result) {
             $io->write('Updated Security.salt value in config/app.php');
+
+
             return;
         }
         $io->write('Unable to update Security.salt value.');

src/Controller/AppController.php

@@ -77,7 +77,7 @@ public function initialize()
               'controller' => 'Auth',
               'action' => 'login'
             ],
-            'unauthorizedRedirect'=>$this->referer()
+            'unauthorizedRedirect' => $this->referer()
 
         ]);
     }

src/Controller/AuthController.php

@@ -28,6 +28,8 @@ public function login()
             $user = $this->Auth->identify();
             if ($user) {
                 $this->Auth->setUser($user);
+
+
                 return $this->redirect($this->Auth->redirectUrl());
             }
             $this->Flash->error(__('Invalid username or password, try again'));
@@ -43,5 +45,4 @@ public function logout()
     {
         return $this->redirect($this->Auth->logout());
     }
-
 }

src/Controller/PetitionController.php

@@ -36,7 +36,8 @@ public function index()
         $this->add();
     }
 
-    public function listSignature(){
+    public function listSignature()
+    {
         $signatures = $this->paginate($this->Signatures);
 
         $this->set(compact('signatures'));
@@ -50,6 +51,8 @@ public function add()
             $petition = $this->Signatures->patchEntity($signature, $this->request->data);
             if ($this->Signatures->save($signature)) {
                 $this->Flash->success(__('The signature has been saved.'));
+
+
                 return $this->redirect(['action' => 'thank']);
             } else {
                 $this->Flash->error(__('The signature could not be saved. Please, try again.'));
@@ -66,8 +69,5 @@ public function add()
      */
     public function thank()
     {
-
     }
-
-
 }