Closed Bug 374330 Opened 17 years ago Closed 17 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: 17 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: