Skip to content

Fix validatePlatform for MariaDB, check for instance of MariaDBPlatform#114

Merged
Hikariii merged 1 commit intoScientaNL:masterfrom
annervisser:fix-validateplatform-for-mariadb-check-for-instance-of-mariadbplatform
Apr 10, 2024
Merged

Fix validatePlatform for MariaDB, check for instance of MariaDBPlatform#114
Hikariii merged 1 commit intoScientaNL:masterfrom
annervisser:fix-validateplatform-for-mariadb-check-for-instance-of-mariadbplatform

Conversation

@annervisser
Copy link
Copy Markdown
Contributor

@annervisser annervisser commented Apr 9, 2024

Fix validatePlatform for MariaDB, check for instance of MariaDBPlatform

Versions of DBAL before 3.3 used or extended the MySQLPlatform when
connecting to MariaDB. Because of this the platform check was
requiring an instance of MySQLPlatform.

This commit fixes MariaDB compatibility with DBAL >=3.3, while remaining
compatible with versions prior to that (for now).

Fixes #113

@annervisser annervisser self-assigned this Apr 9, 2024
@annervisser annervisser force-pushed the fix-validateplatform-for-mariadb-check-for-instance-of-mariadbplatform branch from 1c81f37 to 380b8ce Compare April 9, 2024 12:28
Versions of DBAL before 3.3 used or extended the MySQLPlatform when
connecting to MariaDB. Because of this the platform check was
requiring an instance of MySQLPlatform.

This commit fixes MariaDB compatibility with DBAL >=3.3, while remaining
 compatible with versions prior to that (for now).

Fixes ScientaNL#113
@annervisser annervisser force-pushed the fix-validateplatform-for-mariadb-check-for-instance-of-mariadbplatform branch from 380b8ce to d522183 Compare April 9, 2024 12:30
@annervisser annervisser requested a review from Hikariii April 9, 2024 12:30
@Hikariii Hikariii merged commit 0d9c01a into ScientaNL:master Apr 10, 2024
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.

MariadbJsonFunctionNode checks instance of MySQLPlatform NOT MariaDBPlatform

2 participants