Release PHP 7.4 downgraded

Author: Bellangelo

composer.json

@@ -4,7 +4,7 @@
     "type": "phpstan-extension",
     "license": "MIT",
     "require": {
-        "php": "^8.3"
+        "php": "^7.4 || ^8.0"
     },
     "require-dev": {
         "phpunit/phpunit": "^9.5|^10.0",

src/Rules/Laravel/EnforceCollationRule.php

@@ -24,22 +24,35 @@
  */
 final class EnforceCollationRule extends LaravelRule
 {
+    /**
+     * @readonly
+     */
+    private string $requiredCollation;
     // Same identifier as Phinx (as requested)
-    private const string RULE_IDENTIFIER = 'laravel.schema.requiredCollation';
-
-    private const string MESSAGE_MISSING =
+    /**
+     * @var string
+     */
+    private const RULE_IDENTIFIER = 'laravel.schema.requiredCollation';
+
+    /**
+     * @var string
+     */
+    private const MESSAGE_MISSING =
         'Required: table collation must be "%s". '
         . 'Why: prevents environment-dependent defaults and keeps schema consistent. '
         . 'Fix: set the table collation explicitly in the migration.';
 
-    private const string MESSAGE_WRONG =
+    /**
+     * @var string
+     */
+    private const MESSAGE_WRONG =
         'Required: table collation must be "%s". Found: "%s". '
         . 'Why: prevents environment-dependent defaults and keeps schema consistent. '
         . 'Fix: set the table collation explicitly in the migration.';
 
-    public function __construct(
-        private readonly string $requiredCollation
-    ) {
+    public function __construct(string $requiredCollation)
+    {
+        $this->requiredCollation = $requiredCollation;
     }
 
     public function getNodeType(): string

src/Rules/Laravel/ForbidAfterRule.php

@@ -16,9 +16,15 @@
  */
 final class ForbidAfterRule extends LaravelRule
 {
-    private const string RULE_IDENTIFIER = 'laravel.schema.afterForbidden';
+    /**
+     * @var string
+     */
+    private const RULE_IDENTIFIER = 'laravel.schema.afterForbidden';
 
-    private const string MESSAGE =
+    /**
+     * @var string
+     */
+    private const MESSAGE =
         'Forbidden: column positioning ("after"). '
         . 'Why: can trigger a full table rewrite or long locks depending on the engine. '
         . 'Fix: avoid column ordering in migrations.';

src/Rules/Laravel/ForbidEnumColumnRule.php

@@ -16,9 +16,15 @@
  */
 final class ForbidEnumColumnRule extends LaravelRule
 {
-    private const string RULE_IDENTIFIER = 'laravel.schema.enumColumnForbidden';
+    /**
+     * @var string
+     */
+    private const RULE_IDENTIFIER = 'laravel.schema.enumColumnForbidden';
 
-    private const string MESSAGE =
+    /**
+     * @var string
+     */
+    private const MESSAGE =
         'Forbidden: enum column type. '
         . 'Why: adding or removing enum values requires a full ALTER TABLE, which can cause long locks on large tables. '
         . 'Fix: use a string column with application-level validation instead.';

src/Rules/Laravel/ForbidMultipleTableCreationsRule.php

@@ -16,9 +16,15 @@
  */
 final class ForbidMultipleTableCreationsRule extends LaravelRule
 {
-    private const string RULE_IDENTIFIER = 'laravel.schema.multipleTableCreationsForbidden';
+    /**
+     * @var string
+     */
+    private const RULE_IDENTIFIER = 'laravel.schema.multipleTableCreationsForbidden';
 
-    private const string MESSAGE =
+    /**
+     * @var string
+     */
+    private const MESSAGE =
         'Forbidden: creating multiple tables in a single migration. '
         . 'Why: reduces reviewability and rollback safety. '
         . 'Fix: split into one migration per table.';

src/Rules/Laravel/ForbidRawSqlRule.php

@@ -16,14 +16,23 @@
  */
 final class ForbidRawSqlRule extends LaravelRule
 {
-    private const string RULE_IDENTIFIER = 'laravel.schema.rawSqlForbidden';
+    /**
+     * @var string
+     */
+    private const RULE_IDENTIFIER = 'laravel.schema.rawSqlForbidden';
 
-    private const string MESSAGE =
+    /**
+     * @var string
+     */
+    private const MESSAGE =
         'Forbidden: raw SQL via DB::%s(). '
         . 'Why: raw SQL bypasses the schema builder, making migrations harder to review, less portable, and prone to errors. '
         . 'Fix: use Laravel schema builder methods instead.';
 
-    private const array FORBIDDEN_METHODS = ['statement', 'unprepared'];
+    /**
+     * @var mixed[]
+     */
+    private const FORBIDDEN_METHODS = ['statement', 'unprepared'];
 
     public function getNodeType(): string
     {

src/Rules/Phinx/EnforceCollationRule.php

@@ -17,21 +17,35 @@
  */
 final class EnforceCollationRule extends PhinxRule
 {
+    /**
+     * @readonly
+     */
+    private string $requiredCollation;
     // Same identifier as Laravel
-    private const string RULE_IDENTIFIER = 'phinx.schema.requiredCollation';
+    /**
+     * @var string
+     */
+    private const RULE_IDENTIFIER = 'phinx.schema.requiredCollation';
 
-    private const string MESSAGE_MISSING =
+    /**
+     * @var string
+     */
+    private const MESSAGE_MISSING =
         'Required: table collation must be "%s". '
         . 'Why: prevents environment-dependent defaults and keeps schema consistent. '
         . 'Fix: set the table collation explicitly in the migration.';
 
-    private const string MESSAGE_WRONG =
+    /**
+     * @var string
+     */
+    private const MESSAGE_WRONG =
         'Required: table collation must be "%s". Found: "%s". '
         . 'Why: prevents environment-dependent defaults and keeps schema consistent. '
         . 'Fix: set the table collation explicitly in the migration.';
 
-    public function __construct(private readonly string $requiredCollation)
+    public function __construct(string $requiredCollation)
     {
+        $this->requiredCollation = $requiredCollation;
     }
 
     public function getNodeType(): string

src/Rules/Phinx/ForbidAfterRule.php

@@ -17,9 +17,15 @@
  */
 final class ForbidAfterRule extends PhinxRule
 {
-    private const string RULE_IDENTIFIER = 'phinx.schema.afterForbidden';
+    /**
+     * @var string
+     */
+    private const RULE_IDENTIFIER = 'phinx.schema.afterForbidden';
 
-    private const string MESSAGE =
+    /**
+     * @var string
+     */
+    private const MESSAGE =
         'Forbidden: column positioning ("after"). '
         . 'Why: can trigger a full table rewrite or long locks depending on the engine. '
         . 'Fix: avoid column ordering in migrations.';

src/Rules/Phinx/ForbidEnumColumnRule.php

@@ -16,9 +16,15 @@
  */
 final class ForbidEnumColumnRule extends PhinxRule
 {
-    private const string RULE_IDENTIFIER = 'phinx.schema.enumColumnForbidden';
+    /**
+     * @var string
+     */
+    private const RULE_IDENTIFIER = 'phinx.schema.enumColumnForbidden';
 
-    private const string MESSAGE =
+    /**
+     * @var string
+     */
+    private const MESSAGE =
         'Forbidden: enum column type. '
         . 'Why: adding or removing enum values requires a full ALTER TABLE, which can cause long locks on large tables. '
         . 'Fix: use a string column with application-level validation instead.';

src/Rules/Phinx/ForbidMultipleTableCreationsRule.php

@@ -15,9 +15,15 @@
  */
 final class ForbidMultipleTableCreationsRule extends PhinxRule
 {
-    private const string RULE_IDENTIFIER = 'phinx.schema.multipleTableCreationsForbidden';
+    /**
+     * @var string
+     */
+    private const RULE_IDENTIFIER = 'phinx.schema.multipleTableCreationsForbidden';
 
-    private const string MESSAGE =
+    /**
+     * @var string
+     */
+    private const MESSAGE =
         'Forbidden: creating multiple tables in a single migration. '
         . 'Why: reduces reviewability and rollback safety. '
         . 'Fix: split into one migration per table.';