Merge pull request #4524 from oleibman/issue4521

Recognize "New" Mimetype for Empty File

Author: oleibman

CHANGELOG.md

@@ -29,6 +29,7 @@ and this project adheres to [Semantic Versioning](https://semver.org).
 
 ### Fixed
 
+- Recognize application/x-empty mimetype. [Issue #4521](https://github.com/PHPOffice/PhpSpreadsheet/issues/4521) [PR #4524](https://github.com/PHPOffice/PhpSpreadsheet/pull/4524)
 - Micro-optimization in getSheetByName. [PR #4499](https://github.com/PHPOffice/PhpSpreadsheet/pull/4499)
 - Bug in resizeMatricesExtend. [Issue #4451](https://github.com/PHPOffice/PhpSpreadsheet/issues/4451) [PR #4474](https://github.com/PHPOffice/PhpSpreadsheet/pull/4474)
 

src/PhpSpreadsheet/Reader/Csv.php

@@ -605,6 +605,7 @@ public function canRead(string $filename): bool
             'text/csv',
             'text/plain',
             'inode/x-empty',
+            'application/x-empty', // has now replaced previous
             'text/html',
         ];
 

tests/PhpSpreadsheetTests/Issue4521Test.php

@@ -0,0 +1,56 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests;
+
+use PhpOffice\PhpSpreadsheet\IOFactory;
+use PhpOffice\PhpSpreadsheet\Shared\File;
+use PHPUnit\Framework\TestCase;
+
+class Issue4521Test extends TestCase
+{
+    private string $outfile = '';
+
+    protected int $weirdMimetypeMajor = 8;
+
+    protected int $weirdMimetypeMinor1 = 1;
+
+    protected int $weirdMimetypeMinor2 = 2;
+
+    protected function tearDown(): void
+    {
+        if ($this->outfile !== '') {
+            unlink($this->outfile);
+            $this->outfile = '';
+        }
+    }
+
+    public function testEmptyFile(): void
+    {
+        $this->outfile = File::temporaryFilename();
+        file_put_contents($this->outfile, '');
+        $spreadsheet = IOFactory::load($this->outfile);
+        $sheet = $spreadsheet->getActiveSheet();
+        self::assertSame('A', $sheet->getHighestColumn());
+        self::assertSame(1, $sheet->getHighestRow());
+        $spreadsheet->disconnectWorksheets();
+    }
+
+    public function testCrlfFile(): void
+    {
+        if (PHP_MAJOR_VERSION === $this->weirdMimetypeMajor) {
+            if (
+                PHP_MINOR_VERSION === $this->weirdMimetypeMinor1
+                || PHP_MINOR_VERSION === $this->weirdMimetypeMinor2
+            ) {
+                self::markTestSkipped('Php mimetype bug with this release');
+            }
+        }
+        $this->outfile = File::temporaryFilename();
+        file_put_contents($this->outfile, "\r\n");
+        $spreadsheet = IOFactory::load($this->outfile);
+        $sheet = $spreadsheet->getActiveSheet();
+        self::assertSame('A', $sheet->getHighestColumn());
+        self::assertSame(1, $sheet->getHighestRow());
+        $spreadsheet->disconnectWorksheets();
+    }
+}