552480 - Bug XML add ison attribute for groups in show_bug.cgi

Status: RESOLVED WONTFIX

(Bugzilla :: Creating/Changing Bugs, enhancement)

Description

[Frank Becker]

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

[Frédéric Buclin]

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.

Comment 2

[Frank Becker]

Attachment: patch, V1

this only changes the template

Comment 3

[Frank Becker]

(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)

Comment 4

[Max Kanat-Alexander]

Comment on attachment 432613 [details] [diff] [review]
patch, V1

That's fine.

Comment 5

[Max Kanat-Alexander]

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.

Comment 6

[Frank Becker]

(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

[Frédéric Buclin]

Wontfix per comment 1. Including groups in config.cgi is already requested in bug 449139.