364895 - atom feed for secure bugs needs to list groups

Status: NEW

(Bugzilla :: Creating/Changing Bugs, enhancement)

Description

[timeless]

"@" is *not* discoverable! i had to read the source to figure out what it meant.

The atom feed needs to list the groups the bug is in.

Comment 1

[timeless]

Attachment: is this ok

Comment 2

[Frédéric Buclin]

Comment on attachment 249572 [details] [diff] [review]
is this ok

>         <th>Field</th><th>Value</th>
>+        [% IF bug.secure_mode %]
>       </tr><tr>
>+        <th>Groups</th>

The indentation is incorrect. Everything inside the IF block must be indented accordingly.

Comment 3

[Frédéric Buclin]

Comment on attachment 249572 [details] [diff] [review]
is this ok

>+          [% FOREACH group = bug.groups %]

bug.groups lists all groups the bug *could* belong to, or those which are used for local privs. That's not what you want.


>+            [% IF list_started %]
>+              [% ", " %]
>+            [% ELSE %]
>+              [% list_started = 1 %]
>+            [% END %]

A join() would be much cleaner.

Also, my previous comment still applies.