GetCustomerNotes::getParentNotes() - Code Metrics - Inspection of "Updated Customer Notes" - butcherman/Tech_Bench - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — dev5 ( 5844c4...03a57e )

by Ron

created 2020-04-21 17:56 UTC

GetCustomerNotes::getParentNotes() A

↳ Parent: GetCustomerNotes

Complexity

Conditions	1
Paths	1

Size

Total Lines	6
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	3
c	1
b	0
f	0
nc	1
nop	1
dl	0
loc	6
ccs	4
cts	4
cp	1
crap	1
rs	10

<?php

namespace App\Domains\Customers;

use Illuminate\Support\Facades\Log;

use App\Customers;
use App\CustomerNotes;

class GetCustomerNotes
{
    protected $custID;

    public function __construct($custID)
    {
        $this->custID = $custID;
    }

    //  Get all notes that are assigned to the customer
    public function execute()
    {
        $hasParent = Customers::findOrFail($this->custID)->parent_id;
        $notes = $this->getLocalNotes();

        if($hasParent)
        {
            $parentNotes = $this->getParentNotes($hasParent);
            return $notes->merge($parentNotes);
        }
        return $notes;
    }

    //  Retrieve any notes attached to the customer
    protected function getLocalNotes()
    {
        $notes = CustomerNotes::where('cust_id', $this->custID)->orderBy('urgent', 'desc')->get();

        Log::debug('Customer Notes Query completed for customer ID '.$this->custID.'.  Results - ', array($notes));
        return $notes;
    }

    //  Retrieve any notes attached to the parent customer
    protected function getParentNotes($parentID)
    {
        $parentNotes = CustomerNotes::where('cust_id', $parentID)->where('shared', 1)->orderBy('urgent', 'desc')->get();

        Log::debug('Customer Parent Notes Query completed for Customer ID '.$this->custID.'.  Results - ', array($parentNotes));
        return $parentNotes;
    }
}


1		<?php
2
3		namespace App\Domains\Customers;
4
5		use Illuminate\Support\Facades\Log;
6
7		use App\Customers;
8		use App\CustomerNotes;
9
10		class GetCustomerNotes
11		{
12		protected $custID;
13
14	4	public function __construct($custID)
15		{
16	4	$this->custID = $custID;
17	4	}
18
19		// Get all notes that are assigned to the customer
20	4	public function execute()
21		{
22	4	$hasParent = Customers::findOrFail($this->custID)->parent_id;
23	4	$notes = $this->getLocalNotes();
24
25	4	if($hasParent)
26		{
27	2	$parentNotes = $this->getParentNotes($hasParent);
28	2	return $notes->merge($parentNotes);
29		}
30	2	return $notes;
31		}
32
33		// Retrieve any notes attached to the customer
34	4	protected function getLocalNotes()
35		{
36	4	$notes = CustomerNotes::where('cust_id', $this->custID)->orderBy('urgent', 'desc')->get();
37
38	4	Log::debug('Customer Notes Query completed for customer ID '.$this->custID.'. Results - ', array($notes));
39	4	return $notes;
40		}
41
42		// Retrieve any notes attached to the parent customer
43	2	protected function getParentNotes($parentID)
44		{
45	2	$parentNotes = CustomerNotes::where('cust_id', $parentID)->where('shared', 1)->orderBy('urgent', 'desc')->get();
46
47	2	Log::debug('Customer Parent Notes Query completed for Customer ID '.$this->custID.'. Results - ', array($parentNotes));
48	2	return $parentNotes;
49		}
50		}
51

butcherman / Tech_Bench

Push — dev5 ( 5844c4...03a57e )

GetCustomerNotes::getParentNotes() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like