Absolutely, Jun, this is nothing personal, at all. I know I am less than diplomatic at times (like most of the time), so take that into account as well.
I did a little more digging on the effect of the patch. It actually turns out that the effect would be that any existing variable would skip validation/filtering completely, although post filters will still run. But the main filters are skipped because the variable isn't in the form.
I think Jun can get the effect he wants by doing the following:
- Remove the variable from the environment that calls ad_page_contract.
- Put the variable into the request form as a query var.
- Call ad_page_contract
