From 49293a63bad7c62eacd8c3465dd16fdd5069338a Mon Sep 17 00:00:00 2001 From: Dylan William Hardison Date: Wed, 13 Apr 2016 15:55:50 -0400 Subject: Revert "Bug 1195736 - intermittent internal error: "file error - nav_link: not found" (also manifests as fields_lhs: not found)" Test failures result from this, we will need a different approach. This reverts commit 33f61556746e1729746342d802ca7ea9cea18caf. --- Bugzilla/Error.pm | 9 ++------- Bugzilla/Template.pm | 3 --- 2 files changed, 2 insertions(+), 10 deletions(-) (limited to 'Bugzilla') diff --git a/Bugzilla/Error.pm b/Bugzilla/Error.pm index 927745201..fa898fba4 100644 --- a/Bugzilla/Error.pm +++ b/Bugzilla/Error.pm @@ -99,13 +99,8 @@ sub _throw_error { # and calling $template->process over and over for those errors # is too slow. So instead, we just "die" with a dump of the arguments. if (Bugzilla->error_mode != ERROR_MODE_TEST) { - if (Bugzilla->template->in_process) { - $message = "error inside Bugzilla->template->process: $error"; - } - else { - $template->process($name, $vars, \$message) - || ThrowTemplateError($template->error()); - } + $template->process($name, $vars, \$message) + || ThrowTemplateError($template->error()); } # Let's call the hook first, so that extensions can override diff --git a/Bugzilla/Template.pm b/Bugzilla/Template.pm index 5b3b045ab..56ebd9c21 100644 --- a/Bugzilla/Template.pm +++ b/Bugzilla/Template.pm @@ -665,15 +665,12 @@ $Template::Stash::SCALAR_OPS->{ truncate } = ############################################################################### -our $_in_process = 0; -sub in_process { return $_in_process } sub process { my $self = shift; # All of this current_langs stuff allows template_inner to correctly # determine what-language Template object it should instantiate. my $current_langs = Bugzilla->request_cache->{template_current_lang} ||= []; unshift(@$current_langs, $self->context->{bz_language}); - local $_in_process = 1; my $retval = $self->SUPER::process(@_); shift @$current_langs; return $retval; -- cgit v1.2.3-24-g4f1b