Changeset 3147

Show
Ignore:
Timestamp:
05/03/08 15:53:54 (2 months ago)
Author:
pmjones
Message:

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

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/Solar/Session/Handler/Adapter/Sql.php

    r2985 r3147  
    9292        $this->_sql->disconnect(); 
    9393        $this->_sql = null; 
     94        return true; 
    9495    } 
    9596     
    …… 
    231232        try { 
    232233            $this->_sql->insert($this->_config['table'], $cols); 
     234            return true; 
    233235        } catch (Solar_Sql_Exception $e) { 
    234236            // @todo log this somehow? 
    …… 
    261263        try { 
    262264            $this->_sql->update($this->_config['table'], $cols, $where); 
     265            return true; 
    263266        } catch (Solar_Sql_Exception $e) { 
    264267            // @todo log this somehow?