Docker/Maybe
Docker/Maybe
1
0
Fork 0
mirror of https://github.com/maybe-finance/maybe.git synced 2025-08-04 13:05:19 +02:00
Code Issues Releases Activity Actions

Allow nil current values in trends

Browse source 
I think I might've gotten it wrong before, nils might appear in trends if values are unavailable for snapshots
This commit is contained in:
Jose Farias 2024-04-18 20:52:52 -06:00
parent ec68b9bfe7
commit 5d14d03963
1 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

8
app/models/time_series/trend.rb
View file

@ -8,7 +8,7 @@ class TimeSeries::Trend
validate :values_must_be_of_same_type, :values_must_be_of_known_type
def initialize(current:, previous:, series: nil)
@current = current || 0
@current = current
@previous = previous
@series = series
@ -68,12 +68,12 @@ class TimeSeries::Trend
end
def values_must_be_of_known_type
unless current.is_a?(Money) || current.is_a?(Numeric)
errors.add :current, "must be of type Money or Numeric"
unless current.is_a?(Money) || current.is_a?(Numeric) || current.nil?
errors.add :current, "must be of type Money, Numeric, or nil"
end
unless previous.is_a?(Money) || previous.is_a?(Numeric) || previous.nil?
errors.add :previous, "must be of type Money, Numeric or nil"
errors.add :previous, "must be of type Money, Numeric, or nil"
end
end