minor #35473 [Cache] Don't allow mysqli to be used as it doesn't work (duncan3dc)
This PR was merged into the 4.3 branch.
Discussion
----------
[Cache] Don't allow mysqli to be used as it doesn't work
| Q | A
| ------------- | ---
| Branch? | 4.3
| Bug fix? | yes
| New feature? | no
| Deprecations? | no
| Tickets |
| License | MIT
| Doc PR |
As discussed in #35375 Mysqli doesn't support named parameters, so if you pass a doctrine connection using `mysqli` then you get errors for any queries using named parameters.
This PR ensures a clear error is provided and suggests to use `pdo_mysql` instead
Commits
-------
a7864489b0
Mysqli doesn't support the named parameters used by PdoAdapter
This commit is contained in:
commit
86ccf322e8
@ -395,6 +395,7 @@ trait PdoTrait
|
|||||||
} else {
|
} else {
|
||||||
switch ($this->driver = $this->conn->getDriver()->getName()) {
|
switch ($this->driver = $this->conn->getDriver()->getName()) {
|
||||||
case 'mysqli':
|
case 'mysqli':
|
||||||
|
throw new \LogicException(sprintf('The adapter "%s" does not support the mysqli driver, use pdo_mysql instead.', \get_class($this)));
|
||||||
case 'pdo_mysql':
|
case 'pdo_mysql':
|
||||||
case 'drizzle_pdo_mysql':
|
case 'drizzle_pdo_mysql':
|
||||||
$this->driver = 'mysql';
|
$this->driver = 'mysql';
|
||||||
|
Reference in New Issue
Block a user