CustomerDetailsController::__construct() - Code Metrics - Inspection of "Updated Testing and Documentation for Customers" - butcherman/Tech_Bench - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — dev5 ( 3534c9...5191cd )

by Ron

created 2019-11-15 02:17 UTC

CustomerDetailsController::__construct() A

↳ Parent: CustomerDetailsController

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

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

<?php

namespace App\Http\Controllers\Customers;

use App\Customers;
use App\CustomerFavs;
use App\PhoneNumberType;
use App\CustomerFileTypes;
use Illuminate\Http\Request;
use App\Http\Traits\SystemsTrait;
use Illuminate\Support\Facades\Log;
use Illuminate\Support\Facades\Auth;
use App\Http\Controllers\Controller;
use Illuminate\Support\Facades\Route;

class CustomerDetailsController extends Controller
{
    use SystemsTrait;


    public function __construct()
    {
        $this->middleware('auth');
    }

    //  New Customer Form
    public function create()
    {
        $this->authorize('hasAccess', 'add_customer');

        Log::debug('Route '.Route::currentRouteName().' visited by User ID-'.Auth::user()->user_id);
        return view('customer.newCustomer');
    }

    //  Submit the new customer form
    public function store(Request $request)
    {
        $this->authorize('hasAccess', 'add_customer');

        $request->validate([
            'cust_id'  => 'required|numeric|unique:customers,cust_id',
            'name'     => 'required', // |unique:customers,name',
            'dba_name' => 'nullable',
            'address'  => 'required',
            'city'     => 'required',
            'zip'      => 'required|numeric'
        ]);

        //  Remove any forward slash (/) from the Customer name field
        $request->merge(['name' => str_replace('/', '-', $request->name)]);

        Customers::create([
            'cust_id'  => $request->cust_id,
            'name'     => $request->name,
            'dba_name' => $request->dba_name,
            'address'  => $request->address,
            'city'     => $request->city,
            'state'    => $request->selectedState,
            'zip'      => $request->zip,
        ]);

        Log::debug('Route '.Route::currentRouteName().' visited by User ID-'.Auth::user()->user_id);
        Log::notice('New Customer ID-'.$request->custID.' created by User ID-'.Auth::user()->user_id);

        return response()->json(['success' => true ]);
    }







    //  Show the customer details
    public function details($id, $name)
    public function details(/** @scrutinizer ignore-unused */ $id, $name)
    {
        echo 'customer details';
        die();



        $custDetails = Customers::find($id);
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
        $allSystems  = $this->getAllSystems();

        Log::debug('Route '.Route::currentRouteName().' visited by User ID-'.Auth::user()->user_id);
        if(empty($custDetails))
        {
            Log::info('User ID-'.Auth::user()->user_id.' visited invalid customer ID-'.$id.'-'.$name);
            return view('err.customerNotFound');
        }

        //  Determine if the customer is one of the users bookmarks
        $custFav = CustomerFavs::where('user_id', Auth::user()->user_id)->where('cust_id', $custDetails->cust_id)->first();
        //  Get the types of phone numbers that can be assigned to a customer contact
        $pTypes = PhoneNumberType::all();
        $phoneTypes = [];
        foreach($pTypes as $type)
        {
            $phoneTypes[] = [
                'value' => $type->phone_type_id,
                'text'  => $type->description,
                'icon'  => $type->icon_class
            ];
        }

        //  Get the types of files that can be attached to a file
        $fileTypes = CustomerFileTypes::select('file_type_id as value', 'description as text')->get();

        Log::debug('Customer Details', $custDetails->toArray());
        return view('customer.details', [
            'details'    => $custDetails,
            'isFav'      => empty($custFav) ? false : true,
            'sysList'    => $allSystems,
            'phoneTypes' => $phoneTypes,
            'fileTypes'  => $fileTypes
        ]);
    }

    //  Get the basic details of the customer
    public function show($id)
    {
        $details = Customers::find($id);

        Log::debug('Route '.Route::currentRouteName().' visited by User ID-'.Auth::user()->user_id);
        if(empty($details))
        {
            Log::info('User ID-'.Auth::user()->user_id.' visited invalid customer ID-'.$id);
            return response()->json(['error' => 'Customer Not Found']);
        }

        Log::debug('Customer Details', $details->toArray());
        return response()->json($details);
    }

    //  Update the customer details
    public function update(Request $request, $id)
    {
        $request->validate([
            'name'     => 'required',
            'dba_name' => 'nullable',
            'address'  => 'required',
            'city'     => 'required',
            'state'    => 'required',
            'zip'      => 'required|numeric'
        ]);

        Customers::find($id)->update([
            'name'     => $request->name,
            'dba_name' => $request->dba_name,
            'address'  => $request->address,
            'city'     => $request->city,
            'state'    => $request->state,
            'zip'      => $request->zip
        ]);

        Log::debug('Route '.Route::currentRouteName().' visited by User ID-'.Auth::user()->user_id);
        Log::notice('Customer Details Updated for Customer ID-'.$id.' by User ID-'.Auth::user()->user_id);
        Log::debug('Customer Details Submitted - ', $request->toArray());
        return response()->json([
            'success' => true
        ]);
    }

    /**
     * Remove the specified resource from storage.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function destroy($id)
    public function destroy(/** @scrutinizer ignore-unused */ $id)
    {
        //
    }
}


butcherman / Tech_Bench

Push — dev5 ( 3534c9...5191cd )

CustomerDetailsController::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like