Skip to content

Commit d7df4d4

Browse files
committed
AC-665: Create phpcs static check for ConfigTest
1 parent 277c79f commit d7df4d4

File tree

1 file changed

+17
-6
lines changed

1 file changed

+17
-6
lines changed

Magento2/Sniffs/Legacy/ObsoleteConfigNodesSniff.php

Lines changed: 17 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -49,12 +49,8 @@ public function process(File $phpcsFile, $stackPtr)
4949
self::ERROR_CODE_CONFIG
5050
);
5151
}
52-
$obsoleteNodes = [];
53-
$obsoleteNodesFiles = glob(__DIR__ . '/_files/obsolete_config_nodes*.php');
54-
foreach ($obsoleteNodesFiles as $obsoleteNodesFile) {
55-
$obsoleteNodes = array_merge($obsoleteNodes, include $obsoleteNodesFile);
56-
}
57-
foreach ($obsoleteNodes as $xpath => $suggestion) {
52+
53+
foreach ($this->getObsoleteNodes() as $xpath => $suggestion) {
5854
$matches = $xml->xpath($xpath);
5955
if (empty($matches)) {
6056
continue;
@@ -87,4 +83,19 @@ private function getFormattedXML(File $phpcsFile)
8783
return $doc->saveXML();
8884
}
8985

86+
/**
87+
* Get a list of obsolete nodes
88+
*
89+
* @return array
90+
*/
91+
private function getObsoleteNodes(): array
92+
{
93+
$obsoleteNodes = [];
94+
$obsoleteNodesFiles = glob(__DIR__ . '/_files/obsolete_config_nodes*.php');
95+
foreach ($obsoleteNodesFiles as $obsoleteNodesFile) {
96+
$obsoleteNodes = array_merge($obsoleteNodes, include $obsoleteNodesFile);
97+
}
98+
return $obsoleteNodes;
99+
}
100+
90101
}

0 commit comments

Comments
 (0)