984505 - Link component and product to browse for other bugs in this category

Status: VERIFIED FIXED

(Bugzilla :: User Interface, enhancement)

Description

[sjw]

If you click on Component or Product now, you got just an overview of all components / products. There should be a way to browse for other bugs in this category / other categories in this product. That's what I expected.

If you wishes an overview of all Products and Component, you have still the 'Browse' link on the top.

Comment 1

[Andre Klapper]

Please provide links to what you refer to so we talk about the same. Also, what are "categories"?

Comment 2

[Frédéric Buclin]

(In reply to sjw from comment #0)
> If you click on Component or Product now, you got just an overview of all
> components / products. There should be a way to browse for other bugs in
> this category / other categories in this product. That's what I expected.

I don't see why clicking the Product or Component labels should display bugs in this product or component, while clicking any other label doesn't. This would be a weird behavior.


> If you wishes an overview of all Products and Component, you have still the
> 'Browse' link on the top.

Or said differently, if you wish to see bugs in the same product or component, you have still the "Search" link on the top.

Comment 3

[sjw]

If I reported / viewed a bug in a specific component, I want a way to go back to the bug list of this component. It's not very comfortable to search for it every time.

My idea was to change the existing link on 'Component', because there is no other existing place for it. The current location of this link would be accessible on 'Product' and the current link behind 'Product' is already accessible on 'Browse'

Any better places for these links?


(In reply to Andre Klapper from comment #1)
> Please provide links to what you refer to so we talk about the same. Also,
> what are "categories"?
categories means components and I mean the two links above (field_label_product and field_label_component)

Comment 4

[David Lawrence [:dkl]]

We did something just like this on the Red Hat Bugzilla when I worked there. Here is the original bug:

https://bugzilla.redhat.com/show_bug.cgi?id=458436

We can add something similar to upstream which could be useful.

dkl

Comment 5

[David Lawrence [:dkl]]

Attachment: 984505_1.patch

Comment 6

[Frédéric Buclin]

Comment on attachment 8394833 [details] [diff] [review]
984505_1.patch

>+        [% IF bug.id && field.name == 'component' %]

It should be "IF bug", because if the bug object doesn't exist, you cannot call its "id" attribute (though I think TT is lenient enough to not die).


>+          (<a href="buglist.cgi?component=[% bug.component FILTER uri %]&product=[% bug.product FILTER uri %]"

& -> &amp; Also, I think this link should only return open bugs. Else the list is huge and pretty useless (especially with the 500 bugs limit by default).

Comment 7

[David Lawrence [:dkl]]

Attachment: 984505_2.patch

Comment 8

[Frédéric Buclin]

Comment on attachment 8395060 [details] [diff] [review]
984505_2.patch

I never needed this feature personally, so...

Comment 9

[:glob ✱]

Comment on attachment 8395060 [details] [diff] [review]
984505_2.patch

Review of attachment 8395060 [details] [diff] [review]:
-----------------------------------------------------------------

::: template/en/default/bug/field.html.tmpl
@@ +137,5 @@
>  
> +        [% IF bug && field.name == 'component' %]
> +          (<a href="buglist.cgi?component=[% bug.component FILTER uri %]&amp;product=[% bug.product FILTER uri %]&amp;bug_status=__open__"
> +              target="_blank">Show other [% terms.bugs %]</a>)
> +        [% END %]

the "S" in "Show" should be lowercase to match other links.

it also should be visible to all users, not just those logged in.

Comment 10

[David Lawrence [:dkl]]

Attachment: 984505_3.patch

Comment 11

[:glob ✱]

Comment on attachment 8400002 [details] [diff] [review]
984505_3.patch

Review of attachment 8400002 [details] [diff] [review]:
-----------------------------------------------------------------

r=glob

Comment 12

[David Lawrence [:dkl]]

To ssh://[e-mail]/bugzilla/bugzilla.git
   4538109..dc9486b  master -> master

Comment 13

[sjw]

Looks good :)