phalcon / migrations

Generate or migrate database changes via migrations.
https://docs.phalcon.io/latest/en/db-migrations
BSD 3-Clause "New" or "Revised" License
28 stars 23 forks source link

[BugFix] Migration log index size fix #148

Open rodriguez-loic opened 5 months ago

rodriguez-loic commented 5 months ago

https://github.com/phalcon/migrations/blob/69c1ea982a94e13510201e642e408dea9d554539/src/Migrations.php#L734

This line should either be configurable dynamically or modified back to a value <=191. (maybe a check on the collation settings from the configuration could do it) Because of the evolution of MySQL Engines, the default utf8 charset used today is utf8mb4 which doesn't allow such length for VARCHAR type.

This issue was already fixed in an old version of phalcon devtools: https://github.com/phalcon/phalcon-devtools/pull/1293