Session::del() - Code Metrics - Inspection of "1.4 Beta 1" - XoopsModules25x/xnewsletter - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#27)

by Michael

created 2019-10-05 16:15 UTC

Session::del() A

↳ Parent: Session

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace XoopsModules\Xnewsletter;

/*
 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.
 */
/**
 *  Session class
 *
 * @copyright       The XUUPS Project http://sourceforge.net/projects/xuups/
 * @license         http://www.fsf.org/copyleft/gpl.html GNU public license
 * @package         xnewsletter
 * @since           1.3
 * @author          trabis <[email protected]>
 * @author          Harry Fuecks (PHP Anthology Volume II)
 */
require_once dirname(__DIR__) . '/include/common.php';

/**
 * Class Session
 */
class Session
{
    /**
     * Session constructor<br>
     * Starts the session with session_start()
     * <strong>Note:</strong> that if the session has already started,
     * session_start() does nothing
     */
    protected function __construct()
    {
        @session_start();
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
    }

    /**
     * Sets a session variable
     *
     * @param string $name  name of variable
     * @param mixed  $value value of variable
     *
     * @access public
     */
    public function set($name, $value)
    {
        $_SESSION[$name] = $value;
    }

    /**
     * Fetches a session variable
     *
     * @param string $name name of variable
     *
     * @return mixed value of session variable
     * @access public
     */
    public function get($name)
    {
        if (isset($_SESSION[$name])) {
            return $_SESSION[$name];
        }

        return false;
    }

    /**
     * Deletes a session variable
     *
     * @param string $name name of variable
     *
     * @access public
     */
    public function del($name)
    {
        unset($_SESSION[$name]);
    }

    /**
     * Destroys the whole session
     *
     * @access public
     */
    public function destroy()
    {
        $_SESSION = [];
        session_destroy();
    }

    /**
     * @return Session
     */
    public static function getInstance()
    {
        static $instance;
        if (null === $instance) {
            $instance = new static();
        }

        return $instance;
    }
}


1			<?php
2
3			namespace XoopsModules\Xnewsletter;
4
5			/*
6			You may not change or alter any portion of this comment or credits
7			of supporting developers from this source code or any supporting source code
8			which is considered copyrighted (c) material of the original comment or credit authors.
9
10			This program is distributed in the hope that it will be useful,
11			but WITHOUT ANY WARRANTY; without even the implied warranty of
12			MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
13			*/
14			/**
15			* Session class
16			*
17			* @copyright The XUUPS Project http://sourceforge.net/projects/xuups/
18			* @license http://www.fsf.org/copyleft/gpl.html GNU public license
19			* @package xnewsletter
20			* @since 1.3
21			* @author trabis <[email protected]>
22			* @author Harry Fuecks (PHP Anthology Volume II)
23			*/
24			require_once dirname(__DIR__) . '/include/common.php';
25
26			/**
27			* Class Session
28			*/
29			class Session
30			{
31			/**
32			* Session constructor<br>
33			* Starts the session with session_start()
34			* <strong>Note:</strong> that if the session has already started,
35			* session_start() does nothing
36			*/
37			protected function __construct()
38			{
39			@session_start();
			0 ignored issues – show Security Best Practice introduced 2016-06-01 02:30 UTC by Report Bug Copy Issue Report It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended. If you suppress an error, we recommend checking for the error condition explicitly: // For example instead of @mkdir($dir); // Better use if (@mkdir($dir) === false) { throw new \RuntimeException('The directory '.$dir.' could not be created.'); } Loading history...
40			}
41
42			/**
43			* Sets a session variable
44			*
45			* @param string $name name of variable
46			* @param mixed $value value of variable
47			*
48			* @access public
49			*/
50			public function set($name, $value)
51			{
52			$_SESSION[$name] = $value;
53			}
54
55			/**
56			* Fetches a session variable
57			*
58			* @param string $name name of variable
59			*
60			* @return mixed value of session variable
61			* @access public
62			*/
63			public function get($name)
64			{
65			if (isset($_SESSION[$name])) {
66			return $_SESSION[$name];
67			}
68
69			return false;
70			}
71
72			/**
73			* Deletes a session variable
74			*
75			* @param string $name name of variable
76			*
77			* @access public
78			*/
79			public function del($name)
80			{
81			unset($_SESSION[$name]);
82			}
83
84			/**
85			* Destroys the whole session
86			*
87			* @access public
88			*/
89			public function destroy()
90			{
91			$_SESSION = [];
92			session_destroy();
93			}
94
95			/**
96			* @return Session
97			*/
98			public static function getInstance()
99			{
100			static $instance;
101			if (null === $instance) {
102			$instance = new static();
103			}
104
105			return $instance;
106			}
107			}
108

XoopsModules25x / xnewsletter

Pull Request — master (#27)

Session::del() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like