Closed Bug 374330 Opened 18 years ago Closed 18 years ago

Make it possible for installation templates to be text or HTML

Categories

(Bugzilla :: Installation & Upgrading, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 3.2

People

(Reporter: mkanat, Assigned: mkanat)

References

Details

Attachments

(1 file, 1 obsolete file)

The next step for setup.cgi is to make it so that we can display HTML there instead of just plain text. This means: * setup.cgi will have to have HTML markup * We'll have to html-escape anything we throw at the user. * We have to be able to pick an HTML string instead of a text string for setup.cgi vs. checksetup.pl
Attached patch v1 (obsolete) — Splinter Review
Okay, here's the first version. I'm going to do another version, though, and replace is_web with a new installation_mode called INSTALLATION_MODE_WEB. I'll also move installation_mode into Bugzilla::Install::Util.
Assignee: installation → mkanat
Status: NEW → ASSIGNED
Attached patch v2Splinter Review
Okay. I didn't move installation_mode into Bugzilla::Util, because I want installation_mode to continue to mean what it means. Instead, I made is_web understand Bugzilla->usage_mode for when you're running at the command line. It also works before you have all the requirements loaded, so that's nice.
Attachment #258915 - Attachment is obsolete: true
Attachment #259033 - Flags: review+
Note that if we discover a performance problem, it's possible to speed up install_string by loading every hash into an arrayref, caching that arrayref, and just going though it to look for the strings.
Checking in checksetup.pl; /cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v <-- checksetup.pl new revision: 1.550; previous revision: 1.549 done Checking in setup.cgi; /cvsroot/mozilla/webtools/bugzilla/setup.cgi,v <-- setup.cgi new revision: 1.2; previous revision: 1.1 done Checking in Bugzilla/Install/Util.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Install/Util.pm,v <-- Util.pm new revision: 1.3; previous revision: 1.2 done RCS file: /cvsroot/mozilla/webtools/bugzilla/template/en/default/setup/strings.html.pl,v done Checking in template/en/default/setup/strings.html.pl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/setup/strings.html.pl,v <-- strings.html.pl initial revision: 1.1 done Checking in template/en/default/setup/strings.txt.pl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/setup/strings.txt.pl,v <-- strings.txt.pl new revision: 1.2; previous revision: 1.1 done
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Flags: approval+
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: