Language::load() - Code Metrics - Inspection of "Sync with 2.5.8 changes" - XOOPS/XoopsCore - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#470)

by Richard

created 2016-03-22 02:39 UTC

Language::load() B

↳ Parent: Language

Complexity

Conditions	6
Paths	24

Size

Total Lines	17
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	6.288

Importance

Changes	1
Bugs	1	Features	0

Metric	Value
c	1
b	1
f	0
dl	0
loc	17
rs	8.8571
ccs	8
cts	10
cp	0.8
cc	6
eloc	11
nc	24
nop	3
crap	6.288

<?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.
 */

namespace Xmf;

/**
 * Language
 *
 * @category  Xmf\Language
 * @package   Xmf
 * @author    trabis <[email protected]>
 * @copyright 2011-2016 XOOPS Project (http://xoops.org)
 * @license   GNU GPL 2 or later (http://www.gnu.org/licenses/gpl-2.0.html)
 * @version   Release: 1.0
 * @link      http://xoops.org
 * @since     1.0
 */
class Language
{
    /**
     * Attempt a translation of a simple string
     *
     * @param string $string string to translate
     * @param string $domain language domain
     *
     * @return string translated string
     *
     * @todo do something useful
     */
    public static function translate($string, $domain = null)

    {
        return $string;
    }

    /**
     * load - load a language file
     *
     * @param string $name     name of the language file
     * @param string $domain   domain or module supplying language file
     * @param string $language language folder name
     *
     * @return bool true if loaded, otherwise false
     */
    public static function load($name, $domain = '', $language = null)
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

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

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        if (empty($language)) {
            if (!empty($GLOBALS['xoopsConfig']['language'])) {
                $language = $GLOBALS['xoopsConfig']['language'];
            } else {
                $language = 'english';
            }
        }
        $path = \XoopsBaseConfig::get('root-path') . '/' . ((empty($domain) || 'global' === $domain) ? ''
            : "modules/{$domain}/") . 'language';
        if (!$ret = static::loadFile("{$path}/{$language}/{$name}.php")) {
            $ret = static::loadFile("{$path}/english/{$name}.php");
        }

        return $ret;
    }

    /**
     * Load a file
     *
     * @param string $filename filename to load
     *
     * @return bool true if file exists and was loaded
     *
     * @throws \InvalidArgumentException
     */
    protected static function loadFile($filename)
    {
        if (preg_match('/[^a-z0-9\\/\\\\_.:-]/i', $filename)) {
            throw new \InvalidArgumentException('Security check: Illegal character in filename');
        }
        if (file_exists($filename)) {
            include_once $filename;
            return true;
        }
        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		namespace Xmf;
13
14		/**
15		* Language
16		*
17		* @category Xmf\Language
18		* @package Xmf
19		* @author trabis <[email protected]>
20		* @copyright 2011-2016 XOOPS Project (http://xoops.org)
21		* @license GNU GPL 2 or later (http://www.gnu.org/licenses/gpl-2.0.html)
22		* @version Release: 1.0
23		* @link http://xoops.org
24		* @since 1.0
25		*/
26		class Language
27		{
28		/**
29		* Attempt a translation of a simple string
30		*
31		* @param string $string string to translate
32		* @param string $domain language domain
33		*
34		* @return string translated string
35		*
36		* @todo do something useful
37		*/
38	1	public static function translate($string, $domain = null)
		0 ignored issues – show Unused Code introduced 2015-12-12 03:50 UTC by Report Bug Copy Issue Report The parameter `$domain` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
39		{
40	1	return $string;
41		}
42
43		/**
44		* load - load a language file
45		*
46		* @param string $name name of the language file
47		* @param string $domain domain or module supplying language file
48		* @param string $language language folder name
49		*
50		* @return bool true if loaded, otherwise false
51		*/
52	1	public static function load($name, $domain = '', $language = null)
		0 ignored issues – show Coding Style introduced 2015-12-12 03:50 UTC by Report Bug Copy Issue Report `load` uses the super-global variable `$GLOBALS` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
53		{
54	1	if (empty($language)) {
55	1	if (!empty($GLOBALS['xoopsConfig']['language'])) {
56		$language = $GLOBALS['xoopsConfig']['language'];
57		} else {
58	1	$language = 'english';
59		}
60		}
61	1	$path = \XoopsBaseConfig::get('root-path') . '/' . ((empty($domain) \|\| 'global' === $domain) ? ''
62	1	: "modules/{$domain}/") . 'language';
63	1	if (!$ret = static::loadFile("{$path}/{$language}/{$name}.php")) {
64		$ret = static::loadFile("{$path}/english/{$name}.php");
65		}
66
67	1	return $ret;
68		}
69
70		/**
71		* Load a file
72		*
73		* @param string $filename filename to load
74		*
75		* @return bool true if file exists and was loaded
76		*
77		* @throws \InvalidArgumentException
78		*/
79		protected static function loadFile($filename)
80		{
81		if (preg_match('/[^a-z0-9\\/\\\\_.:-]/i', $filename)) {
82		throw new \InvalidArgumentException('Security check: Illegal character in filename');
83		}
84		if (file_exists($filename)) {
85		include_once $filename;
86		return true;
87		}
88		return false;
89		}
90		}
91

XOOPS / XoopsCore

Pull Request — master (#470)

Language::load() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like