EventLogger::getMetaFields() - Code Metrics - Inspection of "Merge pull request #21 from understand/2.0" - understand/understand-laravel5 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 925ecf...691399 )

by Aivis

created 2018-02-06 16:31 UTC

EventLogger::getMetaFields() A

↳ Parent: EventLogger

Complexity

Conditions	1
Paths	1

Size

Total Lines	12
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	12
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	9
nc	1
nop	0

<?php namespace Understand\UnderstandLaravel5;

use Illuminate\Config\Repository;

class EventLogger
{

    /**
     * Log writer
     *
     * @var Logger
     */
    protected $logger;

    /**
     * Configuration
     *
     * @var array
     */
    protected $config;

    /**
     * @param Logger $logger
     * @param Repository $config
     */
    public function __construct(
        Logger $logger,
        Repository $config
    )
    {
        $this->logger = $logger;
        $this->config = $config;

    }

    /**
     * @param $level
     * @param $message
     * @param $context
     */
    public function logEvent($level, $message, $context)
    {
        // integer, float, string or boolean as message
        if (is_scalar($message))
        {
            $log = [
                'message' => $message
            ];
        }
        else
        {
            $log = (array)$message;
        }

        $log['tags'] = ['laravel_log'];
        $log['level'] = $level;

        if ($context)
        {
            $log['context'] = (array)$context;
        }

        $additionalFields = $this->getMetaFields();
        $customFields = $this->config->get('understand-laravel.events.meta', []);


        $this->logger->log($log, $additionalFields, $customFields);
    }

    /**
     * @return array
     */
    protected function getMetaFields()
    {
        return [
            'session_id' => 'UnderstandFieldProvider::getSessionId',
            'request_id' => 'UnderstandFieldProvider::getProcessIdentifier',
            'user_id' => 'UnderstandFieldProvider::getUserId',
            'env' => 'UnderstandFieldProvider::getEnvironment',
            'url' => 'UnderstandFieldProvider::getUrl',
            'method' => 'UnderstandFieldProvider::getRequestMethod',
            'client_ip' => 'UnderstandFieldProvider::getClientIp',
        ];
    }
}

1			<?php namespace Understand\UnderstandLaravel5;
2
3			use Illuminate\Config\Repository;
4
5			class EventLogger
6			{
7
8			/**
9			* Log writer
10			*
11			* @var Logger
12			*/
13			protected $logger;
14
15			/**
16			* Configuration
17			*
18			* @var array
19			*/
20			protected $config;
21
22			/**
23			* @param Logger $logger
24			* @param Repository $config
25			*/
26			public function __construct(
27			Logger $logger,
28			Repository $config
29			)
30			{
31			$this->logger = $logger;
32			$this->config = $config;
			0 ignored issues – show Documentation Bug introduced 2016-01-13 16:23 UTC by Report Bug Copy Issue Report It seems like `$config` of type `object<Illuminate\Config\Repository>` is incompatible with the declared type `array` of property `$config`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
33			}
34
35			/**
36			* @param $level
37			* @param $message
38			* @param $context
39			*/
40			public function logEvent($level, $message, $context)
41			{
42			// integer, float, string or boolean as message
43			if (is_scalar($message))
44			{
45			$log = [
46			'message' => $message
47			];
48			}
49			else
50			{
51			$log = (array)$message;
52			}
53
54			$log['tags'] = ['laravel_log'];
55			$log['level'] = $level;
56
57			if ($context)
58			{
59			$log['context'] = (array)$context;
60			}
61
62			$additionalFields = $this->getMetaFields();
63			$customFields = $this->config->get('understand-laravel.events.meta', []);
			0 ignored issues – show Bug introduced 2018-02-06 16:33 UTC by Report Bug Copy Issue Report The method `get` cannot be called on `$this->config` (of type `array`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
64
65			$this->logger->log($log, $additionalFields, $customFields);
66			}
67
68			/**
69			* @return array
70			*/
71			protected function getMetaFields()
72			{
73			return [
74			'session_id' => 'UnderstandFieldProvider::getSessionId',
75			'request_id' => 'UnderstandFieldProvider::getProcessIdentifier',
76			'user_id' => 'UnderstandFieldProvider::getUserId',
77			'env' => 'UnderstandFieldProvider::getEnvironment',
78			'url' => 'UnderstandFieldProvider::getUrl',
79			'method' => 'UnderstandFieldProvider::getRequestMethod',
80			'client_ip' => 'UnderstandFieldProvider::getClientIp',
81			];
82			}
83			}

understand / understand-laravel5

Push — master ( 925ecf...691399 )

EventLogger::getMetaFields() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like