mirror of
https://github.com/PHPOffice/PhpSpreadsheet.git
synced 2026-09-13 03:26:26 +00:00
2d9375f847
Fix #4584. As discussed there, MS has implemented this situation in a way that I frankly do not understand. Marking a row with a wrapped cell to *not* use the default row height seems to affect every other populated row. Very odd. Nevertheless, adding a new `customFormat` boolean property to RowDimension seems to be a way to resolve this problem. Note that the Xml generated by Excel already outputs a `customFormat` attribute when a style is applied to a row. This PR just adds that property for when it is needed for rowHeight. It need not, and should not, be set by the application when a style is applied to the row; PhpSpreadsheet will take care of that on its own. Although the problem was raised for Xlsx format (I think), I investigated other relevant output formats as well. Html tends to do its own cell wrapping. I think the results after this change match the results before. Xls already handled the output side of this situation without difficulty. A change was needed on the input side, and is included as part of this PR. Ods did not handle default row height at all. As with other style properties, it is difficult to handle this on the input side, and this PR does nothing to improve that situation. However, we are, at least, able to provide some relief on the output side. Not complete relief - in order to apply the default size to unpopulated rows, we'd need to at least populate them with the `table:number-rows-repeated` attribute, and that has often been a source of problems, so I'm not willing to go there yet. This PR just ensures that all populated rows will have the correct height.
123 lines
4.7 KiB
PHP
123 lines
4.7 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace PhpOffice\PhpSpreadsheetTests\Writer\Xlsx;
|
|
|
|
use PhpOffice\PhpSpreadsheet\Exception as PhpSpreadsheetException;
|
|
use PhpOffice\PhpSpreadsheet\Spreadsheet;
|
|
use PhpOffice\PhpSpreadsheet\Writer\Xlsx as XlsxWriter;
|
|
use PhpOffice\PhpSpreadsheetTests\Functional\AbstractFunctional;
|
|
|
|
class Issue4584Test extends AbstractFunctional
|
|
{
|
|
public function testWriteRowDimensions(): void
|
|
{
|
|
$spreadsheet = new Spreadsheet();
|
|
$sheet = $spreadsheet->getActiveSheet();
|
|
$sheet->getDefaultRowDimension()->setRowHeight(20);
|
|
$sheet->setCellValue('A1', 'hello there world 1');
|
|
$sheet->getStyle('A1')->getAlignment()->setWrapText(true);
|
|
$sheet->getRowDimension(1)->setCustomFormat(true);
|
|
$sheet->setCellValue('A2', 'hello there world 2');
|
|
$sheet->setCellValue('A4', 'hello there world 4');
|
|
$writer = new XlsxWriter($spreadsheet);
|
|
$writerWorksheet = new XlsxWriter\Worksheet($writer);
|
|
$data = $writerWorksheet->writeWorksheet($sheet, []);
|
|
self::assertStringContainsString(
|
|
'<sheetFormatPr customHeight="true" defaultRowHeight="20" outlineLevelRow="0" outlineLevelCol="0"/>',
|
|
$data
|
|
);
|
|
self::assertStringContainsString(
|
|
'<row r="1" spans="1:1" customFormat="1" ht="-1">',
|
|
$data
|
|
);
|
|
self::assertStringContainsString(
|
|
'<row r="2" spans="1:1" customHeight="1">',
|
|
$data
|
|
);
|
|
self::assertStringContainsString(
|
|
'<row r="4" spans="1:1" customHeight="1">',
|
|
$data
|
|
);
|
|
$spreadsheet->disconnectWorksheets();
|
|
}
|
|
|
|
public function testWriteRowDimensionsNoCustom(): void
|
|
{
|
|
$spreadsheet = new Spreadsheet();
|
|
$sheet = $spreadsheet->getActiveSheet();
|
|
$sheet->getDefaultRowDimension()->setRowHeight(20);
|
|
$sheet->setCellValue('A1', 'hello there world 1');
|
|
$sheet->getStyle('A1')->getAlignment()->setWrapText(true);
|
|
//$sheet->getRowDimension(1)->setCustomFormat(true);
|
|
$sheet->setCellValue('A2', 'hello there world 2');
|
|
$sheet->setCellValue('A4', 'hello there world 4');
|
|
$writer = new XlsxWriter($spreadsheet);
|
|
$writerWorksheet = new XlsxWriter\Worksheet($writer);
|
|
$data = $writerWorksheet->writeWorksheet($sheet, []);
|
|
self::assertStringContainsString(
|
|
'<sheetFormatPr customHeight="true" defaultRowHeight="20" outlineLevelRow="0" outlineLevelCol="0"/>',
|
|
$data
|
|
);
|
|
self::assertStringContainsString(
|
|
'<row r="1" spans="1:1">',
|
|
$data
|
|
);
|
|
self::assertStringContainsString(
|
|
'<row r="2" spans="1:1">',
|
|
$data
|
|
);
|
|
self::assertStringContainsString(
|
|
'<row r="4" spans="1:1">',
|
|
$data
|
|
);
|
|
$spreadsheet->disconnectWorksheets();
|
|
}
|
|
|
|
public function testWriteAndReadRowDimension(): void
|
|
{
|
|
$spreadsheet = new Spreadsheet();
|
|
$sheet = $spreadsheet->getActiveSheet();
|
|
$sheet->getDefaultRowDimension()->setRowHeight(20);
|
|
$sheet->setCellValue('A1', 'hello there world 1');
|
|
$sheet->getStyle('A1')->getAlignment()->setWrapText(true);
|
|
$sheet->getRowDimension(1)->setCustomFormat(true);
|
|
$sheet->setCellValue('A2', 'hello there world 2');
|
|
$sheet->setCellValue('A4', 'hello there world 4');
|
|
$reloadedSpreadsheet = $this->writeAndReload($spreadsheet, 'Xlsx');
|
|
$spreadsheet->disconnectWorksheets();
|
|
|
|
$rsheet = $reloadedSpreadsheet->getActiveSheet();
|
|
$rwriter = new XlsxWriter($reloadedSpreadsheet);
|
|
$rwriterWorksheet = new XlsxWriter\Worksheet($rwriter);
|
|
$data = $rwriterWorksheet->writeWorksheet($rsheet, []);
|
|
self::assertStringContainsString(
|
|
'<sheetFormatPr customHeight="true" defaultRowHeight="20" outlineLevelRow="0" outlineLevelCol="0"/>',
|
|
$data
|
|
);
|
|
self::assertStringContainsString(
|
|
'<row r="1" spans="1:1" customFormat="1" ht="-1">',
|
|
$data
|
|
);
|
|
self::assertStringContainsString(
|
|
'<row r="2" spans="1:1" customHeight="1">',
|
|
$data
|
|
);
|
|
self::assertStringContainsString(
|
|
'<row r="4" spans="1:1" customHeight="1">',
|
|
$data
|
|
);
|
|
$reloadedSpreadsheet->disconnectWorksheets();
|
|
}
|
|
|
|
public function testBadOutlineLevel(): void
|
|
{
|
|
$this->expectException(PhpSpreadsheetException::class);
|
|
$this->expectExceptionMessage('Outline level must range between 0 and 7.');
|
|
$spreadsheet = new Spreadsheet();
|
|
$sheet = $spreadsheet->getActiveSheet();
|
|
$sheet->getRowDimension(1)->setOutlineLevel(8);
|
|
}
|
|
}
|