A query without date should display results between now and range_value range_unit ago or 7 days ago

VERIFIED FIXED in 2.4.1

Status

VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: scoobidiver, Assigned: adrian)

Tracking

unspecified
2.4.1

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

7 years ago
On January 19th UTC, https://crash-stats.mozilla.com/report/list?signature=PR_EnumerateAddrInfo gives you:
* at 7:57 UTC results before January 17th
* at 8:05 UTC results before January 18th
A query should display results before the previous full UTC day instead of the previous full PST day
(Assignee)

Comment 1

7 years ago
The new behavior of this report will be to show, by default, crashes between now and 7 days ago. Does this seem like a reasonable behavior, or do you expect this report to show crashes until the day before by default?
(Reporter)

Comment 2

7 years ago
(In reply to Adrian Gaudebert [:adrian] from comment #1)
> Does this seem like a reasonable behavior, or do you expect this report to show
> crashes until the day before by default?
I agree with that.

Maybe this bug is still applicable to https://crash-stats.mozilla.com/topcrasher/byversion/Firefox/9.0.1 where only full days are used.

In addition, when you click on a crash signature in the top crasher list, you are redirected to https://crash-stats.mozilla.com/report/list?range_value=7&range_unit=days&date=2012-01-18&signature=PR_EnumerateAddrInfo&version=Firefox%3A10.0b4 although it should be https://crash-stats.mozilla.com/report/list?range_value=7&range_unit=days&date=2012-01-19&signature=PR_EnumerateAddrInfo&version=Firefox%3A10.0b4

Comment 3

7 years ago
(In reply to Adrian Gaudebert [:adrian] from comment #1)
> The new behavior of this report will be to show, by default, crashes between
> now and 7 days ago.

That sounds good to me. I'd expect just-processed crashes to show up if I don't give it any date param.

And Scoobidiver, the second half of comment #2 is wrong. There is a bug, but it's not that the date field in the URL is wrong, it's that it's wrongly interpreted in the report/list - and that's already being dealt with in bug 718901 and fixed for the next Socorro update.
(Reporter)

Comment 4

7 years ago
Per comment 1 and comment 3, I renamed the bug summary.
Summary: A query should display results before the previous full UTC day instead of the previous full PST day → A query without date should display results between now and range_value range_unit ago or 7 days ago
(Assignee)

Updated

7 years ago
Depends on: 678642
(Assignee)

Comment 5

7 years ago
Fixed with bug 678642. 

Steps to QA
-----------

1. Go to advanced search and search for something without changing the default date parameters (one week before now).
2. Click on one of the signatures with more than 250 results.
3. In the URL, remove everything except the "signature" parameter, so your URL looks like: 
> http://crash-stats.somewhere.tld/report/list?signature=my-awesome-signature
4. Check that you have the same number of results before and after cleaning the URL.
(Assignee)

Updated

7 years ago
Assignee: nobody → adrian
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.4.1
Thx adrian ... qa verified on stage per comment 5's steps.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.