Skip to content

fix(migration): support false on active boolean column#73

Merged
vitormattos merged 3 commits intomainfrom
fix/issue-58-active-bool-compat
Apr 13, 2026
Merged

fix(migration): support false on active boolean column#73
vitormattos merged 3 commits intomainfrom
fix/issue-58-active-bool-compat

Conversation

@vitormattos
Copy link
Copy Markdown
Member

@vitormattos vitormattos commented Apr 13, 2026

Summary

  • make active nullable in migration Version1000Date20260309120000 to avoid DB compatibility rejection when persisting false
  • add migration Version1001Date20260404010000 to adjust existing installs where active was created as not-null

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
  • docker compose exec -u www-data -w /var/www/html/apps-extra/profile_fields nextcloud composer test:php (fails in API suite with 503 Service unavailable, environment-related)

Closes #58

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos vitormattos merged commit 3af0b01 into main Apr 13, 2026
20 checks passed
@vitormattos vitormattos deleted the fix/issue-58-active-bool-compat branch April 13, 2026 17:57
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