Avoid TypeError due to $lastError being boolean/false instead of a string (strict types) (#608)

rmontagud · web-flow · commit 00b2592f8e40 · 2021-11-19T01:10:17.000+01:00
diff --git a/src/PhpImap/Imap.php b/src/PhpImap/Imap.php
@@ -706,7 +706,7 @@ public static function open(
         if (!$result) {
             $lastError = \imap_last_error();
 
-            if ('' !== \trim($lastError)) {
+            if ((\is_string($lastError)) && ('' !== \trim($lastError))) {
                 throw new UnexpectedValueException('IMAP error:'.$lastError);
             }
 

Original file line number	Diff line number	Diff line change
`@@ -706,7 +706,7 @@ public static function open(`
`706`	`706`	`if (!$result) {`
`707`	`707`	`$lastError = \imap_last_error();`
`708`	`708`
`709`		`- if ('' !== \trim($lastError)) {`
	`709`	`+ if ((\is_string($lastError)) && ('' !== \trim($lastError))) {`
`710`	`710`	`throw new UnexpectedValueException('IMAP error:'.$lastError);`
`711`	`711`	`}`
`712`	`712`