Remove BOM when parsing file; fixes pluginsGLPI#74

stonebuzz · Aug 3, 2018 · 31f01dd · 31f01dd
1 parent 28d0d97
commit 31f01dd
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/inc/backendcsv.class.php b/inc/backendcsv.class.php
@@ -198,6 +198,14 @@ function getNumberOfLines() {
    function openFile() {
 
       $this->file_handler = fopen($this->file, 'r');
+
+      // Check if file starts with BOM.
+      // 1. If BOM found, keep the handler moved to 4th char to not include it in data.
+      // 2. If no BOM found, rewind to start of file.
+      $hasBOM = fread($this->file_handler, 3) === pack('CCC', 0xEF, 0xBB, 0xBF);
+      if (!$hasBOM) {
+         fseek($this->file_handler, 0);
+      }
    }