App\Http\Controllers\Customers\BreakCustomerLinkController

Complexity

Total Complexity

1

Size/Duplication

Total Lines	12
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	7	1

<?php

namespace App\Http\Controllers\Customers;

use App\Models\Customer;
use App\Http\Controllers\Controller;

use Illuminate\Support\Facades\Log;
use Illuminate\Support\Facades\Auth;

class BreakCustomerLinkController extends Controller
{
    /**
     *  Break the link between the customer and its parent
     */
    public function __invoke($cust_id)
    {
        $this->authorize('manage', Customer::class);
        Customer::findOrFail($cust_id)->update(['parent_id' => null]);

        Log::channel('cust')->notice('Customer Link broken for Customer ID '.$cust_id.' by '.Auth::user()->username);

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

        return back()->with(['message' => 'Customer Link Broken', 'type' => 'warning']);
    }
}