-

src/Client.php

@@ -41,6 +41,7 @@
 use Symfony\Component\Panther\DomCrawler\Link as PantherLink;
 use Symfony\Component\Panther\Exception\InvalidArgumentException;
 use Symfony\Component\Panther\Exception\LogicException;
+use Symfony\Component\Panther\Exception\RuntimeException;
 use Symfony\Component\Panther\ProcessManager\BrowserManagerInterface;
 use Symfony\Component\Panther\ProcessManager\ChromeManager;
 use Symfony\Component\Panther\ProcessManager\FirefoxManager;
@@ -66,25 +67,25 @@ final class Client extends AbstractBrowser implements WebDriver, JavaScriptExecu
     /**
      * @param string[]|null $arguments
      */
-    public static function createChromeClient(string $chromeDriverBinary = null, array $arguments = null, array $options = [], string $baseUri = null): self
+    public static function createChromeClient(?string $chromeDriverBinary = null, ?array $arguments = null, array $options = [], ?string $baseUri = null): self
     {
         return new self(new ChromeManager($chromeDriverBinary, $arguments, $options), $baseUri);
     }
 
     /**
      * @param string[]|null $arguments
      */
-    public static function createFirefoxClient(string $geckodriverBinary = null, array $arguments = null, array $options = [], string $baseUri = null): self
+    public static function createFirefoxClient(?string $geckodriverBinary = null, ?array $arguments = null, array $options = [], ?string $baseUri = null): self
     {
         return new self(new FirefoxManager($geckodriverBinary, $arguments, $options), $baseUri);
     }
 
-    public static function createSeleniumClient(string $host = null, WebDriverCapabilities $capabilities = null, string $baseUri = null, array $options = []): self
+    public static function createSeleniumClient(?string $host = null, ?WebDriverCapabilities $capabilities = null, ?string $baseUri = null, array $options = []): self
     {
         return new self(new SeleniumManager($host, $capabilities, $options), $baseUri);
     }
 
-    public function __construct(BrowserManagerInterface $browserManager, string $baseUri = null)
+    public function __construct(BrowserManagerInterface $browserManager, ?string $baseUri = null)
     {
         $this->browserManager = $browserManager;
         $this->baseUri = $baseUri;
@@ -254,7 +255,7 @@ public function refreshCrawler(): PantherCrawler
         return $this->crawler = $this->createCrawler();
     }
 
-    public function request(string $method, string $uri, array $parameters = [], array $files = [], array $server = [], string $content = null, bool $changeHistory = true): PantherCrawler
+    public function request(string $method, string $uri, array $parameters = [], array $files = [], array $server = [], ?string $content = null, bool $changeHistory = true): PantherCrawler
     {
         if ('GET' !== $method) {
             throw new InvalidArgumentException('Only the GET method is supported when using WebDriver.');
@@ -268,7 +269,7 @@ public function request(string $method, string $uri, array $parameters = [], arr
 
         foreach (['parameters', 'files', 'server'] as $arg) {
             if ([] !== $$arg) {
-                throw new InvalidArgumentException(sprintf('The parameter "$%s" is not supported when using WebDriver.', $arg));
+                throw new InvalidArgumentException(\sprintf('The parameter "$%s" is not supported when using WebDriver.', $arg));
             }
         }
 
@@ -770,14 +771,14 @@ public function ping(int $timeout = 1000): bool
     }
 
     /**
-     * @return \LogicException|\RuntimeException
+     * @return LogicException|RuntimeException
      */
     private function createException(string $implementableClass): \Exception
     {
         if (null === $this->webDriver) {
-            return new \LogicException(sprintf('WebDriver not started yet. Call method `start()` first before calling any `%s` method.', $implementableClass));
+            return new LogicException(\sprintf('WebDriver not started yet. Call method `start()` first before calling any `%s` method.', $implementableClass));
         }
 
-        return new \RuntimeException(sprintf('"%s" does not implement "%s".', \get_class($this->webDriver), $implementableClass));
+        return new RuntimeException(\sprintf('"%s" does not implement "%s".', \get_class($this->webDriver), $implementableClass));
     }
 }

src/Cookie/CookieJar.php

@@ -133,7 +133,7 @@ private function webDriverToSymfony(WebDriverCookie $cookie): Cookie
         return new Cookie($cookie->getName(), $cookie->getValue(), $expiry, $cookie->getPath(), (string) $cookie->getDomain(), (bool) $cookie->isSecure(), (bool) $cookie->isHttpOnly());
     }
 
