Handle General number format as non date (#221)

If the number format is set to General (id = 0), do no try to format the value as a date

src/Spout/Reader/XLSX/Helper/StyleHelper.php

@@ -171,11 +171,23 @@ class StyleHelper
     protected function doesNumFmtIdIndicateDate($numFmtId)
     {
         return (
-            $this->isNumFmtIdBuiltInDateFormat($numFmtId) ||
+            !$this->doesNumFmtIdIndicateGeneralFormat($numFmtId) &&
-            $this->isNumFmtIdCustomDateFormat($numFmtId)
+            (
+                $this->isNumFmtIdBuiltInDateFormat($numFmtId) ||
+                $this->isNumFmtIdCustomDateFormat($numFmtId)
+            )
         );
     }
 
+    /**
+     * @param int $numFmtId
+     * @return bool Whether the number format ID indicates the "General" format (0 by convention)
+     */
+    protected function doesNumFmtIdIndicateGeneralFormat($numFmtId)
+    {
+        return ($numFmtId === 0);
+    }
+
     /**
      * @param int $numFmtId
      * @return bool Whether the number format ID indicates that the number is a timestamp

tests/Spout/Reader/XLSX/Helper/StyleHelperTest.php

@@ -59,6 +59,16 @@ class StyleHelperTest extends \PHPUnit_Framework_TestCase
         $this->assertFalse($shouldFormatAsDate);
     }
 
+    /**
+     * @return void
+     */
+    public function testShouldFormatNumericValueAsDateWithGeneralFormat()
+    {
+        $styleHelper = $this->getStyleHelperMock([[], ['applyNumberFormat' => true, 'numFmtId' => 0]]);
+        $shouldFormatAsDate = $styleHelper->shouldFormatNumericValueAsDate(1);
+        $this->assertFalse($shouldFormatAsDate);
+    }
+
     /**
      * @return void
      */