13020705351

Committed 28 Jan 2025 10:26PM UTC coverage: 44.039%. Remained the same

Build # 13020705351

Build Type

push

github

Committed by

web-flow

Commit Message

[CLEANUP] Avoid Hungarian notation for `oSelector` (#846)

Part of #756

Run Details

2 of 6 new or added lines in 2 files covered. (33.33%)

761 of 1728 relevant lines covered (44.04%)

10.34 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

0.0

/src/CSSList/CSSBlockList.php

<?php

declare(strict_types=1);

namespace Sabberworm\CSS\CSSList;

use Sabberworm\CSS\Property\Selector;
use Sabberworm\CSS\Rule\Rule;
use Sabberworm\CSS\RuleSet\DeclarationBlock;
use Sabberworm\CSS\RuleSet\RuleSet;
use Sabberworm\CSS\Value\CSSFunction;
use Sabberworm\CSS\Value\Value;
use Sabberworm\CSS\Value\ValueList;

/**
 * A `CSSBlockList` is a `CSSList` whose `DeclarationBlock`s are guaranteed to contain valid declaration blocks or
 * at-rules.
 *
 * Most `CSSList`s conform to this category but some at-rules (such as `@keyframes`) do not.
 */
abstract class CSSBlockList extends CSSList
{
    /**
     * @param int $lineNumber
     */
    public function __construct($lineNumber = 0)
    {
        parent::__construct($lineNumber);
    }

    /**
     * @param array<int, DeclarationBlock> $result
     */
    protected function allDeclarationBlocks(array &$result): void
    {
        foreach ($this->aContents as $mContent) {
            if ($mContent instanceof DeclarationBlock) {
                $result[] = $mContent;
            } elseif ($mContent instanceof CSSBlockList) {
                $mContent->allDeclarationBlocks($result);
            }
        }
    }

    /**
     * @param array<int, RuleSet> $result
     */
    protected function allRuleSets(array &$result): void
    {
        foreach ($this->aContents as $mContent) {
            if ($mContent instanceof RuleSet) {
                $result[] = $mContent;
            } elseif ($mContent instanceof CSSBlockList) {
                $mContent->allRuleSets($result);
            }
        }
    }

    /**
     * @param CSSList|Rule|RuleSet|Value $element
     * @param array<int, Value> $result
     * @param string|null $searchString
     * @param bool $searchInFunctionArguments
     */
    protected function allValues(
        $element,
        array &$result,
        $searchString = null,
        $searchInFunctionArguments = false
    ): void {
        if ($element instanceof CSSBlockList) {
            foreach ($element->getContents() as $oContent) {
                $this->allValues($oContent, $result, $searchString, $searchInFunctionArguments);
            }
        } elseif ($element instanceof RuleSet) {
            foreach ($element->getRules($searchString) as $rule) {
                $this->allValues($rule, $result, $searchString, $searchInFunctionArguments);
            }
        } elseif ($element instanceof Rule) {
            $this->allValues($element->getValue(), $result, $searchString, $searchInFunctionArguments);
        } elseif ($element instanceof ValueList) {
            if ($searchInFunctionArguments || !($element instanceof CSSFunction)) {
                foreach ($element->getListComponents() as $mComponent) {
                    $this->allValues($mComponent, $result, $searchString, $searchInFunctionArguments);
                }
            }
        } else {
            // Non-List `Value` or `CSSString` (CSS identifier)
            $result[] = $element;
        }
    }

    /**
     * @param array<int, Selector> $result
     * @param string|null $specificitySearch
     */
    protected function allSelectors(array &$result, $specificitySearch = null): void
    {
        /** @var array<int, DeclarationBlock> $aDeclarationBlocks */
        $aDeclarationBlocks = [];
        $this->allDeclarationBlocks($aDeclarationBlocks);
        foreach ($aDeclarationBlocks as $oBlock) {
            foreach ($oBlock->getSelectors() as $selector) {
                if ($specificitySearch === null) {
                    $result[] = $selector;
                } else {
                    $sComparator = '===';
                    $aSpecificitySearch = \explode(' ', $specificitySearch);
                    $iTargetSpecificity = $aSpecificitySearch[0];
                    if (\count($aSpecificitySearch) > 1) {
                        $sComparator = $aSpecificitySearch[0];
                        $iTargetSpecificity = $aSpecificitySearch[1];
                    }
                    $iTargetSpecificity = (int) $iTargetSpecificity;
                    $iSelectorSpecificity = $selector->getSpecificity();
                    $bMatches = false;
                    switch ($sComparator) {
                        case '<=':
                            $bMatches = $iSelectorSpecificity <= $iTargetSpecificity;
                            break;
                        case '<':
                            $bMatches = $iSelectorSpecificity < $iTargetSpecificity;
                            break;
                        case '>=':
                            $bMatches = $iSelectorSpecificity >= $iTargetSpecificity;
                            break;
                        case '>':
                            $bMatches = $iSelectorSpecificity > $iTargetSpecificity;
                            break;
                        default:
                            $bMatches = $iSelectorSpecificity === $iTargetSpecificity;
                            break;
                    }
                    if ($bMatches) {
                        $result[] = $selector;
                    }
                }
            }
        }
    }
}

1	<?php
2
3	declare(strict_types=1);
4
5	namespace Sabberworm\CSS\CSSList;
6
7	use Sabberworm\CSS\Property\Selector;
8	use Sabberworm\CSS\Rule\Rule;
9	use Sabberworm\CSS\RuleSet\DeclarationBlock;
10	use Sabberworm\CSS\RuleSet\RuleSet;
11	use Sabberworm\CSS\Value\CSSFunction;
12	use Sabberworm\CSS\Value\Value;
13	use Sabberworm\CSS\Value\ValueList;
14
15	/**
16	* A `CSSBlockList` is a `CSSList` whose `DeclarationBlock`s are guaranteed to contain valid declaration blocks or
17	* at-rules.
18	*
19	* Most `CSSList`s conform to this category but some at-rules (such as `@keyframes`) do not.
20	*/
21	abstract class CSSBlockList extends CSSList
22	{
23	/**
24	* @param int $lineNumber
25	*/
26	public function __construct($lineNumber = 0)	×
27	{
28	parent::__construct($lineNumber);	×
29	}	×
30
31	/**
32	* @param array<int, DeclarationBlock> $result
33	*/
34	protected function allDeclarationBlocks(array &$result): void	×
35	{
36	foreach ($this->aContents as $mContent) {	×
37	if ($mContent instanceof DeclarationBlock) {	×
38	$result[] = $mContent;	×
39	} elseif ($mContent instanceof CSSBlockList) {	×
40	$mContent->allDeclarationBlocks($result);	×
41	}
42	}
43	}	×
44
45	/**
46	* @param array<int, RuleSet> $result
47	*/
48	protected function allRuleSets(array &$result): void	×
49	{
50	foreach ($this->aContents as $mContent) {	×
51	if ($mContent instanceof RuleSet) {	×
52	$result[] = $mContent;	×
53	} elseif ($mContent instanceof CSSBlockList) {	×
54	$mContent->allRuleSets($result);	×
55	}
56	}
57	}	×
58
59	/**
60	* @param CSSList\|Rule\|RuleSet\|Value $element
61	* @param array<int, Value> $result
62	* @param string\|null $searchString
63	* @param bool $searchInFunctionArguments
64	*/
65	protected function allValues(	×
66	$element,
67	array &$result,
68	$searchString = null,
69	$searchInFunctionArguments = false
70	): void {
71	if ($element instanceof CSSBlockList) {	×
72	foreach ($element->getContents() as $oContent) {	×
73	$this->allValues($oContent, $result, $searchString, $searchInFunctionArguments);	×
74	}
75	} elseif ($element instanceof RuleSet) {	×
76	foreach ($element->getRules($searchString) as $rule) {	×
77	$this->allValues($rule, $result, $searchString, $searchInFunctionArguments);	×
78	}
79	} elseif ($element instanceof Rule) {	×
80	$this->allValues($element->getValue(), $result, $searchString, $searchInFunctionArguments);	×
81	} elseif ($element instanceof ValueList) {	×
82	if ($searchInFunctionArguments \|\| !($element instanceof CSSFunction)) {	×
83	foreach ($element->getListComponents() as $mComponent) {	×
84	$this->allValues($mComponent, $result, $searchString, $searchInFunctionArguments);	×
85	}
86	}
87	} else {
88	// Non-List `Value` or `CSSString` (CSS identifier)
89	$result[] = $element;	×
90	}
91	}	×
92
93	/**
94	* @param array<int, Selector> $result
95	* @param string\|null $specificitySearch
96	*/
97	protected function allSelectors(array &$result, $specificitySearch = null): void	×
98	{
99	/** @var array<int, DeclarationBlock> $aDeclarationBlocks */
100	$aDeclarationBlocks = [];	×
101	$this->allDeclarationBlocks($aDeclarationBlocks);	×
102	foreach ($aDeclarationBlocks as $oBlock) {	×
NEW 103	foreach ($oBlock->getSelectors() as $selector) {	×
104	if ($specificitySearch === null) {	×
NEW 105	$result[] = $selector;	×
106	} else {
107	$sComparator = '===';	×
108	$aSpecificitySearch = \explode(' ', $specificitySearch);	×
109	$iTargetSpecificity = $aSpecificitySearch[0];	×
110	if (\count($aSpecificitySearch) > 1) {	×
111	$sComparator = $aSpecificitySearch[0];	×
112	$iTargetSpecificity = $aSpecificitySearch[1];	×
113	}
114	$iTargetSpecificity = (int) $iTargetSpecificity;	×
NEW 115	$iSelectorSpecificity = $selector->getSpecificity();	×
116	$bMatches = false;	×
117	switch ($sComparator) {	×
118	case '<=':	×
119	$bMatches = $iSelectorSpecificity <= $iTargetSpecificity;	×
120	break;	×
121	case '<':	×
122	$bMatches = $iSelectorSpecificity < $iTargetSpecificity;	×
123	break;	×
124	case '>=':	×
125	$bMatches = $iSelectorSpecificity >= $iTargetSpecificity;	×
126	break;	×
127	case '>':	×
128	$bMatches = $iSelectorSpecificity > $iTargetSpecificity;	×
129	break;	×
130	default:
131	$bMatches = $iSelectorSpecificity === $iTargetSpecificity;	×
132	break;	×
133	}
134	if ($bMatches) {	×
NEW 135	$result[] = $selector;	×
136	}
137	}
138	}
139	}
140	}	×
141	}

MyIntervals / PHP-CSS-Parser / 13020705351

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous