[PATCH] D47581: [LNT] Fix use of field index in daily report

[John Brawn via Phabricator via llvm-commits]

john.brawn created this revision.
john.brawn added reviewers: MatzeB, marxin, kristof.beyls, cmatthews.

Links to graph plots in the daily report make use of the field index but haven't been updated to get it via get_field_index, causing these links to be broken. Fix this by using get_field_index.


Repository:
  rL LLVM

https://reviews.llvm.org/D47581

Files:
  lnt/server/ui/templates/reporting/daily_report.html


Index: lnt/server/ui/templates/reporting/daily_report.html
===================================================================
--- lnt/server/ui/templates/reporting/daily_report.html
+++ lnt/server/ui/templates/reporting/daily_report.html
@@ -247,7 +247,7 @@
     <td style="{{ styles.td }}"> </td>
     <td class="machine-name" style="{{ styles.td }}"><a href="{{
       ts_url}}/graph?plot.0={{machine.id}}.{{test.id}}.{{
-        field.index}}&highlight_run={{key_run.id}}">{{
+        report.ts.get_field_index(field)}}&highlight_run={{key_run.id}}">{{
           machine.name}}</a></td>
 {%-    set first_result_shown = false -%}
 {%-    for day_result in day_results|reverse -%}


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D47581.149264.patch
Type: text/x-patch
Size: 698 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180531/e0d8d56b/attachment.bin>