Skip to content

Commit 3cd934a

Browse files
authored
[CLEANUP] Avoid Hungarian notation for iKey (#855)
Part of #756
1 parent 24e69d4 commit 3cd934a

File tree

3 files changed

+17
-17
lines changed

3 files changed

+17
-17
lines changed

src/CSSList/CSSList.php

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -315,9 +315,9 @@ public function insertBefore($item, $sibling): void
315315
*/
316316
public function remove($itemToRemove)
317317
{
318-
$iKey = \array_search($itemToRemove, $this->contents, true);
319-
if ($iKey !== false) {
320-
unset($this->contents[$iKey]);
318+
$key = \array_search($itemToRemove, $this->contents, true);
319+
if ($key !== false) {
320+
unset($this->contents[$key]);
321321
return true;
322322
}
323323
return false;
@@ -334,12 +334,12 @@ public function remove($itemToRemove)
334334
*/
335335
public function replace($oldItem, $newItem)
336336
{
337-
$iKey = \array_search($oldItem, $this->contents, true);
338-
if ($iKey !== false) {
337+
$key = \array_search($oldItem, $this->contents, true);
338+
if ($key !== false) {
339339
if (\is_array($newItem)) {
340-
\array_splice($this->contents, $iKey, 1, $newItem);
340+
\array_splice($this->contents, $key, 1, $newItem);
341341
} else {
342-
\array_splice($this->contents, $iKey, 1, [$newItem]);
342+
\array_splice($this->contents, $key, 1, [$newItem]);
343343
}
344344
return true;
345345
}
@@ -371,7 +371,7 @@ public function removeDeclarationBlockBySelector($mSelector, $bRemoveAll = false
371371
if (!\is_array($mSelector)) {
372372
$mSelector = \explode(',', $mSelector);
373373
}
374-
foreach ($mSelector as $iKey => &$mSel) {
374+
foreach ($mSelector as $key => &$mSel) {
375375
if (!($mSel instanceof Selector)) {
376376
if (!Selector::isValid($mSel)) {
377377
throw new UnexpectedTokenException(
@@ -383,12 +383,12 @@ public function removeDeclarationBlockBySelector($mSelector, $bRemoveAll = false
383383
$mSel = new Selector($mSel);
384384
}
385385
}
386-
foreach ($this->contents as $iKey => $item) {
386+
foreach ($this->contents as $key => $item) {
387387
if (!($item instanceof DeclarationBlock)) {
388388
continue;
389389
}
390390
if ($item->getSelectors() == $mSelector) {
391-
unset($this->contents[$iKey]);
391+
unset($this->contents[$key]);
392392
if (!$bRemoveAll) {
393393
return;
394394
}

src/RuleSet/DeclarationBlock.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ public function setSelectors($mSelector, $oList = null): void
9696
} else {
9797
$this->aSelectors = \explode(',', $mSelector);
9898
}
99-
foreach ($this->aSelectors as $iKey => $mSelector) {
99+
foreach ($this->aSelectors as $key => $mSelector) {
100100
if (!($mSelector instanceof Selector)) {
101101
if ($oList === null || !($oList instanceof KeyFrame)) {
102102
if (!Selector::isValid($mSelector)) {
@@ -106,7 +106,7 @@ public function setSelectors($mSelector, $oList = null): void
106106
'custom'
107107
);
108108
}
109-
$this->aSelectors[$iKey] = new Selector($mSelector);
109+
$this->aSelectors[$key] = new Selector($mSelector);
110110
} else {
111111
if (!KeyframeSelector::isValid($mSelector)) {
112112
throw new UnexpectedTokenException(
@@ -115,7 +115,7 @@ public function setSelectors($mSelector, $oList = null): void
115115
'custom'
116116
);
117117
}
118-
$this->aSelectors[$iKey] = new KeyframeSelector($mSelector);
118+
$this->aSelectors[$key] = new KeyframeSelector($mSelector);
119119
}
120120
}
121121
}
@@ -131,9 +131,9 @@ public function removeSelector($mSelector): bool
131131
if ($mSelector instanceof Selector) {
132132
$mSelector = $mSelector->getSelector();
133133
}
134-
foreach ($this->aSelectors as $iKey => $selector) {
134+
foreach ($this->aSelectors as $key => $selector) {
135135
if ($selector->getSelector() === $mSelector) {
136-
unset($this->aSelectors[$iKey]);
136+
unset($this->aSelectors[$key]);
137137
return true;
138138
}
139139
}

src/RuleSet/RuleSet.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -230,9 +230,9 @@ public function removeRule($mRule): void
230230
if (!isset($this->aRules[$sRule])) {
231231
return;
232232
}
233-
foreach ($this->aRules[$sRule] as $iKey => $rule) {
233+
foreach ($this->aRules[$sRule] as $key => $rule) {
234234
if ($rule === $mRule) {
235-
unset($this->aRules[$sRule][$iKey]);
235+
unset($this->aRules[$sRule][$key]);
236236
}
237237
}
238238
} else {

0 commit comments

Comments
 (0)