Maybe/app/controllers/account/transfers_controller.rb

class Account::TransfersController < ApplicationController
  layout :with_sidebar

  before_action :set_transfer, only: %i[destroy show update]

  def new
    @transfer = Account::Transfer.new
  end

  def show
  end

  def create
    from_account = Current.family.accounts.find(transfer_params[:from_account_id])
    to_account = Current.family.accounts.find(transfer_params[:to_account_id])

    @transfer = Account::Transfer.build_from_accounts from_account, to_account, \
                                             date: transfer_params[:date],
                                             amount: transfer_params[:amount].to_d

    if @transfer.save
      @transfer.entries.each(&:sync_account_later)
      redirect_to transactions_path, notice: t(".success")
    else
      # TODO: this is not an ideal way to handle errors and should eventually be improved.
      # See: https://github.com/hotwired/turbo-rails/pull/367
      flash[:alert] = @transfer.errors.full_messages.to_sentence
      redirect_to transactions_path
    end
  end

  def update
    @transfer.update_entries!(transfer_update_params)
    redirect_back_or_to transactions_url, notice: t(".success")
  end

  def destroy
    @transfer.destroy!
    redirect_back_or_to transactions_url, notice: t(".success")
  end

  private

    def set_transfer
      record = Account::Transfer.find(params[:id])

      unless record.entries.all? { |entry| Current.family.accounts.include?(entry.account) }
        raise ActiveRecord::RecordNotFound
      end

      @transfer = record
    end

    def transfer_params
      params.require(:account_transfer).permit(:from_account_id, :to_account_id, :amount, :date, :name, :excluded)
    end

    def transfer_update_params
      params.require(:account_transfer).permit(:excluded, :notes)
    end
end