Closed
Bug 341689
Opened 19 years ago
Closed 19 years ago
'Edit Attachment As Comment' converts plain text to HTML entities
Categories
(Bugzilla :: Attachments & Requests, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.22
People
(Reporter: mkanat, Assigned: LpSolit)
Details
(Keywords: regression)
Attachments
(1 file)
|
914 bytes,
patch
|
myk
:
review+
|
Details | Diff | Splinter Review |
On the current CVS Bugzilla, if I do "edit attachment as comment" on attachment.cgi, it will take a line line this:
<input>
And convert it to:
<input>
This is particularly irritating when reviewing patches, as you can imagine. For example:
Bugzilla->dbh
becomes:
Bugzilla->dbh
It's not happening to me on bmo, which is currently running 2.20+. I think it's some change to the attachment/edit.html.tmpl template at some point. I'm looking into it.
| Reporter | ||
Comment 1•19 years ago
|
||
Okay. I looked into it, and I have no idea why it's happening. If anybody has any idea, please submit a patch and I'll review it. :-)
| Assignee | ||
Comment 2•19 years ago
|
||
This is a regression due to bug 301764. 2.22 is affected too. My patch applies and has been tested on both 2.22 and tip.
Assignee: attach-and-request → LpSolit
Status: NEW → ASSIGNED
Attachment #225857 -
Flags: review?(mkanat)
| Assignee | ||
Updated•19 years ago
|
Flags: blocking2.22.1?
Target Milestone: Bugzilla 2.24 → Bugzilla 2.22
Version: 2.23 → 2.22
| Assignee | ||
Updated•19 years ago
|
Keywords: regression
Updated•19 years ago
|
Attachment #225857 -
Flags: review?(mkanat) → review+
| Assignee | ||
Updated•19 years ago
|
Flags: approval?
Flags: approval2.22?
Updated•19 years ago
|
Flags: approval?
Flags: approval2.22?
Flags: approval2.22+
Flags: approval+
| Assignee | ||
Comment 3•19 years ago
|
||
tip:
Checking in template/en/default/attachment/edit.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/attachment/edit.html.tmpl,v <-- edit.html.tmpl
new revision: 1.35; previous revision: 1.34
done
2.22:
Checking in template/en/default/attachment/edit.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/attachment/edit.html.tmpl,v <-- edit.html.tmpl
new revision: 1.30.2.2; previous revision: 1.30.2.1
done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
| Assignee | ||
Updated•19 years ago
|
Flags: blocking2.22.1?
You need to log in
before you can comment on or make changes to this bug.
Description
•