Forum OpenACS Development: Re: Issuing a warning for deprecated procs

Collapse
Posted by Jeff Davis on
I think we should not warn on all deprecated procs if only because there are so many that are still used *a lot* like ad_get_user_id (deprecated in favour of [ad_conn user_id] and the logs would be huge. I do think we should warn on procs where we really intend to remove them in the next version and have made an effort to change the existing code to use the new proc.

An example of a deprecated proc where nobody's made the effort to move to the new proc is ad_permission_p (deprecated in favor of permission::get). I guess it's a seperate question whether we should deprecate procs unless we also make the effort to actually get rid of them in the code.

I would change the message to:

ns_log Debug "Deprecated proc $proc_name used"
Since it is less annoyingly long and also is Debug so isn't displayed on a production box.