[Solar-svn] Revision 3147

pmjones at solarphp.com pmjones at solarphp.com
Sat May 3 15:53:55 CDT 2008


Solar_Session_Handler_Adapter_Sql: [FIX] Methods _insert() and _update() now return true on success, as does method close().  Thanks for the bug report, cromulent.

fixes #122


Modified: trunk/Solar/Session/Handler/Adapter/Sql.php
===================================================================
--- trunk/Solar/Session/Handler/Adapter/Sql.php	2008-05-03 20:45:27 UTC (rev 3146)
+++ trunk/Solar/Session/Handler/Adapter/Sql.php	2008-05-03 20:53:54 UTC (rev 3147)
@@ -91,6 +91,7 @@
     {
         $this->_sql->disconnect();
         $this->_sql = null;
+        return true;
     }
     
     /**
@@ -230,6 +231,7 @@
         
         try {
             $this->_sql->insert($this->_config['table'], $cols);
+            return true;
         } catch (Solar_Sql_Exception $e) {
             // @todo log this somehow?
             return false;
@@ -260,6 +262,7 @@
         
         try {
             $this->_sql->update($this->_config['table'], $cols, $where);
+            return true;
         } catch (Solar_Sql_Exception $e) {
             // @todo log this somehow?
             return false;




More information about the Solar-svn mailing list