-    private function getWebDriverCookie(string $name, string $path = '/', string $domain = null): ?WebDriverCookie
+    private function getWebDriverCookie(string $name, string $path = '/', ?string $domain = null): ?WebDriverCookie
     {
         try {
             $cookie = $this->webDriver->manage()->getCookieNamed($name);

src/DomCrawler/Crawler.php

@@ -36,7 +36,7 @@ final class Crawler extends BaseCrawler implements WebDriverElement
     /**
      * @param WebDriverElement[] $elements
      */
-    public function __construct(array $elements = [], WebDriver $webDriver = null, string $uri = null)
+    public function __construct(array $elements = [], ?WebDriver $webDriver = null, ?string $uri = null)
     {
         $this->uri = $uri;
         $this->webDriver = $webDriver;
@@ -179,7 +179,7 @@ public function ancestors(): static
     /**
      * @see https://github.com/symfony/symfony/issues/26432
      */
-    public function children(string $selector = null): static
+    public function children(?string $selector = null): static
     {
         $xpath = 'child::*';
         if (null !== $selector) {
@@ -205,7 +205,7 @@ public function nodeName(): string
         return $this->getElementOrThrow()->getTagName();
     }
 
-    public function text(string $default = null, bool $normalizeWhitespace = true): string
+    public function text(?string $default = null, bool $normalizeWhitespace = true): string
     {
         if (!$normalizeWhitespace) {
             throw new InvalidArgumentException('Panther only supports getting normalized text.');
@@ -222,7 +222,7 @@ public function text(string $default = null, bool $normalizeWhitespace = true):
         }
     }
 
-    public function html(string $default = null): string
+    public function html(?string $default = null): string
     {
         try {
             $element = $this->getElementOrThrow();
@@ -276,19 +276,19 @@ public function filter($selector): static
     public function selectLink($value): static
     {
         return $this->selectFromXpath(
-            sprintf('descendant-or-self::a[contains(concat(\' \', normalize-space(string(.)), \' \'), %1$s) or ./img[contains(concat(\' \', normalize-space(string(@alt)), \' \'), %1$s)]]', self::xpathLiteral(' '.$value.' '))
+            \sprintf('descendant-or-self::a[contains(concat(\' \', normalize-space(string(.)), \' \'), %1$s) or ./img[contains(concat(\' \', normalize-space(string(@alt)), \' \'), %1$s)]]', self::xpathLiteral(' '.$value.' '))
         );
     }
 
     public function selectImage($value): static
     {
-        return $this->selectFromXpath(sprintf('descendant-or-self::img[contains(normalize-space(string(@alt)), %s)]', self::xpathLiteral($value)));
+        return $this->selectFromXpath(\sprintf('descendant-or-self::img[contains(normalize-space(string(@alt)), %s)]', self::xpathLiteral($value)));
     }
 
     public function selectButton($value): static
     {
         return $this->selectFromXpath(
-            sprintf(
+            \sprintf(
                 'descendant-or-self::input[((contains(%1$s, "submit") or contains(%1$s, "button")) and contains(concat(\' \', normalize-space(string(@value)), \' \'), %2$s)) or (contains(%1$s, "image") and contains(concat(\' \', normalize-space(string(@alt)), \' \'), %2$s)) or @id=%3$s or @name=%3$s] | descendant-or-self::button[contains(concat(\' \', normalize-space(string(.)), \' \'), %2$s) or @id=%3$s or @name=%3$s]',
                 'translate(@type, "ABCDEFGHIJKLMNOPQRSTUVWXYZ", "abcdefghijklmnopqrstuvwxyz")',
                 self::xpathLiteral(' '.$value.' '),
@@ -332,7 +332,7 @@ public function images(): array
         return $images;
     }
 
-    public function form(array $values = null, $method = null): Form
+    public function form(?array $values = null, $method = null): Form
     {
         $form = new Form($this->getElementOrThrow(), $this->webDriver);
         if (null !== $values) {
@@ -395,7 +395,7 @@ private function selectFromXpath(string $xpath): self
     /**
      * @param WebDriverElement[]|null $nodes
      */
-    private function createSubCrawler(array $nodes = null): self
+    private function createSubCrawler(?array $nodes = null): self
     {
         return new self($nodes ?? [], $this->webDriver, $this->uri);
     }

src/DomCrawler/Field/ChoiceFormField.php

@@ -54,7 +54,7 @@ public function select($value): void
     public function tick(): void
     {
         if ('checkbox' !== $type = $this->element->getAttribute('type')) {
-            throw new LogicException(sprintf('You cannot tick "%s" as it is not a checkbox (%s).', $this->element->getAttribute('name'), $type));
+            throw new LogicException(\sprintf('You cannot tick "%s" as it is not a checkbox (%s).', $this->element->getAttribute('name'), $type));
         }
 
         $this->setValue(true);
@@ -68,7 +68,7 @@ public function tick(): void
     public function untick(): void
     {
         if ('checkbox' !== $type = $this->element->getAttribute('type')) {
-            throw new LogicException(sprintf('You cannot tick "%s" as it is not a checkbox (%s).', $this->element->getAttribute('name'), $type));
+            throw new LogicException(\sprintf('You cannot tick "%s" as it is not a checkbox (%s).', $this->element->getAttribute('name'), $type));
         }
 
         $this->setValue(false);
@@ -116,7 +116,7 @@ public function setValue($value): void
     {
         if (\is_bool($value)) {
             if ('checkbox' !== $this->type) {
-                throw new InvalidArgumentException(sprintf('Invalid argument of type "%s"', \gettype($value)));
+                throw new InvalidArgumentException(\sprintf('Invalid argument of type "%s"', \gettype($value)));
             }
 
             if ($value) {
@@ -191,12 +191,12 @@ protected function initialize(): void
     {
         $tagName = $this->element->getTagName();
         if ('input' !== $tagName && 'select' !== $tagName) {
-            throw new LogicException(sprintf('A ChoiceFormField can only be created from an input or select tag (%s given).', $tagName));
+            throw new LogicException(\sprintf('A ChoiceFormField can only be created from an input or select tag (%s given).', $tagName));
         }
 
         $type = strtolower((string) $this->element->getAttribute('type'));
         if ('input' === $tagName && 'checkbox' !== $type && 'radio' !== $type) {
-            throw new LogicException(sprintf('A ChoiceFormField can only be created from an input tag with a type of checkbox or radio (given type is %s).', $type));
+            throw new LogicException(\sprintf('A ChoiceFormField can only be created from an input tag with a type of checkbox or radio (given type is %s).', $type));
         }
 
         $this->type = 'select' === $tagName ? 'select' : $type;

src/DomCrawler/Field/FileFormField.php

@@ -68,12 +68,12 @@ protected function initialize(): void
     {
         $tagName = $this->element->getTagName();
         if ('input' !== $tagName) {
-            throw new LogicException(sprintf('A FileFormField can only be created from an input tag (%s given).', $tagName));
+            throw new LogicException(\sprintf('A FileFormField can only be created from an input tag (%s given).', $tagName));
         }
 
         $type = strtolower($this->element->getAttribute('type'));
         if ('file' !== $type) {
-            throw new LogicException(sprintf('A FileFormField can only be created from an input tag with a type of file (given type is %s).', $type));
+            throw new LogicException(\sprintf('A FileFormField can only be created from an input tag with a type of file (given type is %s).', $type));
         }
 
         $value = $this->element->getAttribute('value');

src/DomCrawler/Field/InputFormField.php

@@ -49,7 +49,7 @@ protected function initialize(): void
     {
         $tagName = $this->element->getTagName();
         if ('input' !== $tagName && 'button' !== $tagName) {
-            throw new LogicException(sprintf('An InputFormField can only be created from an input or button tag (%s given).', $tagName));
+            throw new LogicException(\sprintf('An InputFormField can only be created from an input or button tag (%s given).', $tagName));
         }
 
         $type = strtolower((string) $this->element->getAttribute('type'));

src/DomCrawler/Field/TextareaFormField.php

@@ -37,7 +37,7 @@ protected function initialize(): void
     {
         $tagName = $this->element->getTagName();
         if ('textarea' !== $tagName) {
-            throw new LogicException(sprintf('A TextareaFormField can only be created from a textarea tag (%s given).', $tagName));
+            throw new LogicException(\sprintf('A TextareaFormField can only be created from a textarea tag (%s given).', $tagName));
         }
     }
 }

src/DomCrawler/Form.php

@@ -62,7 +62,7 @@ private function setElement(WebDriverElement $element): void
                 try {
                     $form = $this->webDriver->findElement(WebDriverBy::id($formId));
                 } catch (NoSuchElementException $e) {
-                    throw new LogicException(sprintf('The selected node has an invalid form attribute (%s).', $formId));
+                    throw new LogicException(\sprintf('The selected node has an invalid form attribute (%s).', $formId));
                 }
 
                 $this->element = $form;
@@ -78,7 +78,7 @@ private function setElement(WebDriverElement $element): void
                 }
             } while ('form' !== $element->getTagName());
         } elseif ('form' !== $tagName = $element->getTagName()) {
-            throw new LogicException(sprintf('Unable to submit on a "%s" tag.', $tagName));
+            throw new LogicException(\sprintf('Unable to submit on a "%s" tag.', $tagName));
         }
 
         $this->element = $element;
@@ -168,7 +168,7 @@ public function getFiles(): array
                 continue;
             }
 
-            if ($field instanceof Field\FileFormField) {
+            if ($field instanceof FileFormField) {
                 $files[$field->getName()] = $field->getValue();
             }
         }
@@ -272,7 +272,7 @@ protected function getRawUri(): string
     private function getFormElement(string $name): WebDriverElement
     {
         return $this->element->findElement(WebDriverBy::xpath(
-            sprintf('.//input[@name=%1$s] | .//textarea[@name=%1$s] | .//select[@name=%1$s] | .//button[@name=%1$s] | .//input[@name=%2$s] | .//textarea[@name=%2$s] | .//select[@name=%2$s] | .//button[@name=%2$s]', XPathEscaper::escapeQuotes($name), XPathEscaper::escapeQuotes($name.'[]'))
+            \sprintf('.//input[@name=%1$s] | .//textarea[@name=%1$s] | .//select[@name=%1$s] | .//button[@name=%1$s] | .//input[@name=%2$s] | .//textarea[@name=%2$s] | .//select[@name=%2$s] | .//button[@name=%2$s]', XPathEscaper::escapeQuotes($name), XPathEscaper::escapeQuotes($name.'[]'))
         ));
     }
 

src/DomCrawler/Image.php

@@ -30,7 +30,7 @@ final class Image extends BaseImage
     public function __construct(WebDriverElement $element)
     {
         if ('img' !== $tagName = $element->getTagName()) {
-            throw new LogicException(sprintf('Unable to visualize a "%s" tag.', $tagName));
+            throw new LogicException(\sprintf('Unable to visualize a "%s" tag.', $tagName));
         }
 
         $this->element = $element;

src/DomCrawler/Link.php

@@ -31,7 +31,7 @@ public function __construct(WebDriverElement $element, string $currentUri)
     {
         $tagName = $element->getTagName();
         if ('a' !== $tagName && 'area' !== $tagName && 'link' !== $tagName) {
-            throw new LogicException(sprintf('Unable to navigate from a "%s" tag.', $tagName));
+            throw new LogicException(\sprintf('Unable to navigate from a "%s" tag.', $tagName));
         }
 
         $this->element = $element;

src/Exception/ExceptionInterface.php

@@ -9,6 +9,8 @@
  * file that was distributed with this source code.
  */
 
+declare(strict_types=1);
+
 namespace Symfony\Component\Panther\Exception;
 
 /**

src/Exception/InvalidArgumentException.php

@@ -9,6 +9,8 @@
  * file that was distributed with this source code.
  */
 
+declare(strict_types=1);
+
 namespace Symfony\Component\Panther\Exception;
 
 /**

src/Exception/LogicException.php

@@ -9,6 +9,8 @@
  * file that was distributed with this source code.
  */
 
+declare(strict_types=1);
+
 namespace Symfony\Component\Panther\Exception;
 
 /**

src/Exception/RuntimeException.php

@@ -9,6 +9,8 @@
  * file that was distributed with this source code.
  */
 
+declare(strict_types=1);
+
 namespace Symfony\Component\Panther\Exception;
 
 /**

src/ExceptionThrower.php

@@ -22,6 +22,6 @@ trait ExceptionThrower
 {
     private function createNotSupportedException(string $method): \InvalidArgumentException
     {
-        return new \InvalidArgumentException(sprintf('The "%s" method is not supported when using WebDriver.', $method));
+        return new \InvalidArgumentException(\sprintf('The "%s" method is not supported when using WebDriver.', $method));
     }
 }