Merge pull request #178 from armanist/177-Integrate-PHP-CS-Fixer-into-Quantum-Starter-Project

Integrate php cs fixer into quantum starter project

Author: armanist

.github/workflows/php.yml

@@ -42,6 +42,9 @@ jobs:
     - name: Install dependencies
       run: composer update --prefer-dist --no-progress
 
+    - name: Run PHP-CS-Fixer
+      run: composer cs:check
+
     - name: Create env
       run: php qt core:env
 

.gitignore

@@ -1,8 +1,30 @@
 /vendor
 /.idea
 /nbproject/private/
-/shared/store/*
-/shared/emails/*
 .env
 composer.lock
-.phpunit.result.cache
+.phpunit.result.cache
+.php-cs-fixer.cache
+debug.log
+
+# Runtime data
+/shared/store/*
+/shared/emails/*
+/shared/config/auth.php
+/shared/config/modules.php
+
+# Generated modules
+/modules
+
+# Published assets
+/public/assets/DebugBar
+/public/assets/OpenApiUi
+/public/assets/Web
+/public/uploads
+
+# Test runtime
+/tests/_root/.env.testing
+/tests/_root/modules
+/tests/_root/shared/store/*
+/tests/_root/shared/config/auth.php
+/tests/_root/logs

.php-cs-fixer.php

@@ -0,0 +1,36 @@
+<?php
+
+declare(strict_types=1);
+
+use PhpCsFixer\Config;
+use PhpCsFixer\Finder;
+
+$finder = Finder::create()
+    ->in(__DIR__)
+    ->exclude([
+        'vendor',
+        'modules',
+        'public/assets',
+        'public/uploads',
+    ])
+    ->name('*.php');
+
+return (new Config())
+    ->setRiskyAllowed(false)
+    ->setRules([
+        // Base standard
+        '@PSR12' => true,
+
+        // Syntax & consistency
+        'array_syntax' => ['syntax' => 'short'],
+        'binary_operator_spaces' => ['default' => 'single_space'],
+        'cast_spaces' => ['space' => 'single'],
+        'concat_space' => ['spacing' => 'one'],
+        'single_quote' => true,
+        'trailing_comma_in_multiline' => true,
+        'no_unused_imports' => true,
+        'no_extra_blank_lines' => true,
+        'blank_line_after_namespace' => true,
+        'blank_line_after_opening_tag' => true,
+    ])
+    ->setFinder($finder);

composer.json

@@ -5,6 +5,7 @@
     "license": "MIT",
     "type": "project",
     "require": {
+        "php": "^7.4 || ^8.0",
         "quantum/framework": "dev-master",
         "fakerphp/faker": "^1.15",
         "bluemmb/faker-picsum-photos-provider": "^2.0",
@@ -13,7 +14,8 @@
     },
     "require-dev": {
         "phpunit/phpunit": "^9.0",
-        "mockery/mockery": "^1.2"
+        "mockery/mockery": "^1.2",
+        "friendsofphp/php-cs-fixer": "^3.94"
     },
     "autoload": {
         "psr-4": {
@@ -35,6 +37,8 @@
             "php qt install:debugbar",
             "php qt core:version"
         ],
+        "cs:check": "vendor/bin/php-cs-fixer check",
+        "cs:fix": "vendor/bin/php-cs-fixer fix",
         "test": "vendor/bin/phpunit --stderr --coverage-clover coverage.xml"
     },
     "config": {

helpers/functions.php

@@ -26,7 +26,7 @@
  */
 function url_with_lang(string $lang): string
 {
-    if (!in_array($lang, (array)config()->get('lang.supported'))) {
+    if (!in_array($lang, (array) config()->get('lang.supported'))) {
         $lang = config()->get('lang.default');
     }
 
@@ -35,7 +35,7 @@ function url_with_lang(string $lang): string
     }
 
     if (preg_match('/' . preg_quote(current_lang()) . '/', route_uri())) {
-        return base_url() . preg_replace('/' . preg_quote(current_lang(), '/') . "/", $lang, route_uri(), 1);
+        return base_url() . preg_replace('/' . preg_quote(current_lang(), '/') . '/', $lang, route_uri(), 1);
     }
 
     $url = base_url(true);
@@ -47,7 +47,7 @@ function url_with_lang(string $lang): string
     if (!empty(route_prefix()) && $langSegmentIndex == 1) {
         $langSegmentIndex += 1;
 
-        $uri = preg_replace('/' . preg_quote(route_prefix(), '/') . "/", '', route_uri(), 1);
+        $uri = preg_replace('/' . preg_quote(route_prefix(), '/') . '/', '', route_uri(), 1);
     }
 
     $segments = explode('/', $uri);
@@ -103,7 +103,7 @@ function create_user_directory(string $uuid)
 {
     $userDirectory = uploads_dir() . DS . $uuid;
 
-    if(!fs()->isDirectory($userDirectory)) {
+    if (!fs()->isDirectory($userDirectory)) {
         fs()->makeDirectory($userDirectory);
     }
 }
@@ -143,4 +143,4 @@ function nav_ref_decode(?string $ref): string
     $decoded = $ref ? base64_decode(strtr($ref, '-_', '+/'), true) : false;
 
     return '/posts' . ($decoded ? '?' . $decoded : '');
-}
+}

migrations/create_table_comments_1698145440.php

@@ -5,7 +5,6 @@
 
 class Create_table_comments_1698145440 extends QtMigration
 {
-
     public function up(?TableFactory $tableFactory)
     {
         $table = $tableFactory->create('comments');

migrations/create_table_posts_1669639752.php

@@ -5,7 +5,6 @@
 
 class Create_table_posts_1669639752 extends QtMigration
 {
-
     public function up(?TableFactory $tableFactory)
     {
         $table = $tableFactory->create('posts');

migrations/create_table_users_1669639740.php

@@ -5,7 +5,6 @@
 
 class Create_table_users_1669639740 extends QtMigration
 {
-
     public function up(?TableFactory $tableFactory)
     {
         $table = $tableFactory->create('users');

public/index.php

@@ -15,12 +15,13 @@
  * Directory separator
  * -----------------------------------------------------------------------------
  */
-if (!defined('DS')) define('DS', DIRECTORY_SEPARATOR);
-
+if (!defined('DS')) {
+    define('DS', DIRECTORY_SEPARATOR);
+}
 
 /*
  * -----------------------------------------------------------------------------
  * Starting the app
  * -----------------------------------------------------------------------------
  */
-AppFactory::create(AppType::WEB, dirname(__DIR__))->start();
+AppFactory::create(AppType::WEB, dirname(__DIR__))->start();

shared/Commands/CommandValidationTrait.php

@@ -26,7 +26,6 @@
  */
 trait CommandValidationTrait
 {
-
     /**
      * @var Validator
      */
@@ -78,4 +77,4 @@ public function firstError(): ?string
 
         return null;
     }
-}
+}