[
https://issues.apache.org/jira/browse/OODT-478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13425065#comment-13425065
]
Andrew Hart commented on OODT-478:
----------------------------------
Hey Nesbert, first off: thanks very much for this issue & patch!
I looked at the patch, and I like what you're proposing. The only implication,
if I understand the patch correctly, is that some hooks (e.g.: hook_before_all)
will run for both scripts and views, while others (hook_before_header,
hook_before_view, hook_before_footer) will run only for views.
Interestingly, this *should* already have been the case. Concretely, there's
nothing in the existing 'hook_before_send' and 'hook_after_send' hooks (invoked
in the ApplicationResponse's "send" function) that is specific to view
requests, and yet these hooks will never get run for script requests as the
current initialization code only includes the hooks file for view requests.
I've never had to implement either of these hooks so it has personally not come
up, but I can imagine it would certainly cause some confusion.
So, +1(!) for this change, both because I like the idea of a pre-processor
hook, and because it incidentally resolves a latent issue with hook processing
for script requests.
One minor request, though, to make this complete: check out the sample
hooks.php file in trunk/balance/etc/skel/hooks.php. It lists the currently
defined hooks along with a little context about what they each do. Would you
consider adding a similar doc block for your hook to this file? That way people
will better know it exists and how to use it. Feel free to attach an updated
patch to this issue with the change + documentation.
Thanks a lot!
> Balance: New hook for hooks.php
> -------------------------------
>
> Key: OODT-478
> URL: https://issues.apache.org/jira/browse/OODT-478
> Project: OODT
> Issue Type: New Feature
> Components: balance
> Reporter: Nesbert Hidalgo
> Priority: Minor
> Attachments: BalanceHookBeforeAll.patch.diff
>
>
> I would like to recommend adding an additional hook "hook_before_all" to
> Balance. Currently, the hooks (hook_before_header, hook_before_view,
> hook_before_footer, hook_before_send) if defined are called for views only.
> In my case I would like a callback to be executed for both views and scripts.
> This will allow us to execute the same logic for both views and scripts.
> <?php
> // Additional callback for hooks.php
> /**
> * hook_before_all
> *
> * This is the first hook called for both views and scripts (*.do).
> */
> function hook_before_all() {}
> ?>
> Please see attached patch...
> Cheers & Thanks!
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
