Merge pull request #21 from stellarwp/fix/vsprint_php8_incompat

borkweb · web-flow · commit 7cb4534415a4 · 2024-01-22T10:34:35.000-05:00
Pass $data param as an array to addWarning()
diff --git a/StellarWP/Sniffs/XSS/EscapeOutputSniff.php b/StellarWP/Sniffs/XSS/EscapeOutputSniff.php
@@ -320,7 +320,7 @@ public function process( File $phpcsFile, $stackPtr )
 				"Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '%s'",
 				$i,
 				'OutputNotEscaped',
-				$this->tokens[ $i ]['content']
+				array( $this->tokens[ $i ]['content'] )
 			);
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -320,7 +320,7 @@ public function process( File $phpcsFile, $stackPtr )`
`320`	`320`	`"Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '%s'",`
`321`	`321`	`$i,`
`322`	`322`	`'OutputNotEscaped',`
`323`		`- $this->tokens[ $i ]['content']`
	`323`	`+ array( $this->tokens[ $i ]['content'] )`
`324`	`324`	`);`
`325`	`325`	`}`
`326`	`326`