fix: Skip users that still exist in backend

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

fix: Make user removal more resilient

Currently there is a problem if an exception is thrown in `User::delete`,
because at that point the user is already removed from the backend,
but not all data is deleted.

There is no way to recover from this state, as the user is gone no information is available anymore.
This means the data is still available on the server but can not removed by any API anymore.

The solution here is to first set a flag and backup the user home,
this can be used to recover failed user deletions in a way the delete can be re-tried.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #48370 from nextcloud/dependabot/composer/laravel/serializable-closure-1.3.5

build(deps): Bump laravel/serializable-closure from 1.3.3 to 1.3.5

build(deps): Bump laravel/serializable-closure from 1.3.3 to 1.3.5

Signed-off-by: Joas Schilling <coding@schilljs.com>

Merge pull request #48366 from nextcloud/fix/files_sharing/ocm-permissions

fix(files_sharing): Parse OCM share permissions from OCM and not OCS prop

Merge pull request #48153 from nextcloud/feat/shipped/app_api

feat(app_api): make app_api shipped and default enabled

Merge pull request #48375 from nextcloud/bugfix/noid/skip-future-shipped-apps-from-updatenotification-check

Skip future shipped apps from updatenotification check

Merge pull request #48361 from nextcloud/fix/querybuilder/oracle-indentifier-length

fix(QueryBuilder): Restrict identifier length to 30 characters due to Oracle limitations

Merge pull request #48359 from nextcloud/artonge/feat/compare_hashed_password_when_updating_global_cred_in_files_external

fix: Use placeholder for external storage password

fix(files_sharing): Check if propfind response is valid before accessing share permission props

Signed-off-by: provokateurin <kate@provokateurin.de>

refactor(files_sharing): Make permissions prop checks less error prone

Signed-off-by: provokateurin <kate@provokateurin.de>

fix(updatenotification): Communicate app_api as shipped in 30+

Signed-off-by: Joas Schilling <coding@schilljs.com>

fix(updatenotification): Communicate twofactor_nextcloud_notification as shipped in 30+

Signed-off-by: Joas Schilling <coding@schilljs.com>

fix(updatenotification): Communicate files_downloadlimit as shipped in 29+

Signed-off-by: Joas Schilling <coding@schilljs.com>

Merge pull request #48343 from nextcloud/dependabot/composer/giggsey/libphonenumber-for-php-lite-8.13.45

chore(deps): Bump giggsey/libphonenumber-for-php-lite to 8.13.45

Merge pull request #48354 from nextcloud/artonge/fix/storage_full_warning

fix: Show storage full warning when storage is actually full

chore(deps): Bump giggsey/libphonenumber-for-php-lite to 8.13.45

Signed-off-by: Joas Schilling <coding@schilljs.com>

fix(files_sharing): Parse OCM share permissions from OCM and not OCS prop

Signed-off-by: provokateurin <kate@provokateurin.de>

Merge pull request #48345 from nextcloud/fix/files_external/forbidden-exception

fix(files_external): Catch correct exception

fix(files_external): Catch correct exception

SMB#getFileInfo used to throw an icewind/smb exception, but nowadays throws \OCP\Files\ForbiddenException. This fixes downstream methods to catch the new exception.

Signed-off-by: Marcel Klehr <mklehr@gmx.net>

fix(QueryBuilder): Restrict identifier length to 30 characters due to Oracle limitations

Signed-off-by: provokateurin <kate@provokateurin.de>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

fix: Show storage full warning when storage is actually full

< 0 values for "free" do not mean that the storage is full: https://github.com/nextcloud/server/blob/6befdd6dd7cd20fe7f7036e4665bcfbb783d6803/lib/public/Files/FileInfo.php#L31-L42

Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

Merge pull request #48353 from nextcloud/enh/30551/weather-status-support-more-codes

[WeatherStatus] Support more weather codes, change icons, fix style

fix: Use hashed password in files_external settings

Signed-off-by: Louis Chemineau <louis@chmn.me>

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

fix(weatherstatus): adjustments from review comments

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>

fix(weatherstatus): fix location marker icon with dark theme, only use icons from @mdi/svg

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>

fix(weatherstatus): make a difference between 'unknown weather code' and 'no weather info'

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>

feat(weatherstatus): support all weather codes from api.met.no, use the icons provided by met.no, cleanup UI

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>

Fix #30551: Added snowfall option to weather status

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>

Merge pull request #48356 from nextcloud/bugfix/noid/document-icon-requirements

fix(icons): Document icon requirements for notifications and activities

fix(icons): Document icon requirements for notifications and activities

Signed-off-by: Joas Schilling <coding@schilljs.com>

feat: add app_api to shipped and default enabled apps

Signed-off-by: Andrey Borysenko <andrey18106x@gmail.com>

Merge pull request #48331 from nextcloud/fix/duplicated-conflict-resolution

fix(files): Fix having to resolve conflicts twice when dropping files

Merge pull request #48344 from nextcloud/bugfix/noid/more-reliable-tests

More reliable user_status tests

fix(tests): Check that the revert was successful

Signed-off-by: Joas Schilling <coding@schilljs.com>

fix(tests): Also check that the backup status works

Signed-off-by: Joas Schilling <coding@schilljs.com>

fix(tests): Make timing test more reliable

Signed-off-by: Joas Schilling <coding@schilljs.com>

Merge pull request #48327 from nextcloud/dependabot/composer/web-auth/webauthn-lib-4.9.1

chore(deps): Bump web-auth/webauthn-lib from 4.8.5 to 4.9.1

chore(deps): Bump web-auth/webauthn-lib from 4.8.5 to 4.9.1

Signed-off-by: Joas Schilling <coding@schilljs.com>

Merge pull request #48333 from nextcloud/automated/noid/master-update-ca-cert-bundle

[master] fix(security): Update CA certificate bundle

fix(security): Update CA certificate bundle

Signed-off-by: GitHub <noreply@github.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

fix(files): Fix having to resolve conflicts twice when dropping files

Signed-off-by: Christopher Ng <chrng8@gmail.com>

Merge pull request #48244 from nextcloud/fix/sharing-entry-link

fix(files): Link share URL was not generated due to typo

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

fix(files): Link share URL was not generated due to typo

`toen` vs `token`

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #48323 from nextcloud/chore/legacy-updatenotification

refactor(updatenotification): Migrate legacy code

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

refactor(updatenotification): Migrate legacy code

1. Remove hook usage and just provide an initial state
2. Replace jQuery code with modern non-deprecated frontend code

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #32018 from nextcloud/cleanup/event/trashbin

Port files trashbin events to IEventDispatcher/IEventListener

Merge pull request #48307 from nextcloud/fix/gracefully-parse-trusted-certificates

Merge pull request #48311 from nextcloud/fix/app-discover

fix: Ensure app discover section is returned as list

fix: gracefully parse non-standard trusted certificates

Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>

fix: Ensure app discover section is returned as list

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #47349 from nextcloud/feat/31420/bidi-backend-support

Add bidirectional text support - Backend

fix(tests): Pause trashbin for lock tests in ViewTest using rmdir and unlink

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Merge pull request #44945 from nextcloud/rakekniven-patch-1

fix(i18n): Changed grammar

fix(tests): Fix tests now that trashbin listens to events properly

Hooks are cleared in test bootstrap so switching to events activates
 them in tests.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

chore: Use SPDX headers and property promotion in new files

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

feat(trashbin): Port files trashbin events to IEventDispatcher/IEventListener

oc_hooks is deprecated and will at some point be removed

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Signed-off-by: Thomas Citharel <tcit@tcit.fr>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Merge pull request #48268 from nextcloud/debt/noid/ignore-docker-image-lock-file

fix: skip the integrity check for nextcloud-init-sync.lock

fix(rtl): Make clear that the direction is based on language

Signed-off-by: Joas Schilling <coding@schilljs.com>

ci: Restrict RTL characters to RTL languages

Signed-off-by: Joas Schilling <coding@schilljs.com>

fix(rtl): Fix list of RTL languages

Signed-off-by: Joas Schilling <coding@schilljs.com>

fix(rtl): Add unittest for language direction

Signed-off-by: Mostafa Ahangarha <ahangarha@riseup.net>

feat(rtl): Set layout direction based on language

Signed-off-by: ali ghorbani <ghorbani.ali.developer@gmail.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

chore(assets): Recompile assets

Signed-off-by: Joas Schilling <coding@schilljs.com>

fix(translations): Changed grammar

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>

Merge pull request #48274 from nextcloud/feat/noid/priority-notifications

feat(prioritynotifications): Allow some apps to mark notifications as priority

Merge pull request #48111 from nextcloud/fix/move-storage-constructor-to-specific-interface

fix: Move storage constructor to specific interface

Merge pull request #48223 from nextcloud/feat/add-rector-config

Add rector config

fix(tests): Fix test to remove call to non-existing constructor

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Merge pull request #48175 from nextcloud/followup/48086/fix-more-activity-providers

fix(activity): Update more activity providers to use new exceptions

Merge pull request #48215 from nextcloud/bugfix/noid/run-all-unit-tests

fix(tests): Fix test selection and run unit tests of DAV and user_status

fix(activity): Update more activity providers to use new exceptions

Signed-off-by: Joas Schilling <coding@schilljs.com>

Merge pull request #48207 from nextcloud/artonge/fix/prevent_missing_users_from_crashing_disabled_users_search

fix: Don't crash if disabled user is missing in the database

chore(rector): Move rector.php into the build folder

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Merge pull request #48275 from nextcloud/dependabot/github_actions/github-actions-560ea78344

chore(deps): Bump the github-actions group across 1 directory with 3 updates

chore: use a proper `@param` tag for IConstructableStorage constructor parameter

Co-authored-by: Kate <26026535+provokateurin@users.noreply.github.com>
Signed-off-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com>

fix(users): Don't crash if disabled user is missing in the database

Signed-off-by: Louis Chemineau <louis@chmn.me>

chore(deps): Bump the github-actions group across 1 directory with 3 updates

Bumps the github-actions group with 3 updates in the / directory: [actions/setup-node](https://github.com/actions/setup-node), [cypress-io/github-action](https://github.com/cypress-io/github-action) and [peter-evans/create-pull-request](https://github.com/peter-evans/create-pull-request).

Updates `actions/setup-node` from 4.0.3 to 4.0.4
- [Release notes](https://github.com/actions/setup-node/releases)
- [Commits](https://github.com/actions/setup-node/compare/1e60f620b9541d16bece96c5465dc8ee9832be0b...0a44ba7841725637a19e28fa30b79a866c81b0a6)

Updates `cypress-io/github-action` from 6.7.5 to 6.7.6
- [Release notes](https://github.com/cypress-io/github-action/releases)
- [Changelog](https://github.com/cypress-io/github-action/blob/master/CHANGELOG.md)
- [Commits](https://github.com/cypress-io/github-action/compare/496e7dc0edc421a9de8a36a31c793340e00c61bf...0da3c06ed8217b912deea9d8ee69630baed1737e)

Updates `peter-evans/create-pull-request` from 6.1.0 to 7.0.5
- [Release notes](https://github.com/peter-evans/create-pull-request/releases)
- [Commits](https://github.com/peter-evans/create-pull-request/compare/c5a7806660adbe173f04e3e038b0ccdcd758773c...5e914681df9dc83aa4e4905692ca88beb2f9e91f)

---
updated-dependencies:
- dependency-name: actions/setup-node
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: github-actions
- dependency-name: cypress-io/github-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: github-actions
- dependency-name: peter-evans/create-pull-request
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <support@github.com>

fix: Move storage constructor to specific interface

That allows Wrappers to use DI and not care about the constructor

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Merge pull request #48266 from nextcloud/fix/psalm/enabled-find-unused-baseline-entry

fix(psalm): Enable findUnusedBaselineEntry

Signed-off-by: provokateurin <kate@provokateurin.de>

feat(prioritynotifications): Allow some apps to mark notifications as priority

They will be still send as push during DND.
Apps are currently limited to:
- twofactor_nextcloud_notification to help with login
- spreed which will only set it for pushes in manually picked conversations

Signed-off-by: Joas Schilling <coding@schilljs.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

fix: skip the integrity check for nextcloud-init-sync.lock

nextcloud-init-sync.lock is used by nextcloud/docker to prevent running the initialization script on multiple containers at the same time.

Ref: https://github.com/nextcloud/docker/issues/2299.

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #48252 from nextcloud/dependabot/npm_and_yarn/sass-1.79.3

chore(deps-dev): Bump sass from 1.77.8 to 1.79.3

Bumps [sass](https://github.com/sass/dart-sass) from 1.77.8 to 1.79.3.
- [Release notes](https://github.com/sass/dart-sass/releases)
- [Changelog](https://github.com/sass/dart-sass/blob/main/CHANGELOG.md)
- [Commits](https://github.com/sass/dart-sass/compare/1.77.8...1.79.3)

---
updated-dependencies:
- dependency-name: sass
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

Merge pull request #48251 from nextcloud/dependabot/npm_and_yarn/vitest/coverage-v8-2.1.1

Merge pull request #48255 from nextcloud/dependabot/npm_and_yarn/tslib-2.7.0

Merge pull request #48246 from nextcloud/bug/noid/profile-clear-not-working

fix(profiler): clear command does not work

Merge pull request #48254 from nextcloud/dependabot/npm_and_yarn/nextcloud/axios-2.5.1

Merge pull request #48253 from nextcloud/dependabot/npm_and_yarn/focus-trap-7.6.0

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>