Ok, I apologize for not soliciting comments for this. I did not make the right judgement in committing this.
Can I ask an opinion which one to do:
- reject the patch and revert back to old. Wherein the tcl var gets overridden by the default value of the ad_page_contract.
- check if the value exists and throw an error.
- check if the value exists and throw an error. And add a switch not to clobber, so no error is thrown.
Thanks. I apologize of the lack of judgement on this one. Also I don't think I have shown any habits of changing this toolkit for my own good.
