When an error in PoStreamWriter occurs an exception is supposed to be thrown. The exception class does not exist, however, which then fatals directly and additionally masks the error.
Throw a \RuntimeException.
Additionally the patch fixes 3 code style issues in PoStreamWriter:
Needs work
10.1 ✨
Last updated
Used to track the progress of issues reviewed by the Drupal Needs Review Queue Initiative.
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
Can we get a simple test to show this exception is returned with the change.
Thanks.