Normalize on Camelcase Html for consistency with loading

We need to match MiqReport::Formatters:Html and MiqReport::Generator::Html or change them to also be HTML. I decided on Html because HTML would also require us to teach zeitwerk another acronym.
jrafanie · May 2, 2023 · 9e0de20 · 9e0de20
1 parent 14a845f
commit 9e0de20
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/manageiq/reporting/formatter.rb b/lib/manageiq/reporting/formatter.rb
@@ -31,7 +31,7 @@ module ReportFormatter
   deprecate_constant 'C3Charting',        'ManageIQ::Reporting::Formatter::C3Charting'
   deprecate_constant 'ChartCommon',       'ManageIQ::Reporting::Formatter::ChartCommon'
   deprecate_constant 'Converter',         'ManageIQ::Reporting::Formatter::Converter'
-  deprecate_constant 'ReportHTML',        'ManageIQ::Reporting::Formatter::HTML'
+  deprecate_constant 'ReportHTML',        'ManageIQ::Reporting::Formatter::Html'
   deprecate_constant 'ReportRenderer',    'ManageIQ::Reporting::Formatter::ReportRenderer'
   deprecate_constant 'ReportText',        'ManageIQ::Reporting::Formatter::Text'
   deprecate_constant 'ReportTimeline',    'ManageIQ::Reporting::Formatter::Timeline'

diff --git a/lib/manageiq/reporting/formatter/html.rb b/lib/manageiq/reporting/formatter/html.rb
@@ -1,7 +1,7 @@
 module ManageIQ
   module Reporting
     module Formatter
-      class HTML < Ruport::Formatter
+      class Html < Ruport::Formatter
         renders :html, :for => ManageIQ::Reporting::Formatter::ReportRenderer
 
         def build_html_title