SessionStorage::__construct() - Code Metrics - Inspection of "- rebuild github access" - ggoffy/wggithub - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 0deb8f...f54565 )

by Goffy

created 2020-12-21 07:51 UTC

SessionStorage::__construct() A

↳ Parent: SessionStorage

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	1
nc	1
nop	1
dl	0
loc	3
rs	10
c	0
b	0
f	0

<?php

namespace XoopsModules\Wggithub\Github\Storages;

use XoopsModules\Wggithub\Github;


/**
 * Session storage which uses $_SESSION directly. Session must be started already before use.
 *
 * @author  Miloslav Hůla (https://github.com/milo)
 */
class SessionStorage extends Github\Sanity implements ISessionStorage
{
	const SESSION_KEY = 'milo.github-api';

	/** @var string */
	private $sessionKey;


	/**
	 * @param  string
	 */
	public function __construct($sessionKey = self::SESSION_KEY)
	{
		$this->sessionKey = $sessionKey;
	}


	/**
	 * @param  string
	 * @param  mixed
	 * @return self
	 */
	public function set($name, $value)
	{
		if ($value === NULL) {
			return $this->remove($name);
		}

		$this->check(__METHOD__);
		$_SESSION[$this->sessionKey][$name] = $value;

		return $this;
	}


	/**
	 * @param  string
	 * @return mixed
	 */
	public function get($name)
	{
		$this->check(__METHOD__);

		return isset($_SESSION[$this->sessionKey][$name])
			? $_SESSION[$this->sessionKey][$name]
			: NULL;
	}


	/**
	 * @param  string
	 * @return self
	 */
	public function remove($name)
	{
		$this->check(__METHOD__);

		unset($_SESSION[$this->sessionKey][$name]);

		return $this;
	}


	/**
	 * @param  string
	 */
	private function check($method)
	{
		if (!isset($_SESSION)) {
			trigger_error("Start session before using $method().", E_USER_WARNING);
		}
	}

}


1			<?php
2
3			namespace XoopsModules\Wggithub\Github\Storages;
4
5			use XoopsModules\Wggithub\Github;
6
7
8			/**
9			* Session storage which uses $_SESSION directly. Session must be started already before use.
10			*
11			* @author Miloslav Hůla (https://github.com/milo)
12			*/
13			class SessionStorage extends Github\Sanity implements ISessionStorage
14			{
15			const SESSION_KEY = 'milo.github-api';
16
17			/** @var string */
18			private $sessionKey;
19
20
21			/**
22			* @param string
23			*/
24			public function __construct($sessionKey = self::SESSION_KEY)
25			{
26			$this->sessionKey = $sessionKey;
27			}
28
29
30			/**
31			* @param string
32			* @param mixed
33			* @return self
34			*/
35			public function set($name, $value)
36			{
37			if ($value === NULL) {
38			return $this->remove($name);
39			}
40
41			$this->check(__METHOD__);
42			$_SESSION[$this->sessionKey][$name] = $value;
43
44			return $this;
45			}
46
47
48			/**
49			* @param string
50			* @return mixed
51			*/
52			public function get($name)
53			{
54			$this->check(__METHOD__);
55
56			return isset($_SESSION[$this->sessionKey][$name])
57			? $_SESSION[$this->sessionKey][$name]
58			: NULL;
59			}
60
61
62			/**
63			* @param string
64			* @return self
65			*/
66			public function remove($name)
67			{
68			$this->check(__METHOD__);
69
70			unset($_SESSION[$this->sessionKey][$name]);
71
72			return $this;
73			}
74
75
76			/**
77			* @param string
78			*/
79			private function check($method)
80			{
81			if (!isset($_SESSION)) {
82			trigger_error("Start session before using $method().", E_USER_WARNING);
83			}
84			}
85
86			}
87

ggoffy / wggithub

Push — master ( 0deb8f...f54565 )

SessionStorage::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like