Merge pull request #78 from magento-commerce/imported-svera-magento-coding-standard-290

svera · web-flow · commit 1bb41745d9f2 · 2021-09-28T14:26:49.000+02:00
[Imported] AC-669: Fixed directory name checking too lax
diff --git a/Magento2/Sniffs/Legacy/InstallUpgradeSniff.php b/Magento2/Sniffs/Legacy/InstallUpgradeSniff.php
@@ -64,7 +64,10 @@ public function process(File $phpcsFile, $stackPtr)
             }
         }
 
-        if (preg_match('/(sql|data)/', $fileInfo->getPath()) === 1) {
+        $folders = array_filter(explode('/', $fileInfo->getPath()));
+        $folderName = array_pop($folders);
+
+        if ($folderName === 'data' || $folderName === 'sql') {
             $phpcsFile->addError(
                 $fileInfo->getFilename()." is in an invalid directory ".$fileInfo->getPath().":\n"
                 . "- Create a data patch within module's Setup/Patch/Data folder for data upgrades.\n"
diff --git a/Magento2/Tests/Legacy/_files/InstallUpgradeUnitTest/another-data-folder/file3.inc b/Magento2/Tests/Legacy/_files/InstallUpgradeUnitTest/another-data-folder/file3.inc
@@ -0,0 +1 @@
+<?php
diff --git a/Magento2/Tests/Legacy/_files/InstallUpgradeUnitTest/another-sql-folder/file4.inc b/Magento2/Tests/Legacy/_files/InstallUpgradeUnitTest/another-sql-folder/file4.inc
@@ -0,0 +1 @@
+<?php

Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,10 @@ public function process(File $phpcsFile, $stackPtr)`
`64`	`64`	`}`
`65`	`65`	`}`
`66`	`66`
`67`		`- if (preg_match('/(sql\|data)/', $fileInfo->getPath()) === 1) {`
	`67`	`+ $folders = array_filter(explode('/', $fileInfo->getPath()));`
	`68`	`+ $folderName = array_pop($folders);`
	`69`	`+`
	`70`	`+ if ($folderName === 'data' \|\| $folderName === 'sql') {`
`68`	`71`	`$phpcsFile->addError(`
`69`	`72`	`$fileInfo->getFilename()." is in an invalid directory ".$fileInfo->getPath().":\n"`
`70`	`73`	`. "- Create a data patch within module's Setup/Patch/Data folder for data upgrades.\n"`