remove unused imports using php-cs-fixer (#562)

Author: simonLeary42

.gitignore

@@ -28,6 +28,7 @@ deployment/**/*
 !deployment/templates_overrides/README.md
 
 .phpunit.result.cache
+.php-cs-fixer.cache
 
 node_modules
 # copied from node_modules using the copy-files-from-to NPM package

.php-cs-fixer.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+use PhpCsFixer\Config;
+use PhpCsFixer\Finder;
+use PhpCsFixer\Runner\Parallel\ParallelConfigFactory;
+
+return (new Config())
+    ->setParallelConfig(ParallelConfigFactory::detect()) // @TODO 4.0 no need to call this manually
+    ->setRules(["no_unused_imports" => true])
+    // by default, Fixer looks for `*.php` files excluding `./vendor/`
+    ->setFinder((new Finder())->in(__DIR__)->exclude([__DIR__ . "/resources/lib/phpopenldaper"]));

.pre-commit-config.yaml

@@ -49,6 +49,12 @@ repos:
           (?x)^(
               resources/mail/user_flag_.*\.php|
           )$
+      - id: php-cs-fixer
+        name: PHP CS Fixer
+        entry: ./vendor/bin/php-cs-fixer fix
+        language: system
+        files: \.php$
+        pass_filenames: false
 
   - repo: https://github.com/rbubley/mirrors-prettier
     rev: v3.6.2

composer.json

@@ -10,7 +10,8 @@
         "robiningelbrecht/phpunit-coverage-tools": "<2",
         "squizlabs/php_codesniffer": "*",
         "phpstan/phpstan": "^2.1",
-        "rawr/phpunit-data-provider": "^3.3"
+        "rawr/phpunit-data-provider": "^3.3",
+        "friendsofphp/php-cs-fixer": "^3.92"
     },
     "config": {
         "platform": {