Userlog - Code Metrics - Inspection of "Merge pull request #6 from mambax7/master" - XoopsModules25x/userlog - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 507df7...d00737 )

by Michael

created 2017-09-05 10:59 UTC

Userlog C

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	343
Duplicated Lines	0 %

Coupling/Cohesion

Components	6
Dependencies	1

Importance

Changes

Metric	Value
dl	0
loc	343
rs	5.5555
c	0
b	0
f	0
wmc	56
lcom	6
cbo	1

20 Methods

Rating	Name	Size	Complexity
A	__construct()	6	2
A	getInstance()	9	2
A	getLogModule()	8	2
B	getModules()	26	6
A	getUser()	8	2
A	getGroupList()	8	2
A	getBrowsCap()	8	2
A	setConfig()	10	2
A	getAllLogFiles()	17	3
A	getWorkingFile()	6	1
B	getFromKeys()	17	5
A	getSinceTime()	8	2
B	formatTime()	12	5
A	getCookie()	6	1
A	probCheck()	8	1
D	patchLoginHistory()	29	9
A	initLogModule()	12	3
A	initUser()	10	3
A	initGroupList()	7	1
A	initBrowsCap()	18	2

How to fix Complexity

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

/**
 *  userlog module
 *
 * @copyright       XOOPS Project (https://xoops.org)
 * @license         GNU GPL 2 (http://www.gnu.org/licenses/old-licenses/gpl-2.0.html)
 * @package         userlog class
 * @since           1
 * @author          irmtfan ([email protected])
 * @author          XOOPS Project <www.xoops.org> <www.xoops.ir>
 */

use Xmf\Request;

defined('XOOPS_ROOT_PATH') || exit('Restricted access.');
require_once __DIR__ . '/phpbrowscap/Browscap.php';

// The Browscap class is in the phpbrowscap namespace, so import it
use phpbrowscap\Browscap;

/**
 * Class Userlog
 */
class Userlog extends \Xmf\Module\Helper
namespace YourVendor;

class YourClass { }
{
    public $logmodule;
    public $user;
    public $debugArray   = [];
    public $logext       = 'log';
    public $cookiePrefix = '';
    public $groupList;
    public $browscap;

    /**
     * @param $debug
     */
    protected function __construct($debug)
    {
        $this->debug        = $debug;
        $this->dirname      = USERLOG_DIRNAME;
        $this->cookiePrefix = USERLOG_DIRNAME . '_' . ($this->getUser() ? $this->getUser()->getVar('uid') : '');
    }

    /**
     * @param bool $debug
     *
     * @return Userlog
     */
    public static function getInstance($debug = false)
    {
        static $instance;
        if (null === $instance) {
            $instance = new static($debug);
        }

        return $instance;
    }

    /**
     * @return null|\XoopsModule
     */
    public function getLogModule()
    {
        if (null === $this->logmodule) {
            $this->initLogModule();
        }

        return $this->logmodule;
    }

    /**
     * @param array $dirnames
     * @param null  $otherCriteria
     * @param bool  $asObj
     *
     * @return mixed
     */
    public function getModules($dirnames = [], $otherCriteria = null, $asObj = false)
    {
        // get all dirnames
        /** @var XoopsModuleHandler $moduleHandler */
        $moduleHandler = xoops_getHandler('module');
        $criteria      = new CriteriaCompo();
        if (count($dirnames) > 0) {
            foreach ($dirnames as $mDir) {
                $criteria->add(new Criteria('dirname', $mDir), 'OR');
            }
        }
        if (!empty($otherCriteria)) {
            $criteria->add($otherCriteria);
        }
        $criteria->add(new Criteria('isactive', 1), 'AND');
        $modules = $moduleHandler->getObjects($criteria, true);
        if ($asObj) {
            return $modules;
        }
        $dirNames['system-root'] = _YOURHOME;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
        foreach ($modules as $module) {
            $dirNames[$module->dirname()] = $module->name();
        }

        return $dirNames;
    }

