phalcon / phalcon/migrations

[BugFix] Migration log index size fix

Open
#148 0 comments 0 reactions 1 assignee View on GitHub

@niden is already working on this.

Since Jul 4, 2026.

Dominant language
PHP
Stars
28
Forks
25
Avg merge
9h 39m
Merged PRs (30d)
2

Description

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

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.