Fix a Minz_Session TODO

Not use additional variable to manipulate session variables
Fix coding style
This commit is contained in:
Marien Fressinaud
2014-08-12 21:09:13 +02:00
parent ed7d8aa44f
commit 08a9009c19

View File

@@ -2,17 +2,11 @@
/**
* La classe Session gère la session utilisateur
* C'est un singleton
*/
class Minz_Session {
/**
* $session stocke les variables de session
*/
private static $session = array (); //TODO: Try to avoid having another local copy
/**
* Initialise la session, avec un nom
* Le nom de session est utilisé comme nom pour les cookies et les URLs (i.e. PHPSESSID).
* Le nom de session est utilisé comme nom pour les cookies et les URLs(i.e. PHPSESSID).
* Il ne doit contenir que des caractères alphanumériques ; il doit être court et descriptif
*/
public static function init($name) {
@@ -22,10 +16,6 @@ class Minz_Session {
// démarre la session
session_name($name);
session_start();
if (isset($_SESSION)) {
self::$session = $_SESSION;
}
}
@@ -34,8 +24,8 @@ class Minz_Session {
* @param $p le paramètre à récupérer
* @return la valeur de la variable de session, false si n'existe pas
*/
public static function param ($p, $default = false) {
return isset(self::$session[$p]) ? self::$session[$p] : $default;
public static function param($p, $default = false) {
return isset($_SESSION[$p]) ? $_SESSION[$p] : $default;
}
@@ -44,13 +34,11 @@ class Minz_Session {
* @param $p le paramètre à créer ou modifier
* @param $v la valeur à attribuer, false pour supprimer
*/
public static function _param ($p, $v = false) {
public static function _param($p, $v = false) {
if ($v === false) {
unset ($_SESSION[$p]);
unset (self::$session[$p]);
unset($_SESSION[$p]);
} else {
$_SESSION[$p] = $v;
self::$session[$p] = $v;
}
}
@@ -59,15 +47,15 @@ class Minz_Session {
* Permet d'effacer une session
* @param $force si à false, n'efface pas le paramètre de langue
*/
public static function unset_session ($force = false) {
$language = self::param ('language');
public static function unset_session($force = false) {
$language = self::param('language');
session_destroy();
self::$session = array ();
$_SESSION = array();
if (!$force) {
self::_param ('language', $language);
Minz_Translate::reset ();
self::_param('language', $language);
Minz_Translate::reset();
}
}