On Fri, Oct 3, 2008 at 12:29 PM, Brion Vibber <brion(a)wikimedia.org> wrote:
Perhaps it would be better to fix the setHeaders()
issue, so it's called
prior to execute()?
Would moving it up a level to executePath() and removing the child calls
fix this? If so, should be fairly trivial.
While we're at it, moving the hook calls up a level and making the default
execute() a no-op might be ideal. If subclasses are required to implement
execute, which they are, then they shouldn't have to be redoing what the
parent execute() did (by either doing it manually or calling
parent::execute()).
-Chad