Closed
Bug 552480
Opened 14 years ago
Closed 13 years ago
Bug XML add ison attribute for groups in show_bug.cgi
Categories
(Bugzilla :: Creating/Changing Bugs, enhancement)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: Frank, Unassigned)
Details
Attachments
(1 file)
1.13 KB,
patch
|
mkanat
:
review-
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_6_2; de-de) AppleWebKit/531.22.7 (KHTML, like Gecko) Version/4.0.5 Safari/531.22.7 Build Identifier: 3.6rc1 Bug XML only contains group names and group IDs only for groups with ison = 1 Reproducible: Always Steps to Reproduce: 1. change a bug so that not all groups listed in the html are checked (ison=1) 2. add the &ctype=xml to the url Actual Results: the xml onliy include the groups where ison=1 Expected Results: for each group i want to have an attribute ison with the value We need this for Mylyn to support the change what users can view this bug
Comment 1•14 years ago
|
||
Hum, this is not the right place to list groups which the bug doesn't belong to. This information should be obtained from config.cgi.
Updated•14 years ago
|
Severity: normal → enhancement
Version: unspecified → 3.6
Reporter | ||
Comment 2•14 years ago
|
||
this only changes the template
Attachment #432613 -
Flags: review?(mkanat)
Reporter | ||
Comment 3•14 years ago
|
||
(In reply to comment #1) > Hum, this is not the right place to list groups which the bug doesn't belong > to. This information should be obtained from config.cgi. Why? The html has the same information. There is a checkbox for every group (checked/unchecked)
Updated•14 years ago
|
Attachment #432613 -
Flags: review?(mkanat) → review+
Comment 4•14 years ago
|
||
Comment on attachment 432613 [details] [diff] [review] patch, V1 That's fine.
Comment 5•14 years ago
|
||
Comment on attachment 432613 [details] [diff] [review] patch, V1 Oh, wait, seeing LpSolit's comment, I agree. This is not the place to list groups that the bug is not in.
Attachment #432613 -
Flags: review+ → review-
Reporter | ||
Comment 6•14 years ago
|
||
(In reply to comment #5) > (From update of attachment 432613 [details] [diff] [review]) > Oh, wait, seeing LpSolit's comment, I agree. This is not the place to list > groups that the bug is not in. Should I change the patch that I use config.cgi? This results in the following steps an client must do to get this work. 1) get the actual show_bug.cgi with format xml (only groups with ison are included) 2) read the config (http request or chached from disk) 3) add all groups from config.cgi not in show_bug.cgi with ison=0 On the other side we only have to do step 1) Thougths?
Comment 7•13 years ago
|
||
Wontfix per comment 1. Including groups in config.cgi is already requested in bug 449139.
Status: UNCONFIRMED → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•