Merge pull request #4 from 77web/feat/phpstan

77web · web-flow · commit 35864f6fa2cb · 2023-09-19T22:55:02.000+09:00
Introduced phpstan
diff --git a/composer.json b/composer.json
@@ -4,7 +4,8 @@
     "type": "library",
     "require-dev": {
         "phpunit/phpunit": "^10.3",
-        "symplify/easy-coding-standard": "^12.0"
+        "symplify/easy-coding-standard": "^12.0",
+        "phpstan/phpstan": "^1.10"
     },
     "license": "MIT",
     "autoload": {
diff --git a/phpstan.neon b/phpstan.neon
@@ -0,0 +1,6 @@
+parameters:
+    paths:
+        - src
+    level: max
+    checkGenericClassInNonGenericObjectType: false
+
diff --git a/src/TsTzRangeType.php b/src/TsTzRangeType.php
@@ -28,6 +28,9 @@ public function getSQLDeclaration(array $column, AbstractPlatform $platform)
      */
     public function convertToPHPValue($value, AbstractPlatform $platform)
     {
+        if (!is_string($value)) {
+            throw new \RuntimeException('tstzrange expects only string. unexpected value from DB: ' . $value);
+        }
         if (!preg_match('/(\[|\()(.*)\,(.*)(\]|\))/', $value, $matches)) {
             throw new \RuntimeException('unexpected value from DB: ' . $value);
         }

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,9 @@ public function getSQLDeclaration(array $column, AbstractPlatform $platform)`
`28`	`28`	`*/`
`29`	`29`	`public function convertToPHPValue($value, AbstractPlatform $platform)`
`30`	`30`	`{`
	`31`	`+ if (!is_string($value)) {`
	`32`	`+ throw new \RuntimeException('tstzrange expects only string. unexpected value from DB: ' . $value);`
	`33`	`+ }`
`31`	`34`	`if (!preg_match('/(\[\|\()(.)\,(.)(\]\|\))/', $value, $matches)) {`
`32`	`35`	`throw new \RuntimeException('unexpected value from DB: ' . $value);`
`33`	`36`	`}`