handle money move without To homie
This commit is contained in:
@@ -17,6 +17,9 @@ class MoneyMovesController < ApplicationController
|
||||
else
|
||||
error_response(:bad_request)
|
||||
end
|
||||
|
||||
rescue StandardError
|
||||
error_response(:bad_request)
|
||||
end
|
||||
|
||||
def money_move_params
|
||||
|
||||
@@ -4,13 +4,15 @@ class MoneyMove < ApplicationRecord
|
||||
def self.create_move(params)
|
||||
return false unless valid_money_move?(params)
|
||||
|
||||
move_from = create_move_from(params) if double_money_move?(params)
|
||||
move_to = create_move_to(params)
|
||||
move_from = create_move_from(params) if params[:from_homie_id].present?
|
||||
move_to = create_move_to(params) if params[:to_homie_id].present?
|
||||
|
||||
MoneyMove.transaction do
|
||||
move_from.save! if move_from.present?
|
||||
move_to.save!
|
||||
move_to.save! if move_to.present?
|
||||
end
|
||||
|
||||
true
|
||||
end
|
||||
|
||||
def self.create_settle_record(homie_id, amount)
|
||||
@@ -27,24 +29,27 @@ class MoneyMove < ApplicationRecord
|
||||
private
|
||||
|
||||
def valid_money_move?(params)
|
||||
params[:amount].present? && params[:to_homie_id].present?
|
||||
end
|
||||
|
||||
def double_money_move?(params)
|
||||
params[:from_homie_id].present? && params[:to_homie_id].present?
|
||||
params[:amount].present? && (params[:to_homie_id].present? || params[:from_homie_id].present?)
|
||||
end
|
||||
|
||||
def from_source_name(params)
|
||||
if double_money_move?(params)
|
||||
if params[:from_homie_id].present?
|
||||
Homie.find(params[:from_homie_id]).name
|
||||
else
|
||||
'External'
|
||||
end
|
||||
end
|
||||
|
||||
def to_source_name(params)
|
||||
if params[:to_homie_id].present?
|
||||
Homie.find(params[:to_homie_id]).name
|
||||
else
|
||||
'External'
|
||||
end
|
||||
end
|
||||
|
||||
def create_move_from(params)
|
||||
to_homie_name = Homie.find(params[:to_homie_id]).name
|
||||
new_desc = "[#{to_homie_name}] #{params[:description]}".lstrip
|
||||
new_desc = "[#{to_source_name(params)}] #{params[:description]}".lstrip
|
||||
MoneyMove.new(
|
||||
homie_id: params[:from_homie_id],
|
||||
amount: -params[:amount].to_d,
|
||||
|
||||
Reference in New Issue
Block a user