Assets - Code Metrics - bueltge/multisite-global-media - Measure and Improve Code Quality continuously with Scrutinizer

Assets A
last analyzed 2020-04-24 12:24 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Importance

Changes	4
Bugs	0	Features	0

Metric	Value
wmc	5
eloc	21
c	4
b	0
f	0
dl	0
loc	59
rs	10

3 Methods

Rating	Name	Size	Complexity
A	enqueueScripts()	15	2
A	enqueueStyles()	14	2
A	__construct()	3	1

<?php # -*- coding: utf-8 -*-
declare(strict_types=1);

namespace MultisiteGlobalMedia;

/**
 * Class Assets
 */
class Assets
{

    /**
     * @var PluginProperties
     */
    private $pluginProperties;

    /**
     * Assets constructor
     *
     * @param PluginProperties $pluginProperties
     */
    public function __construct(PluginProperties $pluginProperties)
    {
        $this->pluginProperties = $pluginProperties;
    }

    /**
     * Enqueue script for media modal
     *
     * @since  2015-01-26
     */
    public function enqueueScripts()
    {
        if ('post' !== get_current_screen()->base) {
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
            return;
        }

        $scriptFile = '/assets/js/global-media.js';
        wp_register_script(
            'global_media',
            $this->pluginProperties->dirUrl() . $scriptFile,
            ['media-views'],
            filemtime($this->pluginProperties->dirPath() . $scriptFile),
            true
        );
        wp_enqueue_script('global_media');
    }

    /**
     * Enqueue script for media modal
     *
     * @since   2015-02-27
     */
    public function enqueueStyles()
    {
        if ('post' !== get_current_screen()->base) {
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
            return;
        }

        $styleFile = '/assets/css/global-media.css';
        wp_register_style(
            'global_media',
            $this->pluginProperties->dirUrl() . $styleFile,
            [],
            filemtime($this->pluginProperties->dirPath() . $styleFile)
        );
        wp_enqueue_style('global_media');
    }
}


1			<?php # -- coding: utf-8 --
2			declare(strict_types=1);
3
4			namespace MultisiteGlobalMedia;
5
6			/**
7			* Class Assets
8			*/
9			class Assets
10			{
11
12			/**
13			* @var PluginProperties
14			*/
15			private $pluginProperties;
16
17			/**
18			* Assets constructor
19			*
20			* @param PluginProperties $pluginProperties
21			*/
22			public function __construct(PluginProperties $pluginProperties)
23			{
24			$this->pluginProperties = $pluginProperties;
25			}
26
27			/**
28			* Enqueue script for media modal
29			*
30			* @since 2015-01-26
31			*/
32			public function enqueueScripts()
33			{
34			if ('post' !== get_current_screen()->base) {
			0 ignored issues – show Bug introduced 2020-01-02 16:30 UTC by Report Bug Copy Issue Report Are you sure the usage of `get_current_screen()` is correct as it seems to always return `null`. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
35			return;
36			}
37
38			$scriptFile = '/assets/js/global-media.js';
39			wp_register_script(
40			'global_media',
41			$this->pluginProperties->dirUrl() . $scriptFile,
42			['media-views'],
43			filemtime($this->pluginProperties->dirPath() . $scriptFile),
44			true
45			);
46			wp_enqueue_script('global_media');
47			}
48
49			/**
50			* Enqueue script for media modal
51			*
52			* @since 2015-02-27
53			*/
54			public function enqueueStyles()
55			{
56			if ('post' !== get_current_screen()->base) {
			0 ignored issues – show Bug introduced 2020-01-02 16:30 UTC by Report Bug Copy Issue Report Are you sure the usage of `get_current_screen()` is correct as it seems to always return `null`. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
57			return;
58			}
59
60			$styleFile = '/assets/css/global-media.css';
61			wp_register_style(
62			'global_media',
63			$this->pluginProperties->dirUrl() . $styleFile,
64			[],
65			filemtime($this->pluginProperties->dirPath() . $styleFile)
66			);
67			wp_enqueue_style('global_media');
68			}
69			}
70

bueltge / multisite-global-media

Assets A last analyzed 2020-04-24 12:24 UTC

Complexity

Size/Duplication

Importance

3 Methods

Duplication Side-by-Side

Filter issues like

Assets A
last analyzed 2020-04-24 12:24 UTC