Skip to content

fix(SchemaChecker): replay migrations for disabled apps in 'expected' schema - #63701

Open
Antreesy wants to merge 3 commits into
masterfrom
fix/63351/schema-checker-disabled
Open

fix(SchemaChecker): replay migrations for disabled apps in 'expected' schema#63701
Antreesy wants to merge 3 commits into
masterfrom
fix/63351/schema-checker-disabled

Conversation

@Antreesy

@Antreesy Antreesy commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Summary

Also replay migrations for disabled apps
⚠️ Only in expected schema, it's a dry run, no live db is touched

  • otherwise tables from disabled apps rendered as 'unexpected'
  • should only be true for actually removed apps
  • do not exit with 1 if there are only findings in disabled apps
B A
image image

TODO

  • ...

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@Antreesy Antreesy added this to the Nextcloud 35 milestone Aug 27, 2026
@Antreesy Antreesy self-assigned this Aug 27, 2026
@Antreesy
Antreesy requested a review from a team as a code owner August 27, 2026 09:48
@Antreesy
Antreesy requested review from Altahrim, icewind1991, leftybournes and salmart-dev and removed request for a team August 27, 2026 09:48
@Antreesy Antreesy changed the title fix(SchemaChecker): also replay migrations for disabled apps fix(SchemaChecker): replay migrations for disabled apps in 'expected' schema Aug 27, 2026
Comment thread lib/private/DB/SchemaChecker.php Outdated
- otherwise tables from disabled apps rendered as 'unexpected'
- should only be true for actually removed apps
- do not exit with 1 if there are only findings in disabled apps

Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
Antreesy force-pushed the fix/63351/schema-checker-disabled branch from c354f8e to 6693ef0 Compare August 31, 2026 14:09
Comment thread core/Command/Db/CheckSchema.php Outdated
Comment thread core/Command/Upgrade.php Outdated
Comment thread lib/private/DB/SchemaChecker.php Outdated
Comment thread lib/private/DB/SchemaChecker.php Outdated
Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
- db:schema:check now prints disabled-app findings only with -v.
- Upgrade no longer reports disabled-app schema drift at all.

Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
Antreesy requested a review from come-nc September 1, 2026 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants