diff --git a/src/QROptionsTrait.php b/src/QROptionsTrait.php index 88f24bce3..47ffe2ed2 100644 --- a/src/QROptionsTrait.php +++ b/src/QROptionsTrait.php @@ -389,34 +389,6 @@ trait QROptionsTrait{ $this->version = $version !== Version::AUTO ? max(1, min(40, $version)) : Version::AUTO; } - /** - * sets the error correction level - * - * @todo: accept string values (PHP8+) - * @see https://github.com/chillerlan/php-qrcode/discussions/160 - * - * @throws \chillerlan\QRCode\QRCodeException - */ - protected function set_eccLevel(int $eccLevel):void{ - - if((0b11 & $eccLevel) !== $eccLevel){ - throw new QRCodeException(sprintf('Invalid error correct level: %s', $eccLevel)); - } - - $this->eccLevel = $eccLevel; - } - - /** - * sets/clamps the mask pattern - */ - protected function set_maskPattern(int $maskPattern):void{ - - if($maskPattern !== MaskPattern::AUTO){ - $this->maskPattern = max(0, min(7, $maskPattern)); - } - - } - /** * sets/clamps the quiet zone size */ diff --git a/tests/QROptionsTest.php b/tests/QROptionsTest.php index 51f076b79..d3f8fd0a4 100644 --- a/tests/QROptionsTest.php +++ b/tests/QROptionsTest.php @@ -68,38 +68,6 @@ final class QROptionsTest extends TestCase{ $this::assertSame($expectedMax, $o->versionMax); } - /** - * @return int[][] - */ - public function MaskPatternProvider():array{ - return [ - 'exceed max' => [42, 7,], - 'exceed min' => [-42, 0], - 'default (-1)' => [MaskPattern::AUTO, -1], - ]; - } - - /** - * Tests the $maskPattern clamping - * - * @dataProvider MaskPatternProvider - */ - public function testMaskPatternClamp(int $maskPattern, int $expected):void{ - $o = new QROptions(['maskPattern' => $maskPattern]); - - $this::assertSame($expected, $o->maskPattern); - } - - /** - * Tests if an exception is thrown on an incorrect ECC level - */ - public function testInvalidEccLevelException():void{ - $this->expectException(QRCodeException::class); - $this->expectExceptionMessage('Invalid error correct level: 42'); - - $o = new QROptions(['eccLevel' => 42]); - } - /** * @return int[][][] */