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.
