mirror of
https://github.com/PHPOffice/PhpSpreadsheet.git
synced 2026-09-02 05:27:44 +00:00
oleibman
9698573e32
It requires an index to an array which needs to be supplied as a string, even though the array is indexed exclusively by int. It is called only once in the code base, by some inefficient code which becomes much simpler when restructured and eliminates the call altogether. It is not independently tested. The use case for its existence is not clear. It should probably just be deprecated. However, for now, I'm just allowing the parameter to be int|string, rather than just string, and adding tests.
89 lines
3.2 KiB
PHP
89 lines
3.2 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace PhpOffice\PhpSpreadsheetTests\Worksheet;
|
|
|
|
use PhpOffice\PhpSpreadsheet\Exception as SpreadsheetException;
|
|
use PhpOffice\PhpSpreadsheet\Spreadsheet;
|
|
use PhpOffice\PhpSpreadsheet\Worksheet\Worksheet;
|
|
use PHPUnit\Framework\Attributes\DataProvider;
|
|
use PHPUnit\Framework\TestCase;
|
|
|
|
class CloneTest extends TestCase
|
|
{
|
|
#[DataProvider('providerCopyClone')]
|
|
public function testUnattachedIndex(string $type): void
|
|
{
|
|
$spreadsheet = new Spreadsheet();
|
|
$sheet1 = $spreadsheet->getActiveSheet();
|
|
$sheet1->getCell('A1')->setValue(10);
|
|
$sheet1->getCell('A2')->setValue(30);
|
|
if ($type === 'copy') {
|
|
$sheet2 = $sheet1->copy();
|
|
} else {
|
|
$sheet2 = clone $sheet1;
|
|
}
|
|
$sheet2->getCell('A1')->setValue(20);
|
|
self::assertSame(0, $spreadsheet->getIndex($sheet1));
|
|
$idx = $spreadsheet->getIndex($sheet2, true);
|
|
self::assertSame(-1, $idx);
|
|
$sheet2->setTitle('clone');
|
|
$spreadsheet->addSheet($sheet2);
|
|
$idx = $spreadsheet->getIndex($sheet2, true);
|
|
self::assertSame(1, $idx);
|
|
self::assertSame(10, $spreadsheet->getSheet(0)->getCell('A1')->getValue());
|
|
self::assertSame(20, $spreadsheet->getSheet(1)->getCell('A1')->getValue());
|
|
self::assertSame(30, $spreadsheet->getSheet(0)->getCell('A2')->getValue());
|
|
self::assertSame(30, $spreadsheet->getSheet(1)->getCell('A2')->getValue());
|
|
$spreadsheet->disconnectWorksheets();
|
|
}
|
|
|
|
public static function providerCopyClone(): array
|
|
{
|
|
return [
|
|
['copy'],
|
|
['clone'],
|
|
];
|
|
}
|
|
|
|
public function testGetCloneIndex(): void
|
|
{
|
|
$this->expectException(SpreadsheetException::class);
|
|
$this->expectExceptionMessage('Sheet does not exist');
|
|
$spreadsheet = new Spreadsheet();
|
|
$sheet1 = $spreadsheet->getActiveSheet();
|
|
$sheet1->getCell('A1')->setValue(10);
|
|
$sheet2 = clone $sheet1;
|
|
$spreadsheet->getSheet($spreadsheet->getIndex($sheet2))
|
|
->setCellValue('A1', 100);
|
|
}
|
|
|
|
public function testSerialize1(): void
|
|
{
|
|
$spreadsheet = new Spreadsheet();
|
|
$sheet1 = $spreadsheet->getActiveSheet();
|
|
$sheet1->getCell('A1')->setValue(10);
|
|
$serialized = serialize($sheet1);
|
|
$newSheet = unserialize($serialized);
|
|
self::assertInstanceOf(Worksheet::class, $newSheet);
|
|
self::assertSame(10, $newSheet->getCell('A1')->getValue());
|
|
self::assertNotSame($newSheet, $sheet1);
|
|
self::assertNotNull($newSheet->getParent());
|
|
self::assertNotSame($newSheet->getParent(), $sheet1->getParent());
|
|
$newSheet->getParent()->disconnectWorksheets();
|
|
$spreadsheet->disconnectWorksheets();
|
|
}
|
|
|
|
public function testSerialize2(): void
|
|
{
|
|
$sheet1 = new Worksheet();
|
|
$sheet1->getCell('A1')->setValue(10);
|
|
$serialized = serialize($sheet1);
|
|
$newSheet = unserialize($serialized);
|
|
self::assertInstanceOf(Worksheet::class, $newSheet);
|
|
self::assertSame(10, $newSheet->getCell('A1')->getValue());
|
|
self::assertNotSame($newSheet, $sheet1);
|
|
}
|
|
}
|