Skip to content

[stable32] fix(Apps): fix install command check on existing apps - #55832

Merged
nickvergessen merged 1 commit into
stable32from
backport/55395/stable32
Oct 28, 2025
Merged

[stable32] fix(Apps): fix install command check on existing apps#55832
nickvergessen merged 1 commit into
stable32from
backport/55395/stable32

Conversation

@backportbot

Copy link
Copy Markdown

Backport of PR #55395

@backportbot
backportbotBot requested a review from a team as a code ownerOctober 17, 2025 14:22
@backportbot
backportbotBot requested review from CarlSchwan, blizzz, come-nc, leftybournes, nickvergessen and salmart-dev and removed request for a teamOctober 17, 2025 14:22
@backportbotbackportbotBot added this to the Nextcloud 32.0.1 milestone Oct 17, 2025
This was referenced Oct 17, 2025
- AppManager::isInstalled() is misleading, as it checks only whether it is
enabled. But an app might not be present in some edge cases.
- AppManager::getAppPath() does however only check whether an app dir is
present, independent of the enabled-state.
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz
blizzzforce-pushed the backport/55395/stable32 branch from 170ad95 to 8afffd8CompareOctober 27, 2025 20:45
@nickvergessen
nickvergessen merged commit fcc36ab into stable32Oct 28, 2025
193 of 198 checks passed
@nickvergessen
nickvergessen deleted the backport/55395/stable32 branch October 28, 2025 07:26
@blizzzblizzz mentioned this pull request Nov 10, 2025
9 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugfeature: occ

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@blizzz@come-nc@nickvergessen@Altahrim