Whenever I use $nid variable in addition processing php code, the form gone nut. I have to rename the variable to $__nid to make it work.

I suggest we separate the scope of evaluate function to protect variable

Comments

quicksketch’s picture

quicksketch
he/him
commented
Status: Active » Closed (fixed)

The scope is actually intentionally open so that $form_state and $node variables may be modified. However we're not using $nid any more in the processing so hopefully this will prevent this particular conflict. In the 3.x branch I'm working on moving the additional submit/validate code out of Webform entirely, making this a non-issue.