Merge pull request #201 from scholieren-com/bugfix/wrong-parameters-for-exception

oohnoitz · web-flow · commit b828c6569b95 · 2021-08-12T08:22:44.000-05:00
Fix PDOException $code data type
diff --git a/src/Drivers/Pdo/Connection.php b/src/Drivers/Pdo/Connection.php
@@ -25,7 +25,8 @@ public function query($query)
         try {
             $statement->execute();
         } catch (PDOException $exception) {
-            throw new DatabaseException($exception->getMessage() . ' [' . $query . ']', $exception->getCode(), $exception);
+            throw new DatabaseException('[' . $exception->getCode() . '] ' . $exception->getMessage() . ' [' . $query . ']',
+                (int)$exception->getCode(), $exception);
         }
 
         return new ResultSet(new ResultSetAdapter($statement));

Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,8 @@ public function query($query)`
`25`	`25`	`try {`
`26`	`26`	`$statement->execute();`
`27`	`27`	`} catch (PDOException $exception) {`
`28`		`- throw new DatabaseException($exception->getMessage() . ' [' . $query . ']', $exception->getCode(), $exception);`
	`28`	`+ throw new DatabaseException('[' . $exception->getCode() . '] ' . $exception->getMessage() . ' [' . $query . ']',`
	`29`	`+ (int)$exception->getCode(), $exception);`
`29`	`30`	`}`
`30`	`31`
`31`	`32`	`return new ResultSet(new ResultSetAdapter($statement));`