]> source.dussan.org Git - nextcloud-server.git/commitdiff
use fetch method instead of fetchRow because fetchRow is only an owncloud internal...
authorBernhard Posselt <dev@bernhard-posselt.com>
Tue, 13 May 2014 23:07:19 +0000 (01:07 +0200)
committerBernhard Posselt <dev@bernhard-posselt.com>
Tue, 13 May 2014 23:09:48 +0000 (01:09 +0200)
lib/public/appframework/db/mapper.php
tests/lib/appframework/db/MapperTestUtility.php

index 21ccb686221669b1c7535259fc5080a6be15803e..a23149e796b52c82cb1cba564875565313114443 100644 (file)
@@ -225,12 +225,12 @@ abstract class Mapper {
         */
        protected function findOneQuery($sql, array $params=array(), $limit=null, $offset=null){
                $result = $this->execute($sql, $params, $limit, $offset);
-               $row = $result->fetchRow();
+               $row = $result->fetch();
 
                if($row === false || $row === null){
                        throw new DoesNotExistException('No matching entry found');
                }
-               $row2 = $result->fetchRow();
+               $row2 = $result->fetch();
                //MDB2 returns null, PDO and doctrine false when no row is available
                if( ! ($row2 === false || $row2 === null )) {
                        throw new MultipleObjectsReturnedException('More than one result');
@@ -264,7 +264,7 @@ abstract class Mapper {
 
                $entities = array();
                
-               while($row = $result->fetchRow()){
+               while($row = $result->fetch()){
                        $entities[] = $this->mapRowToEntity($row);
                }
 
index 4c81d4cd27bdc849e74ec1e83af9070283b2db9f..fc0e5c2c445e6213aa758d69fc2a6cd064120cee 100644 (file)
@@ -51,7 +51,7 @@ abstract class MapperTestUtility extends \PHPUnit_Framework_TestCase {
                        ->getMock();
 
                $this->query = $this->getMock('Query', array('execute', 'bindValue'));
-               $this->pdoResult = $this->getMock('Result', array('fetchRow'));
+               $this->pdoResult = $this->getMock('Result', array('fetch'));
                $this->queryAt = 0;
                $this->prepareAt = 0;
                $this->iterators = array();
@@ -65,7 +65,7 @@ abstract class MapperTestUtility extends \PHPUnit_Framework_TestCase {
         * @param array $arguments the expected arguments for the prepare query
         * method
         * @param array $returnRows the rows that should be returned for the result
-        * of the database query. If not provided, it wont be assumed that fetchRow
+        * of the database query. If not provided, it wont be assumed that fetch
         * will be called on the result
         */
        protected function setMapperResult($sql, $arguments=array(), $returnRows=array(),
@@ -77,7 +77,7 @@ abstract class MapperTestUtility extends \PHPUnit_Framework_TestCase {
                $fetchAt = $this->fetchAt;
 
                $this->pdoResult->expects($this->any())
-                       ->method('fetchRow')
+                       ->method('fetch')
                        ->will($this->returnCallback(
                                function() use ($iterators, $fetchAt){
                                        $iterator = $iterators[$fetchAt];