Skip to content

Commit 15dd156

Browse files
committed
AC-3483:: SVC false-positive: modifying system.xml file from another module
1 parent fea14fe commit 15dd156

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

src/Analyzer/SystemXml/Analyzer.php

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ private function getBaseDir($filePath)
164164
* @param string $excludeFile The file to exclude from the search.
165165
* @return array An array of paths to system.xml files, excluding the specified file.
166166
*/
167-
private function getSystemXmlFiles($magentoBaseDir, $excludeFile = null)
167+
private function getSystemXmlFiles($magentoBaseDir, $excludeFile = null): array
168168
{
169169
$systemXmlFiles = [];
170170
$directoryToSearch = [
@@ -195,7 +195,7 @@ private function getSystemXmlFiles($magentoBaseDir, $excludeFile = null)
195195
* @param $nodePath
196196
* @return array|null
197197
*/
198-
private function extractSectionGroupField($nodePath)
198+
private function extractSectionGroupField($nodePath): ?array
199199
{
200200
$parts = explode('/', $nodePath);
201201

@@ -218,7 +218,7 @@ private function extractSectionGroupField($nodePath)
218218
* @return array
219219
* @throws \ReflectionException
220220
*/
221-
private function getNodeData($node)
221+
private function getNodeData($node): array
222222
{
223223
$data = [];
224224

@@ -406,7 +406,6 @@ private function isDuplicatedFieldInXml(
406406
[
407407
'status' => 'duplicate',
408408
'field' => $fieldId
409-
410409
]
411410
];
412411
}

0 commit comments

Comments
 (0)