🔧 estimated bit length was correct, add a safety margin instead

chillerlan · Feb 27, 2024 · da5bdb8 · da5bdb8
1 parent 5371453
commit da5bdb8
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 6 deletions.
diff --git a/src/Data/QRData.php b/src/Data/QRData.php
@@ -165,9 +165,9 @@ public function estimateTotalBitLength():int{
 
 			// it seems that in some cases the estimated total length is not 100% accurate,
 			// so we substract 4 bits from the total when not in mixed mode
-#			if(count($this->dataSegments) <= 1){
-#				$length -= 4;
-#			}
+			if(count($this->dataSegments) <= 1){
+				$length -= 4;
+			}
 
 			// we've got a match!
 			// or let's see if there's a higher version number available
@@ -195,7 +195,7 @@ public function getMinimumVersion():Version{
 
 		// guess the version number within the given range
 		for($version = $this->options->versionMin; $version <= $this->options->versionMax; $version++){
-			if($total <= $this->maxBitsForEcc[$version]){
+			if($total <= ($this->maxBitsForEcc[$version] - 4)){
 				return new Version($version);
 			}
 		}

diff --git a/tests/Data/DataInterfaceTestAbstract.php b/tests/Data/DataInterfaceTestAbstract.php
@@ -11,6 +11,7 @@
 namespace chillerlan\QRCodeTest\Data;
 
 use chillerlan\QRCode\Common\{EccLevel, MaskPattern, Mode, Version};
+use PHPUnit\Framework\ExpectationFailedException;
 use chillerlan\QRCode\Data\{QRCodeDataException, QRData, QRDataModeInterface, QRMatrix};
 use chillerlan\QRCode\QROptions;
 use chillerlan\QRCodeTest\QRMaxLengthTrait;
@@ -187,7 +188,13 @@ public function testGetMinimumVersion(Version $version, EccLevel $eccLevel, stri
 
 		$minimumVersionNumber = $this->QRData->getMinimumVersion()->getVersionNumber();
 
-		$this::assertSame($version->getVersionNumber(), $minimumVersionNumber);
+		try{
+			$this::assertSame($version->getVersionNumber(), $minimumVersionNumber);
+		}
+		catch(ExpectationFailedException $e){
+			$this::assertSame(($version->getVersionNumber() + 1), $minimumVersionNumber, 'safety margin');
+		}
+
 		// verify the encoded data
 		$this::assertSame($this->dataMode::DATAMODE, $bitBuffer->read(4));
 		$this::assertSame($str, $this->dataMode::decodeSegment($bitBuffer, $minimumVersionNumber));

diff --git a/tests/Data/QRDataTest.php b/tests/Data/QRDataTest.php
@@ -86,7 +86,8 @@ public function testEstimateTotalBitLength():void{
 
 		$qrData = new QRData($options, [new Byte($str)]);
 
-		$this::assertSame(980, $qrData->estimateTotalBitLength());
+		$this::assertSame(976, $qrData->estimateTotalBitLength());
+		$this::assertSame(11, $qrData->getMinimumVersion()->getVersionNumber()); // version adjusted to 11
 	}
 
 }