Graceful preg error handling when parsing names

GrahamCampbell · GrahamCampbell · commit 5c8934c0f2f3 · 2020-05-22T16:40:19.000+01:00
diff --git a/src/Parser.php b/src/Parser.php
@@ -3,6 +3,7 @@
 namespace Dotenv;
 
 use Dotenv\Exception\InvalidFileException;
+use Dotenv\Regex\Regex;
 
 class Parser
 {
@@ -87,7 +88,7 @@ private static function parseName($name)
      */
     private static function isValidName($name)
     {
-        return preg_match('~\A[a-zA-Z0-9_.]+\z~', $name) === 1;
+        return Regex::match('~\A[a-zA-Z0-9_.]+\z~', $name)->success()->getOrElse(0) === 1;
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`namespace Dotenv;`
`4`	`4`
`5`	`5`	`use Dotenv\Exception\InvalidFileException;`
	`6`	`+use Dotenv\Regex\Regex;`
`6`	`7`
`7`	`8`	`class Parser`
`8`	`9`	`{`
`@@ -87,7 +88,7 @@ private static function parseName($name)`
`87`	`88`	`*/`
`88`	`89`	`private static function isValidName($name)`
`89`	`90`	`{`
`90`		`- return preg_match('~\A[a-zA-Z0-9_.]+\z~', $name) === 1;`
	`91`	`+ return Regex::match('~\A[a-zA-Z0-9_.]+\z~', $name)->success()->getOrElse(0) === 1;`
`91`	`92`	`}`
`92`	`93`
`93`	`94`	`/**`