Hi Dave,
I have done the checking on the clobber on the latter part of the patch. So its in there already. Since after making the patch. The default value override the local var value. Hehehe.
Hi Peter,
We actually use pretty much all except adding an if. And 2 procs to set and check the status of ad_conn was added. Also I think it will be redundant if the case is like this:
ad_page_contract {} {foo:notnull}
ad_include_contract {} {foo:notnull}
All,
I will create a new patch that has a switch. I think that would be best.
How about this:
ad_page_contract {} {} {} -some_nice_name_that_says_apply_contract_to_includes_too
Hmmm maybe
ad_page_contract {} {} {} -apply_to_page_n_includes
How about that?