Merge pull request #199 from box/check_valid_resource_on_close

Check file handle is valid before manipulating it

Author: adrilo

src/Spout/Writer/ODS/Internal/Worksheet.php

@@ -218,6 +218,10 @@ protected function getCellContent($cellValue, $styleIndex, $numTimesValueRepeate
      */
     public function close()
     {
+        if (!is_resource($this->sheetFilePointer)) {
+            return;
+        }
+
         fclose($this->sheetFilePointer);
     }
 }

src/Spout/Writer/XLSX/Helper/SharedStringsHelper.php

@@ -88,6 +88,10 @@ public function writeString($string)
      */
     public function close()
     {
+        if (!is_resource($this->sharedStringsFilePointer)) {
+            return;
+        }
+
         fwrite($this->sharedStringsFilePointer, '</sst>');
 
         // Replace the default strings count with the actual number of shared strings in the file header

src/Spout/Writer/XLSX/Internal/Worksheet.php

@@ -178,6 +178,10 @@ public function addRow($dataRow, $style)
      */
     public function close()
     {
+        if (!is_resource($this->sheetFilePointer)) {
+            return;
+        }
+
         fwrite($this->sheetFilePointer, '</sheetData>');
         fwrite($this->sheetFilePointer, '</worksheet>');
         fclose($this->sheetFilePointer);