Skip to content

fix(core): don't report changes in dry run of db:add-missing-* commands - #65577

Open
dualfroz wants to merge 1 commit into
nextcloud:masterfrom
dualfroz:fix/39383-dry-run-output
Open

dualfroz wants to merge 1 commit into
nextcloud:masterfrom
dualfroz:fix/39383-dry-run-output

Conversation

@dualfroz

Copy link
Copy Markdown

Summary

with --dry-run nothing gets changed, but the output still says "this can take some time..." and "table updated successfully", so it looks exactly like a real run (see the last comment on the issue). dry run now prints which indices are missing or would be removed and skips the success line, the SQL is still printed at the end. did the same for db:add-missing-columns and db:add-missing-primary-keys, they had the same pattern.

  • Tests are included

With --dry-run nothing is changed, but the commands still printed "this can take some time..." and "table updated successfully", which reads exactly like a real run.

Signed-off-by: dualfroz <me@dualfroz.com>
@dualfroz
dualfroz requested a review from a team as a code owner October 16, 2026 09:52
@dualfroz
dualfroz requested review from Altahrim, CarlSchwan, leftybournes and sorbaugh and removed request for a team October 16, 2026 09:52

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

--dry-run flag in occ db:add-missing-indices partly ignored

1 participant