Fix MySQL/MariaDB platform detection for DBAL 4 compatibility

[henryfoster]

Summary

This PR updates the MySQL platform detection logic in ORMDatabaseTool to ensure compatibility with Doctrine DBAL 4.

The Issue

In Doctrine DBAL 4, MariaDBPlatform no longer extends MySQLPlatform but extends AbstractMySQLPlatform directly. This caused the instanceof MySQLPlatform check in isMysql() to return false on MariaDB setups, preventing the bundle from correctly disabling/enabling foreign key checks when purging the database with truncate.

The Fix

I've updated isMysql() to use the already existing getPlatformName() method from AbstractDbalDatabaseTool.
This method already has the check for the newer AbstractMySQLPlatform which is the highest common ancestor of both MySQLPlatform and MariaDBPlatform.

This ensures that the platform check isMysql() inside disableForeignKeyChecksIfApplicable and enableForeignKeyChecksIfApplicable returns true and foreign key checks are disabled/enabled correctly when called using a MariaDB database with DBAL 4.

Testing

• All tests passed.

[alexisLefebvre]

Thanks for the PR.

PHPStan reports an error about AbstractPlatform that is not used here, that is strange.

I’m not sure that DBAL 2.x or 3.2.x could be installed other dependencies may require newer versions.

What issue did you encounter? Could you please share your composer.json?

[henryfoster]

Hi @alexisLefebvre I have updated my commit with what I think is a better solution.
I also updated the description and hope the issue is now clearer. Let me know if it is not.

What issue did you encounter?

When using MariaDB with DBAL 4, isMysql() returned false, so foreign key checks were not disabled before truncating, which caused constraint violations during fixture purging.

I can't share the composer.json but I could create a small reproducer on the weekend if that would help.

[Copilot]

Pull request overview

This PR updates the MySQL/MariaDB platform detection logic in ORMDatabaseTool to ensure compatibility with Doctrine DBAL 4, where MariaDBPlatform no longer extends MySQLPlatform but extends AbstractMySQLPlatform directly.

Changes:

• Replaced instanceof MySQLPlatform check with string comparison using getPlatformName() method
• Removed unused MySQLPlatform import

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[alexisLefebvre]

Thanks!

Here is a release with this fix: https://github.com/liip/LiipTestFixturesBundle/releases/tag/3.8.1