Do not use AlphaNum when string contains comma

eigan · codemasher · commit e5145b19a28c · 2025-09-19T17:43:25.000+02:00
Using dash in this context will create a range between `+` and `.` which allows the following characters: `+`, `,`, `-`, `.`. The ASCII range is as follow: + (43), , (44), - (45), . (46). Comma is not supported and will then crash in `write()`. (cherry picked from commit 03e8485)
diff --git a/src/Data/AlphaNum.php b/src/Data/AlphaNum.php
@@ -44,7 +44,7 @@ public function getLengthInBits():int{
 	 * @inheritDoc
 	 */
 	public static function validateString(string $string):bool{
-		return (bool)preg_match('/^[A-Z\d %$*+-.:\/]+$/', $string);
+		return (bool)preg_match('/^[A-Z\d %$*+\-.:\/]+$/', $string);
 	}
 
 	/**

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ public function getLengthInBits():int{`
`44`	`44`	`* @inheritDoc`
`45`	`45`	`*/`
`46`	`46`	`public static function validateString(string $string):bool{`
`47`		`- return (bool)preg_match('/^[A-Z\d %$*+-.:\/]+$/', $string);`
	`47`	`+ return (bool)preg_match('/^[A-Z\d %$*+\-.:\/]+$/', $string);`
`48`	`48`	`}`
`49`	`49`
`50`	`50`	`/**`