mirror of
https://github.com/PHPOffice/PhpSpreadsheet.git
synced 2026-09-15 04:26:25 +00:00
oleibman
5082b35709
Fix #4629. Excel Xml can specify Conditional Formatting in 2 ways - either using a `conditionalFormatting` tag as a child of `worksheet`, or in a slightly different manner as a child of `worksheet->extLst`. Although these are both handled by PhpSpreadsheet, the former is lost if there exists a latter whose cell range (`sqref`) matches it. This PR merges the former with the latter in that circumstance, preserving both. The issue also notes that `font` and `stopIfTrue` are not handled correctly. Support for those was missing in the `extlst` handling, and is now added with this PR.
43 lines
1.7 KiB
PHP
43 lines
1.7 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace PhpOffice\PhpSpreadsheetTests\Reader\Xlsx;
|
|
|
|
use PhpOffice\PhpSpreadsheet\Reader\Xlsx as XlsxReader;
|
|
use PHPUnit\Framework\TestCase;
|
|
|
|
class Issue4629Test extends TestCase
|
|
{
|
|
public function testExternalAndInternalCondionalStyles(): void
|
|
{
|
|
$infile = 'tests/data/Reader/XLSX/issue.4629.xlsx';
|
|
$reader = new XlsxReader();
|
|
$spreadsheet = $reader->load($infile);
|
|
$sheet = $spreadsheet->getSheetByNameOrThrow('top');
|
|
$conditionals = $sheet->getStyle('A1:A20')->getConditionalStyles();
|
|
self::assertCount(3, $conditionals);
|
|
|
|
$conditional = $conditionals[0];
|
|
self::assertSame('expression', $conditional->getConditionType());
|
|
self::assertFalse($conditional->getStopIfTrue());
|
|
self::assertSame(['$A1<>$B1'], $conditional->getConditions());
|
|
self::assertSame(2, $conditional->getPriority());
|
|
|
|
$conditional = $conditionals[1];
|
|
self::assertSame('expression', $conditional->getConditionType());
|
|
self::assertFalse($conditional->getStopIfTrue());
|
|
self::assertSame(['AND($A1="cheese", $C1="yogurt")'], $conditional->getConditions());
|
|
self::assertSame(3, $conditional->getPriority());
|
|
|
|
$conditional = $conditionals[2]; // defined within <ext>
|
|
self::assertSame('expression', $conditional->getConditionType());
|
|
self::assertTrue($conditional->getStopIfTrue());
|
|
self::assertSame(['A1<>bottom!A1'], $conditional->getConditions());
|
|
self::assertSame(1, $conditional->getPriority());
|
|
self::assertSame('FF9C5700', $conditional->getStyle()->getFont()->getColor()->getArgb());
|
|
|
|
$spreadsheet->disconnectWorksheets();
|
|
}
|
|
}
|