On Fri, Oct 3, 2008 at 12:29 PM, Brion Vibber brion@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