Skip to content

Commit 9b370c9

Browse files
committed
MC-19921: Create static tests for object variables inside E-mail templates
1 parent 862042a commit 9b370c9

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

dev/tests/static/framework/Magento/Sniffs/Html/HtmlDirectiveSniff.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -190,6 +190,9 @@ private function validateDefinedVariables(File $phpcsFile, string $templateText)
190190

191191
$extraDefinedVariables = array_diff($definedVariables, $this->usedVariables);
192192
foreach ($extraDefinedVariables as $extraDefinedVariable) {
193+
if (substr($extraDefinedVariable, 0, 4) !== 'var ') {
194+
continue;
195+
}
193196
$phpcsFile->addError(
194197
'Template @vars comment block contains a variable not used in the template.' . PHP_EOL
195198
. 'Extra variable: ' . $extraDefinedVariable,

dev/tests/static/framework/tests/unit/testsuite/Magento/Sniffs/Html/_files/test-html-directive.html

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@
4040
"var foo.goodFor|stillfine":"Some Variable",
4141
"var foo.badFor()":"Some Variable",
4242
"var foo.badFor()|alsobad":"Some Variable",
43+
"foobar baz":"Some other directive as a variable",
4344
"var bad.multiline()":""
4445
} @-->
4546

0 commit comments

Comments
 (0)