Browse Source

CLDC-1944: Fix linter errors

pull/2280/head
Robert Sullivan 2 years ago
parent
commit
b9b8bb320d
  1. 2
      app/controllers/form_controller.rb
  2. 2
      app/models/validations/sales/sale_information_validations.rb

2
app/controllers/form_controller.rb

@ -86,7 +86,7 @@ private
year = params[@log.model_name.param_key]["#{question.id}(1i)"] year = params[@log.model_name.param_key]["#{question.id}(1i)"]
next unless [day, month, year].any?(&:present?) next unless [day, month, year].any?(&:present?)
result[question.id] = if Date.valid_date?(year.to_i, month.to_i, day.to_i) && year.to_i > 0 result[question.id] = if Date.valid_date?(year.to_i, month.to_i, day.to_i) && year.to_i.positive?
Date.new(year.to_i, month.to_i, day.to_i) Date.new(year.to_i, month.to_i, day.to_i)
else else
Date.new(0, 1, 1) Date.new(0, 1, 1)

2
app/models/validations/sales/sale_information_validations.rb

@ -4,7 +4,7 @@ module Validations::Sales::SaleInformationValidations
include MoneyFormattingHelper include MoneyFormattingHelper
def validate_practical_completion_date(record) def validate_practical_completion_date(record)
return unless !record.hodate.blank? && date_valid?("hodate", record) return unless record.hodate.present? && date_valid?("hodate", record)
return if record.saledate.blank? return if record.saledate.blank?
if record.hodate > record.saledate if record.hodate > record.saledate

Loading…
Cancel
Save