Bug 223541: Make flags show up correctly in "View all attachments" mode.

attachment.cgi

@@ -768,6 +768,7 @@ sub viewall
      $a{'description'}, $a{'ispatch'}, $a{'isobsolete'}, $a{'isprivate'},
      $a{'datasize'}) = FetchSQLData();
     $a{'isviewable'} = isViewable($a{'contenttype'});
+    $a{'flags'} = Bugzilla::Flag::match({ 'attach_id' => $a{'attachid'} });
 
     # Add the hash representing the attachment to the array of attachments.
     push @attachments, \%a;

template/en/default/attachment/show-multiple.html.tmpl

@@ -61,11 +61,18 @@
       <td valign="top">[% a.datasize FILTER unitconvert %]</td>
 
       <td valign="top">
-        [% IF a.statuses.size == 0 %]
-          <i>none</i>
+        [% IF a.flags.size == 0 %]
+          <i>no flags</i>
         [% ELSE %]
-          [% FOREACH s = a.statuses %]
-            [% s FILTER html FILTER replace('\s', '&nbsp;') %]<br>
+          [% FOREACH flag = a.flags %]
+            [% IF flag.setter %]
+              [% flag.setter.nick FILTER html %]:
+            [% END %]
+            [%+ flag.type.name FILTER html %][% flag.status %]
+            [% IF flag.status == "?" && flag.requestee %]
+              ([% flag.requestee.nick FILTER html %])
+            [% END %]
+            [% ", " IF !loop.last %]
           [% END %]
         [% END %]
       </td>

template/en/default/filterexceptions.pl

@@ -450,6 +450,7 @@
 
 'attachment/show-multiple.html.tmpl' => [
   'a.attachid', 
+  'flag.status'
 ],
 
 'attachment/updated.html.tmpl' => [