Summary
Three classes contain an identical stray \" after the __getMetaString() call in their simple-layout rendering branch, producing invalid HTML like <div class="vf__multifielditem""> (note the extra ").
Affected files
classes/ValidFormBuilder/File.php:129: $strOutput = "<div{$this->__getMetaString()}\">\n";
classes/ValidFormBuilder/Group.php:153: $strOutput = "<div{$this->__getMetaString()}\">\n";
classes/ValidFormBuilder/Password.php:167: $strOutput = "<div{$this->__getMetaString()}\">\n";
All three are in the else (simple-layout) branch of __toHtml().
Expected fix
Remove the stray \" in each file:
- $strOutput = "<div{$this->__getMetaString()}\">\n";
+ $strOutput = "<div{$this->__getMetaString()}>\n";
Impact
Low. Browsers tolerate the extra quote character, but it produces invalid HTML that may confuse DOM parsers, accessibility tools, or strict HTML validators.
Discovered while writing unit tests in #154.
Summary
Three classes contain an identical stray
\"after the__getMetaString()call in their simple-layout rendering branch, producing invalid HTML like<div class="vf__multifielditem"">(note the extra").Affected files
All three are in the
else(simple-layout) branch of__toHtml().Expected fix
Remove the stray
\"in each file:Impact
Low. Browsers tolerate the extra quote character, but it produces invalid HTML that may confuse DOM parsers, accessibility tools, or strict HTML validators.
Discovered while writing unit tests in #154.