Fix missing stylesheet extension

[hayesr]

Referencing the file without an extension may have worked in older versions of Prince/Rails/etc. Now it seems necessary.

Core PR ManageIQ/manageiq#14793 depends on this change.

Related to this BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1447940

And part of the fixes in #1090

Current look of PDFs

/cc @dclarizio

[miq-bot]

Checked commit hayesr@7c50807 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. ⭐

[h-kataria]

@hayesr can you add before/after screenshots.

[hayesr]

@h-kataria I included the current state of PDFs, but this change alone will make no visual difference. Without ManageIQ/manageiq#14793 the path that is sent to Prince is broken and you get the unstyled version. (and without this PR the core PR will result in errors shown in the PDF)

[h-kataria]

@hayesr thx for the screenshot and info. i will merge this PR and once core PR is merged PDF should look nice and beautiful :-)

[simaishi]

Fine backport details:

$ git log -1
commit 05b8cd191aaceb294a808d908f10c2bc62d153fd
Author: Harpreet Kataria <hkataria@redhat.com>
Date:   Thu May 11 15:59:39 2017 -0400

    Merge pull request #1332 from hayesr/fix_pdf_stylesheet_ext
    
    Fix missing stylesheet extension
    (cherry picked from commit 6d21b2eca5efd704e0da9ede5455eb2779972ecc)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1460815

[simaishi]

Euwe backport (to manageiq repo) details:

$ git log -1
commit b744bb6ce376e7abe972fa45695497ed60d4fd18
Author: Harpreet Kataria <hkataria@redhat.com>
Date:   Thu May 11 15:59:39 2017 -0400

    Merge pull request #1332 from hayesr/fix_pdf_stylesheet_ext
    
    Fix missing stylesheet extension
    (cherry picked from commit 6d21b2eca5efd704e0da9ede5455eb2779972ecc)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1465081

[simaishi]

Travis is failing for Euwe branch now... @hayesr @h-kataria would you mind taking a look?

Failures:
  1) EmsInfraController#show download pdf file should match proper title of report
     Failure/Error: pdf_data = PdfGenerator.pdf_from_string(html_string, "pdf_summary.css")
       #<PdfGenerator (class)> received :pdf_from_string with unexpected arguments
         expected: ("", "pdf_summary")
              got: ("", "pdf_summary.css")
        Please stub a default value first if message might be received with other args as well. 
     # ./app/controllers/application_controller/report_downloads.rb:187:in `set_summary_pdf_data'
     # ./app/controllers/ems_common.rb:12:in `show_download'
     # ./app/controllers/ems_common.rb:135:in `show'
     # ./spec/controllers/ems_common_controller_spec.rb:312:in `block (4 levels) in <top (required)>'

  2) EmsInfraController#show download pdf file should not contains string 'ManageIQ' in the title of summary report
     Failure/Error: pdf_data = PdfGenerator.pdf_from_string(html_string, "pdf_summary.css")
       #<PdfGenerator (class)> received :pdf_from_string with unexpected arguments
         expected: ("", "pdf_summary")
              got: ("", "pdf_summary.css")
        Please stub a default value first if message might be received with other args as well. 
     # ./app/controllers/application_controller/report_downloads.rb:187:in `set_summary_pdf_data'
     # ./app/controllers/ems_common.rb:12:in `show_download'
     # ./app/controllers/ems_common.rb:135:in `show'
     # ./spec/controllers/ems_common_controller_spec.rb:312:in `block (4 levels) in <top (required)>'