Changeset 1059

Show
Ignore:
Timestamp:
08/25/08 22:22:36 (5 months ago)
Author:
laurentj
Message:

ticket #665: bug fix in jDaoFactoryBase::findBy : it should not ignore limit=0. p=Greut

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/lib/jelix/CREDITS

    r1058 r1059  
    122122 - bug fix in jXmlRpc.class.php (#192) 
    123123 
     124Greut 
     125 - bug fix in jDaoFactoryBase::findBy : it should not ignore limit=0 (#665) 
     126 
    124127Frédéric Guillot (aka fg) 
    125128 - patch for multiple authentification in a same session (#207) 

  • trunk/lib/jelix/dao/jDaoFactoryBase.class.php

    r1026 r1059  
    66 * @contributor Loic Mathaud 
    77 * @contributor Julien Issler 
     8 * @contributor Thomas, Greut 
    89 * @copyright   2005-2007 Laurent Jouanneau 
    910 * @copyright   2007 Loic Mathaud 
    1011 * @copyright   2007-2008 Julien Issler 
    11  * @copyright   2008 Thomas 
     12 * @copyright   2008 Thomas, 2008 Greut 
    1213 * @link        http://www.jelix.org 
    1314 * @licence     http://www.gnu.org/licenses/lgpl.html GNU Lesser General Public Licence, see LICENCE file 
    …… 
    238239     * @return jDbResultSet 
    239240     */ 
    240     final public function findBy ($searchcond, $limitOffset=0, $limitCount=0){ 
     241    final public function findBy ($searchcond, $limitOffset=0, $limitCount=null){ 
    241242        $query = $this->_selectClause.$this->_fromClause.$this->_whereClause; 
    242243        if ($searchcond->hasConditions ()){ 
    …… 
    247248        $query.= $this->_createOrderClause($searchcond); 
    248249 
    249         if($limitCount != 0){ 
     250        if($limitCount !== null){ 
    250251            $rs = $this->_conn->limitQuery ($query, $limitOffset, $limitCount); 
    251252        }else{ 
Download in other formats: Unified Diff Zip Archive