Skip to content

Commit d0498af

Browse files
committed
Correct result table access in async daily report
Result table is indexed by the field's name, not display_name. Adjust the async report accordingly. Change-Id: Id7f4dbb07a1619a62fa2a611afce277cafe33815
1 parent b4b3fe8 commit d0498af

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

lnt/server/ui/templates/reporting/daily_report_async.html

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@ <h3>Result Table ({{ field.display_name }})</h3>
120120
<th style="{{ styles.th }}">Sparkline</th>
121121
</tr>
122122
</thead>
123-
<tbody id="rt-{{ field.display_name }}">
123+
<tbody id="rt-{{ field.name }}">
124124
</tbody></table>
125125
{%- endfor -%}
126126
{% endif %}
@@ -285,6 +285,7 @@ <h3>Result Table ({{ field.display_name }})</h3>
285285
const FIELDS = [
286286
{% for field in report.fields|reverse %}
287287
{
288+
name: "{{ field.name }}",
288289
display_name: "{{ field.display_name }}",
289290
index: "{{ report.ts.get_field_index(field) }}"
290291
},
@@ -295,7 +296,7 @@ <h3>Result Table ({{ field.display_name }})</h3>
295296
// we cannot just append to the DOM directly since we want the result table
296297
// to be sorted by some priority metric (largest abs percentage delta).
297298
var KEY_RUNS = {}; // { machine.id -> run.id }
298-
var RESULTS = {}; // { field.display_name -> { test.id -> { name, results: { machine.id -> day results }}}}
299+
var RESULTS = {}; // { field.name -> { test.id -> { name, results: { machine.id -> day results }}}}
299300

300301
function test_priority_compare(a, b) {
301302
const a_res = a[1].results;
@@ -313,7 +314,7 @@ <h3>Result Table ({{ field.display_name }})</h3>
313314
}
314315

315316
function regenerate_results_tables_for_field(field, results) {
316-
var rt_table_id = "rt-" + field.display_name;
317+
var rt_table_id = "rt-" + field.name;
317318
var rt_table = document.getElementById(rt_table_id);
318319
rt_table.innerHTML = '';
319320
for (const [rt_test_id, rt_value] of Object.entries(results).sort(test_priority_compare)) {
@@ -362,7 +363,7 @@ <h3>Result Table ({{ field.display_name }})</h3>
362363

363364
function regenerate_results_tables() {
364365
for (const field of FIELDS) {
365-
regenerate_results_tables_for_field(field, RESULTS[field.display_name]);
366+
regenerate_results_tables_for_field(field, RESULTS[field.name]);
366367
}
367368
}
368369

@@ -408,12 +409,12 @@ <h3>Result Table ({{ field.display_name }})</h3>
408409
var today_key_run = res.data.machine_runs['' + machine.id]["0"];
409410
if (today_key_run) {
410411
KEY_RUNS[machine.id] = today_key_run.id;
411-
// RESULTS: { field.display_name -> { test.id -> { name, results: { machine.id -> day results }}}}
412+
// RESULTS: { field.name -> { test.id -> { name, results: { machine.id -> day results }}}}
412413
for (const field of FIELDS) {
413-
RESULTS[field.display_name] = RESULTS[field.display_name] || {};
414-
var field_results = RESULTS[field.display_name];
414+
RESULTS[field.name] = RESULTS[field.name] || {};
415+
var field_results = RESULTS[field.name];
415416
// merge new data into (possibly pre-existing from other results) field_results object.
416-
var rt_data = res.data['result_table'][field.display_name]
417+
var rt_data = res.data['result_table'][field.name]
417418
if (rt_data) {
418419
for (const [rt_test_id, rt_value] of Object.entries(rt_data)) {
419420
var rt_test_name = res.data.reporting_tests['' + rt_test_id].name;

0 commit comments

Comments
 (0)