Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.3] Use \Doctrine\DBAL\Driver\PDOConnection if available #14363

Merged
merged 1 commit into from
Jul 21, 2016
Merged

[5.3] Use \Doctrine\DBAL\Driver\PDOConnection if available #14363

merged 1 commit into from
Jul 21, 2016

Conversation

mzur
Copy link
Contributor

@mzur mzur commented Jul 18, 2016

Follow-up of #14359 to target the 5.3 branch.

@@ -3,6 +3,7 @@
namespace Illuminate\Database\Connectors;

use PDO;
use Doctrine\DBAL\Driver\PDOConnection;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sort by length.

If doctrine/dbal is installed the PDOConnection class provided by
the package should be used instead of just PDO. Else e.g. detecting
the database version doesn't work.
@josebailo
Copy link

Is this fix still working? I want to rename a column in a table with a json column but throws me the error:

[Doctrine\DBAL\DBALException]
Unknown database type json requested, Doctrine\DBAL\Platforms\MySQL57Platform may not support it.

Even it doesn't let me change any column...

@mzur
Copy link
Contributor Author

mzur commented Nov 22, 2016

Looks like DBAL doesn't support the JSON type for MySQL yet (see here). It should work with PostgreSQL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants