minor #14498 Update doctrine fetchAll (deprecated) (qdequippe)

javiereguiluz · javiereguiluz · commit f803d1ed8b3d · 2020-10-30T13:34:23.000+01:00
This PR was submitted for the 5.x branch but it was merged into the 4.4 branch instead. Discussion ---------- Update doctrine fetchAll (deprecated) Since doctrine/dbal 2.11 [Deprecated FetchMode and the corresponding methods](https://github.com/doctrine/dbal/blob/2.12.x/UPGRADE.md#deprecated-fetchmode-and-the-corresponding-methods) > The Statement::fetchAll() method is deprecated in favor of Result::fetchAllNumeric(), ::fetchAllAssociative() and ::fetchFirstColumn() Commits ------- d865835 Update doctrine fetchAll deprecated
diff --git a/doctrine.rst b/doctrine.rst
@@ -812,7 +812,7 @@ In addition, you can query directly with SQL if you need to::
         $stmt->execute(['price' => $price]);
 
         // returns an array of arrays (i.e. a raw data set)
-        return $stmt->fetchAll();
+        return $stmt->fetchAllAssociative();
     }
 
 With SQL, you will get back raw data, not objects (unless you use the `NativeQuery`_

Original file line number	Diff line number	Diff line change
`@@ -812,7 +812,7 @@ In addition, you can query directly with SQL if you need to::`
`812`	`812`	`$stmt->execute(['price' => $price]);`
`813`	`813`
`814`	`814`	`// returns an array of arrays (i.e. a raw data set)`
`815`		`- return $stmt->fetchAll();`
	`815`	`+ return $stmt->fetchAllAssociative();`
`816`	`816`	`}`
`817`	`817`
`818`	`818`	With SQL, you will get back raw data, not objects (unless you use the `NativeQuery`_