Update patch set 1
Patch Set 1:
This makes sense. But shouldn't $formHtmlHasRun be initialized to
> false somewhere?
Yeah, you are right. And you already took care of this :)
l 882 // Flag to keep track of formHTML() runs.
l 883 $formHtmlHasRun = false;
This was already in the code, albeit unused. Looks like you originally had the same idea, so I took credit for it ;)
Reviewer: Oetterer <2791@e9e9afe9-4712-486d-8885-f54b72dd1951>