Don't format chargeback rate column in MiqReport::Formatting

ManageIQ · Feb 5, 2019 · bad74a9 · bad74a9
1 parent 5810a00
commit bad74a9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/app/models/chargeback.rb b/app/models/chargeback.rb
@@ -268,6 +268,10 @@ def self.default_column_for_format(col)
     end
   end
 
+  def self.rate_column?(col)
+    col.ends_with?("_rate")
+  end
+
   private
 
   def relevant_fields

diff --git a/app/models/miq_report/formatting.rb b/app/models/miq_report/formatting.rb
@@ -37,6 +37,8 @@ def format(col, value, options = {})
       end
     end
 
+    # TODO: remove this and update storing column format to instance of report in UI (this requires migration)
+    options[:format] = :_none_ if Chargeback.db_is_chargeback?(db) && Chargeback.rate_column?(col.to_s)
     col = Chargeback.default_column_for_format(col.to_s) if Chargeback.db_is_chargeback?(db)
 
     format = options.delete(:format)