[TASK] Set line number to null by default (#1288)

No longer allow or support `0` as a default line or column number.

Part of #974

Author: JakeQZ

CHANGELOG.md

@@ -32,6 +32,7 @@ Please also have a look at our
 
 ### Changed
 
+- The default line (and column) number is now `null` (not zero) (#1288)
 - `setPosition()` (in `Rule` and other classes) now has fluent interface,
   returning itself (#1259)
 - `RuleSet::removeRule()` now only allows `Rule` as the parameter

src/CSSList/AtRuleBlockList.php

@@ -24,9 +24,9 @@ class AtRuleBlockList extends CSSBlockList implements AtRule
 
     /**
      * @param non-empty-string $type
-     * @param int<0, max> $lineNumber
+     * @param int<1, max>|null $lineNumber
      */
-    public function __construct(string $type, string $arguments = '', int $lineNumber = 0)
+    public function __construct(string $type, string $arguments = '', ?int $lineNumber = null)
     {
         parent::__construct($lineNumber);
         $this->type = $type;

src/CSSList/CSSList.php

@@ -48,9 +48,9 @@ abstract class CSSList implements CSSElement, CSSListItem, Positionable
     protected $contents = [];
 
     /**
-     * @param int<0, max> $lineNumber
+     * @param int<1, max>|null $lineNumber
      */
-    public function __construct(int $lineNumber = 0)
+    public function __construct(?int $lineNumber = null)
     {
         $this->setPosition($lineNumber);
     }

src/Comment/Comment.php

@@ -21,9 +21,9 @@ class Comment implements Positionable, Renderable
     protected $commentText;
 
     /**
-     * @param int<0, max> $lineNumber
+     * @param int<1, max>|null $lineNumber
      */
-    public function __construct(string $commentText = '', int $lineNumber = 0)
+    public function __construct(string $commentText = '', ?int $lineNumber = null)
     {
         $this->commentText = $commentText;
         $this->setPosition($lineNumber);

src/Parsing/SourceException.php

@@ -12,9 +12,9 @@ class SourceException extends \Exception implements Positionable
     use Position;
 
     /**
-     * @param int<0, max> $lineNumber
+     * @param int<1, max>|null $lineNumber
      */
-    public function __construct(string $message, int $lineNumber = 0)
+    public function __construct(string $message, ?int $lineNumber = null)
     {
         $this->setPosition($lineNumber);
         if ($lineNumber !== 0) {

src/Parsing/UnexpectedTokenException.php

@@ -11,9 +11,9 @@ class UnexpectedTokenException extends SourceException
 {
     /**
      * @param 'literal'|'identifier'|'count'|'expression'|'search'|'custom' $matchType
-     * @param int<0, max> $lineNumber
+     * @param int<1, max>|null $lineNumber
      */
-    public function __construct(string $expected, string $found, string $matchType = 'literal', int $lineNumber = 0)
+    public function __construct(string $expected, string $found, string $matchType = 'literal', ?int $lineNumber = null)
     {
         $message = "Token “{$expected}” ({$matchType}) not found. Got “{$found}”.";
         if ($matchType === 'search') {

src/Position/Position.php

@@ -48,15 +48,14 @@ public function getColumnNumber(): ?int
     }
 
     /**
-     * @param int<0, max>|null $lineNumber
+     * @param int<1, max>|null $lineNumber
      * @param int<0, max>|null $columnNumber
      *
      * @return $this fluent interface
      */
     public function setPosition(?int $lineNumber, ?int $columnNumber = null): Positionable
     {
-        // The conditional is for backwards compatibility (backcompat); `0` will not be allowed in future.
-        $this->lineNumber = $lineNumber !== 0 ? $lineNumber : null;
+        $this->lineNumber = $lineNumber;
         $this->columnNumber = $columnNumber;
 
         return $this;

src/Position/Positionable.php

@@ -29,9 +29,7 @@ public function getLineNo(): int;
     public function getColumnNumber(): ?int;
 
     /**
-     * @param int<0, max>|null $lineNumber
-     *        Providing zero for this parameter is deprecated in version 8.9.0, and will not be supported from v9.0.
-     *        Use `null` instead when no line number is available.
+     * @param int<1, max>|null $lineNumber
      * @param int<0, max>|null $columnNumber
      *
      * @return $this fluent interface

src/Property/CSSNamespace.php

@@ -31,9 +31,9 @@ class CSSNamespace implements AtRule, Positionable
 
     /**
      * @param CSSString|URL $url
-     * @param int<0, max> $lineNumber
+     * @param int<1, max>|null $lineNumber
      */
-    public function __construct($url, ?string $prefix = null, int $lineNumber = 0)
+    public function __construct($url, ?string $prefix = null, ?int $lineNumber = null)
     {
         $this->url = $url;
         $this->prefix = $prefix;

src/Property/Charset.php

@@ -29,9 +29,9 @@ class Charset implements AtRule, Positionable
     private $charset;
 
     /**
-     * @param int<0, max> $lineNumber
+     * @param int<1, max>|null $lineNumber
      */
-    public function __construct(CSSString $charset, int $lineNumber = 0)
+    public function __construct(CSSString $charset, ?int $lineNumber = null)
     {
         $this->charset = $charset;
         $this->setPosition($lineNumber);