[CssSelector] Add types to private properties

Signed-off-by: Alexander M. Turek <[email protected]>

Author: derrabus

CssSelectorConverter.php

@@ -26,11 +26,11 @@
  */
 class CssSelectorConverter
 {
-    private $translator;
-    private $cache;
+    private Translator $translator;
+    private array $cache;
 
-    private static $xmlCache = [];
-    private static $htmlCache = [];
+    private static array $xmlCache = [];
+    private static array $htmlCache = [];
 
     /**
      * @param bool $html Whether HTML support should be enabled. Disable it for XML documents

Node/AbstractNode.php

@@ -23,17 +23,10 @@
  */
 abstract class AbstractNode implements NodeInterface
 {
-    /**
-     * @var string
-     */
-    private $nodeName;
+    private string $nodeName;
 
     public function getNodeName(): string
     {
-        if (null === $this->nodeName) {
-            $this->nodeName = preg_replace('~.*\\\\([^\\\\]+)Node$~', '$1', static::class);
-        }
-
-        return $this->nodeName;
+        return $this->nodeName ??= preg_replace('~.*\\\\([^\\\\]+)Node$~', '$1', static::class);
     }
 }

Node/AttributeNode.php

@@ -23,11 +23,11 @@
  */
 class AttributeNode extends AbstractNode
 {
-    private $selector;
-    private $namespace;
-    private $attribute;
-    private $operator;
-    private $value;
+    private NodeInterface $selector;
+    private ?string $namespace;
+    private string $attribute;
+    private string $operator;
+    private ?string $value;
 
     public function __construct(NodeInterface $selector, ?string $namespace, string $attribute, string $operator, ?string $value)
     {

Node/ClassNode.php

@@ -23,8 +23,8 @@
  */
 class ClassNode extends AbstractNode
 {
-    private $selector;
-    private $name;
+    private NodeInterface $selector;
+    private string $name;
 
     public function __construct(NodeInterface $selector, string $name)
     {

Node/CombinedSelectorNode.php

@@ -23,9 +23,9 @@
  */
 class CombinedSelectorNode extends AbstractNode
 {
-    private $selector;
-    private $combinator;
-    private $subSelector;
+    private NodeInterface $selector;
+    private string $combinator;
+    private NodeInterface $subSelector;
 
     public function __construct(NodeInterface $selector, string $combinator, NodeInterface $subSelector)
     {

Node/ElementNode.php

@@ -23,8 +23,8 @@
  */
 class ElementNode extends AbstractNode
 {
-    private $namespace;
-    private $element;
+    private ?string $namespace;
+    private ?string $element;
 
     public function __construct(string $namespace = null, string $element = null)
     {

Node/FunctionNode.php

@@ -25,9 +25,9 @@
  */
 class FunctionNode extends AbstractNode
 {
-    private $selector;
-    private $name;
-    private $arguments;
+    private NodeInterface $selector;
+    private string $name;
+    private array $arguments;
 
     /**
      * @param Token[] $arguments

Node/HashNode.php

@@ -23,8 +23,8 @@
  */
 class HashNode extends AbstractNode
 {
-    private $selector;
-    private $id;
+    private NodeInterface $selector;
+    private string $id;
 
     public function __construct(NodeInterface $selector, string $id)
     {

Node/NegationNode.php

@@ -23,8 +23,8 @@
  */
 class NegationNode extends AbstractNode
 {
-    private $selector;
-    private $subSelector;
+    private NodeInterface $selector;
+    private NodeInterface $subSelector;
 
     public function __construct(NodeInterface $selector, NodeInterface $subSelector)
     {

Node/PseudoNode.php

@@ -23,8 +23,8 @@
  */
 class PseudoNode extends AbstractNode
 {
-    private $selector;
-    private $identifier;
+    private NodeInterface $selector;
+    private string $identifier;
 
     public function __construct(NodeInterface $selector, string $identifier)
     {

Node/SelectorNode.php

@@ -23,8 +23,8 @@
  */
 class SelectorNode extends AbstractNode
 {
-    private $tree;
-    private $pseudoElement;
+    private NodeInterface $tree;
+    private ?string $pseudoElement;
 
     public function __construct(NodeInterface $tree, string $pseudoElement = null)
     {

Node/Specificity.php

@@ -29,9 +29,9 @@ class Specificity
     public const B_FACTOR = 10;
     public const C_FACTOR = 1;
 
-    private $a;
-    private $b;
-    private $c;
+    private int $a;
+    private int $b;
+    private int $c;
 
     public function __construct(int $a, int $b, int $c)
     {

Parser/Handler/HashHandler.php

@@ -29,8 +29,8 @@
  */
 class HashHandler implements HandlerInterface
 {
-    private $patterns;
-    private $escaping;
+    private TokenizerPatterns $patterns;
+    private TokenizerEscaping $escaping;
 
     public function __construct(TokenizerPatterns $patterns, TokenizerEscaping $escaping)
     {

Parser/Handler/IdentifierHandler.php

@@ -29,8 +29,8 @@
  */
 class IdentifierHandler implements HandlerInterface
 {
-    private $patterns;
-    private $escaping;
+    private TokenizerPatterns $patterns;
+    private TokenizerEscaping $escaping;
 
     public function __construct(TokenizerPatterns $patterns, TokenizerEscaping $escaping)
     {

Parser/Handler/NumberHandler.php

@@ -28,7 +28,7 @@
  */
 class NumberHandler implements HandlerInterface
 {
-    private $patterns;
+    private TokenizerPatterns $patterns;
 
     public function __construct(TokenizerPatterns $patterns)
     {

Parser/Handler/StringHandler.php

@@ -31,8 +31,8 @@
  */
 class StringHandler implements HandlerInterface
 {
-    private $patterns;
-    private $escaping;
+    private TokenizerPatterns $patterns;
+    private TokenizerEscaping $escaping;
 
     public function __construct(TokenizerPatterns $patterns, TokenizerEscaping $escaping)
     {

Parser/Parser.php

@@ -27,7 +27,7 @@
  */
 class Parser implements ParserInterface
 {
-    private $tokenizer;
+    private Tokenizer $tokenizer;
 
     public function __construct(Tokenizer $tokenizer = null)
     {

Parser/Reader.php

@@ -23,9 +23,9 @@
  */
 class Reader
 {
-    private $source;
-    private $length;
-    private $position = 0;
+    private string $source;
+    private int $length;
+    private int $position = 0;
 
     public function __construct(string $source)
     {

Parser/Token.php

@@ -31,9 +31,9 @@ class Token
     public const TYPE_NUMBER = 'number';
     public const TYPE_STRING = 'string';
 
-    private $type;
-    private $value;
-    private $position;
+    private ?string $type;
+    private ?string $value;
+    private ?int $position;
 
     public function __construct(?string $type, ?string $value, ?int $position)
     {

Parser/TokenStream.php

@@ -29,27 +29,16 @@ class TokenStream
     /**
      * @var Token[]
      */
-    private $tokens = [];
+    private array $tokens = [];
 
     /**
      * @var Token[]
      */
-    private $used = [];
+    private array $used = [];
 
-    /**
-     * @var int
-     */
-    private $cursor = 0;
-
-    /**
-     * @var Token|null
-     */
-    private $peeked;
-
-    /**
-     * @var bool
-     */
-    private $peeking = false;
+    private int $cursor = 0;
+    private ?Token $peeked;
+    private bool $peeking = false;
 
     /**
      * Pushes a token.

Parser/Tokenizer/Tokenizer.php

@@ -31,7 +31,7 @@ class Tokenizer
     /**
      * @var Handler\HandlerInterface[]
      */
-    private $handlers;
+    private array $handlers;
 
     public function __construct()
     {

Parser/Tokenizer/TokenizerEscaping.php

@@ -23,7 +23,7 @@
  */
 class TokenizerEscaping
 {
-    private $patterns;
+    private TokenizerPatterns $patterns;
 
     public function __construct(TokenizerPatterns $patterns)
     {

Parser/Tokenizer/TokenizerPatterns.php

@@ -23,18 +23,18 @@
  */
 class TokenizerPatterns
 {
-    private $unicodeEscapePattern;
-    private $simpleEscapePattern;
-    private $newLineEscapePattern;
-    private $escapePattern;
-    private $stringEscapePattern;
-    private $nonAsciiPattern;
-    private $nmCharPattern;
-    private $nmStartPattern;
-    private $identifierPattern;
-    private $hashPattern;
-    private $numberPattern;
-    private $quotedStringPattern;
+    private string $unicodeEscapePattern;
+    private string $simpleEscapePattern;
+    private string $newLineEscapePattern;
+    private string $escapePattern;
+    private string $stringEscapePattern;
+    private string $nonAsciiPattern;
+    private string $nmCharPattern;
+    private string $nmStartPattern;
+    private string $identifierPattern;
+    private string $hashPattern;
+    private string $numberPattern;
+    private string $quotedStringPattern;
 
     public function __construct()
     {

XPath/Extension/NodeExtension.php

@@ -31,7 +31,7 @@ class NodeExtension extends AbstractExtension
     public const ATTRIBUTE_NAME_IN_LOWER_CASE = 2;
     public const ATTRIBUTE_VALUE_IN_LOWER_CASE = 4;
 
-    private $flags;
+    private int $flags;
 
     public function __construct(int $flags = 0)
     {

XPath/Translator.php

@@ -30,23 +30,23 @@
  */
 class Translator implements TranslatorInterface
 {
-    private $mainParser;
+    private ParserInterface $mainParser;
 
     /**
      * @var ParserInterface[]
      */
-    private $shortcutParsers = [];
+    private array $shortcutParsers = [];
 
     /**
      * @var Extension\ExtensionInterface[]
      */
-    private $extensions = [];
+    private array $extensions = [];
 
-    private $nodeTranslators = [];
-    private $combinationTranslators = [];
-    private $functionTranslators = [];
-    private $pseudoClassTranslators = [];
-    private $attributeMatchingTranslators = [];
+    private array $nodeTranslators = [];
+    private array $combinationTranslators = [];
+    private array $functionTranslators = [];
+    private array $pseudoClassTranslators = [];
+    private array $attributeMatchingTranslators = [];
 
     public function __construct(ParserInterface $parser = null)
     {

XPath/XPathExpr.php

@@ -23,9 +23,9 @@
  */
 class XPathExpr
 {
-    private $path;
-    private $element;
-    private $condition;
+    private string $path;
+    private string $element;
+    private string $condition;
 
     public function __construct(string $path = '', string $element = '*', string $condition = '', bool $starPrefix = false)
     {