From 147f2fc9d9341e99dc17c3a267b80b7fc22a9699 Mon Sep 17 00:00:00 2001 From: Dylan William Hardison Date: Mon, 10 Apr 2017 21:53:51 -0400 Subject: Revert "Bug 1352913 - Extensions must register for template_before_process()" This reverts commit ecbdca8c3f06dd420db6a960c8808615dae6848a. --- Bugzilla/Hook.pm | 44 +++++++++++++++----------------------------- 1 file changed, 15 insertions(+), 29 deletions(-) (limited to 'Bugzilla/Hook.pm') diff --git a/Bugzilla/Hook.pm b/Bugzilla/Hook.pm index e28c01a40..00bd316ab 100644 --- a/Bugzilla/Hook.pm +++ b/Bugzilla/Hook.pm @@ -11,42 +11,18 @@ use 5.10.1; use strict; use warnings; -use Scalar::Util qw(blessed); - sub process { - my ($name, $args, $extensions) = @_; - - $extensions //= Bugzilla->extensions; - - my $hook_stack = Bugzilla->request_cache->{hook_stack} ||= []; - push @$hook_stack, $name; - - foreach my $extension (@$extensions) { - if (my $hook = $extension->can($name)) { - $hook->($extension, $args); - } - } + my ($name, $args) = @_; - pop @$hook_stack; -} - -sub collect_wants { - my ($name) = @_; - my %result; + _entering($name); foreach my $extension (@{ Bugzilla->extensions }) { - my $hook = $extension->can($name); - if ($hook) { - my $wants = $hook->($extension); - foreach my $want (keys %$wants) { - if ($wants->{$want}) { - $result{ $want }{ blessed $extension } = 1; - } - } + if ($extension->can($name)) { + $extension->$name($args); } } - return \%result; + _leaving($name); } sub in { @@ -55,6 +31,16 @@ sub in { return $hook_name eq $currently_in ? 1 : 0; } +sub _entering { + my ($hook_name) = @_; + my $hook_stack = Bugzilla->request_cache->{hook_stack} ||= []; + push(@$hook_stack, $hook_name); +} + +sub _leaving { + pop @{ Bugzilla->request_cache->{hook_stack} }; +} + 1; __END__ -- cgit v1.2.3-24-g4f1b