mcndave/spout
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix DIRECTORY_SEPARATOR bug occurring on Windows

Browse Source 
Fixes #3:
The sheetN.xml files location is dynamically retrieved when parsing
[Content_Types].xml. In this file, the location is like "/xl/worksheets/sheet1.xml".

Because the zip stream wrapper expects the zip entry name to be like
"xl/worksheets/sheet1/xml" (without the first "/"), this first "/"
needs to be trimmed. It was trimmed using:
  ltrim($this->dataXmlFilePath, DIRECTORY_SEPARATOR);
which obvisously does not work on Windows platform where DIRECTORY_SEPARATOR is "\".
Replacing DIRECTORY_SEPARATOR by '/' solves this issue.
This commit is contained in:
Adrien Loison 2015-04-03 21:00:00 -07:00
parent 4ca1fc5851
commit 39c72a91b4
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Spout/Reader/Internal/XLSX/Worksheet.php
View File

@ -31,7 +31,7 @@ class Worksheet
*/ */
public function getDataXmlFilePath() public function getDataXmlFilePath()
{ {
return ltrim($this->dataXmlFilePath, DIRECTORY_SEPARATOR); return ltrim($this->dataXmlFilePath, '/');
} }
/** /**