Bug 1447940 - Formatting of Provider summary PDF file generated from provider summary page is very broken
Summary: Formatting of Provider summary PDF file generated from provider summary page ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.7.0
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: GA
: 5.9.0
Assignee: Eric Winchell
QA Contact: Jan Krocil
URL:
Whiteboard:
Depends On:
Blocks: 1460815 1465081
TreeView+ depends on / blocked
 
Reported: 2017-05-04 09:32 UTC by Jan Krocil
Modified: 2019-08-06 20:04 UTC (History)
7 users (show)

Fixed In Version: 5.9.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1460815 1465081 (view as bug list)
Environment:
Last Closed: 2018-03-06 14:53:06 UTC
Category: Bug
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Broken summary PDF (14.99 KB, application/pdf)
2017-05-04 09:32 UTC, Jan Krocil
no flags Details

Description Jan Krocil 2017-05-04 09:32:45 UTC
Created attachment 1276204 [details]
Broken summary PDF

Description of problem:
SSIA

Version-Release number of selected component (if applicable):
5.8.0.12

How reproducible:
Always

Steps to Reproduce:
1. Add a provider (EC2 in my case)
2. Go to it's summary page
3. Click the "Download summary in PDF format" button

Actual results:
See attached file

Expected results:
Formatting is correct - with tables and graphics, not just text

Additional info:

Comment 2 Jan Krocil 2017-05-04 09:52:47 UTC
This is also broken in 5.7.2.1 but it used to work in 5.6.z. Changing the "Version" to "5.7.0" and adding Regression keyword.

Comment 5 CFME Bot 2017-05-11 20:03:32 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/7c50807e91a5d291382bc969df4865a6fa789770

commit 7c50807e91a5d291382bc969df4865a6fa789770
Author:     Eric Hayes <eric>
AuthorDate: Thu May 11 11:39:18 2017 -0700
Commit:     Eric Hayes <eric>
CommitDate: Thu May 11 12:02:11 2017 -0700

    Fix missing extension
    
    https://github.com/ManageIQ/manageiq/pull/14793
    https://bugzilla.redhat.com/show_bug.cgi?id=1447940

 app/controllers/application_controller/report_downloads.rb | 2 +-
 spec/controllers/ems_common_controller_spec.rb             | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

Comment 6 CFME Bot 2017-05-12 11:37:03 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/46719a32a8acb6136e046107b242f7c36124f52b

commit 46719a32a8acb6136e046107b242f7c36124f52b
Author:     Eric Hayes <eric>
AuthorDate: Tue Apr 18 09:06:56 2017 -0700
Commit:     Eric Hayes <eric>
CommitDate: Thu May 11 11:14:45 2017 -0700

    Find correct stylesheet path in plugins
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1447940

 lib/pdf_generator.rb | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

Comment 8 CFME Bot 2017-06-02 03:06:37 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/1bd4255b3914728f0d82763f8d633950ea82524b

commit 1bd4255b3914728f0d82763f8d633950ea82524b
Author:     Eric Hayes <eric>
AuthorDate: Mon May 15 08:33:30 2017 -0700
Commit:     Eric Hayes <eric>
CommitDate: Thu Jun 1 14:34:51 2017 -0400

    Compile and write stylesheets for PDF rendering
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1447940

 lib/asset_writer.rb                       | 33 +++++++++++++++++++++++++++++
 lib/pdf_generator.rb                      | 11 ++--------
 lib/pdf_generator/prince_pdf_generator.rb | 11 +++++++++-
 spec/lib/pdf_generator_spec.rb            | 35 +++++++++++++++++++++++++++++++
 4 files changed, 80 insertions(+), 10 deletions(-)
 create mode 100644 lib/asset_writer.rb
 create mode 100644 spec/lib/pdf_generator_spec.rb

Comment 20 Jan Krocil 2017-10-30 13:28:07 UTC
Verified fixed in 5.9.0.4 - 5.9.0.4.20171024163837_ef71ea6; formatting is now correct.

Works only for cloud providers. There is a separate bug for the button missing in infra providers.


Note You need to log in before you can comment on or make changes to this bug.