Red Hat Bugzilla – Bug 1205023
Disable Download button for reports when no records are found
Last modified: 2016-06-29 10:53:24 EDT
Created attachment 1005642 [details] screenshot where download button is not greyed out Description of problem: ----------------------- Download button should be disabled for reports when no records are found. Version-Release number of selected component (if applicable): --------------------- 5.4.0.0.12 How reproducible: ---------------- Always Steps to Reproduce: ------------------ 1. 2. 3. Actual results: ---------------- Expected results: ---------------- Additional info: ----------------
Zita, Can you look into this one, i this might have already been fixed, we can close this as dup if that's the case. Let me know if you have questions. Thanks, ~Harpreet
https://github.com/ManageIQ/manageiq/pull/7692
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/f83a301ddf7b1ed25ab3cef903d1c380bfe18d3e commit f83a301ddf7b1ed25ab3cef903d1c380bfe18d3e Author: Zita Nemeckova <znemecko@redhat.com> AuthorDate: Mon Apr 4 16:50:30 2016 +0200 Commit: Zita Nemeckova <znemecko@redhat.com> CommitDate: Tue Apr 5 10:20:27 2016 +0200 Disable Download button for reports when no records are found https://bugzilla.redhat.com/show_bug.cgi?id=1205023 Download button should be disabled for reports when no records are found. app/helpers/application_helper/button/basic.rb | 2 +- app/helpers/application_helper/button/report_download_choice.rb | 5 +++++ app/helpers/application_helper/toolbar/report_view.rb | 1 + 3 files changed, 7 insertions(+), 1 deletion(-) create mode 100644 app/helpers/application_helper/button/report_download_choice.rb
https://github.com/ManageIQ/manageiq/pull/7849
Verified in 5.6.0.1-beta2.20160413141124_e25ac0e
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHBA-2016:1348