LinkCustomerController::removeParent() - Code Metrics - Inspection of "Updated Manage Customer" - butcherman/Tech_Bench - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — dev6 ( 3fa4bc...200ba5 )

by Ron

created 2021-09-05 21:08 UTC

LinkCustomerController::removeParent() A

↳ Parent: LinkCustomerController

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	5
c	1
b	0
f	0
dl	0
loc	8
ccs	0
cts	0
cp	0
rs	10
cc	1
nc	1
nop	1
crap	2

<?php

namespace App\Http\Controllers\Customers;

use App\Models\Customer;
use App\Events\CustomerLinkedEvent;
use App\Http\Controllers\Controller;
use App\Http\Requests\Customers\LinkCustomerRequest;

class LinkCustomerController extends Controller
{
    /**
     * Link or Unlink the customer to a parent customer
     */
    public function __invoke(LinkCustomerRequest $request)
    {
        if($request->add)
        {
            $msg = $this->addParent($request->cust_id, $request->parent_id);
        }
        else
        {
            $msg = $this->removeParent($request->cust_id);
        }

        return back()->with($msg);
    }

    /**
     * Add a link for multi-site customer
     */
    protected function addParent($cust, $parent)
    {
        //  Determine if the selected Parent ID has a parent ID of its own
        $p = Customer::findOrFail($parent);
        if($p->parent_id)
        {
            $parent = $p->parent_id;
        }

        Customer::find($cust)->update([
            'parent_id' => $parent,
        ]);

        event(new CustomerLinkedEvent($cust, true));
        return [
            'message' => 'Customer successfully linked',
            'type'    => 'success',
        ];
    }

    /**
     * Remove a multi-site link
     */
    protected function removeParent($cust)
    {
        Customer::findOrFail($cust)->update(['parent_id' => null]);

        event(new CustomerLinkedEvent($cust, false));
        return [
            'message' => 'Customer link removed',
            'type'    => 'warning',
        ];
    }
}


1		<?php
2
3		namespace App\Http\Controllers\Customers;
4
5		use App\Models\Customer;
6		use App\Events\CustomerLinkedEvent;
7		use App\Http\Controllers\Controller;
8		use App\Http\Requests\Customers\LinkCustomerRequest;
9
10		class LinkCustomerController extends Controller
11		{
12		/**
13		* Link or Unlink the customer to a parent customer
14		*/
15		public function __invoke(LinkCustomerRequest $request)
16	3	{
17		if($request->add)
18	3	{
19		$msg = $this->addParent($request->cust_id, $request->parent_id);
20		}
21	2	else
22	2	{
23		$msg = $this->removeParent($request->cust_id);
24	1	}
25
26		return back()->with($msg);
27	2	}
28	2
29		/**
30	2	* Add a link for multi-site customer
31		*/
32	2	protected function addParent($cust, $parent)
33		{
34		// Determine if the selected Parent ID has a parent ID of its own
35		$p = Customer::findOrFail($parent);
36		if($p->parent_id)
37		{
38		$parent = $p->parent_id;
39		}
40
41		Customer::find($cust)->update([
42		'parent_id' => $parent,
43		]);
44
45		event(new CustomerLinkedEvent($cust, true));
46		return [
47		'message' => 'Customer successfully linked',
48		'type' => 'success',
49		];
50		}
51
52		/**
53		* Remove a multi-site link
54		*/
55		protected function removeParent($cust)
56		{
57		Customer::findOrFail($cust)->update(['parent_id' => null]);
58
59		event(new CustomerLinkedEvent($cust, false));
60		return [
61		'message' => 'Customer link removed',
62		'type' => 'warning',
63		];
64		}
65		}
66

butcherman / Tech_Bench

Push — dev6 ( 3fa4bc...200ba5 )

LinkCustomerController::removeParent() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like