Skip to content

Restored interface methods on obsolete IBackgroundTaskQueue#19223

Merged
kjac merged 2 commits intorelease/15.4.1from
v15/bugfix/restore-interface-methods
May 5, 2025
Merged

Restored interface methods on obsolete IBackgroundTaskQueue#19223
kjac merged 2 commits intorelease/15.4.1from
v15/bugfix/restore-interface-methods

Conversation

@AndyButland
Copy link
Contributor

@AndyButland AndyButland commented May 3, 2025

Prerequisites

  • I have added steps to test this contribution in the description below

Resolves umbraco/Umbraco.Forms.Issues#1393

Description

This PR addresses the inadvertent breaking change introduced in 15.4 (see discussion in comment on linked issue).

Although this didn't get picked up by the usual static package validation checks, it wasn't sufficient to inherit from the newly introduced interface in the core project and expect the methods defined on the obsolete one to be found. Rather we need to keep the methods on the obsoleted interface too.

Testing

I've verified the following using a standalone install of the CMS with Forms, comparing before and after using the packages generated from the build pipeline for this PR.

Install CMS 15.4 and Forms 15.1.1 and after submitting a form a missing method exception will be thrown.

With this PR in place, you should find that it no longer will be and the Form's Examine index will be populated as expected with the form submission information.

Also verify that the publish with descendants works without error (as this uses the introduced interface in core).

Copilot AI review requested due to automatic review settings May 3, 2025 15:32
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR restores the interface methods on the obsolete IBackgroundTaskQueue to maintain backward compatibility with the earlier interface definition.

  • Added QueueBackgroundWorkItem and DequeueAsync methods with appropriate documentation.
  • Ensures the obsolete interface continues to support legacy method signatures.

@AndyButland AndyButland changed the title Restored interface methods on obsolete IBackgroundTaskQueue. Restored interface methods on obsolete IBackgroundTaskQueue May 5, 2025
Copy link
Contributor

@kjac kjac left a comment

Choose a reason for hiding this comment

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

Works like a charm 👍

@kjac kjac merged commit 0ad020f into release/15.4.1 May 5, 2025
21 checks passed
@kjac kjac deleted the v15/bugfix/restore-interface-methods branch May 5, 2025 05:38
@kjac kjac added release/15.4.1 status/regression A previously working feature that has broken or changed behavior unexpectedly labels May 5, 2025
AndyButland added a commit that referenced this pull request May 5, 2025
* Restore interface methods on obsolete IBackgroundTaskQueue.

* Fixed typos in comment.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release/15.4.1 status/regression A previously working feature that has broken or changed behavior unexpectedly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants