adulau/cakephp2-php8
1
0
Fork 0
mirror of https://github.com/kamilwylegala/cakephp2-php8.git synced 2024-11-15 03:18:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix usage of non-existant property.

Browse source 
Use the declared `_result` property instead. I would normally add
a test, but getting to this path requires a query to fail which is not
simple to simulate in the test harness.

Fixes #11615
This commit is contained in:
mark_story 2018-02-04 11:48:31 -05:00
parent 49725a8f86
commit 5816143493
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/Cake/Model/Datasource/DboSource.php
View file

@ -470,7 +470,7 @@ class DboSource extends DataSource {
$query = $this->_connection->prepare($sql, $prepareOptions); $query = $this->_connection->prepare($sql, $prepareOptions);
$query->setFetchMode(PDO::FETCH_LAZY); $query->setFetchMode(PDO::FETCH_LAZY);
if (!$query->execute($params)) { if (!$query->execute($params)) {
$this->_results = $query; $this->_result = $query;
$query->closeCursor(); $query->closeCursor();
return false; return false;
} }