bug #60057 [Mailer] Fix Trying to access array offset on value of type null error by adding null checking (khushaalan)

fabpot · fabpot · commit 75d99910034d · 2025-04-03T08:47:24.000+02:00
This PR was submitted for the 7.2 branch but it was squashed and merged into the 6.4 branch instead. Discussion ---------- [Mailer] Fix `Trying to access array offset on value of type null` error by adding null checking | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | Fix `Trying to access array offset on value of type null` error by adding null checking | License | MIT Its a very simple bug fix.. basically the `error_get_last()` doesn't have the 'message' and causing this issue: ![image](https://github.com/user-attachments/assets/95b51b87-0011-4913-a67a-ce91195d4f2e) so i just added Null Checking and it seems to be working now: ![image](https://github.com/user-attachments/assets/0d2f7dae-8854-421c-a946-10426b15ee06) this is happens when using Mailtrap and when it reaches its quota.. Commits ------- a303fba bug #60094 [DoctrineBridge] Fix support for entities that leverage native lazy objects (nicolas-grekas)
diff --git a/src/Symfony/Component/Mailer/Transport/Smtp/Stream/AbstractStream.php b/src/Symfony/Component/Mailer/Transport/Smtp/Stream/AbstractStream.php
@@ -87,7 +87,7 @@ public function readLine(): string
                 throw new TransportException(sprintf('Connection to "%s" has been closed unexpectedly.', $this->getReadConnectionDescription()));
             }
             if (false === $line) {
-                throw new TransportException(sprintf('Unable to read from connection to "%s": ', $this->getReadConnectionDescription()).error_get_last()['message']);
+                throw new TransportException(sprintf('Unable to read from connection to "%s": ', $this->getReadConnectionDescription().error_get_last()['message'] ?? ''));
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ public function readLine(): string`
`87`	`87`	`throw new TransportException(sprintf('Connection to "%s" has been closed unexpectedly.', $this->getReadConnectionDescription()));`
`88`	`88`	`}`
`89`	`89`	`if (false === $line) {`
`90`		`- throw new TransportException(sprintf('Unable to read from connection to "%s": ', $this->getReadConnectionDescription()).error_get_last()['message']);`
	`90`	`+ throw new TransportException(sprintf('Unable to read from connection to "%s": ', $this->getReadConnectionDescription().error_get_last()['message'] ?? ''));`
`91`	`91`	`}`
`92`	`92`	`}`
`93`	`93`