Description of problem: Summary Screens: Download Summary to PDF toolbar button is missing Version-Release number of selected component (if applicable): How reproducible: 100% Steps to Reproduce: 1. Go to any CI summary screen, download to PDF button is missing 2. 3. Actual results: Download to PDF button is missing Expected results: Should have PDF button available for download. Additional info:
https://github.com/ManageIQ/manageiq/pull/9952
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/69bffac3e40e6108aca4d3d5cdfd0f42f0d2e65e commit 69bffac3e40e6108aca4d3d5cdfd0f42f0d2e65e Author: Harpreet Kataria <hkataria> AuthorDate: Wed Jul 20 13:50:21 2016 -0400 Commit: Harpreet Kataria <hkataria> CommitDate: Wed Jul 20 16:14:11 2016 -0400 Moved restful controller check into a mixin in helpers @showtype should also be passed in to ToolbarChooser.new call. Due to @showtype not being passed in @view_context.send(:restful?) was not getting called at all, after @showtype was passed in @view_context.send(:restful?) call was throwing an error, to fix the error moved restful controller check from toolbar_builder into a mixin so it can be used from other places such as toolbar_chooser to determine which summary_view_tb should be rendered. https://bugzilla.redhat.com/show_bug.cgi?id=1358449 app/helpers/application_helper.rb | 2 ++ app/helpers/application_helper/toolbar_builder.rb | 17 ++--------------- app/helpers/application_helper/toolbar_chooser.rb | 7 +++++-- app/helpers/restful_controller_mixin.rb | 9 +++++++++ 4 files changed, 18 insertions(+), 17 deletions(-) create mode 100644 app/helpers/restful_controller_mixin.rb
Verified in version 5.7.0.3.20160927165516_075d0f3