mirror of
https://github.com/PHPOffice/PhpSpreadsheet.git
synced 2026-09-07 08:37:05 +00:00
oleibman
29b07a7c86
* Namespace-Aware Code for SheetViewOptions, SheetProtection (minor break) This is fairly straightforward for SheetViewOptions. SheetProtection is a bit less straightforward. All the attributes should allow for null as well as boolean; this is especially important because Excel defaults to true for some of them and false for others (existing code always treated attributes missing from the XML as false). DocBlocks are updated to indicate the defaults. The test for isProtectionEnabled is changed to reflect this reality. Also, PhpSpreadsheet has been using true/false when writing these, but Excel uses 1/0. Because of these differences, this is a breaking change, but the consequences of the breaks should be minor. Two unit test results needed to change. Remaining areas in Reader/Xlsx which still use the namespace-unaware code include conditional formatting (internal or external), auto filters, unparsed loaded data, data validation (internal or external), alternate content, and header/footer images. * Corrections to Xls Reader and Writer New test and changed sample to go with code changes. Also doc change.
41 lines
1.7 KiB
PHP
41 lines
1.7 KiB
PHP
<?php
|
|
|
|
namespace PhpOffice\PhpSpreadsheetTests\Worksheet;
|
|
|
|
use PhpOffice\PhpSpreadsheet\Worksheet\Protection;
|
|
use PHPUnit\Framework\TestCase;
|
|
|
|
class ProtectionTest extends TestCase
|
|
{
|
|
public function testVerifyPassword(): void
|
|
{
|
|
$protection = new Protection();
|
|
self::assertTrue($protection->verify('foo'), 'non-protected always pass');
|
|
|
|
$protection->setSheet(true);
|
|
self::assertTrue($protection->verify('foo'), 'no password will always pass 1');
|
|
self::assertTrue($protection->verify('xyz'), 'no password will always pass 2');
|
|
|
|
$protection->setPassword('foo', true);
|
|
self::assertSame('foo', $protection->getPassword(), 'was not stored as-is, without hashing');
|
|
self::assertFalse($protection->verify('foo'), 'setting already hashed password will not match');
|
|
|
|
$protection->setPassword('foo');
|
|
self::assertSame('CC40', $protection->getPassword(), 'was hashed');
|
|
self::assertTrue($protection->verify('foo'), 'setting non-hashed password will hash it and not match');
|
|
|
|
$protection->setAlgorithm(Protection::ALGORITHM_MD5);
|
|
self::assertFalse($protection->verify('foo'), 'changing algorithm will not match anymore');
|
|
|
|
$protection->setPassword('foo');
|
|
$hash1 = $protection->getPassword();
|
|
$protection->setPassword('foo');
|
|
$hash2 = $protection->getPassword();
|
|
|
|
self::assertSame(24, mb_strlen($hash1)); // @phpstan-ignore-line
|
|
self::assertSame(24, mb_strlen($hash2)); // @phpstan-ignore-line
|
|
self::assertNotSame($hash1, $hash2, 'was hashed with automatic salt');
|
|
self::assertTrue($protection->verify('foo'), 'setting password again, will hash with proper algorithm and will match');
|
|
}
|
|
}
|