Fixes #36587 - rename Host - Applicable Errata report

* ...and fix comment typos
theforeman · Aug 7, 2023 · 75d8ea3 · 75d8ea3
1 parent 7c41286
commit 75d8ea3
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 3 deletions.
diff --git a/...rt_templates/host_-_applicable_errata.erb → ...ort_templates/host_-_available_errata.erb b/...rt_templates/host_-_applicable_errata.erb → ...ort_templates/host_-_available_errata.erb
@@ -1,5 +1,5 @@
 <%#
-name: Host - Applicable Errata
+name: Host - Available Errata
 snippet: false
 template_inputs:
 - name: Hosts filter
@@ -27,7 +27,7 @@ template_inputs:
 model: ReportTemplate
 require:
 - plugin: katello
- version: 4.9.0
+  version: 4.9.0
 -%>
 <%- report_headers 'Host', 'Operating System', 'Environment', 'Erratum', 'Type', 'Published', 'Available since', 'Severity', 'Packages', 'CVEs', 'Reboot suggested' -%>
 <%- errata_filter = input('Errata filter') %>

diff --git a/app/views/unattended/report_templates/host_-_registered_content_hosts.erb b/app/views/unattended/report_templates/host_-_registered_content_hosts.erb
@@ -21,7 +21,7 @@ template_inputs:
 model: ReportTemplate
 require:
 - plugin: katello
- version: 4.9.0
+  version: 4.9.0
 -%>
 <%- if input('Installability') == 'applicable' -%>
     <%- report_headers 'Name', 'Ip', 'Operating System', 'Subscriptions', 'Applicable Errata', 'Owner', 'Kernel', 'Latest kernel available' -%>

diff --git a/db/migrate/20230803180552_rename_applicable_errata_report_template.rb b/db/migrate/20230803180552_rename_applicable_errata_report_template.rb
@@ -0,0 +1,20 @@
+class RenameApplicableErrataReportTemplate < ActiveRecord::Migration[6.1]
+  TEMPLATE_NAMES = {
+    "Host - Applicable Errata" => "Host - Available Errata",
+  }
+
+  def up
+    TEMPLATE_NAMES.each do |from, to|
+      token = SecureRandom.base64(5)
+      ReportTemplate.unscoped.find_by(name: to)&.update_columns(:name => "#{to} Backup #{token}")
+      ReportTemplate.unscoped.find_by(name: from)&.update_columns(:name => to)
+    end
+  end
+
+  def down
+    TEMPLATE_NAMES.each do |from, to|
+      ReportTemplate.unscoped.find_by(name: from)&.delete
+      ReportTemplate.unscoped.find_by(name: to)&.update_columns(:name => from)
+    end
+  end
+end