Fix missing checks
This commit is contained in:
parent
4d1e2c5164
commit
16e65a3b03
|
|
@ -190,11 +190,15 @@ def validate_set_timer_end_params
|
||||||
|
|
||||||
if timer_end_seconds.present?
|
if timer_end_seconds.present?
|
||||||
begin
|
begin
|
||||||
parsed_time = Time.zone.now + timer_end_seconds.to_i
|
timer_end_seconds = Integer(timer_end_seconds)
|
||||||
params[:timer_end] = parsed_time
|
|
||||||
rescue ArgumentError
|
rescue ArgumentError
|
||||||
return render json: { error: 'Invalid seconds format' }, status: :unprocessable_entity
|
return render json: { error: 'Invalid seconds format' }, status: :unprocessable_entity
|
||||||
end
|
end
|
||||||
|
|
||||||
|
return render json: { error: 'Timer end must be in the future' }, status: :unprocessable_entity if timer_end_seconds <= 0
|
||||||
|
|
||||||
|
parsed_time = Time.zone.now + timer_end_seconds
|
||||||
|
params[:timer_end] = parsed_time
|
||||||
elsif timer_end.present?
|
elsif timer_end.present?
|
||||||
begin
|
begin
|
||||||
parsed_time = Time.zone.parse(timer_end)
|
parsed_time = Time.zone.parse(timer_end)
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue