Tylercd100\LERN\Components\Recorder

Complexity

Total Complexity

20

Size/Duplication

Total Lines	116
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

86.67%

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	record()	0	19	1
A	canCollect()	0	6	4
A	getUserId()	0	8	3
A	getMethod()	0	8	3
A	getData()	0	8	3
A	getUrl()	0	8	3
A	getStatusCode()	0	7	2

<?php

namespace Tylercd100\LERN\Components;

use Exception;
use Tylercd100\LERN\Models\ExceptionModel;
use Symfony\Component\HttpKernel\Exception\HttpExceptionInterface;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\Request;
use Illuminate\Support\Facades\Input;

class Recorder {

    /**
     * @var array
     */
    protected $config = [];

    /**
     * The constructor
     */
    public function __construct(){
        $this->config = config('lern.record');

It seems like config('lern.record') of type * is incompatible with the declared type array of property $config.

    }

    /**
     * Records an Exception to the database
     * @param  Exception $e The exception you want to record
     * @return Tylercd100\LERN\Models\ExceptionModel
     */
    public function record(Exception $e)
    {
        $opts = [
            'class'       => get_class($e),
            'file'        => $e->getFile(),
            'line'        => $e->getLine(),
            'code'        => $e->getCode(),
            'message'     => $e->getMessage(),
            'trace'       => $e->getTraceAsString(),
        ];

        $opts['status_code'] = $this->getStatusCode($e);
        $opts['user_id'] = $this->getUserId($e);

The call to Recorder::getUserId() has too many arguments starting with $e.

        $opts['method'] = $this->getMethod($e);

The call to Recorder::getMethod() has too many arguments starting with $e.

        $opts['data'] = $this->getData($e);

The call to Recorder::getData() has too many arguments starting with $e.

        $opts['url'] = $this->getUrl($e);

The call to Recorder::getUrl() has too many arguments starting with $e.

        return ExceptionModel::create($opts);
    }

    /**
     * Checks the config to see if you can collect certain information
     * @param  string $type the config value you want to check
     * @return boolean      
     */
    private function canCollect($type){
        if(!empty($this->config) && !empty($this->config['collect']) && !empty($this->config['collect'][$type])){
            return $this->config['collect'][$type] === true;
        }
        return false;
    }

    /**
     * Gets the ID of the User that is logged in
     * @return integer|null The ID of the User or Null if not logged in
     */
    protected function getUserId(){
        $user = Auth::user();
        if($this->canCollect('user_id') && is_object($user)) {
            return $user->id;
        } else {
            return null;
        }
    }

    /**
     * Gets the Method of the Request
     * @return string|null Possible values are null or GET, POST, DELETE, PUT, etc...
     */
    protected function getMethod(){
        $method = Request::method();
        if($this->canCollect('method') && !empty($method)) {
            return $method;
        } else {
            return null;
        }
    }

    /**
     * Gets the input data of the Request
     * @return array|null The Input data or null
     */
    protected function getData(){
        $data = Input::all();
        if($this->canCollect('data') && is_array($data)) {
            return $data;
        } else {
            return null;
        }
    }

    /**
     * Gets the URL of the Request
     * @return string|null Returns a URL string or null
     */
    protected function getUrl(){
        $url = Request::url();
        if($this->canCollect('url') && is_array($url)) {
            return $url;
        } else {
            return null;
        }
    }

    /**
     * Gets the status code of the Exception
     * @param  Exception $e The Exception to check
     * @return string|integer The status code value
     */
    protected function getStatusCode(Exception $e){
        if ($e instanceof HttpExceptionInterface) {
            return $e->getStatusCode();
        } else {
            return 0;
        }
    }
}