Update the CI chain (#78)

* Updated CI and fixed CS

* Update the GitHub CI

.github/workflows/ci.yaml

@@ -2,40 +2,31 @@ name: CI
 
 on:
     pull_request: ~
-    push:
-        branches:
-            - '*'
-        tags:
-            - '*'
-    schedule:
-        - cron: 0 13 * * MON
 
 jobs:
     cs:
         name: Coding Style
         runs-on: ubuntu-latest
-        if: github.event_name != 'push'
         steps:
             - name: Setup PHP
               uses: shivammathur/setup-php@v2
 
             - name: Checkout
               uses: actions/checkout@v2
 
-            - name: Install ecs
-              run: cd tools/ecs && composer install --no-interaction --no-suggest
+            - name: Install the dependencies
+              run: composer install --no-interaction --no-suggest
 
             - name: Run the CS fixer
               run: composer cs-fixer
 
     tests:
         name: PHP ${{ matrix.php }}
         runs-on: ubuntu-latest
-        if: github.event_name != 'push'
         strategy:
             fail-fast: false
             matrix:
-                php: [8.1, 8.2]
+                php: [8.1, 8.2, 8.3]
         steps:
             - name: Setup PHP
               uses: shivammathur/setup-php@v2
@@ -45,9 +36,6 @@ jobs:
             - name: Checkout
               uses: actions/checkout@v2
 
-            - name: Install phpunit
-              run: cd tools/phpunit && composer install --no-interaction --no-suggest
-
             - name: Install the dependencies
               run: composer install --no-interaction --no-suggest
 
@@ -57,11 +45,10 @@ jobs:
     prefer-lowest-tests:
         name: PHP ${{ matrix.php }} --prefer-lowest
         runs-on: ubuntu-latest
-        if: github.event_name != 'push'
         strategy:
             fail-fast: false
             matrix:
-                php: [8.1, 8.2]
+                php: [8.1, 8.2, 8.3]
         steps:
             - name: Setup PHP
               uses: shivammathur/setup-php@v2
@@ -71,9 +58,6 @@ jobs:
             - name: Checkout
               uses: actions/checkout@v2
 
-            - name: Install phpunit
-              run: cd tools/phpunit && composer install --no-interaction --no-suggest
-
             - name: Install the dependencies
               run: composer update --prefer-lowest --prefer-stable --no-interaction --no-suggest
 

.gitignore

@@ -1,5 +1,3 @@
 /vendor/
-/tools/*/vendor
 /composer.lock
-/.php-cs-fixer.cache
-/tools/phpunit/.phpunit.result.cache
+/.phpunit.result.cache

composer.json

@@ -46,16 +46,20 @@
     "contao-manager-plugin": "Terminal42\\MultipageFormsBundle\\ContaoManager\\Plugin"
   },
   "scripts": {
-    "cs-fixer": "@php tools/ecs/vendor/bin/ecs check config/ contao/ src/ tests/ --config tools/ecs/config.php --fix --ansi",
-    "unit-tests": "@php tools/phpunit/vendor/bin/phpunit -c tools/phpunit/phpunit.xml.dist"
+    "unit-tests": "@php vendor/bin/phpunit"
   },
   "config": {
     "allow-plugins": {
       "contao-components/installer": false,
-      "contao/manager-plugin": false
+      "contao/manager-plugin": false,
+      "php-http/discovery": true,
+      "terminal42/contao-build-tools": true
     }
   },
   "require-dev": {
-    "contao/manager-plugin": "^2.12"
+    "contao/manager-plugin": "^2.12",
+    "terminal42/contao-build-tools": "@dev",
+    "phpunit/phpunit": "^9.6",
+    "contao/test-case": "^5.0"
   }
 }

tools/phpunit/phpunit.xml.dist → phpunit.xml.dist

@@ -2,7 +2,7 @@
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
          xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
          backupGlobals="false"
-         bootstrap="../../vendor/autoload.php"
+         bootstrap="vendor/autoload.php"
          colors="true"
 >
     <php>
@@ -12,7 +12,7 @@
 
     <testsuites>
         <testsuite name="MultipageForms Test Suite">
-            <directory>../../tests</directory>
+            <directory>tests</directory>
         </testsuite>
     </testsuites>
 </phpunit>

src/Controller/FrontendModule/StepsController.php

@@ -18,8 +18,10 @@
 #[AsFrontendModule('mp_form_steps', template: 'mod_mp_form_steps')]
 class StepsController extends AbstractFrontendModuleController
 {
-    public function __construct(private ContaoFramework $contaoFramework, private FormManagerFactoryInterface $formManagerFactory)
-    {
+    public function __construct(
+        private readonly ContaoFramework $contaoFramework,
+        private readonly FormManagerFactoryInterface $formManagerFactory,
+    ) {
     }
 
     protected function getResponse(Template $template, ModuleModel $model, Request $request): Response

src/EventListener/CompileFormFieldsListener.php

@@ -15,8 +15,10 @@
 #[AsHook('compileFormFields')]
 class CompileFormFieldsListener
 {
-    public function __construct(private FormManagerFactoryInterface $formManagerFactory, private RequestStack $requestStack)
-    {
+    public function __construct(
+        private readonly FormManagerFactoryInterface $formManagerFactory,
+        private readonly RequestStack $requestStack,
+    ) {
     }
 
     /**

src/EventListener/InsertTagsListener.php

@@ -11,7 +11,7 @@
 #[AsHook('replaceInsertTags')]
 class InsertTagsListener
 {
-    public function __construct(private FormManagerFactoryInterface $formManagerFactory)
+    public function __construct(private readonly FormManagerFactoryInterface $formManagerFactory)
     {
     }
 
@@ -28,14 +28,11 @@ public function __invoke(string $tag)
 
         $manager = $this->formManagerFactory->forFormId((int) $formId);
 
-        switch ($type) {
-            case 'step':
-                return $this->getStepValue($manager, $value);
-            case 'field_value':
-                return $manager->getDataOfAllSteps()->getAllSubmitted()[$value] ?? '';
-        }
-
-        return '';
+        return match ($type) {
+            'step' => $this->getStepValue($manager, $value),
+            'field_value' => $manager->getDataOfAllSteps()->getAllSubmitted()[$value] ?? '',
+            default => '',
+        };
     }
 
     private function getStepValue(FormManager $manager, string $value): string

src/EventListener/LoadFormFieldListener.php

@@ -13,7 +13,7 @@
 #[AsHook('loadFormField')]
 class LoadFormFieldListener
 {
-    public function __construct(private FormManagerFactoryInterface $formManagerFactory)
+    public function __construct(private readonly FormManagerFactoryInterface $formManagerFactory)
     {
     }
 
@@ -38,7 +38,7 @@ public function __invoke(Widget $widget, string $formId, array $formData, Form $
         // 3. The widget default value itself
         if (!$postData->has($widget->name)) {
             $widget->value = $stepData->getSubmitted()->get(
-                $widget->name, $stepData->getOriginalPostData()->get($widget->name, $widget->value)
+                $widget->name, $stepData->getOriginalPostData()->get($widget->name, $widget->value),
             );
         }
 

src/EventListener/PrepareFomDataListener.php

@@ -16,8 +16,10 @@
 #[AsHook('prepareFormData')]
 class PrepareFomDataListener
 {
-    public function __construct(private FormManagerFactoryInterface $formManagerFactory, private RequestStack $reqestStack)
-    {
+    public function __construct(
+        private readonly FormManagerFactoryInterface $formManagerFactory,
+        private readonly RequestStack $reqestStack,
+    ) {
     }
 
     /**
@@ -77,7 +79,7 @@ public function __invoke(array &$submitted, array &$labels, array $fields, Form
     private function getUploadedFiles($hook = []): FileParameterBag
     {
         // Contao 5
-        if (0 !== \count($hook)) {
+        if (0 !== (is_countable($hook) ? \count($hook) : 0)) {
             return new FileParameterBag($hook);
         }
 

src/FormManager.php

@@ -21,10 +21,13 @@
 class FormManager
 {
     private FormModel $formModel;
+
     private string $sessionRef;
+
     private string $storageIdentifier;
 
     private bool $prepared = false;
+
     private bool $preparing = false;
 
     /**
@@ -40,13 +43,13 @@ class FormManager
     private bool $isValidFormFieldCombination = true;
 
     public function __construct(
-        private int $formId,
-        private Request $request,
-        private ContaoFramework $contaoFramework,
-        private StorageInterface $storage,
-        private StorageIdentifierGeneratorInterface $storageIdentifierGenerator,
-        private SessionReferenceGeneratorInterface $sessionReferenceGenerator,
-        private UrlParser $urlParser,
+        private readonly int $formId,
+        private readonly Request $request,
+        private readonly ContaoFramework $contaoFramework,
+        private readonly StorageInterface $storage,
+        private readonly StorageIdentifierGeneratorInterface $storageIdentifierGenerator,
+        private readonly SessionReferenceGeneratorInterface $sessionReferenceGenerator,
+        private readonly UrlParser $urlParser,
     ) {
     }
 
@@ -230,7 +233,7 @@ public function getGetParamForSessionReference()
     /**
      * @throws RedirectResponseException
      */
-    public function redirectToStep(int $step): void
+    public function redirectToStep(int $step): never
     {
         $this->prepare();
         $this->validateStep($step);
@@ -354,7 +357,7 @@ private function loadFormFieldModels(): void
                 // Ignore the name of form fields which do not use a name (see contao/core-bundle #1268)
                 if (
                     $formFieldModel->name && isset($GLOBALS['TL_DCA']['tl_form_field']['palettes'][$formFieldModel->type])
-                    && preg_match('/[,;]name[,;]/', $GLOBALS['TL_DCA']['tl_form_field']['palettes'][$formFieldModel->type])
+                    && preg_match('/[,;]name[,;]/', (string) $GLOBALS['TL_DCA']['tl_form_field']['palettes'][$formFieldModel->type])
                 ) {
                     $formFieldModels[$formFieldModel->name] = $formFieldModel;
                 } else {
@@ -467,7 +470,7 @@ private function initSessionReference(bool $forceNew = false): void
 
         $this->sessionRef = $this->request->query->get(
             $this->getGetParamForSessionReference(),
-            $this->sessionReferenceGenerator->generate($this)
+            $this->sessionReferenceGenerator->generate($this),
         );
     }
 }

src/FormManagerFactory.php

@@ -19,7 +19,9 @@
 class FormManagerFactory implements FormManagerFactoryInterface
 {
     private StorageInterface|null $storage = null;
+
     private StorageIdentifierGeneratorInterface|null $storageIdentifierGenerator = null;
+
     private SessionReferenceGeneratorInterface|null $sessionReferenceGenerator = null;
 
     /**
@@ -28,9 +30,9 @@ class FormManagerFactory implements FormManagerFactoryInterface
     private array $managers = [];
 
     public function __construct(
-        private ContaoFramework $contaoFramework,
-        private RequestStack $requestStack,
-        private UrlParser $urlParser,
+        private readonly ContaoFramework $contaoFramework,
+        private readonly RequestStack $requestStack,
+        private readonly UrlParser $urlParser,
     ) {
     }
 
@@ -78,7 +80,7 @@ public function forFormId(int $id): FormManager
             $storage,
             $storageIdentifierGenerator,
             $sessionReferenceGenerator,
-            $this->urlParser
+            $this->urlParser,
         );
 
         if ($storage instanceof FormManagerAwareInterface) {

src/Step/FileParameterBag.php

@@ -17,9 +17,10 @@ class FileParameterBag extends ParameterBag
      */
     public function set(string $name, mixed $value): self
     {
-        if (\is_array($value) &&
-            \array_key_exists('tmp_name', $value) &&
-            \is_string($value['tmp_name'])
+        if (
+            \is_array($value)
+            && \array_key_exists('tmp_name', $value)
+            && \is_string($value['tmp_name'])
             && is_uploaded_file($value['tmp_name'])
         ) {
             $target = (new Filesystem())->tempnam(sys_get_temp_dir(), 'nc');

src/Step/StepData.php

@@ -6,8 +6,13 @@
 
 class StepData
 {
-    private function __construct(private int $step, private ParameterBag $submitted, private ParameterBag $labels, private ParameterBag $files, private ParameterBag $originalPostData)
-    {
+    private function __construct(
+        private readonly int $step,
+        private ParameterBag $submitted,
+        private ParameterBag $labels,
+        private ParameterBag $files,
+        private ParameterBag $originalPostData,
+    ) {
     }
 
     /**

src/Storage/SessionReferenceGenerator/FixedSessionReferenceGenerator.php

@@ -8,7 +8,7 @@
 
 class FixedSessionReferenceGenerator implements SessionReferenceGeneratorInterface
 {
-    public function __construct(private string $identifier)
+    public function __construct(private readonly string $identifier)
     {
     }
 

src/Storage/SessionStorage.php

@@ -10,9 +10,9 @@
 
 class SessionStorage implements StorageInterface
 {
-    public const SESSION_KEY = 'contao.mp_forms';
+    final public const SESSION_KEY = 'contao.mp_forms';
 
-    public function __construct(private Request $request)
+    public function __construct(private readonly Request $request)
     {
     }
 

src/Storage/StorageIdentifierGenerator/FixedStorageIdentifierGenerator.php

@@ -8,7 +8,7 @@
 
 class FixedStorageIdentifierGenerator implements StorageIdentifierGeneratorInterface
 {
-    public function __construct(private string $identifier)
+    public function __construct(private readonly string $identifier)
     {
     }