Skip to content

Fix one core migration: Only add taskprocessing columns if they don't exist - #63350

Merged
joshtrichards merged 1 commit into
masterfrom
fix/noid/taskprocessing-webhook-migration
Aug 19, 2026
Merged

Fix one core migration: Only add taskprocessing columns if they don't exist#63350
joshtrichards merged 1 commit into
masterfrom
fix/noid/taskprocessing-webhook-migration

Conversation

@julien-nc

@julien-nc julien-nc commented Aug 18, 2026

Copy link
Copy Markdown
Member

Make OC\Core\Migrations\Version30000Date20240717111406 safer when adding columns in taskprocessing_tasks.

This migration step could not be executed more than once because it always tried to add 2 columns, whether they exist or not.

Checklist

AI (if applicable)

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

…n adding columns in taskprocessing_tasks

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
@julien-nc julien-nc added this to the Nextcloud 35 milestone Aug 18, 2026
@julien-nc
julien-nc requested a review from a team as a code owner August 18, 2026 08:27
@julien-nc julien-nc added the bug label Aug 18, 2026
@julien-nc
julien-nc requested review from leftybournes, provokateurin and salmart-dev and removed request for a team August 18, 2026 08:27
@julien-nc julien-nc added the 3. to review Waiting for reviews label Aug 18, 2026
@julien-nc
julien-nc requested a review from come-nc August 18, 2026 08:27
@julien-nc

Copy link
Copy Markdown
Member Author

/backport to stable30

@julien-nc

Copy link
Copy Markdown
Member Author

/backport to stable31

@julien-nc

Copy link
Copy Markdown
Member Author

/backport to stable32

@julien-nc

Copy link
Copy Markdown
Member Author

/backport to stable33

@julien-nc

Copy link
Copy Markdown
Member Author

/backport to stable34

@nextcloud-bot nextcloud-bot mentioned this pull request Aug 18, 2026

@joshtrichards joshtrichards left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should maybe do audit for these in all existing migrations -- this has come up several times before IIRC...

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.

[Bug]: Upgrade fails with 'The column "webhook_uri" on table "oc_taskprocessing_tasks" already exists.'

3 participants