173622 - Move template handling code into a module

Status: RESOLVED FIXED

(Bugzilla :: Bugzilla-General, defect)

Description

[Bradley Baetz (:bbaetz)]

We should move the template/vars stuff into a module. This is because:

- we shouldn't be running the code from a require'd file
- global variables suck
- its cleaner/nicer/more efficient

I have bits of this done, but its not completed.

Comment 1

[Bradley Baetz (:bbaetz)]

Attachment: v1

I've been running with this for about a month now.

The only substantial change  in the patch is to use cgi.user_agent instead of
pushing a user_agent var all the time. This is a CGI instance, not a
Bugzilla::CGI instance. Not that it matters in this case; if anyone ever does
care we'd have to write a wrpaper plugin. Its possible I may do this as part of
the relogin + confirm stuff, whic currently uses FORM/MFORM, but thats not this
bug.

The rest is just code movement. Note that the 'default' vars stuff is now a
VARAIBLES option to the template, rather than in the vars hash, which makes it
easier to remove the global-ness of that later.

Comment 2

[Bradley Baetz (:bbaetz)]

Comment on attachment 106788 [details] [diff] [review]
v1

myk - you originally did the template stuff.

There will be a minor change or two to the actual template defn to deal with
other patches, but thats jsut going to be applying to Template.pm what is
applied to globals.pl

The top hunk was needed, but please don't ask me to remember why. Its 'correct'
in any event, though

Comment 3

[Bradley Baetz (:bbaetz)]

Attachment: take 2

OK, this uses a different scheme, as I discussed on developers@b.o

Theres now a semi-persistable Bugzilla object. Bugzilla.pm has more POD than
code, so it should be easy to comment on the design for that.

Additional changes from the previous patch are:

- to use this new object in the templates via a plugin (which will mean
creating a new directory - any objections?).
- collectstats is changed to call duplicates.cgi with GATEWAY_INTERFACE set, so
that it has a CGI object
- Bugzilla::Template no longer caches the template, since Bugzilla.pm is doing
that for us. That makes the code a bit nicer too, btw

This patch is larger than it looks - a lot of stuff is just movement of code

Comment 4

[Bradley Baetz (:bbaetz)]

Attachment: v3

Fixes bitrot from both the help checkin, and the time filter changes.

Comment 5

[Bradley Baetz (:bbaetz)]

Attachment: v4

Weekly update to fix more cvs conflicts.

Comment 6

[Bradley Baetz (:bbaetz)]

Attachment: v5

Fix cvs conflicts

Comment 7

[Dave Miller [:justdave]]

Comment on attachment 109337 [details] [diff] [review]
v5

my head hurts trying to look at the code in this patch, but bbaetz tells me
it's mostly existing code just moving around.  I don't see anything in it
that's jumping out and screaming at me.  I installed this patch on Syndicomm 2
days ago, and it's actually been getting heavy use because we're in the middle
of a cross-continent server move and we're using it for a task list.  Haven't
noticed any problems.

Comment 8

[Dave Miller [:justdave]]

Oh, and I forgot to mention, we had a checkin on the 16th that broke
show_bug.cgi on Perl 5.6.0 because of weird taint problems...  this patch fixes
that, so it would be really really nice to have this in before we roll 2.17.2
tomorrow morning :-)

Comment 9

[Dave Miller [:justdave]]

Joel: see comment 8

Comment 10

[Joel Peshkin]

Comment on attachment 109337 [details] [diff] [review]
v5

r=joel

I've diverged enough with my production site that I can only try this on my
little one, but it seems pretty innocuous.

Comment 11

[Bradley Baetz (:bbaetz)]

Checked in!

Comment 12

[Bradley Baetz (:bbaetz)]

Somehow the checkin message ended up as '/tmp/cvshseAPa', so I cvs admin -m'd
the files to fix that. Bonsai won't pick that up, though.