1612287 - Issue with negation operator in query search

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: General, defect, P1)

Description

[Calixte Denizet (:calixte)]

The following query:
https://bugzilla.mozilla.org/buglist.cgi?bug_id_type=anyexact&query_format=advanced&n1=1&list_id=15088954&f1=component&bug_id=1580658&o1=changedafter&v1=1970-01-01
should return bug 1580658 only if component for this bug has never been changed.
But the component has been changed several times and this query returns it.
If I remove the negation, then the bug is not found, so it look likes n=1 is equivalent to false.

Comment 1

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

I believe that the query should be: https://mzl.la/2RZV27n and use the 'ever changed' operator.

:dkl, am I constructing it correctly?

Comment 2

[Marco Castelluccio [:marco]]

(In reply to Emma Humphries, Bugmaster ☕️🎸🧞‍♀️✨ (she/her) [:emceeaich] (UTC-8) needinfo? me from comment #1)

I believe that the query should be: https://mzl.la/2RZV27n and use the 'ever changed' operator.

:dkl, am I constructing it correctly?

This query has f1=component&o1=everchanged, so the bug should be in the result list (but it isn't).
If we negate it, n1=1&f1=component&o1=everchanged, the bug is in the result list (but it shouldn't).

Comment 3

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

Marking this as a P1 blocker because this is preventing bug bug scans.

Comment 4

[Sylvestre Ledru [:Sylvestre]]

Emma, do we have an eta for this?
thanks

can we backout the patch which caused the issue?

Comment 5

[David Lawrence [:dkl]]

I have found the issue and is due to a 2020 year rollover bug with Date::Parse which is used by BMO. Instead of the date defaulting to 1970-01-01 when choosing 'ever changed' in your search, the date is being changed to 2070-01-01. This of course will return results the opposite of what you are looking for.

This will require a new BMO deployment to fix and I am in testing right now.

Comment 6

[David Lawrence [:dkl]]

Attachment: GitHub Pull Request

Comment 7

[Calixte Denizet (:calixte)]

:dkl, any ETA ?
This bug is really a blocker since auto_nag cannot run without the fix.

Comment 8

[David Lawrence [:dkl]]

Merged to master. Will be live soon.