Skip to content

[stable32] refactor: flatten nesting with early returns#2580

Merged
miaulalala merged 1 commit intostable32from
backport/2564/stable32
May 10, 2026
Merged

[stable32] refactor: flatten nesting with early returns#2580
miaulalala merged 1 commit intostable32from
backport/2564/stable32

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot Bot commented May 7, 2026

Backport of PR #2564

@miaulalala miaulalala enabled auto-merge May 7, 2026 12:22
@cypress
Copy link
Copy Markdown

cypress Bot commented May 7, 2026

Activity    Run #3738

Run Properties:  status check passed Passed #3738  •  git commit 4b03e2ccc6: [stable32] refactor: flatten nesting with early returns
Project Activity
Branch Review backport/2564/stable32
Run status status check passed Passed #3738
Run duration 02m 03s
Commit git commit 4b03e2ccc6: [stable32] refactor: flatten nesting with early returns
Committer backportbot[bot]
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 1
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 9
View all changes introduced in this branch ↗︎

- Data::setOffsetFromSince(): invert guards with early returns and extract
  the repeated fallback fetch into getFirstKnownActivityHeader()
- MailQueueHandler::getAffectedUsers(): early return in the restrictEmails
  branch and extract the duplicated result loop into fetchAffectedUsers()

Signed-off-by: Anna Larch <anna@nextcloud.com>
AI-Assisted-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@miaulalala miaulalala force-pushed the backport/2564/stable32 branch from 7368843 to cad5534 Compare May 10, 2026 09:01
@miaulalala miaulalala merged commit 2c5f60f into stable32 May 10, 2026
52 checks passed
@miaulalala miaulalala deleted the backport/2564/stable32 branch May 10, 2026 09:06
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.

1 participant