From 8a044b085424526e6dae760135315bb604a813ae Mon Sep 17 00:00:00 2001 From: madflow Date: Wed, 17 Aug 2016 22:05:42 +0200 Subject: [PATCH] Fix finding the correct borderId --- src/Spout/Writer/XLSX/Helper/StyleHelper.php | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/src/Spout/Writer/XLSX/Helper/StyleHelper.php b/src/Spout/Writer/XLSX/Helper/StyleHelper.php index a917fd2..e01f7ac 100644 --- a/src/Spout/Writer/XLSX/Helper/StyleHelper.php +++ b/src/Spout/Writer/XLSX/Helper/StyleHelper.php @@ -95,14 +95,19 @@ class StyleHelper extends AbstractStyleHelper $border = $style->getBorder(); $serializedBorder = serialize($border); - if (isset($this->registeredBorders[$serializedBorder])) { - $this->styleIdToBorderMappingTable[$styleId] = $this->registeredBorders[$serializedBorder]; + $isBorderAlreadyRegistered = isset($this->registeredBorders[$serializedBorder]); + + if ($isBorderAlreadyRegistered) { + $registeredStyleId = $this->registeredBorders[$serializedBorder]; + $registeredBorderId = $this->styleIdToBorderMappingTable[$registeredStyleId]; + $this->styleIdToBorderMappingTable[$styleId] = $registeredBorderId; } else { $this->registeredBorders[$serializedBorder] = $styleId; $this->styleIdToBorderMappingTable[$styleId] = count($this->registeredBorders); } - } else { // If no border should be applied - the mapping is the default border: 0 + } else { + // If no border should be applied - the mapping is the default border: 0 $this->styleIdToBorderMappingTable[$styleId] = 0; } }