Fixes coding style in the last 4 commits (uses S:: instead of Session::).

Signed-off-by: Vincent Zanotti <vincent.zanotti@polytechnique.org>

diff --git a/classes/csvimporter.php b/classes/csvimporter.php
index e065195..522544f 100644 (file)
--- a/classes/csvimporter.php
+++ b/classes/csvimporter.php
@@ -330,7 +330,7 @@ class CSVImporter
                 $fields[]     = $key;
             }
             if ($current == 'valid' && Env::has('csv_valid')) {
-                if (!Session::has_xsrf_token()) {
+                if (!S::has_xsrf_token()) {
                     $page->kill("L'opération n'a pas pu être effectuée, merci de réessayer.");
                 }
                 $this->run($_SESSION['csv_action'], $insert, $update);

diff --git a/classes/pltableeditor.php b/classes/pltableeditor.php
index cbd944a..35d5aae 100644 (file)
--- a/classes/pltableeditor.php
+++ b/classes/pltableeditor.php
@@ -193,7 +193,7 @@ class PLTableEditor
         $page->changeTpl('core/table-editor.tpl');
         $list = true;
 
-        if ($action == 'delete' && Session::has_xsrf_token()) {
+        if ($action == 'delete' && S::has_xsrf_token()) {
             if (!isset($this->delete_action)) {
                 foreach ($this->jtables as $table => $j)
                     XDB::execute("DELETE FROM {$table} WHERE {$j['joinid']} = {?}{$j['joinextra']}", $id);
@@ -240,7 +240,7 @@ class PLTableEditor
             }
             $list = false;
         }
-        if ($action == 'update' && Session::has_xsrf_token()) {
+        if ($action == 'update' && S::has_xsrf_token()) {
             $values = "";
             $cancel = false;
             foreach ($this->vars as $field => $descr) {

diff --git a/modules/admin.php b/modules/admin.php
index 336f848..e57113a 100644 (file)
--- a/modules/admin.php
+++ b/modules/admin.php
@@ -413,7 +413,7 @@ class AdminModule extends PLModule
 
             // Check if there was a submission
             foreach($_POST as $key => $val) {
-                if (!Session::has_xsrf_token()) {
+                if (!S::has_xsrf_token()) {
                     $page->kill("L'opération de modification de l'utilisateur a échouée, merci de réessayer.");
                 }
                 switch ($key) {
@@ -823,7 +823,7 @@ class AdminModule extends PLModule
 
         $page->assign('promo',$promo);
 
-        if ($validate && Session::has_xsrf_token()) {
+        if ($validate && S::has_xsrf_token()) {
             $new_deces = array();
             $res = XDB::iterRow("SELECT user_id,matricule,nom,prenom,deces FROM auth_user_md5 WHERE promo = {?}", $promo);
             while (list($uid,$mat,$nom,$prenom,$deces) = $res->next()) {
@@ -927,7 +927,7 @@ class AdminModule extends PLModule
 
         if(Env::has('uid') && Env::has('type') && Env::has('stamp')) {
             $req = Validate::get_typed_request(Env::v('uid'), Env::v('type'), Env::v('stamp'));
-            if($req && Session::has_xsrf_token()) {
+            if($req && S::has_xsrf_token()) {
                 $req->handle_formu();
             } else if ($req) {
                 $page->trig("L'opération a échoué, merci de réessayer.");
@@ -1115,7 +1115,7 @@ class AdminModule extends PLModule
         switch (Post::v('action')) {
           case 'create':
             if (trim(Post::v('ipN')) != '') {
-                if (!Session::has_xsrf_token()) {
+                if (!S::has_xsrf_token()) {
                     $page->trig("L'ajout d'une IP à surveiller a échoué, merci de réessayer.");
                     break;
                 }
@@ -1127,7 +1127,7 @@ class AdminModule extends PLModule
             break;
 
           case 'edit':
-            if (!Session::has_xsrf_token()) {
+            if (!S::has_xsrf_token()) {
                 $page->trig("L'édition de l'IP a échoué, merci de réessayer.");
                 break;
             }
@@ -1139,7 +1139,7 @@ class AdminModule extends PLModule
 
           default:
             if ($action == 'delete' && !is_null($ip)) {
-                if (Session::has_xsrf_token()) {
+                if (S::has_xsrf_token()) {
                     Xdb::execute('DELETE FROM ip_watch WHERE ip = {?}', ip_to_uint($ip));
                 } else {
                     $page->trig("La suppression de l'adresse IP a échouée, merci de réssayer.");

diff --git a/modules/auth/auth.inc.php b/modules/auth/auth.inc.php
index 83921e1..a1a562f 100644 (file)
--- a/modules/auth/auth.inc.php
+++ b/modules/auth/auth.inc.php
@@ -60,12 +60,12 @@ function gpex_make($chlg, $privkey, $datafields, $charset)
                                     WHERE  uid = {?} AND diminutif = {?}",
                                   S::v('uid'), $_GET['group']);
                 $perms = $res->fetchOneCell();
-                if (Session::has_perms()) {
+                if (S::has_perms()) {
                     $perms = 'admin';
                 }
             } else {
                 // if no group asked, return main rights
-                $perms = Session::has_perms()?'admin':'membre';
+                $perms = S::has_perms() ? 'admin' : 'membre';
             }
             $params .= gpex_prepare_param($val, $perms, $tohash, $charset);
         }

diff --git a/modules/email.php b/modules/email.php
index 2642477..371ed49 100644 (file)
--- a/modules/email.php
+++ b/modules/email.php
@@ -223,7 +223,7 @@ class EmailModule extends PLModule
             $redirect->modify_one_email_redirect($email, $rewrite);
         }
 
-        if (Env::has('emailop') && Session::has_xsrf_token()) {
+        if (Env::has('emailop') && S::has_xsrf_token()) {
             $actifs = Env::v('emails_actifs', Array());
             print_r(Env::v('emails_rewrite'));
             if (Env::v('emailop') == "ajouter" && Env::has('email')) {

diff --git a/modules/googleapps.php b/modules/googleapps.php
index 72e3488..14baa8e 100644 (file)
--- a/modules/googleapps.php
+++ b/modules/googleapps.php
@@ -68,7 +68,7 @@ class GoogleAppsModule extends PLModule
                 } else if ($subaction == 'nosync') {
                     $account->set_password_sync(false);
                 } else if (Post::has('response2') && !$account->sync_password) {
-                    if (Session::has_xsrf_token()) {
+                    if (S::has_xsrf_token()) {
                         $account->set_password(Post::v('response2'));
                     } else {
                         $page->trig("Le changement de ton mot de passe Google Apps a échoué, merci de réessayer.");
@@ -77,7 +77,7 @@ class GoogleAppsModule extends PLModule
             }
 
             if ($action == 'suspend' && Post::has('suspend') && $account->active()) {
-                if (!Session::has_xsrf_token()) {
+                if (!S::has_xsrf_token()) {
                     $page->trig("La demande de suspension de ton compte a échouée, merci de réessayer.");
                 } else if ($account->pending_update_suspension) {
                     $page->trig("Ton compte est déjà en cours de désactivation.");
@@ -107,7 +107,7 @@ class GoogleAppsModule extends PLModule
                     $password = Post::v('response2');
                 }
 
-                if (Session::has_xsrf_token()) {
+                if (S::has_xsrf_token()) {
                     $account->create($password_sync, $password, $redirect_mails);
                     $page->trig("La demande de création de ton compte Google Apps a bien été enregistrée.");
                 } else {

diff --git a/modules/platal.php b/modules/platal.php
index a58398b..1c3f71d 100644 (file)
--- a/modules/platal.php
+++ b/modules/platal.php
@@ -187,7 +187,7 @@ class PlatalModule extends PLModule
     {
         global $globals;
 
-        if (Post::has('response2') && Session::has_xsrf_token())  {
+        if (Post::has('response2') && S::has_xsrf_token())  {
             require_once 'secure_hash.inc.php';
 
             $_SESSION['password'] = $password = Post::v('response2');

diff --git a/modules/xnet.php b/modules/xnet.php
index c05c939..b1dd12b 100644 (file)
--- a/modules/xnet.php
+++ b/modules/xnet.php
@@ -80,7 +80,7 @@ class XnetModule extends PLModule
                                         Get::v('del'));
             list($id, $nom, $domain) = $res->fetchOneRow();
             $page->assign('nom', $nom);
-            if ($id && Post::has('del') && Session::has_xsrf_token()) {
+            if ($id && Post::has('del') && S::has_xsrf_token()) {
                 XDB::query('DELETE FROM groupex.membres WHERE asso_id={?}', $id);
                 $page->trig('membres supprimés');
 
@@ -111,7 +111,7 @@ class XnetModule extends PLModule
             }
         }
 
-        if (Post::has('diminutif') && Session::has_xsrf_token()) {
+        if (Post::has('diminutif') && S::has_xsrf_token()) {
             XDB::query('INSERT INTO groupex.asso (id,diminutif)
                                  VALUES(NULL,{?})', Post::v('diminutif'));
             pl_redirect('../'.Post::v('diminutif').'/edit');

diff --git a/modules/xnetgrp.php b/modules/xnetgrp.php
index b064196..85009c3 100644 (file)
--- a/modules/xnetgrp.php
+++ b/modules/xnetgrp.php
@@ -221,7 +221,7 @@ class XnetGrpModule extends PLModule
         global $globals;
         $page->changeTpl('xnetgrp/edit.tpl');
 
-        if (Post::has('submit') && Session::has_xsrf_token()) {
+        if (Post::has('submit') && S::has_xsrf_token()) {
             if (S::has_perms()) {
                 if (Post::v('mail_domain') && (strstr(Post::v('mail_domain'), '.') === false)) {
                     $page->trig("le domaine doit être un FQDN (aucune modif effectuée) !!!");

diff --git a/plugins/function.xsrf_token.php b/plugins/function.xsrf_token.php
index 0258495..caa0a10 100644 (file)
--- a/plugins/function.xsrf_token.php
+++ b/plugins/function.xsrf_token.php
@@ -20,7 +20,7 @@
  ***************************************************************************/
 
 function smarty_function_xsrf_token($params, &$smarty) {
-    return Session::v('xsrf_token', '');
+    return S::v('xsrf_token', '');
 }
 
 // vim:set et sw=4 sts=4 sws=4 foldmethod=marker enc=utf-8:

diff --git a/plugins/function.xsrf_token_field.php b/plugins/function.xsrf_token_field.php
index 28f09b8..8eb2430 100644 (file)
--- a/plugins/function.xsrf_token_field.php
+++ b/plugins/function.xsrf_token_field.php
@@ -20,8 +20,8 @@
  ***************************************************************************/
 
 function smarty_function_xsrf_token_field($params, &$smarty) {
-    if (Session::has('xsrf_token')) {
-        return '<input type="hidden" name="token" value="' . Session::v('xsrf_token') . '" />';
+    if (S::has('xsrf_token')) {
+        return '<input type="hidden" name="token" value="' . S::v('xsrf_token') . '" />';
     }
     return '';
 }