Forum OpenACS Development: Re: Re: RFC: Callbacks

Collapse
15: Re: Re: RFC: Callbacks (response to 6)
Posted by Andrew Piskorski on
Please do not define Tcl procs via the "namespace eval foo { pages_of_procs_here }" mechanism. That makes finding the procs in the files via Emacs or ctags unreasonably difficult. (And it also adds an unnecessary and annoying additional level of indentation.) Instead, simply define each proc using its fully-qualified name, e.g.:
ad_proc -callback dotlrn::acs_user::foo::bar { ... }

As far as the actual topic of this thread - yes, good OpenACS callback support would be excellent, I applaud this effort. I have no particular thoughts or feedback on it myself, but between Jeff and Andrew working on it I'm pretty sure you'll end up with something good. :)