Merge pull request #8788 from weirdan/fix-xml-report-crashes-on-8.1

Fixes #8321
vimeo · Nov 28, 2022 · 6fff6df · 6fff6df
2 parents 9d597cf + 874eb7d
commit 6fff6df
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/Psalm/Report/XmlReport.php b/src/Psalm/Report/XmlReport.php
@@ -29,13 +29,19 @@ static function (IssueData $issue_data): array {
                             );
                         }
 
+                        // replace null values, as XML serializers tend to have problems with them
+                        $issue_data['taint_trace'] ??= '';
+
                         if (null !== $issue_data['other_references']) {
                             $issue_data['other_references'] = array_map(
                                 static fn(DataFlowNodeData $reference): array => (array) $reference,
                                 $issue_data['other_references']
                             );
                         }
 
+                        // replace null values, as XML serializers tend to have problems with them
+                        $issue_data['other_references'] ??= '';
+
                         return $issue_data;
                     },
                     $this->issues_data