JsProvider::getJsConfig() - Code Metrics - Inspection of "Feature/js config" - understand/understand-laravel5 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#39)

unknown

created 2019-07-11 13:57 UTC

JsProvider::getJsConfig() A

↳ Parent: JsProvider

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	13
rs	9.8333
c	0
b	0
f	0
cc	1
nc	1
nop	0

<?php namespace Understand\UnderstandLaravel5;

use Illuminate\Foundation\Application;
use Understand\UnderstandLaravel5\Facades\UnderstandFieldProvider;

class JsProvider
{
    /**
     * @var Application
     */
    protected $app;

    /**
     * @return void

     */
    public function __construct(Application $app)
    {
        $this->app = $app;
    }

    /**
     * Get the Url of the JS bundle
     *
     * @return string
     */
    public function getJsBundleUrl()
    {
        return $this->app['config']->get('understand-laravel.cdn', 'https://cdn.understand.io/understand-js/beta/bundle.min.js');
    }

    /**
     * Get the JS configuration
     *
     * @return array
     */
    public function getJsConfig()
    {
        return [
            'env' => UnderstandFieldProvider::getEnvironment(),
            'token' => $this->app['config']->get('understand-laravel.token'),
            'context' => [
                'session_id' => UnderstandFieldProvider::getSessionId(),
                'request_id' => UnderstandFieldProvider::getProcessIdentifier(),
                'user_id' => UnderstandFieldProvider::getUserId(),
                'client_ip' => UnderstandFieldProvider::getClientIp()
            ]
        ];
    }
}

1			<?php namespace Understand\UnderstandLaravel5;
2
3			use Illuminate\Foundation\Application;
4			use Understand\UnderstandLaravel5\Facades\UnderstandFieldProvider;
5
6			class JsProvider
7			{
8			/**
9			* @var Application
10			*/
11			protected $app;
12
13			/**
14			* @return void
			0 ignored issues – show Comprehensibility Best Practice introduced 2019-07-11 13:59 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
15			*/
16			public function __construct(Application $app)
17			{
18			$this->app = $app;
19			}
20
21			/**
22			* Get the Url of the JS bundle
23			*
24			* @return string
25			*/
26			public function getJsBundleUrl()
27			{
28			return $this->app['config']->get('understand-laravel.cdn', 'https://cdn.understand.io/understand-js/beta/bundle.min.js');
29			}
30
31			/**
32			* Get the JS configuration
33			*
34			* @return array
35			*/
36			public function getJsConfig()
37			{
38			return [
39			'env' => UnderstandFieldProvider::getEnvironment(),
40			'token' => $this->app['config']->get('understand-laravel.token'),
41			'context' => [
42			'session_id' => UnderstandFieldProvider::getSessionId(),
43			'request_id' => UnderstandFieldProvider::getProcessIdentifier(),
44			'user_id' => UnderstandFieldProvider::getUserId(),
45			'client_ip' => UnderstandFieldProvider::getClientIp()
46			]
47			];
48			}
49			}

understand / understand-laravel5

Pull Request — master (#39)

JsProvider::getJsConfig() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like