App\Http\Controllers\OpenPayroll\DeductionController::store()

Complexity

Conditions	1
Paths	1

Size

Total Lines	24
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function store(Request $request)
    {
        $this->validate($request, [
            'payslip' => 'required',
            'type'    => 'required',
            'amount'  => 'required',
        ]);

        $payslip = \App\Models\OpenPayroll\Payslip::with('payroll', 'user')->findByHashSlugOrId($request->payslip);
        $type    = \App\Models\OpenPayroll\DeductionType::find($request->type);

        \App\Models\OpenPayroll\Deduction::create([
            'user_id'           => $payslip->user_id,

The property user_id does not seem to exist on Illuminate\Database\Eloquent\Builder.

            'payroll_id'        => $payslip->payroll_id,

The property payroll_id does not seem to exist on Illuminate\Database\Eloquent\Builder.

            'payslip_id'        => $payslip->id,

The property id does not seem to exist on Illuminate\Database\Eloquent\Builder.

            'deduction_type_id' => $request->type,
            'name'              => $type->name,

The property name does not exist on App\Models\OpenPayroll\DeductionType. Since you implemented __get, consider adding a @property annotation.

            'description'       => $request->description ?? $type->name,
            'amount'            => money()->toMachine($request->amount),
        ]);

        swal()->success('Deduction', 'You have successfully added new earning.');

The function swal was not found. Maybe you did not declare it correctly or list all dependencies?

        return redirect()->route('open-payroll.payslip.show', $request->payslip);

The expression return redirect()->route('open-payroll.payslip.show', $request->payslip) returns the type Illuminate\Http\RedirectResponse which is incompatible with the documented return type Illuminate\Http\Response.

    }