Skip to content

Commit

Permalink
[FINE] Prefer rr.report over rr.report_results in SavedReportPagi…
Browse files Browse the repository at this point in the history
…ng mixin

https://bugzilla.redhat.com/show_bug.cgi?id=1594387

Gaprindashvili Backport commit from:

    #4143

Original SHA:

   b5260fd

Rest of original message below...

* * *

This not only makes it so we are consistent between the main controller
and the mixin, but this is also much faster to use this over
`MiqReportResult#report_results` when there is a large `binary_blob`
associated with the `MiqReportResult` record.
  • Loading branch information
NickLaMuro committed Jun 25, 2018
1 parent adba577 commit 02fd904
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion app/controllers/application_controller.rb
Expand Up @@ -297,7 +297,7 @@ def saved_report_paging
@sb[:pages][:perpage] = settings(:perpage, :reports)

rr = MiqReportResult.find(@sb[:pages][:rr_id])
@html = report_build_html_table(rr.report_results,
@html = report_build_html_table(rr.report,
rr.html_rows(:page => @sb[:pages][:current],
:per_page => @sb[:pages][:perpage]).join)

Expand Down

0 comments on commit 02fd904

Please sign in to comment.