Changeset 1623

Timestamp:

Dec 1, 2006, 2:46:32 AM (17 years ago)

Author:

rvelices

Message:

• sessions are always started (even for visitors)
• thumbnail order saved in the session instead of cookie

Location:

trunk

Files:

• include/functions_session.inc.php (modified) (2 diffs)
• include/section_init.inc.php (modified) (2 diffs)
• include/user.inc.php (modified) (2 diffs)
• index.php (modified) (2 diffs)

trunk/include/functions_session.inc.php

@@ -82 +82 @@
 function cookie_path()
 {
-  if ( isset($_SERVER['REDIRECT_SCRIPT_NAME']) and 
+  if ( isset($_SERVER['REDIRECT_SCRIPT_NAME']) and
        !empty($_SERVER['REDIRECT_SCRIPT_NAME']) )
   {
@@ -222 +222 @@
   return true;
 }
+
+
+/**
+ * persistently stores a variable for the current session
+ * currently we use standard php sessions but it might change
+ * @return boolean true on success
+ * @see pwg_get_session_var, pwg_unset_session_var
+ */
+function pwg_set_session_var($var, $value)
+{
+  if ( !isset($_SESSION) )
+    return false;
+  $_SESSION['pwg_'.$var] = $value;
+  return true;
+}
+
+/**
+ * retrieves the value of a persistent variable for the current session
+ * currently we use standard php sessions but it might change
+ * @return mixed
+ * @see pwg_set_session_var, pwg_unset_session_var
+ */
+function pwg_get_session_var($var, $default = null)
+{
+  if (isset( $_SESSION['pwg_'.$var] ) )
+  {
+    return $_SESSION['pwg_'.$var];
+  }
+  return $default;
+}
+
+/**
+ * deletes a persistent variable for the current session
+ * currently we use standard php sessions but it might change
+ * @return boolean true on success
+ * @see pwg_set_session_var, pwg_get_session_var
+ */
+function pwg_unset_session_var($var)
+{
+  if ( !isset($_SESSION) )
+    return false;
+  unset( $_SESSION['pwg_'.$var] );
+  return true;
+}
+
 ?>

trunk/include/section_init.inc.php

@@ -298 +298 @@
 $page['nb_image_page'] = $user['nb_image_page'];
 
-if (isset($_COOKIE['pwg_image_order'])
-    and is_numeric($_COOKIE['pwg_image_order'])
-    and $_COOKIE['pwg_image_order'] > 0)
+if (pwg_get_session_var('image_order',0) > 0)
 {
   $orders = get_category_preferred_image_orders();
@@ -306 +304 @@
   $conf['order_by'] = str_replace(
     'ORDER BY ',
-    'ORDER BY '.$orders[ $_COOKIE['pwg_image_order'] ][1].',',
+    'ORDER BY '.$orders[ pwg_get_session_var('image_order',0) ][1].',',
     $conf['order_by']
     );

trunk/include/user.inc.php

@@ -44 +44 @@
     redirect(make_index_url());
   }
-  elseif (empty($_SESSION['pwg_uid']))
-  { // timeout
-    setcookie(session_name(),'',0,
-        ini_get('session.cookie_path'),
-        ini_get('session.cookie_domain')
-      );
-  }
-  else
+  elseif (!empty($_SESSION['pwg_uid']))
   {
     $user['id'] = $_SESSION['pwg_uid'];
   }
 }
-
 
 // Now check the auto-login
@@ -64 +56 @@
 }
 
+if (session_id()=="")
+{
+  session_start();
+}
 
 // using Apache authentication override the above user search

trunk/index.php

@@ -41 +41 @@
 if (isset($_GET['image_order']))
 {
-  setcookie(
-    'pwg_image_order',
-    $_GET['image_order'] > 0 ? $_GET['image_order'] : '',
-    0, cookie_path()
-    );
-
+  if ( (int)$_GET['image_order'] > 0)
+  {
+    pwg_set_session_var('image_order', (int)$_GET['image_order']);
+  }
+  else
+  {
+    pwg_unset_session_var('image_order');
+  }
   redirect(
     duplicate_index_url(
@@ -261 +263 @@
   $template->assign_block_vars( 'preferred_image_order', array() );
 
-  $order_idx = isset($_COOKIE['pwg_image_order'])
-    ? $_COOKIE['pwg_image_order']
-    : 0
-    ;
+  $order_idx = pwg_get_session_var( 'image_order', 0 );
 
   $orders = get_category_preferred_image_orders();