Closed
Bug 236926
Opened 21 years ago
Closed 21 years ago
buglist.cgi isn't using $cgi->header
Categories
(Bugzilla :: Query/Bug List, defect)
Bugzilla
Query/Bug List
Tracking
()
RESOLVED
FIXED
Bugzilla 2.18
People
(Reporter: Maniac, Assigned: justdave)
References
Details
Attachments
(1 file)
(deleted),
patch
|
kiko
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7b) Gecko/20040302 Firefox/0.8.0+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7b) Gecko/20040302 Firefox/0.8.0+
I use bugzilla-2.17.7. Every script except buglist.cgi somehow automagically
knows that I configured my Apache to send charset=utf-8. Buglist.cgi sends
ISO-8859-5 instead.
I've found and fixed the problem on my installation:
showbug.cgi, line 363
---------------------
was:
print "Content-Type: text/html\n\n";
replaced with:
print Bugzilla->cgi->header();
showbug.cgi, line 860
---------------------
was:
$contenttype = "text/html";
replaced with:
$contenttype = "text/html; charset=utf-8";
Sorry, I'm not so familiar with cvs to make a proper patch. And also I'm at all
not good at Perl and do not know where to get just a server's charset (not
entire header) to avoid hard-coding 'utf-8'. But I hope just finding places will
help :-).
Reproducible: Always
Steps to Reproduce:
Assignee | ||
Comment 1•21 years ago
|
||
Doh, he's right. The file in question is not showbug.cgi but buglist.cgi (he
probably typoed in the original description. His line numbers are pretty close
though, and we are indeed still printing manual headers there instead of using
$cgi->header :(
Assignee: myk → justdave
Blocks: mod_perl
Status: UNCONFIRMED → NEW
Component: User Interface → Query/Bug List
Ever confirmed: true
Flags: blocking2.18+
OS: Windows 2000 → All
Hardware: PC → All
Summary: buglist.cgi doesn't honor HTTP charset → buglist.cgi isn't using $cgi->header
Target Milestone: --- → Bugzilla 2.18
Assignee | ||
Comment 2•21 years ago
|
||
Actually, the second one (line 860 in the above description) is correct as-is.
We're only overriding the content type there, not the charset. Perl's CGI
module handles charset separately and builds the header when sending it.
For your problem above, the correct place to fix that is the line where we do
$cgi->charset(''); in Bugzilla/CGI.pm (put the charset you want inside those
quotes).
Assignee | ||
Updated•21 years ago
|
Attachment #144717 -
Flags: review?(myk)
Comment 3•21 years ago
|
||
Comment on attachment 144717 [details] [diff] [review]
Patch v1
Note other files that still want ->header():
chart.cgi
sanitycheck.cgi
Attachment #144717 -
Flags: review?(myk) → review+
Updated•21 years ago
|
Status: NEW → ASSIGNED
Flags: approval?
Assignee | ||
Updated•21 years ago
|
Flags: approval? → approval+
Assignee | ||
Comment 4•21 years ago
|
||
Checking in buglist.cgi;
/cvsroot/mozilla/webtools/bugzilla/buglist.cgi,v <-- buglist.cgi
new revision: 1.249; previous revision: 1.248
done
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•