Merge pull request #68 from magento-ecg/MAGINT-40-Error

alexkaranda · web-flow · commit ab09fd7f66ac · 2022-09-02T11:03:44.000+03:00
Uncaught TypeError: vsprintf(): Argument #2 () must be of type array,…
diff --git a/EcgM2/Sniffs/Templates/EscapedOutputSniff.php b/EcgM2/Sniffs/Templates/EscapedOutputSniff.php
@@ -81,7 +81,12 @@ public function process(File $phpcsFile, $stackPtr)
         if (!$this->isExpectedTokenSequence($stackPtr, $stackClosingPtr)) {
             $error = 'Unescaped output is forbidden! %s';
             $invalidCodeSequence = $this->prepareInvalidCodeSequence($stackPtr, $stackClosingPtr);
-            $phpcsFile->addError($error, $stackPtr, 'Unescaped output', $invalidCodeSequence);
+            $phpcsFile->addError(
+                $error,
+                $stackPtr,
+                'Unescaped output',
+                [$invalidCodeSequence]
+            );
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,12 @@ public function process(File $phpcsFile, $stackPtr)`
`81`	`81`	`if (!$this->isExpectedTokenSequence($stackPtr, $stackClosingPtr)) {`
`82`	`82`	`$error = 'Unescaped output is forbidden! %s';`
`83`	`83`	`$invalidCodeSequence = $this->prepareInvalidCodeSequence($stackPtr, $stackClosingPtr);`
`84`		`- $phpcsFile->addError($error, $stackPtr, 'Unescaped output', $invalidCodeSequence);`
	`84`	`+ $phpcsFile->addError(`
	`85`	`+ $error,`
	`86`	`+ $stackPtr,`
	`87`	`+ 'Unescaped output',`
	`88`	`+ [$invalidCodeSequence]`
	`89`	`+ );`
`85`	`90`	`}`
`86`	`91`	`}`
`87`	`92`