Browse Source

Merge pull request #12121 from cakephp/buffered-query-properties

Remove redundant code.
Mark Story 7 years ago
parent
commit
4af60730ef
2 changed files with 2 additions and 14 deletions
  1. 1 1
      appveyor.yml
  2. 1 13
      src/Database/Statement/BufferedStatement.php

+ 1 - 1
appveyor.yml

@@ -46,7 +46,7 @@ install:
   - echo extension=php_wincache.dll >> php.ini
   - echo wincache.enablecli = 1 >> php.ini
   - cd C:\projects\cakephp
-  - appveyor DownloadFile https://getcomposer.org/composer.phar
+  - curl -fsS https://getcomposer.org/composer.phar -o composer.phar
   - php composer.phar install --prefer-dist --no-interaction --ansi --no-progress
   - php -i | grep "ICU version"
 

+ 1 - 13
src/Database/Statement/BufferedStatement.php

@@ -42,7 +42,7 @@ class BufferedStatement extends StatementDecorator
      *
      * @var bool
      */
-    protected $_allFetched = true;
+    protected $_allFetched = false;
 
     /**
      * Current record pointer
@@ -52,18 +52,6 @@ class BufferedStatement extends StatementDecorator
     protected $_counter = 0;
 
     /**
-     * Constructor
-     *
-     * @param \Cake\Database\StatementInterface|null $statement Statement implementation such as PDOStatement
-     * @param \Cake\Database\Driver|null $driver Driver instance
-     */
-    public function __construct($statement = null, $driver = null)
-    {
-        parent::__construct($statement, $driver);
-        $this->_reset();
-    }
-
-    /**
      * Execute the statement and return the results.
      *
      * @param array|null $params list of values to be bound to query