If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Attached patches should appear as (text/plain; patch)

RESOLVED FIXED in Bugzilla 2.16

Status

()

Bugzilla
Bugzilla-General
P2
enhancement
RESOLVED FIXED
17 years ago
5 years ago

People

(Reporter: timeless, Assigned: justdave)

Tracking

unspecified
Bugzilla 2.16

Details

Comment hidden (empty)

Comment 1

17 years ago
Why?
Because right now they appear as (text/plain) which makes them indistiguishable 
from other text files.  Bugzilla does keep track of whether it's a patch or not 
(it's a separate radio button when you're uploading the attachment), so why not 
display it?

Comment 3

17 years ago
True, I didn't notice that. But in createattachment.cgi it says "text/plain,
diffs", we should use the same wording in each place (but I'm not sure which
one is better).

The initial comment is a bit terse :) - that's why I was asking.
Target Milestone: --- → Bugzilla 2.16

Comment 4

17 years ago
How about just showing (patch)?
Depends on: 84338
I agree they should be just patch - a patch should never have another MIME type
to my knowledge.
Priority: -- → P2
Patches are now distinctly identified on the patch list.  Timeless, if you
concurr, please mark this fixed.
->New (old by now?) product Bugzilla 
Component: Bugzilla → Bugzilla-General
Product: Webtools → Bugzilla
Version: other → unspecified
Timeless has nothing to say, and I agree with Dave, so marking fixed.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.