Skip to content

fix(migration): shorten index name for DB compatibility#74

Merged
vitormattos merged 2 commits intomainfrom
fix/issue-65-index-name-too-long
Apr 13, 2026
Merged

fix(migration): shorten index name for DB compatibility#74
vitormattos merged 2 commits intomainfrom
fix/issue-65-index-name-too-long

Conversation

@vitormattos
Copy link
Copy Markdown
Member

@vitormattos vitormattos commented Apr 13, 2026

Summary

  • shorten the index identifier for (active, sort_order) in migration Version1000Date20260309120000
  • prevents installation failure on DB engines with stricter identifier length limits

Validation

  • docker compose exec -u www-data -w /var/www/html/apps-extra/profile_fields nextcloud composer cs:fix
  • docker compose exec -u www-data -w /var/www/html/apps-extra/profile_fields nextcloud composer psalm
  • docker compose exec -u www-data -w /var/www/html/apps-extra/profile_fields nextcloud composer test:unit
  • docker compose exec -u www-data -w /var/www/html/apps-extra/profile_fields nextcloud composer test:php:controller-integration

Closes #65

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos vitormattos merged commit 4cbb9ac into main Apr 13, 2026
2 checks passed
@vitormattos vitormattos deleted the fix/issue-65-index-name-too-long branch April 13, 2026 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant