Cleanup and refactor
This commit is contained in:
@ -1,20 +1,17 @@
|
||||
class TransfersController < ApplicationController
|
||||
def create
|
||||
# ap ["Logged user", current_user.username]
|
||||
# ap ["Current organization", current_organization.name]
|
||||
|
||||
if admin?
|
||||
transfer_params = params.require(:transfer).permit(:source, :destination, :amount, :reason, :post_id)
|
||||
@source = Account.find(transfer_params[:source])
|
||||
@source = if admin?
|
||||
Account.find(transfer_params[:source])
|
||||
else
|
||||
transfer_params = params.require(:transfer).permit(:destination, :amount, :reason, :post_id)
|
||||
@source = current_user.members.find_by(organization: current_organization).account
|
||||
current_user.members.find_by(organization: current_organization).account
|
||||
end
|
||||
|
||||
# ap transfer_params
|
||||
# ap @source
|
||||
# ap Account.find(transfer_params[:destination])
|
||||
Transfer.create(transfer_params.merge source: @source)
|
||||
redirect_to Account.find(transfer_params[:destination]).accountable.user
|
||||
end
|
||||
|
||||
private
|
||||
|
||||
def transfer_params
|
||||
params.require(:transfer).permit(*[:destination, :amount, :reason, :post_id, (:source if admin?)].compact)
|
||||
end
|
||||
end
|
||||
|
||||
Reference in New Issue
Block a user