Adminetic\Website\Http\Livewire\Admin\Payment\PaymentPanel

Complexity

Total Complexity

10

Size/Duplication

Total Lines	76
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	updatedAmount()	0	3	2
A	mount()	0	5	2
A	update()	0	19	2
A	store()	0	18	2
A	render()	0	5	1
A	setEdit()	0	4	1

<?php

namespace Adminetic\Website\Http\Livewire\Admin\Payment;

use Adminetic\Website\Models\Admin\Payment;
use Illuminate\Support\Facades\Auth;
use Livewire\Component;

class PaymentPanel extends Component
{
    public $toggle_modal = false;

    public $model;
    public $type;

    public $amount;
    public $particular;

    public $payment_id;

    public function mount($model, $type)
    {
        $this->model = $model;
        $this->type = $type;
        $this->particular = ('An '.($this->type ? 'income' : 'expense').' of '.currency().$this->amount.' is registered under '.class_basename($this->model).' : '.$this->model->track_id);
    }

    public function updatedAmount()
    {
        $this->particular = ('An '.($this->type ? 'income' : 'expense').' of '.currency().$this->amount.' is registered under '.class_basename($this->model).' : '.$this->model->track_id);
    }

    public function store()
    {
        $this->validate([
            'amount' => 'required|numeric',
        ]);

        $particular = $this->particular ?? ('An '.($this->type ? 'income' : 'expense').' of '.currency().$this->amount.' is registered under '.class_basename($this->model).' : '.$this->model->track_id);

        $this->model->payments()->create([
            'user_id' => Auth::user()->id,

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

            'type' => $this->type,
            'amount' => $this->amount,
            'particular' => $particular,
        ]);
        $this->amount = null;

        $this->model->refresh();
        $this->emitSelf('payment_success', $particular);
    }

    public function update()
    {
        $this->validate([
            'amount' => 'required|numeric',
        ]);

        $particular = $this->particular ?? ('An '.($this->type ? 'income' : 'expense').' of '.currency().$this->amount.' is registered under '.class_basename($this->model).' : '.$this->model->track_id.'(Updated)');

        Payment::find($this->payment_id)->update([
            'user_id' => Auth::user()->id,

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

            'type' => $this->type,
            'amount' => $this->amount,
            'particular' => $particular,
        ]);
        $this->amount = null;
        $this->payment_id = null;

        $this->model->refresh();
        $this->emitSelf('payment_success', $particular);
    }

    public function setEdit(Payment $payment)
    {
        $this->payment_id = $payment->id;
        $this->amount = $payment->amount;

The property amount does not seem to exist on Adminetic\Website\Models\Admin\Payment.

    }

    public function render()
    {
        $payments = $this->model->payments;

        return view('website::livewire.admin.payment.payment-panel', compact('payments'));
    }
}