Changeset 1077

Timestamp:

09/06/08 15:55:06 (4 months ago)

Author:

laurentj

Message:

little source improvements into jSession

Files:

• trunk/build/manifests/jelix-no-opt.mn (modified) (1 diff)
• trunk/lib/jelix/core/defaultconfig.ini.php (modified) (1 diff)
• trunk/lib/jelix/core/jCoordinator.class.php (modified) (2 diffs)
• trunk/lib/jelix/core/jSession.class.php (modified) (4 diffs)

trunk/build/manifests/jelix-no-opt.mn

@@ -21 +21 @@
-* jSelector.class.php
-* jUrl.class.php
- 
+*
-
-cd lib/jelix/core/selector

trunk/lib/jelix/core/defaultconfig.ini.php

@@ -234 +234 @@
-; You can change the session name by setting the following parameter (only accepts alpha-numeric chars) :
-; name = "mySessionName"
+
+name=
+
-;
-; Use alternative storage engines for sessions
-
-
+
+
+
+
-;
-; storage = "files"

trunk/lib/jelix/core/jCoordinator.class.php

@@ -137 +137 @@
-        jSession::start();
-
-this->
-this->
+
+
-
-        if(empty($this->moduleName)){
@@ -151 +151 @@
-        }
-
-verification du module
+module check
-        if($gJConfig->checkTrustedModules && !in_array($this->moduleName,$gJConfig->_trustedModules)){
-            throw new jException('jelix~errors.module.untrusted',$this->moduleName);

trunk/lib/jelix/core/jSession.class.php

@@ -4 +4 @@
-* @subpackage core
-* @author     Julien Issler
-
-
+ Laurent Jouanneau
+, 2008 Laurent Jouanneau
-* @link       http://www.jelix.org
-* @licence    GNU Lesser General Public Licence see LICENCE file or http://www.gnu.org/licenses/lgpl.html
@@ -27 +27 @@
-    public static function start(){
-
+        $params = & $GLOBALS['gJConfig']->sessions;
+
-        // do not start the session if the request is made from the command line or if sessions are disabled in configuration
-GLOBALS['gJConfig']->session
+param
-            return false;
-        }
-
-        $params = $GLOBALS['gJConfig']->sessions;
-
-        //make sure that the session cookie is only for the current application
@@ -38 +38 @@
-            session_set_cookie_params ( 0 , $GLOBALS['gJConfig']->urlengine['basePath']);
-
-isset($params['storage'])
+$params['storage'] != ''
-
-            /* on debian/ubuntu (maybe others), garbage collector launch probability is set to 0
@@ -71 +71 @@
-        }
-
-
+
+
-            if(!preg_match('#^[a-zA-Z0-9]+$#',$params['name'])){
-                // regexp check because session name can only be alpha numeric according to the php documentation
-                throw new jException('jelix~errors.jsession.name.invalid');
-            }
+#endif
-            session_name($params['name']);
-        }