mirror of
https://github.com/PHPOffice/PhpSpreadsheet.git
synced 2026-09-07 08:37:05 +00:00
oleibman
e9cf27354d
Successor to PR #3523. There are 494 single-line changes (`public function provider` to `public static function provider`) in this PR. None of these were made manually; they were all created with the following script (adapted from https://stackoverflow.com/questions/25909820/how-to-recursively-iterate-through-files-in-php): ```php $dir = 'C:/git/unit10prep2/tests/PhpSpreadsheetTests'; $it = new RecursiveDirectoryIterator($dir); // Loop through files foreach(new RecursiveIteratorIterator($it) as $file) { if ($file->getExtension() === 'php') { $contents = file_get_contents($file); $new = preg_replace('/public function (\\w*)([Pp])rovider/', 'public static function $1$2rovider', $contents); if ($new !== $contents) { echo "changing $file\n"; file_put_contents($file, $new); } } } ``` After this PR, there will be one more, with a small number of test changes, and enabling PhpUnit 10 for Php 8.1+.
47 lines
1.7 KiB
PHP
47 lines
1.7 KiB
PHP
<?php
|
|
|
|
namespace PhpOffice\PhpSpreadsheetTests\Functional;
|
|
|
|
use PhpOffice\PhpSpreadsheet\Spreadsheet;
|
|
|
|
class ReadBlankCellsTest extends AbstractFunctional
|
|
{
|
|
public static function providerSheetFormat(): array
|
|
{
|
|
return [
|
|
['Xlsx'],
|
|
['Xls'],
|
|
// ['Ods'], // Broken. Requires fix in Ods reader.
|
|
// ['Csv'], // never reads blank cells
|
|
// ['Html'], // never reads blank cells
|
|
];
|
|
}
|
|
|
|
/**
|
|
* Test generate file with some empty cells.
|
|
*
|
|
* @dataProvider providerSheetFormat
|
|
*
|
|
* @param mixed $format
|
|
*/
|
|
public function testXlsxLoadWithNoBlankCells($format): void
|
|
{
|
|
$spreadsheet = new Spreadsheet();
|
|
$spreadsheet->getActiveSheet()->getCell('B2')->setValue('');
|
|
$spreadsheet->getActiveSheet()->getCell('C1')->setValue('C1');
|
|
$spreadsheet->getActiveSheet()->getCell('C3')->setValue('C3');
|
|
|
|
$reloadedSpreadsheet = $this->writeAndReload($spreadsheet, $format);
|
|
self::assertTrue($reloadedSpreadsheet->getActiveSheet()->getCellCollection()->has('B2'));
|
|
self::assertFalse($reloadedSpreadsheet->getActiveSheet()->getCellCollection()->has('C2'));
|
|
self::assertTrue($reloadedSpreadsheet->getActiveSheet()->getCellCollection()->has('C3'));
|
|
|
|
$reloadedSpreadsheet = $this->writeAndReload($spreadsheet, $format, function ($reader): void {
|
|
$reader->setReadEmptyCells(false);
|
|
});
|
|
self::assertFalse($reloadedSpreadsheet->getActiveSheet()->getCellCollection()->has('B2'));
|
|
self::assertFalse($reloadedSpreadsheet->getActiveSheet()->getCellCollection()->has('C2'));
|
|
self::assertTrue($reloadedSpreadsheet->getActiveSheet()->getCellCollection()->has('C3'));
|
|
}
|
|
}
|