Merge pull request #33 from Codeception/issue-32

Naktibalda · web-flow · commit 4ec1208d6d5f · 2022-03-05T21:26:44.000+02:00
Null safety in destructor
diff --git a/src/Codeception/Lib/Driver/Db.php b/src/Codeception/Lib/Driver/Db.php
@@ -91,7 +91,7 @@ public function __construct(string $dsn, string $user = null, string $password =
 
     public function __destruct()
     {
-        if ($this->dbh->inTransaction()) {
+        if ($this->dbh !== null && $this->dbh->inTransaction()) {
             $this->dbh->rollBack();
         }
 

Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,7 @@ public function __construct(string $dsn, string $user = null, string $password =`
`91`	`91`
`92`	`92`	`public function __destruct()`
`93`	`93`	`{`
`94`		`- if ($this->dbh->inTransaction()) {`
	`94`	`+ if ($this->dbh !== null && $this->dbh->inTransaction()) {`
`95`	`95`	`$this->dbh->rollBack();`
`96`	`96`	`}`
`97`	`97`