    /**
     * @return null
     */
    public function getUser()
    {
        if (null === $this->user) {
            $this->initUser();
        }

        return $this->user;
    }

    /**
     * @return null|array
     */
    public function getGroupList()
    {
        if (null === $this->groupList) {
            $this->initGroupList();
        }

        return $this->groupList;
    }

    /**
     * @return null
     */
    public function getBrowsCap()
    {
        if (null === $this->browscap) {
            $this->initBrowsCap();
        }

        return $this->browscap;
    }


    /**
     * @param null $name
     * @param null $value
     *
     * @return mixed
     */
    public function setConfig($name = null, $value = null)
    {
        if (null === $this->configs) {
            $this->initConfig();
        }
        $this->configs[$name] = $value;
        $this->addLog("Setting config '{$name}' : " . $this->configs[$name]);

        return $this->configs[$name];
    }

    /**
     * @return array
     */
    public function getAllLogFiles()
    {
        $logPaths    = $this->object->getInfo('log_paths');
        $currentPath = $this->getConfig('logfilepath');
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $allFiles    = [];
        $totalFiles  = 0;
        foreach ($logPaths as $path) {
            $folderHandler                           = XoopsFile::getHandler('folder', $path . '/' . USERLOG_DIRNAME);
            $allFiles[$path . '/' . USERLOG_DIRNAME] = $folderHandler->find('.*' . $this->logext);
            $totalFiles                              += count($allFiles[$path . '/' . USERLOG_DIRNAME]);
        }
        if (empty($totalFiles)) {
            return [[], 0];
        }

        return [$allFiles, $totalFiles];
    }

    /**
     * @return string
     */
    public function getWorkingFile()
    {
        $logFileName = $this->getConfig('logfilepath') . '/' . USERLOG_DIRNAME . '/' . $this->getConfig('logfilename');

        return $logFileName . '.' . $this->logext;
    }

    /**
     * @param      $array
     * @param null $keys
     *
     * @return array
     */
    public function getFromKeys($array, $keys = null)
    {
        if (empty($keys)) {
            return $array;
        } // all keys
        $keyarr = is_string($keys) ? explode(',', $keys) : $keys;
        if (empty($keyarr[0])) {
            return $array;
        } // all keys
        $keyarr = array_intersect(array_keys($array), $keyarr); // keys should be in array
        $ret    = [];
        foreach ($keyarr as $key) {
            $ret[$key] = $array[$key];
        }

        return $ret;
    }

    /**
     * @param int $since
     *
     * @return int
     */
    public function getSinceTime($since = 1) // one day
    {
        if ($since > 0) {
            return (int)$since * 24 * 3600;
        }

        return (int)abs($since) * 3600;
    }

    /**
     * @param null   $intTime
     * @param string $dateFormat
     * @param null|string $timeoffset
     *
     * @return bool|string
     */
    public function formatTime($intTime = null, $dateFormat = 'c', $timeoffset = null)
    {
        if (empty($intTime)) {
            return false;
        }
        if ('custom' === $dateFormat || 'c' === $dateFormat) {
            $dateFormat = $this->getConfig('format_date');
        }
        xoops_load('XoopsLocal');

        return class_exists('XoopsLocal') ? XoopsLocal::formatTimestamp($intTime, $dateFormat, $timeoffset) : XoopsLocale::formatTimestamp($intTime, $dateFormat, $timeoffset); // use XoopsLocale in xoops26
    }

    /**
     * @param string $name
     *
     * @return array
     */
    public function getCookie($name = 'TOGGLE')
    {
        $toggles = Request::getString($this->cookiePrefix . $name, null, 'cookie');

        return explode(',', $toggles);
    }

    /**
     * @param int $prob
     *
     * @return bool
     */
    public function probCheck($prob = 11)
    {
        mt_srand((double)microtime() * 1000000);
        // check probabillity 11 means 10%, 100 means 100%
        $ret = mt_rand(1, 100) > $prob;

        return $ret;
    }

    /**
     * @param null $post
     * @param int  $uid
     * @param bool $unsetPass
     *
     * @return null
     */
    public function patchLoginHistory($post = null, $uid = 0, $unsetPass = true)
    {
        if ($uid > 0 || empty($post['pass']) || empty($post['uname'])) {
            return $post;
        }
        $postPatch                = $post;
        $postPatch['login_patch'] = 1;
        if ($unsetPass) {
            $postPatch['pass'] = 'unset_pass';
            if (isset($postPatch['vpass'])) {
                $postPatch['vpass'] = 'unset_vpass';
            }
        }
        $memberHandler = xoops_getHandler('member');
        $loginSuccess  = $memberHandler->loginUser($post['uname'], $post['pass']); // check login to find if this user is exist in database
        // only for successful login/register
        if (is_object($loginSuccess)) {
            $postPatch['success'] = 1;
            $postPatch['uid']     = $loginSuccess->getVar('uid');
            if (0 < ($level = $loginSuccess->getVar('level'))) {
                $postPatch['level'] = $level;
            }
            if (0 < ($last_visit = $loginSuccess->getVar('last_login'))) {
                $postPatch['last_visit'] = $last_visit;
            }
        }

        return $postPatch;
    }


    private function initLogModule()
    {
        global $xoopsModule;
function myFunction($a, $b) {
    // Do something
}
        if (isset($xoopsModule) && is_object($xoopsModule)) {
            $this->logmodule = $xoopsModule;
        } else {
            $hModule         = xoops_getHandler('module');
            $this->logmodule = $hModule->getByDirname('system');
            $this->logmodule->setVar('dirname', 'system-root');
        }
        $this->addLog('INIT LOGMODULE');
    }

    private function initUser()
    {
        global $xoopsUser;
function myFunction($a, $b) {
    // Do something
}
        if (isset($xoopsUser) && is_object($xoopsUser)) {
            $this->user = $xoopsUser;
        } else {
            $this->user = null;
        }
        $this->addLog('INIT USER');
    }

    private function initGroupList()
    {
        /** @var \XoopsMemberHandler $groupHandler */
        $groupHandler    = xoops_getHandler('member');
        $this->groupList = $groupHandler->getGroupList();
        $this->addLog('INIT GROUP LIST');
    }

    /**
     * @return bool
     * @throws \phpbrowscap\Exception
     */
    private function initBrowsCap()
    {
        $browscapCache = XOOPS_CACHE_PATH . '/browscap';
        // force to create file if not exist
        $folderHandler = XoopsFile::getHandler('folder', $browscapCache, true);
        if (!$folderHandler->pwd()) {
            // Errors Warning: mkdir() [function.mkdir]: Permission denied in file /class/file/folder.php line 529
            $this->addLog("Cannot create folder ({$browscapCache})");

            return false;
        }
        // Creates a new Browscap object (loads or creates the cache)
        // $bc = new Browscap('path/to/the/cache/dir');

        $this->browscap = new Browscap($browscapCache);
        $this->addLog('INIT BrowsCap');

        return true;
    }
}


Push — master ( 507df7...d00737 )

Userlog C

Complexity

Size/Duplication

Coupling/Cohesion

Importance

20 Methods

How to fix Complexity

Complex Class

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1			<?php
			0 ignored issues – show Coding Style Compatibility introduced 2016-08-28 18:11 UTC by Report Bug Copy Issue Report For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 34 and the first side effect is on line 25. The PSR-1: Basic Coding Standard recommends that a file should either introduce new symbols, that is classes, functions, constants or similar, or have side effects. Side effects are anything that executes logic, like for example printing output, changing ini settings or writing to a file. The idea behind this recommendation is that merely auto-loading a class should not change the state of an application. It also promotes a cleaner style of programming and makes your code less prone to errors, because the logic is not spread out all over the place. To learn more about the PSR-1, please see the PHP-FIG site on the PSR-1. Loading history...
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			* userlog module
14			*
15			* @copyright XOOPS Project (https://xoops.org)
16			* @license GNU GPL 2 (http://www.gnu.org/licenses/old-licenses/gpl-2.0.html)
17			* @package userlog class
18			* @since 1
19			* @author irmtfan ([email protected])
20			* @author XOOPS Project <www.xoops.org> <www.xoops.ir>
21			*/
22
23			use Xmf\Request;
24
25			defined('XOOPS_ROOT_PATH') \|\| exit('Restricted access.');
26			require_once __DIR__ . '/phpbrowscap/Browscap.php';
27
28			// The Browscap class is in the phpbrowscap namespace, so import it
29			use phpbrowscap\Browscap;
30
31			/**
32			* Class Userlog
33			*/
34			class Userlog extends \Xmf\Module\Helper
			0 ignored issues – show Coding Style Compatibility introduced 2017-09-05 09:43 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
35			{
36			public $logmodule;
37			public $user;
38			public $debugArray = [];
39			public $logext = 'log';
40			public $cookiePrefix = '';
41			public $groupList;
42			public $browscap;
43
44			/**
45			* @param $debug
46			*/
47			protected function __construct($debug)
48			{
49			$this->debug = $debug;
50			$this->dirname = USERLOG_DIRNAME;
51			$this->cookiePrefix = USERLOG_DIRNAME . '_' . ($this->getUser() ? $this->getUser()->getVar('uid') : '');
52			}
53
54			/**
55			* @param bool $debug
56			*
57			* @return Userlog
58			*/
59			public static function getInstance($debug = false)
60			{
61			static $instance;
62			if (null === $instance) {
63			$instance = new static($debug);
64			}
65
66			return $instance;
67			}
68
69			/**
70			* @return null\|\XoopsModule
71			*/
72			public function getLogModule()
73			{
74			if (null === $this->logmodule) {
75			$this->initLogModule();
76			}
77
78			return $this->logmodule;
79			}
80
81			/**
82			* @param array $dirnames
83			* @param null $otherCriteria
84			* @param bool $asObj
85			*
86			* @return mixed
87			*/
88			public function getModules($dirnames = [], $otherCriteria = null, $asObj = false)
89			{
90			// get all dirnames
91			/** @var XoopsModuleHandler $moduleHandler */
92			$moduleHandler = xoops_getHandler('module');
93			$criteria = new CriteriaCompo();
94			if (count($dirnames) > 0) {
95			foreach ($dirnames as $mDir) {
96			$criteria->add(new Criteria('dirname', $mDir), 'OR');
97			}
98			}
99			if (!empty($otherCriteria)) {
100			$criteria->add($otherCriteria);
101			}
102			$criteria->add(new Criteria('isactive', 1), 'AND');
103			$modules = $moduleHandler->getObjects($criteria, true);
104			if ($asObj) {
105			return $modules;
106			}
107			$dirNames['system-root'] = _YOURHOME;
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-08-28 18:11 UTC by Report Bug Copy Issue Report `$dirNames` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$dirNames = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
108			foreach ($modules as $module) {
109			$dirNames[$module->dirname()] = $module->name();
110			}
111
112			return $dirNames;
113			}
114
115			/**
116			* @return null
117			*/
118			public function getUser()
119			{
120			if (null === $this->user) {
121			$this->initUser();
122			}
123
124			return $this->user;
125			}
126
127			/**
128			* @return null\|array
129			*/
130			public function getGroupList()
131			{
132			if (null === $this->groupList) {
133			$this->initGroupList();
134			}
135
136			return $this->groupList;
137			}
138
139			/**
140			* @return null
141			*/
142			public function getBrowsCap()
143			{
144			if (null === $this->browscap) {
145			$this->initBrowsCap();
146			}
147
148			return $this->browscap;
149			}
150
151
152			/**
153			* @param null $name
154			* @param null $value
155			*
156			* @return mixed
157			*/
158			public function setConfig($name = null, $value = null)
159			{
160			if (null === $this->configs) {
161			$this->initConfig();
162			}
163			$this->configs[$name] = $value;
164			$this->addLog("Setting config '{$name}' : " . $this->configs[$name]);
165
166			return $this->configs[$name];
167			}
168
169			/**
170			* @return array
171			*/
172			public function getAllLogFiles()
173			{
174			$logPaths = $this->object->getInfo('log_paths');
175			$currentPath = $this->getConfig('logfilepath');
			0 ignored issues – show Unused Code introduced 2016-08-28 18:11 UTC by Report Bug Copy Issue Report `$currentPath` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
176			$allFiles = [];
177			$totalFiles = 0;
178			foreach ($logPaths as $path) {
179			$folderHandler = XoopsFile::getHandler('folder', $path . '/' . USERLOG_DIRNAME);
180			$allFiles[$path . '/' . USERLOG_DIRNAME] = $folderHandler->find('.*' . $this->logext);
181			$totalFiles += count($allFiles[$path . '/' . USERLOG_DIRNAME]);
182			}
183			if (empty($totalFiles)) {
184			return [[], 0];
185			}
186
187			return [$allFiles, $totalFiles];
188			}
189
190			/**
191			* @return string
192			*/
193			public function getWorkingFile()
194			{
195			$logFileName = $this->getConfig('logfilepath') . '/' . USERLOG_DIRNAME . '/' . $this->getConfig('logfilename');
196
197			return $logFileName . '.' . $this->logext;
198			}
199
200			/**
201			* @param $array
202			* @param null $keys
203			*
204			* @return array
205			*/
206			public function getFromKeys($array, $keys = null)
207			{
208			if (empty($keys)) {
209			return $array;
210			} // all keys
211			$keyarr = is_string($keys) ? explode(',', $keys) : $keys;
212			if (empty($keyarr[0])) {
213			return $array;
214			} // all keys
215			$keyarr = array_intersect(array_keys($array), $keyarr); // keys should be in array
216			$ret = [];
217			foreach ($keyarr as $key) {
218			$ret[$key] = $array[$key];
219			}
220
221			return $ret;
222			}
223
224			/**
225			* @param int $since
226			*
227			* @return int
228			*/
229			public function getSinceTime($since = 1) // one day
230			{
231			if ($since > 0) {
232			return (int)$since * 24 * 3600;
233			}
234
235			return (int)abs($since) * 3600;
236			}
237
238			/**
239			* @param null $intTime
240			* @param string $dateFormat
241			* @param null\|string $timeoffset
242			*
243			* @return bool\|string
244			*/
245			public function formatTime($intTime = null, $dateFormat = 'c', $timeoffset = null)
246			{
247			if (empty($intTime)) {
248			return false;
249			}
250			if ('custom' === $dateFormat \|\| 'c' === $dateFormat) {
251			$dateFormat = $this->getConfig('format_date');
252			}
253			xoops_load('XoopsLocal');
254
255			return class_exists('XoopsLocal') ? XoopsLocal::formatTimestamp($intTime, $dateFormat, $timeoffset) : XoopsLocale::formatTimestamp($intTime, $dateFormat, $timeoffset); // use XoopsLocale in xoops26
256			}
257
258			/**
259			* @param string $name
260			*
261			* @return array
262			*/
263			public function getCookie($name = 'TOGGLE')
264			{
265			$toggles = Request::getString($this->cookiePrefix . $name, null, 'cookie');
266
267			return explode(',', $toggles);
268			}
269
270			/**
271			* @param int $prob
272			*
273			* @return bool
274			*/
275			public function probCheck($prob = 11)
276			{
277			mt_srand((double)microtime() * 1000000);
278			// check probabillity 11 means 10%, 100 means 100%
279			$ret = mt_rand(1, 100) > $prob;
280
281			return $ret;
282			}
283
284			/**
285			* @param null $post
286			* @param int $uid
287			* @param bool $unsetPass
288			*
289			* @return null
290			*/
291			public function patchLoginHistory($post = null, $uid = 0, $unsetPass = true)
292			{
293			if ($uid > 0 \|\| empty($post['pass']) \|\| empty($post['uname'])) {
294			return $post;
295			}
296			$postPatch = $post;
297			$postPatch['login_patch'] = 1;
298			if ($unsetPass) {
299			$postPatch['pass'] = 'unset_pass';
300			if (isset($postPatch['vpass'])) {
301			$postPatch['vpass'] = 'unset_vpass';
302			}
303			}
304			$memberHandler = xoops_getHandler('member');
305			$loginSuccess = $memberHandler->loginUser($post['uname'], $post['pass']); // check login to find if this user is exist in database
306			// only for successful login/register
307			if (is_object($loginSuccess)) {
308			$postPatch['success'] = 1;
309			$postPatch['uid'] = $loginSuccess->getVar('uid');
310			if (0 < ($level = $loginSuccess->getVar('level'))) {
311			$postPatch['level'] = $level;
312			}
313			if (0 < ($last_visit = $loginSuccess->getVar('last_login'))) {
314			$postPatch['last_visit'] = $last_visit;
315			}
316			}
317
318			return $postPatch;
319			}
320
321
322			private function initLogModule()
323			{
324			global $xoopsModule;
			0 ignored issues – show Compatibility Best Practice introduced 2016-08-28 18:11 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
325			if (isset($xoopsModule) && is_object($xoopsModule)) {
326			$this->logmodule = $xoopsModule;
327			} else {
328			$hModule = xoops_getHandler('module');
329			$this->logmodule = $hModule->getByDirname('system');
330			$this->logmodule->setVar('dirname', 'system-root');
331			}
332			$this->addLog('INIT LOGMODULE');
333			}
334
335			private function initUser()
336			{
337			global $xoopsUser;
			0 ignored issues – show Compatibility Best Practice introduced 2016-08-28 18:11 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
338			if (isset($xoopsUser) && is_object($xoopsUser)) {
339			$this->user = $xoopsUser;
340			} else {
341			$this->user = null;
342			}
343			$this->addLog('INIT USER');
344			}
345
346			private function initGroupList()
347			{
348			/** @var \XoopsMemberHandler $groupHandler */
349			$groupHandler = xoops_getHandler('member');
350			$this->groupList = $groupHandler->getGroupList();
351			$this->addLog('INIT GROUP LIST');
352			}
353
354			/**
355			* @return bool
356			* @throws \phpbrowscap\Exception
357			*/
358			private function initBrowsCap()
359			{
360			$browscapCache = XOOPS_CACHE_PATH . '/browscap';
361			// force to create file if not exist
362			$folderHandler = XoopsFile::getHandler('folder', $browscapCache, true);
363			if (!$folderHandler->pwd()) {
364			// Errors Warning: mkdir() [function.mkdir]: Permission denied in file /class/file/folder.php line 529
365			$this->addLog("Cannot create folder ({$browscapCache})");
366
367			return false;
368			}
369			// Creates a new Browscap object (loads or creates the cache)
370			// $bc = new Browscap('path/to/the/cache/dir');
			0 ignored issues – show Unused Code Comprehensibility introduced 2016-08-28 18:11 UTC by Report Bug Copy Issue Report `50%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
371			$this->browscap = new Browscap($browscapCache);
372			$this->addLog('INIT BrowsCap');
373
374			return true;
375			}
376			}
377

XoopsModules25x / userlog

Push — master ( 507df7...d00737 )

Userlog C

Complexity

Size/Duplication

Coupling/Cohesion

Importance

20 Methods

How to fix Complexity

Complex Class

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

Duplication Side-by-Side

Filter issues like