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)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.2
People
(Reporter: mkanat, Assigned: mkanat)
References
Details
Attachments
(1 file, 1 obsolete file)
10.56 KB,
patch
|
mkanat
:
review+
|
Details | Diff | Splinter Review |
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
Assignee | ||
Comment 1•18 years ago
|
||
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
Assignee | ||
Comment 2•18 years ago
|
||
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+
Assignee | ||
Comment 3•18 years ago
|
||
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.
Assignee | ||
Comment 4•18 years ago
|
||
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.
Description
•