From 7000e3f71fd1822c3e222212785340a7d2fb1936 Mon Sep 17 00:00:00 2001
From: Jakub Vrana <vrana@php.net>
Date: Wed, 6 Sep 2006 11:50:00 +0000
Subject: [PATCH] Improve example

git-svn-id: https://svn.php.net/repository/phpdoc/en/trunk@219459 c90b9560-bf6c-de11-be94-00142212c4b1
---
 .../functions/session-set-save-handler.xml    | 42 +++++++++----------
 1 file changed, 20 insertions(+), 22 deletions(-)

diff --git a/reference/session/functions/session-set-save-handler.xml b/reference/session/functions/session-set-save-handler.xml
index a785dac5e9..d511b88267 100644
--- a/reference/session/functions/session-set-save-handler.xml
+++ b/reference/session/functions/session-set-save-handler.xml
@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="iso-8859-1"?>
-<!-- $Revision: 1.13 $ -->
+<!-- $Revision: 1.14 $ -->
 <!-- splitted from ./en/functions/session.xml, last change in rev 1.23 -->
   <refentry id="function.session-set-save-handler">
    <refnamediv>
@@ -68,10 +68,9 @@
 <?php
 function open($save_path, $session_name) 
 {
-  global $sess_save_path, $sess_session_name;
-       
+  global $sess_save_path;
+  
   $sess_save_path = $save_path;
-  $sess_session_name = $session_name;
   return(true);
 }
 
@@ -82,25 +81,21 @@ function close()
 
 function read($id) 
 {
-  global $sess_save_path, $sess_session_name;
-
+  global $sess_save_path;
+  
   $sess_file = "$sess_save_path/sess_$id";
-  if ($fp = @fopen($sess_file, "r")) {
-    $sess_data = fread($fp, filesize($sess_file));
-    return($sess_data);
-  } else {
-    return(""); // Must return "" here.
-  }
-
+  return (string) @file_get_contents($sess_file);
 }
 
 function write($id, $sess_data) 
 {
-  global $sess_save_path, $sess_session_name;
-
+  global $sess_save_path;
+  
   $sess_file = "$sess_save_path/sess_$id";
   if ($fp = @fopen($sess_file, "w")) {
-    return(fwrite($fp, $sess_data));
+    $return = fwrite($fp, $sess_data);
+    fclose($fp);
+    return $return;
   } else {
     return(false);
   }
@@ -109,18 +104,21 @@ function write($id, $sess_data)
 
 function destroy($id) 
 {
-  global $sess_save_path, $sess_session_name;
-       
+  global $sess_save_path;
+  
   $sess_file = "$sess_save_path/sess_$id";
   return(@unlink($sess_file));
 }
 
-/*********************************************
- * WARNING - You will need to implement some *
- * sort of garbage collection routine here.  *
- *********************************************/
 function gc($maxlifetime) 
 {
+  global $sess_save_path;
+  
+  foreach (glob("$sess_save_path/sess_*") as $filename) {
+    if (filemtime($filename) + $maxlifetime < time()) {
+      @unlink($filename);
+    }
+  }
   return true;
 }