Skip to content

[stable29] fix(IntegrityCheck): Ensure the check is run if no results are available - #46298

Merged
susnux merged 1 commit into
stable29from
backport/46174/stable29
Jul 10, 2024
Merged

[stable29] fix(IntegrityCheck): Ensure the check is run if no results are available#46298
susnux merged 1 commit into
stable29from
backport/46174/stable29

Conversation

@backportbot

Copy link
Copy Markdown

Backport of PR #46174

@backportbotbackportbotBot added bug 3. to review Waiting for reviews feature: settings php Pull requests that update Php code labels Jul 4, 2024
@backportbotbackportbotBot added this to the Nextcloud 29.0.4 milestone Jul 4, 2024
@susnux

susnux commented Jul 4, 2024

Copy link
Copy Markdown
Contributor
  • TODO: Remove union types not support on stable29...

@susnuxsusnux added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Jul 4, 2024
If there are no cached results the current implementation was also returning an empty array,
but this was the same as when there was a successful run.
So to distinguish this we return `null` if there are *no* results.
In this case we need to rerun the integrity checker.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnuxforce-pushed the backport/46174/stable29 branch from 1d784b8 to 57c889fCompareJuly 5, 2024 10:32
@susnuxsusnux added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Jul 5, 2024
@susnux

Copy link
Copy Markdown
Contributor

Issues resolved, can be reviewed

@AltahrimAltahrim mentioned this pull request Jul 10, 2024
@susnux
susnux merged commit 2071f4d into stable29Jul 10, 2024
@susnux
susnux deleted the backport/46174/stable29 branch July 10, 2024 17:01
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugfeature: settingsphpPull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@susnux@AndyScherzinger