Closed Bug 891311 Opened 11 years ago Closed 11 years ago

Text in the "My Requests" page is misleading about how the AND/OR radio button works

Categories

(Bugzilla :: Attachments & Requests, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 4.4

People

(Reporter: LpSolit, Assigned: LpSolit)

References

Details

Attachments

(1 file, 1 obsolete file)

In the "My Requests" page, the description of the AND/OR radio button says:

"The logical conjunction/disjunction between the requester and the requestee"

But in fact, the radio button applies to all fields, not the requester vs requestee fields only. So if you set requester = foo, requestee = bar, product = baz with OR, you get:

requester = foo OR requestee = bar OR product = baz

instead of

(requester = foo OR requestee = bar) AND product = baz

The current behavior is the expected one, as it has been implemented this way in Bugzilla 2.18, see bug 179881. So what's wrong is the description of the radio button which should make clear that it affects all fields.
Attached patch patch, v1 (obsolete) — Splinter Review
Assignee: attach-and-request → LpSolit
Status: NEW → ASSIGNED
Attachment #772607 - Flags: review?(glob)
Comment on attachment 772607 [details] [diff] [review]
patch, v1

i've always thought that ui was clumsy; if we're going to change it, let's fix up the wording or make it self-descriptive.

i think it would be clearer to have a <select> instead of a <radio>, with the two options being something like "Show requests matching ALL set fields" and "Show requests matching ANY set fields" (i'm not particularly attached to this wording).

the behaviour of OR isn't a "logical conjunction between fields" as this implies all fields are OR'd.  if you have the requster and requestee set, with all other fields set to any, the results are limited to just "requester OR requestee".

perhaps instead of using Any/all as the default value for the selects, it should be "-" or similar to indicate that field isn't being used as part of the query.
Attachment #772607 - Flags: review?(glob) → review-
To be honest, the way OR works is really counter-intuitive. I want to track all review requests related to me in the Bugzilla product, so I select product = Bugzilla, flag = review, requester = requestee = LpSolit, and the list I get is totally useless, because everything is OR'ed. IMO, OR only makes sense for the requester vs requestee field. Any objection to change the way it currently works? As there was no UI for AND/OR in the past, I doubt this change affects a lot of people.
justdave, see my previous comment.
Flags: needinfo?(justdave)
hmmm...  the usecase for that is sort of fulfilled by the dashboard page these days (although that's a bmo extension and not part of core).

In any case, I kinda like the dropdown idea, because that makes it match the UI used for that concept in the custom search section of the search page.

It also seems like only applying it to the requestor and requestee and making everything else be AND anyway would fit the vast majority of the use-cases on there. If we really want to allow ORing those other fields then we need a more-involved UI I think
Flags: needinfo?(justdave)
Attached patch patch, v2Splinter Review
Attachment #772607 - Attachment is obsolete: true
Attachment #790257 - Flags: review?(dkl)
Comment on attachment 790257 [details] [diff] [review]
patch, v2

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

Looks good and works as expected. r=dkl
Attachment #790257 - Flags: review?(dkl) → review+
Flags: approval?
Flags: approval4.4?
Flags: approval?
Flags: approval4.4?
Flags: approval4.4+
Flags: approval+
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified request.cgi
modified template/en/default/request/queue.html.tmpl
Committed revision 8766.

Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.4/
modified request.cgi
modified template/en/default/request/queue.html.tmpl
Committed revision 8616.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: