cssParser::parseColor() - Code Metrics - XoopsModules25x/apcal - Measure and Improve Code Quality continuously with Scrutinizer

cssParser::parseColor() A
last analyzed 2020-02-19 10:05 UTC

↳ Parent: cssParser

Complexity

Conditions	5
Paths	6

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
nc	6
nop	2
dl	0
loc	13
rs	9.5222
c	0
b	0
f	0

<?php
/*
 * You may not change or alter any portion of this comment or credits
 * of supporting developers from this source code or any supporting source code
 * which is considered copyrighted (c) material of the original comment or credit authors.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
 */

/**
 * @copyright   {@link http://xoops.org/ XOOPS Project}
 * @license     {@link http://www.fsf.org/copyleft/gpl.html GNU public license}
 * @package
 * @since
 * @author       XOOPS Development Team,
 * @author       GIJ=CHECKMATE (PEAK Corp. http://www.peak.ne.jp/)
 * @author       Antiques Promotion (http://www.antiquespromotion.ca)
 */

class cssParser
{
    public $_css = '';

    /**
     * cssParser constructor.
     * @param string $filename
     */
    public function __construct($filename = 'style.css')
    {
        global $xoopsConfig;

        $this->_css = file_get_contents(XOOPS_THEME_PATH . '/' . $xoopsConfig['theme_set'] . '/' . $filename);
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
    }

    /**
     * @param $selector
     * @param $style
     * @return bool|string
     */
    public function parseColor($selector, $style)
    {
        $match = array();

        if (preg_match('/[\.\S]*' . $selector . '([^a-zA-Z0-9]{1}[\.\S]*{|{)([\s\S][^{}]+)(})/', $this->_css, $match)) {
            preg_match('/[^-]*' . $style . '([^;]*):([^;]*)#([a-zA-Z0-9]+)/', $match[2], $match);
        }
        if (is_array($match) && array_key_exists(3, $match)) {
            return '#' . (strlen($match[3]) == 3 ? $match[3][0] . $match[3][0] . $match[3][1] . $match[3][1] . $match[3][2] . $match[3][2] : $match[3]);
        }

        return false;
    }
}


1			<?php
2			/*
3			* You may not change or alter any portion of this comment or credits
4			* of supporting developers from this source code or any supporting source code
5			* which is considered copyrighted (c) material of the original comment or credit authors.
6			*
7			* This program is distributed in the hope that it will be useful,
8			* but WITHOUT ANY WARRANTY; without even the implied warranty of
9			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
10			*/
11
12			/**
13			* @copyright {@link http://xoops.org/ XOOPS Project}
14			* @license {@link http://www.fsf.org/copyleft/gpl.html GNU public license}
15			* @package
16			* @since
17			* @author XOOPS Development Team,
18			* @author GIJ=CHECKMATE (PEAK Corp. http://www.peak.ne.jp/)
19			* @author Antiques Promotion (http://www.antiquespromotion.ca)
20			*/
21
22			class cssParser
23			{
24			public $_css = '';
25
26			/**
27			* cssParser constructor.
28			* @param string $filename
29			*/
30			public function __construct($filename = 'style.css')
31			{
32			global $xoopsConfig;
33
34			$this->_css = file_get_contents(XOOPS_THEME_PATH . '/' . $xoopsConfig['theme_set'] . '/' . $filename);
			0 ignored issues – show Documentation Bug introduced 2017-03-23 08:16 UTC by Report Bug Copy Issue Report It seems like `file_get_contents(XOOPS_...et'] . '/' . $filename)` can also be of type `boolean`. However, the property `$_css` is declared as type `string`. Maybe add an additional type check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly. For example, imagine you have a variable `$accountId` that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to the `id` property of an instance of the `Account` class. This class holds a proper account, so the id value must no longer be false. Either this assignment is in error or a type check should be added for that assignment. class Id { public $id; public function __construct($id) { $this->id = $id; } } class Account { /** @var Id $id */ public $id; } $account_id = false; if (starsAreRight()) { $account_id = new Id(42); } $account = new Account(); if ($account instanceof Id) { $account->id = $account_id; } Loading history...
35			}
36
37			/**
38			* @param $selector
39			* @param $style
40			* @return bool\|string
41			*/
42			public function parseColor($selector, $style)
43			{
44			$match = array();
45
46			if (preg_match('/[\.\S]' . $selector . '([^a-zA-Z0-9]{1}[\.\S]{\|{)([\s\S][^{}]+)(})/', $this->_css, $match)) {
47			preg_match('/[^-]' . $style . '([^;]):([^;]*)#([a-zA-Z0-9]+)/', $match[2], $match);
48			}
49			if (is_array($match) && array_key_exists(3, $match)) {
50			return '#' . (strlen($match[3]) == 3 ? $match[3][0] . $match[3][0] . $match[3][1] . $match[3][1] . $match[3][2] . $match[3][2] : $match[3]);
51			}
52
53			return false;
54			}
55			}
56

XoopsModules25x / apcal

cssParser::parseColor() A last analyzed 2020-02-19 10:05 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

cssParser::parseColor() A
last analyzed 2020-02-19 10:05 UTC