Report > Radar graph: Uncaught TypeError

Contribute Bug

AI-generated summary

A bug has been reported where clicking the “radar” display mode in any report results in a blank graph and a JavaScript error: Uncaught TypeError: i is undefined Julien_Palard. Screenshots of the error were provided by Arkshine. pmusaraj confirmed the issue and noted that the radar chart was intended only for specific datasets requiring three variables, and was introduced in November via PR #24274. Roman, who added the feature, acknowledged the bug and submitted a fix in PR #25368, restricting radar chart availability to appropriate report types.

1

In any report, when I cilck the third display mode (“radar”) I’m getting a blank “graph” and:

Uncaught TypeError: i is undefined
    collapse report.js:56
    n admin-report-radar.js:56
    _renderChart admin-report-radar.js:52
    _scheduleChartRendering admin-report-radar.js:44
    invoke backburner.js.js:280
    flush backburner.js.js:197
    flush backburner.js.js:358
    _end backburner.js.js:798
    end backburner.js.js:589
    _runExpiredTimers backburner.js.js:905
    setTimeout handler*setTimeout backburner.js.js:47
    _installTimerTimeout backburner.js.js:948
    _scheduleExpiredTimers backburner.js.js:928
    _runExpiredTimers backburner.js.js:904
    setTimeout handler*setTimeout backburner.js.js:47
    _installTimerTimeout backburner.js.js:948
    _reinstallTimerTimeout backburner.js.js:932
    _later backburner.js.js:865
    debounce backburner.js.js:716
    debounce Ember
    s debounce.js:23
    didReceiveAttrs admin-report-radar.js:34
    Ember 5
    <anonymous> runtime.js:2676
    evaluate runtime.js:1052
    evaluateSyscall runtime.js:4204
    evaluateInner runtime.js:4175
    evaluateOuter runtime.js:4168
    next runtime.js:4999
    _execute runtime.js:4986
    execute runtime.js:4979
    handleException runtime.js:4313
    handleException runtime.js:4521
    throw runtime.js:4260
    evaluate runtime.js:2088
    _execute runtime.js:4247
    execute runtime.js:4232
    rerender runtime.js:4547
    Ember 2
report.js:56
    collapse report.js:56
    n admin-report-radar.js:56
    map self-hosted:221
    _renderChart admin-report-radar.js:52
    _scheduleChartRendering admin-report-radar.js:44
    invoke backburner.js.js:280
    flush backburner.js.js:197
    flush backburner.js.js:358
    _end backburner.js.js:798
    end backburner.js.js:589
    _runExpiredTimers backburner.js.js:905
    (Async: setTimeout handler)
    setTimeout backburner.js.js:47
    _installTimerTimeout backburner.js.js:948
    _scheduleExpiredTimers backburner.js.js:928
    _runExpiredTimers backburner.js.js:904
    (Async: setTimeout handler)
    setTimeout backburner.js.js:47
    _installTimerTimeout backburner.js.js:948
    _reinstallTimerTimeout backburner.js.js:932
    _later backburner.js.js:865
    debounce backburner.js.js:716
    debounce Ember
    s debounce.js:23
    didReceiveAttrs admin-report-radar.js:34
    Ember 5
    <anonymous> runtime.js:2676
    evaluate runtime.js:1052
    evaluateSyscall runtime.js:4204
    evaluateInner runtime.js:4175
    evaluateOuter runtime.js:4168
    next runtime.js:4999
    _execute runtime.js:4986
    execute runtime.js:4979
    handleException runtime.js:4313
    handleException runtime.js:4521
    throw runtime.js:4260
    evaluate runtime.js:2088
    _execute runtime.js:4247
    execute runtime.js:4232
    rerender runtime.js:4547
    Ember 2

I’m running 3.2.0.beta5-dev d0486a72ac , I know I’m a few commits late but I see no commit mentionning “radar”. I’ll try to update but at night in case of downtime during upgrade.

2 Likes
2

Here is some additional error output:

image
image788×231 10.3 KB

image
image702×324 17.2 KB

1 Like
3

Thanks, I can repro as well.

I believe the radar chart is supposed to only show for a specific subset of reports, it requires a particular dataset (and it’s likely not useful for most types of datasets).

It was introduced in November via this PR: DEV: Add support for radar charts when displaying reports. by romanrizzi · Pull Request #24274 · discourse/discourse · GitHub

We should have a fix shortly.

1 Like
6

I added this bug while introducing support for radar-type reports. It shouldn’t be widely available because it works with three variables. Fixed it here:

https://github.com/discourse/discourse/pull/25368

4 Likes
8

This topic was automatically closed after 20 hours. New replies are no longer allowed.