Handle null path better

uuf6429 · uuf6429 · commit 8dea1d2a0d78 · 2025-02-12T23:15:10.000+01:00
diff --git a/src/Behat/Gherkin/Node/FeatureNode.php b/src/Behat/Gherkin/Node/FeatureNode.php
@@ -254,13 +254,11 @@ public function getLine()
      * @param string $file A file path
      *
      * @return bool
-     *
-     * @see https://github.com/symfony/filesystem/blob/master/Filesystem.php
      */
     protected function isAbsolutePath($file)
     {
         if ($file === null) {
-            @trigger_error(sprintf('Calling "%s()" with a null in the $file argument is deprecated since Symfony 4.4.', __METHOD__), E_USER_DEPRECATED);
+            throw new InvalidArgumentException('The provided file path must not be null.');
         }
 
         return strspn($file, '/\\', 0, 1)

Original file line number	Diff line number	Diff line change
`@@ -254,13 +254,11 @@ public function getLine()`
`254`	`254`	`* @param string $file A file path`
`255`	`255`	`*`
`256`	`256`	`* @return bool`
`257`		`- *`
`258`		`- * @see https://github.com/symfony/filesystem/blob/master/Filesystem.php`
`259`	`257`	`*/`
`260`	`258`	`protected function isAbsolutePath($file)`
`261`	`259`	`{`
`262`	`260`	`if ($file === null) {`
`263`		`- @trigger_error(sprintf('Calling "%s()" with a null in the $file argument is deprecated since Symfony 4.4.', __METHOD__), E_USER_DEPRECATED);`
	`261`	`+ throw new InvalidArgumentException('The provided file path must not be null.');`
`264`	`262`	`}`
`265`	`263`
`266`	`264`	`return strspn($file, '/\\', 0, 1)`