Commit d57c8562 authored by mkanat%bugzilla.org's avatar mkanat%bugzilla.org

Bug 498318: Speed up field-descs.none.tmpl

Patch by Max Kanat-Alexander <mkanat@bugzilla.org> r=bbaetz, a=mkanat
parent 6a23b833
...@@ -781,6 +781,14 @@ sub create { ...@@ -781,6 +781,14 @@ sub create {
# Allow templates to generate a token themselves. # Allow templates to generate a token themselves.
'issue_hash_token' => \&Bugzilla::Token::issue_hash_token, 'issue_hash_token' => \&Bugzilla::Token::issue_hash_token,
# A way for all templates to get at Field data, cached.
'bug_fields' => sub {
my $cache = Bugzilla->request_cache;
$cache->{template_bug_fields} ||=
{ map { $_->name => $_ } Bugzilla->get_fields() };
return $cache->{template_bug_fields};
},
# These don't work as normal constants. # These don't work as normal constants.
DB_MODULE => \&Bugzilla::Constants::DB_MODULE, DB_MODULE => \&Bugzilla::Constants::DB_MODULE,
REQUIRED_MODULES => REQUIRED_MODULES =>
......
...@@ -91,14 +91,10 @@ ...@@ -91,14 +91,10 @@
Description here, by copying their Description from the Description here, by copying their Description from the
database. If you want to override this for your language database. If you want to override this for your language
or your installation, just use a hook. %] or your installation, just use a hook. %]
[%# Also create the bug_fields hash. %]
[% UNLESS Param('shutdownhtml') %] [% UNLESS Param('shutdownhtml') %]
[% USE Bugzilla %] [% FOREACH bz_field = bug_fields.values %]
[% SET bug_fields = {} %]
[% FOREACH bz_field = Bugzilla.get_fields() %]
[% SET field_descs.${bz_field.name} = bz_field.description [% SET field_descs.${bz_field.name} = bz_field.description
IF !field_descs.${bz_field.name}.defined %] IF !field_descs.${bz_field.name}.defined %]
[% SET bug_fields.${bz_field.name} = bz_field %]
[% END %] [% END %]
[% END %] [% END %]
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment