805739 - [socorro-crashstats] no maximum-query-date-range warning/reset message if you try to search 31 or more days

Status: VERIFIED FIXED

(Socorro :: Webapp, task)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

http://crash-stats-new-dev.allizom.org/query/?product=Firefox&version=Firefox%3A19.0a1&platform=windows&range_value=31&range_unit=days&date=10%2F26%2F2012+06%3A53%3A09&query_search=signature&query_type=contains&query=&reason=&build_id=&process_type=any&hang_type=any# no longer displays the "Uh oh! Something went wrong...

The maximum query date range you can perform is 30 days. Admins may log in to increase query date range limits. Query results have been narrowed to the default range of 14 days." message

Compare with prod:

https://crash-stats.mozilla.com/query?product=Firefox&version=Firefox%3A19.0a1&platform=windows&range_value=31&range_unit=days&date=10%2F26%2F2012+06%3A53%3A09&query_search=signature&query_type=contains&query=&reason=&build_id=&process_type=any&hang_type=any

Comment 1

[[DEACTIVATED] Adrian Gaudebert]

PR: https://github.com/mozilla/socorro-crashstats/pull/238

Comment 2

[[github robot]]

Commits pushed to master at https://github.com/mozilla/socorro-crashstats

https://github.com/mozilla/socorro-crashstats/commit/c1dd47ae96befd911fdd07aabbce2304b60afeb7
Fixes bug 805739 - Limit query date range.

https://github.com/mozilla/socorro-crashstats/commit/6e5e71fd78a0022150b3fc954fad8b77c07be049
Merge pull request #238 from AdrianGaudebert/805739-query-time-range-limit

Fixes bug 805739 - Limit query date range.

Comment 3

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED, using:

http://crash-stats-new-dev.allizom.org/query/?product=Firefox&version=Firefox%3A19.0a1&platform=win&range_value=31&range_unit=days&date=11%2F01%2F2012+18%3A27%3A59&query_search=signature&query_type=contains&query=&reason=&build_id=&process_type=any&hang_type=any

Behavior is identical to prod: query is still run as range_value=31, but on another press of Filter Crash Reports, searches with a range_value of 14.

Comment 4

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Post-fix screenshot