[TASK] Make constructors of CSS element classes @internal

oliverklee · oliverklee · commit b7f855337999 · 2025-02-09T15:05:36.000+01:00
These classes are intended to be instantiated only from within this library. Fixes #858
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -21,6 +21,7 @@ Please also have a look at our
 
 ### Changed
 
+- Make constructors of CSS element classes `@internal` (#888)
 - Make all non-private properties `@internal` (#886)
 - Use more native type declarations and strict mode
   (#641, #772, #774, #778, #804, #841, #873, #875)
diff --git a/src/CSSList/AtRuleBlockList.php b/src/CSSList/AtRuleBlockList.php
@@ -26,6 +26,8 @@ class AtRuleBlockList extends CSSBlockList implements AtRule
      * @param string $type
      * @param string $arguments
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($type, $arguments = '', $lineNumber = 0)
     {
diff --git a/src/CSSList/CSSBlockList.php b/src/CSSList/CSSBlockList.php
@@ -22,6 +22,8 @@ abstract class CSSBlockList extends CSSList
 {
     /**
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($lineNumber = 0)
     {
diff --git a/src/CSSList/CSSList.php b/src/CSSList/CSSList.php
@@ -56,6 +56,8 @@ abstract class CSSList implements Renderable, Commentable
 
     /**
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($lineNumber = 0)
     {
diff --git a/src/CSSList/Document.php b/src/CSSList/Document.php
@@ -20,6 +20,8 @@ class Document extends CSSBlockList
 {
     /**
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($lineNumber = 0)
     {
diff --git a/src/Comment/Comment.php b/src/Comment/Comment.php
@@ -26,6 +26,8 @@ class Comment implements Renderable
     /**
      * @param string $commentText
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($commentText = '', $lineNumber = 0)
     {
diff --git a/src/Parsing/Anchor.php b/src/Parsing/Anchor.php
@@ -21,6 +21,8 @@ class Anchor
 
     /**
      * @param int $iPosition
+     *
+     * @internal since 8.8.0
      */
     public function __construct($iPosition, ParserState $parserState)
     {
diff --git a/src/Property/CSSNamespace.php b/src/Property/CSSNamespace.php
@@ -38,6 +38,8 @@ class CSSNamespace implements AtRule
      * @param string $mUrl
      * @param string|null $sPrefix
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($mUrl, $sPrefix = null, $lineNumber = 0)
     {
diff --git a/src/Property/Charset.php b/src/Property/Charset.php
@@ -39,6 +39,8 @@ class Charset implements AtRule
 
     /**
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct(CSSString $oCharset, $lineNumber = 0)
     {
diff --git a/src/Property/Import.php b/src/Property/Import.php
@@ -40,6 +40,8 @@ class Import implements AtRule
     /**
      * @param string $mediaQuery
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct(URL $location, $mediaQuery, $lineNumber = 0)
     {
diff --git a/src/Property/Selector.php b/src/Property/Selector.php
@@ -87,6 +87,8 @@ public static function isValid($sSelector)
     /**
      * @param string $sSelector
      * @param bool $bCalculateSpecificity
+     *
+     * @internal since 8.8.0
      */
     public function __construct($sSelector, $bCalculateSpecificity = false)
     {
diff --git a/src/Rule/Rule.php b/src/Rule/Rule.php
@@ -64,6 +64,8 @@ class Rule implements Renderable, Commentable
      * @param string $sRule
      * @param int $lineNumber
      * @param int $iColNo
+     *
+     * @internal since 8.8.0
      */
     public function __construct($sRule, $lineNumber = 0, $iColNo = 0)
     {
diff --git a/src/RuleSet/AtRuleSet.php b/src/RuleSet/AtRuleSet.php
@@ -29,6 +29,8 @@ class AtRuleSet extends RuleSet implements AtRule
      * @param string $sType
      * @param string $sArgs
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($sType, $sArgs = '', $lineNumber = 0)
     {
diff --git a/src/RuleSet/DeclarationBlock.php b/src/RuleSet/DeclarationBlock.php
@@ -31,6 +31,8 @@ class DeclarationBlock extends RuleSet
 
     /**
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($lineNumber = 0)
     {
diff --git a/src/RuleSet/RuleSet.php b/src/RuleSet/RuleSet.php
@@ -45,6 +45,8 @@ abstract class RuleSet implements Renderable, Commentable
 
     /**
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($lineNumber = 0)
     {
diff --git a/src/Value/CSSFunction.php b/src/Value/CSSFunction.php
@@ -25,6 +25,8 @@ class CSSFunction extends ValueList
      * @param RuleValueList|array<array-key, Value|string> $aArguments
      * @param string $sSeparator
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($sName, $aArguments, $sSeparator = ',', $lineNumber = 0)
     {
diff --git a/src/Value/CSSString.php b/src/Value/CSSString.php
@@ -25,6 +25,8 @@ class CSSString extends PrimitiveValue
     /**
      * @param string $sString
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($sString, $lineNumber = 0)
     {
diff --git a/src/Value/CalcRuleValueList.php b/src/Value/CalcRuleValueList.php
@@ -10,6 +10,8 @@ class CalcRuleValueList extends RuleValueList
 {
     /**
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($lineNumber = 0)
     {
diff --git a/src/Value/Color.php b/src/Value/Color.php
@@ -18,6 +18,8 @@ class Color extends CSSFunction
     /**
      * @param array<array-key, Value|string> $colorValues
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct(array $colorValues, $lineNumber = 0)
     {
diff --git a/src/Value/LineName.php b/src/Value/LineName.php
@@ -14,6 +14,8 @@ class LineName extends ValueList
     /**
      * @param array<int, Value|string> $aComponents
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct(array $aComponents = [], $lineNumber = 0)
     {
diff --git a/src/Value/PrimitiveValue.php b/src/Value/PrimitiveValue.php
@@ -8,6 +8,8 @@ abstract class PrimitiveValue extends Value
 {
     /**
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($lineNumber = 0)
     {
diff --git a/src/Value/RuleValueList.php b/src/Value/RuleValueList.php
@@ -14,6 +14,8 @@ class RuleValueList extends ValueList
     /**
      * @param string $sSeparator
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($sSeparator = ',', $lineNumber = 0)
     {
diff --git a/src/Value/Size.php b/src/Value/Size.php
@@ -72,6 +72,8 @@ class Size extends PrimitiveValue
      * @param string|null $sUnit
      * @param bool $bIsColorComponent
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($fSize, $sUnit = null, $bIsColorComponent = false, $lineNumber = 0)
     {
diff --git a/src/Value/URL.php b/src/Value/URL.php
@@ -22,6 +22,8 @@ class URL extends PrimitiveValue
 
     /**
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct(CSSString $oURL, $lineNumber = 0)
     {
diff --git a/src/Value/Value.php b/src/Value/Value.php
@@ -25,6 +25,8 @@ abstract class Value implements Renderable
 
     /**
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($lineNumber = 0)
     {
diff --git a/src/Value/ValueList.php b/src/Value/ValueList.php
@@ -32,6 +32,8 @@ abstract class ValueList extends Value
      * @param array<array-key, Value|string>|Value|string $aComponents
      * @param string $sSeparator
      * @param int $lineNumber
+     *
+     * @internal since 8.8.0
      */
     public function __construct($aComponents = [], $sSeparator = ',', $lineNumber = 0)
     {

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,8 @@ class AtRuleBlockList extends CSSBlockList implements AtRule`
`26`	`26`	`* @param string $type`
`27`	`27`	`* @param string $arguments`
`28`	`28`	`* @param int $lineNumber`
	`29`	`+ *`
	`30`	`+ * @internal since 8.8.0`
`29`	`31`	`*/`
`30`	`32`	`public function __construct($type, $arguments = '', $lineNumber = 0)`
`31`	`33`	`{`
Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,8 @@ abstract class CSSBlockList extends CSSList`
`22`	`22`	`{`
`23`	`23`	`/**`
`24`	`24`	`* @param int $lineNumber`
	`25`	`+ *`
	`26`	`+ * @internal since 8.8.0`
`25`	`27`	`*/`
`26`	`28`	`public function __construct($lineNumber = 0)`
`27`	`29`	`{`
Original file line number	Diff line number	Diff line change
`@@ -56,6 +56,8 @@ abstract class CSSList implements Renderable, Commentable`
`56`	`56`
`57`	`57`	`/**`
`58`	`58`	`* @param int $lineNumber`
	`59`	`+ *`
	`60`	`+ * @internal since 8.8.0`
`59`	`61`	`*/`
`60`	`62`	`public function __construct($lineNumber = 0)`
`61`	`63`	`{`
Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,8 @@ class Document extends CSSBlockList`
`20`	`20`	`{`
`21`	`21`	`/**`
`22`	`22`	`* @param int $lineNumber`
	`23`	`+ *`
	`24`	`+ * @internal since 8.8.0`
`23`	`25`	`*/`
`24`	`26`	`public function __construct($lineNumber = 0)`
`25`	`27`	`{`
Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,8 @@ class Comment implements Renderable`
`26`	`26`	`/**`
`27`	`27`	`* @param string $commentText`
`28`	`28`	`* @param int $lineNumber`
	`29`	`+ *`
	`30`	`+ * @internal since 8.8.0`
`29`	`31`	`*/`
`30`	`32`	`public function __construct($commentText = '', $lineNumber = 0)`
`31`	`33`	`{`
Original file line number	Diff line number	Diff line change
`@@ -21,6 +21,8 @@ class Anchor`
`21`	`21`
`22`	`22`	`/**`
`23`	`23`	`* @param int $iPosition`
	`24`	`+ *`
	`25`	`+ * @internal since 8.8.0`
`24`	`26`	`*/`
`25`	`27`	`public function __construct($iPosition, ParserState $parserState)`
`26`	`28`	`{`
Original file line number	Diff line number	Diff line change
`@@ -38,6 +38,8 @@ class CSSNamespace implements AtRule`
`38`	`38`	`* @param string $mUrl`
`39`	`39`	`* @param string\|null $sPrefix`
`40`	`40`	`* @param int $lineNumber`
	`41`	`+ *`
	`42`	`+ * @internal since 8.8.0`
`41`	`43`	`*/`
`42`	`44`	`public function __construct($mUrl, $sPrefix = null, $lineNumber = 0)`
`43`	`45`	`{`
Original file line number	Diff line number	Diff line change
`@@ -39,6 +39,8 @@ class Charset implements AtRule`
`39`	`39`
`40`	`40`	`/**`
`41`	`41`	`* @param int $lineNumber`
	`42`	`+ *`
	`43`	`+ * @internal since 8.8.0`
`42`	`44`	`*/`
`43`	`45`	`public function __construct(CSSString $oCharset, $lineNumber = 0)`
`44`	`46`	`{`
Original file line number	Diff line number	Diff line change
`@@ -40,6 +40,8 @@ class Import implements AtRule`
`40`	`40`	`/**`
`41`	`41`	`* @param string $mediaQuery`
`42`	`42`	`* @param int $lineNumber`
	`43`	`+ *`
	`44`	`+ * @internal since 8.8.0`
`43`	`45`	`*/`
`44`	`46`	`public function __construct(URL $location, $mediaQuery, $lineNumber = 0)`
`45`	`47`	`{`