fixes #1655 issue (#1656)

kalyabin · web-flow · commit 2c927b1ca5ae · 2020-12-10T23:10:01.000+01:00
Resolve problem with incorrectly defined hyperlinks
diff --git a/src/PhpSpreadsheet/Reader/Xlsx/Hyperlinks.php b/src/PhpSpreadsheet/Reader/Xlsx/Hyperlinks.php
@@ -41,7 +41,7 @@ private function setHyperlink(SimpleXMLElement $hyperlink, Worksheet $worksheet)
         foreach (Coordinate::extractAllCellReferencesInRange($hyperlink['ref']) as $cellReference) {
             $cell = $worksheet->getCell($cellReference);
             if (isset($linkRel['id'])) {
-                $hyperlinkUrl = $this->hyperlinks[(string) $linkRel['id']];
+                $hyperlinkUrl = $this->hyperlinks[(string) $linkRel['id']] ?? null;
                 if (isset($hyperlink['location'])) {
                     $hyperlinkUrl .= '#' . (string) $hyperlink['location'];
                 }

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ private function setHyperlink(SimpleXMLElement $hyperlink, Worksheet $worksheet)`
`41`	`41`	`foreach (Coordinate::extractAllCellReferencesInRange($hyperlink['ref']) as $cellReference) {`
`42`	`42`	`$cell = $worksheet->getCell($cellReference);`
`43`	`43`	`if (isset($linkRel['id'])) {`
`44`		`- $hyperlinkUrl = $this->hyperlinks[(string) $linkRel['id']];`
	`44`	`+ $hyperlinkUrl = $this->hyperlinks[(string) $linkRel['id']] ?? null;`
`45`	`45`	`if (isset($hyperlink['location'])) {`
`46`	`46`	`$hyperlinkUrl .= '#' . (string) $hyperlink['location'];`
`47`	`47`	`}`