Make dbup compatible with php 8.1

.github/workflows/ci.yml

@@ -0,0 +1,29 @@
+name: CI
+
+on: [push]
+
+jobs:
+  build-test:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4.1.6
+      - name: Composer (php-actions)
+        uses: php-actions/composer@v6
+      - name: PHP Static Analysis
+        uses: php-actions/composer@v6
+        with:
+          command: phpstan
+      - name: PHP Check style
+        uses: php-actions/composer@v6
+        with:
+          command: phpcs
+      - name: PHP Mess detector
+        uses: php-actions/composer@v6
+        with:
+          command: phpmd
+      - name: PHP Unit tests
+        uses: php-actions/composer@v6
+        with:
+          command: phpunit

.gitignore

@@ -2,9 +2,15 @@
 /sql/*
 /.dbup/*
 /.idea/*
+/.vscode/*
 /.DS_Store
 /phpunit.phar
 /composer.phar
 /php-cs-fixer.phar
-/phpunit.xml
 /composer.lock
+
+.phpunit.result.cache
+
+.phpunit.cache/
+
+dbup.phar

CHANGELOG.md

@@ -1,7 +1,13 @@
 CHANGELOG
 =========
 
+* 0.6 (2024-05-30)
+
+ * 220e42d feat: added a few analyzers 
+ * fe92f76 fix: type hints and other changes, from static analyzers
+
 * 0.5 (2014-10-06)
+
  * c44b0f8 Fix url of phpunit.phar
  * 327bbf1 Option --ini is optional in CreateCommand.
  * 4bdd259 Fixing timstamp pattern to day with leading zeros in CreateCommand.

composer.json

@@ -1,20 +1,24 @@
 {
     "name": "brtriver/dbup",
+    "version": "0.6",
     "description": "simple migration tool with PDO",
     "keywords": ["migration", "database"],
     "homepage": "http://1ms.jp/",
     "type": "library",
     "require": {
-        "php": ">=5.4.0",
+        "php": ">=8.1.0",
         "ext-pdo": "*",
-        "symfony/console": "~2.5.0",
-        "symfony/finder": "~2.5.0",
-        "symfony/event-dispatcher": "~2.5.0"
+        "symfony/console": "^3.2",
+        "symfony/finder": "^7.0",
+        "symfony/event-dispatcher": "^3.2"
     },
     "require-dev": {
-        "phpunit/phpunit": "4.2.*",
-        "phake/phake": "v1.0.3",
-        "hamcrest/hamcrest-php": "1.2.0"
+        "phpunit/phpunit": "^11.1",
+        "phake/phake": "^4.5",
+        "hamcrest/hamcrest-php": "^2.0",
+        "phpstan/phpstan": "^1.11",
+        "squizlabs/php_codesniffer": "^3.10",
+        "phpmd/phpmd": "^2.15"
     },
     "suggest": {
         "ext-pdo-mysql": "In order to use ikou with MySQL databases.",
@@ -29,7 +33,19 @@
         }
     ],
     "autoload": {
-        "psr-0": { "Dbup": "src/" }
+        "psr-0": {
+            "Dbup": [
+                "src/",
+                "tests/"
+            ]
+        }
     },
-    "bin": ["dbup"]
-}
+    "bin": ["dbup"],
+    "scripts": {
+        "phpstan" : "vendor/bin/phpstan analyse -c phpstan.neon --memory-limit=-1",
+        "phpcs" : "vendor/bin/phpcs --standard=phpcs.xml --no-cache -s -n --file-list=phpcs.list",
+        "phpcbf" : "vendor/bin/phpcbf --standard=PSR12 --file-list=phpcs.list",
+        "phpmd" : "vendor/bin/phpmd --ignore-violations-on-exit src,dbup text phpmd.xml",
+        "phpunit" : "vendor/bin/phpunit"
+    }
+}

dbup

@@ -1,4 +1,3 @@
-#!/usr/bin/env php
 <?php
 // bin/dbup
 

phpcs.list

@@ -0,0 +1,2 @@
+src
+dbup

phpcs.xml

@@ -0,0 +1,14 @@
+<ruleset name="dbup">
+    <description>dbup coding standard based on PSR12.</description>
+
+    <!-- Include the whole Zend standard -->
+    <rule ref="PSR12" >
+        <exclude name="PSR1.Methods.CamelCapsMethodName.NotCamelCaps" />
+        <exclude name="Squiz.Classes.ValidClassName.NotCamelCaps" />
+        <exclude name="Squiz.Scope.MethodScope.Missing" />
+        <exclude name="PSR2.Classes.PropertyDeclaration.ScopeMissing" />
+        <exclude name="Generic.NamingConventions.UpperCaseConstantName.ClassConstantNotUpperCase" />
+        <exclude name="PSR12.Files.FileHeader.IncorrectOrder" />
+        <exclude name="PSR2.Classes.PropertyDeclaration.VarUsed" />
+    </rule>
+</ruleset>

phpmd.xml

@@ -0,0 +1,16 @@
+<?xml version="1.0"?>
+<ruleset name="dbup rule set"
+         xmlns="https://pmd.sf.net/ruleset/1.0.0"
+         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:schemaLocation="http://pmd.sf.net/ruleset/1.0.0 https://pmd.sf.net/ruleset_xml_schema.xsd"
+         xsi:noNamespaceSchemaLocation="https://pmd.sf.net/ruleset_xml_schema.xsd">
+    <description>
+        dbup rules for PHP Mess Detector
+    </description>
+    <!-- Import the entire unused code rule set -->
+    <rule ref="rulesets/cleancode.xml">
+        <exclude name="StaticAccess" />
+        <exclude name="ElseExpression" />
+        <exclude name="BooleanArgumentFlag" />
+    </rule>
+</ruleset>

phpstan.neon

@@ -0,0 +1,6 @@
+parameters:
+	phpVersion: 80100
+	level: 4
+	paths:
+		- src
+		- dbup

phpunit.xml

@@ -0,0 +1,16 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/11.1/phpunit.xsd"
+         backupGlobals="false"
+         colors="false"
+         processIsolation="false"
+         stopOnFailure="false"
+         bootstrap="vendor/autoload.php"
+         cacheDirectory=".phpunit.cache"
+         backupStaticProperties="false">
+  <testsuites>
+    <testsuite name="Dbup Test Suite">
+      <directory>./tests/Dbup/</directory>
+    </testsuite>
+  </testsuites>
+</phpunit>