Closed
Bug 242161
Opened 22 years ago
Closed 22 years ago
process_bug.cgi doesn't contain patchviewer ("diff") like show_bug.cgi
Categories
(Bugzilla :: Attachments & Requests, defect)
Bugzilla
Attachments & Requests
Tracking
()
RESOLVED
FIXED
Bugzilla 2.18
People
(Reporter: steffen.wilberg, Assigned: bugzilla)
References
()
Details
Attachments
(1 file, 1 obsolete file)
|
842 bytes,
patch
|
goobix
:
review+
|
Details | Diff | Splinter Review |
Steps to reproduce:
1. Have a buglist.
2. Modify one bug, so that process_bug.cgi is displayed with the next bug in
your list.
Notice that patches have an "edit" button, but no "diff" button.
3. Click on the bug number, so that show_bug.cgi displays the same bug.
Notice that the patch now has the "diff" button.
This is a bit confusing. I first thought something's wrong with the patch.
Comment 1•22 years ago
|
||
Probably has something to do with a template parameter not getting passed
through. I think we currently have a similar problem with the CSS file.
Target Milestone: --- → Bugzilla 2.20
Comment on attachment 149649 [details]
Add PatchViewer check into process_bug
wrong file - ignore!
Attachment #149649 -
Attachment is obsolete: true
Attachment #149649 -
Attachment is patch: false
Attachment #149650 -
Flags: review?
Updated•22 years ago
|
Attachment #149650 -
Flags: review? → review+
Updated•22 years ago
|
Flags: approval?
Updated•22 years ago
|
Flags: approval? → approval+
Target Milestone: Bugzilla 2.20 → Bugzilla 2.18
Comment 5•22 years ago
|
||
Checking in process_bug.cgi;
/cvsroot/mozilla/webtools/bugzilla/process_bug.cgi,v <-- process_bug.cgi
new revision: 1.205; previous revision: 1.204
done
Changing resolution to FIXED.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
(In reply to comment #1)
> Probably has something to do with a template parameter not getting passed
> through. I think we currently have a similar problem with the CSS file.
Dave: Do you have more info on the similar problem with css?
Comment 7•22 years ago
|
||
no, but that's worthy of another bug, which might actually already be filed.
Something to do with obsolete attachments and resolved dependencies not showing
with strike-through.
(In reply to comment #7)
> no, but that's worthy of another bug, which might actually already be filed.
I couldn't find any existing bugs related to what you mention here...
> Something to do with obsolete attachments and resolved dependencies not showing
> with strike-through.
Obsolete attachments and resolved dependencies seemed to be stroked-through(!) for me, on
show_bug and process_bug, so unless anyone can point me at a bug, I'm going to let this one go..
Updated•13 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
•