Bug XML add ison attribute for groups in show_bug.cgi

RESOLVED WONTFIX

Status

()

--
enhancement
RESOLVED WONTFIX
9 years ago
8 years ago

People

(Reporter: Frank, Unassigned)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

9 years ago
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

9 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

9 years ago
Severity: normal → enhancement
Version: unspecified → 3.6
(Reporter)

Comment 2

9 years ago
Created attachment 432613 [details] [diff] [review]
patch, V1

this only changes the template
Attachment #432613 - Flags: review?(mkanat)
(Reporter)

Comment 3

9 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

9 years ago
Attachment #432613 - Flags: review?(mkanat) → review+
Comment on attachment 432613 [details] [diff] [review]
patch, V1

That's fine.
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

9 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

8 years ago
Wontfix per comment 1. Including groups in config.cgi is already requested in bug 449139.
Status: UNCONFIRMED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.