[UI] ADU - If no ADU results plot causes error breaking UI

VERIFIED FIXED in 18

Status

--
major
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: espressive, Assigned: espressive)

Tracking

unspecified

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
If one loads Crashes by ADU and there is no data for the graph it causes the following JS error:

"uncaught exception: Invalid dimensions for plot, width = null, height = null"

Because of this error all other JS does not execute and hence the UI is broken as a user can no longer select different reports for example, switching to by OS.

Only draw plot if there is data available.

Comment 1

7 years ago
Commits pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/0fb909f5817c4d1ae4a23f30dfd5571384bb9f25
only plotting content if data is available fixes bug 779103

https://github.com/mozilla/socorro/commit/cb426b820991354268316562fa29c906141f9ede
Merge pull request #756 from ossreleasefeed/adu-only-plot-if-data-available-779103

only plotting content if data is available fixes bug 779103

Updated

7 years ago
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Comment 2

7 years ago
Sent pull request: https://github.com/mozilla/socorro/pull/756

TEST CASE
---------

If you navigate to crashes per ADU and there is no data or you generate a new report for which there is no data, a JS error will no longer be thrown and thus the interface will still be fully functional.
Target Milestone: 19 → 18

Comment 4

7 years ago
Right, as the front page has data, https://crash-stats.allizom.org/daily?p=Firefox&v[]= should also show some, but there is none at all.
(Assignee)

Comment 5

7 years ago
[:mbrandt] This is because the needed changes for crashes per ADU has not been made for Mobeta so, this still uses and old service that will return no data. This bug is specific to the JS error that was thrown when this page is loaded without data.
Thanks espressive
Status: REOPENED → RESOLVED
Last Resolved: 7 years ago7 years ago
Resolution: --- → FIXED
Created attachment 659127 [details]
Post-fix screenshot
You need to log in before you can comment on or make changes to this bug.