Remove duplicate unit test in sharing that checks for group permission updates

* it was skipped before anyways
* it is covered for example in build/integration/sharing_features/sharing-v1-part3.feature#L517-L548 (see https://github.com/nextcloud/server/blob/54f8f75f6f82ca102b1cfdb88caf0d06e6c9ba31/build/integration/sharing_features/sharing-v1-part3.feature#L517-L548)
* more permission updates are tested in the webdav section, where an OCS API is called and the WebDAV response is checked for the properly changed permission

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #26188 from nextcloud-pr-bot/automated/noid/psalm-baseline-update

[Automated] Update psalm-baseline.xml

Update psalm baseline

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

[tx-robot] updated from transifex

Merge pull request #24384 from nextcloud/cleanup/remove-old-migration-check

Remove dead code from checking core/apps before upgrades

Skip db migration simulation for core schema changes

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Merge pull request #25924 from nextcloud/bugfix/noid/only-mark-migrations-as-installed-after-executing-them-in-case-the-developer-broke-them

Only mark migrations as installed after execution

Merge pull request #25676 from nextcloud/drop/utl-is-ie

Drop \OCP\Util::isIe

Merge pull request #26160 from nextcloud/enh/eslint

Merge pull request #24580 from nextcloud/enh/noid/db-convert

DB conversion: improve console output

Merge pull request #26161 from nextcloud/bugfix/noid/improve-matching-of-phonebook-searches

Improve search results when only phonebook-matches can we autocompleted

Fix comments filesplugin

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Merge pull request #26076 from nextcloud/enh/noid/ldap-occ-user-reset

adds ldap user:reset command

DB conversion: improve console output

Signed-off-by: Simon Spannagel <simonspa@kth.se>

Merge pull request #26172 from nextcloud/techdebt/noid/code-checker-is-noop

occ app:check-code is noop

Fix TerserPlugin

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Lint fix

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Add eslint testing

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Merge pull request #26165 from nextcloud/dependabot/composer/vimeo/psalm-4.6.4

occ app:check-code is noop

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

adds ldap user:reset command

- allows to delete data of existing LDAP users, which otherwise is safe
  guarded
- ensures that the user is not being deleted on LDAP through a plugin

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Also limit to user group in case enumeration is enabled for groups and phonenumbers

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

Merge pull request #26131 from nextcloud/fix/initialstate-templates

Fix default missing initial state for templates

Bump vimeo/psalm from 4.6.2 to 4.6.4

Bumps [vimeo/psalm](https://github.com/vimeo/psalm) from 4.6.2 to 4.6.4.
- [Release notes](https://github.com/vimeo/psalm/releases)
- [Commits](https://github.com/vimeo/psalm/compare/4.6.2...4.6.4)

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

Fix parameter types in docs

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

Add missing index on the user column

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

Improve search results when only phonebook-matches can we autocompleted

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

Merge pull request #26153 from nextcloud/psalm/26111/fix-undefined-exception-class

Use correct exception type hint in catch statement

[tx-robot] updated from transifex

Use correct exception type hint in catch statement

Found by psalm update 4.6.2 to 4.6.3 in #26111

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #26137 from nextcloud/block-__groupfolders-uid

dont allow creating users with __groupfolders as uid

dont allow creating users with __groupfolders as uid

Fixes https://github.com/nextcloud/groupfolders/issues/338

Signed-off-by: Robin Appelman <robin@icewind.nl>

Merge pull request #26013 from nextcloud/object-store-copy-cache-id

Return the fileid from `copyFromCache` and use it instead of doing an extra query

Merge pull request #26143 from nextcloud/techdebt/noid/fix-cs

Fix CS in CacheTest

Merge pull request #26144 from nextcloud/funding

Add funding info

Add funding info

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>

Fix CS in CacheTest

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

[tx-robot] updated from transifex

Merge pull request #25136 from nextcloud/cachejail-search-filter

do cachejail search filtering in sql

Fix default missing initial state for templates

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Merge pull request #26003 from nextcloud/enh/simplefile/getid

Add getID function to the simplefile implementation

Merge pull request #26115 from nextcloud/enh/noid/log-share-exceptions

Log exceptions when creating share

Log exceptions when creating share

Because the exceptions don't always contain a useful message for the UI,
but also because in some cases we need to find out what went wrong.

In some setups, a ShareNotFoundException might happen during creation
when we try to re-read the just written share. Usually related to Galera
Cluster where node syncing wait is not enabled.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #26097 from nextcloud/dependabot/npm_and_yarn/css-loader-5.1.2

Merge pull request #26100 from nextcloud/dependabot/npm_and_yarn/babel/preset-env-7.13.10

Bump css-loader from 5.1.1 to 5.1.2

Bumps [css-loader](https://github.com/webpack-contrib/css-loader) from 5.1.1 to 5.1.2.
- [Release notes](https://github.com/webpack-contrib/css-loader/releases)
- [Changelog](https://github.com/webpack-contrib/css-loader/blob/master/CHANGELOG.md)
- [Commits](https://github.com/webpack-contrib/css-loader/compare/v5.1.1...v5.1.2)

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

Merge pull request #26099 from nextcloud/dependabot/npm_and_yarn/chai-4.3.4

[tx-robot] updated from transifex

Bump @babel/preset-env from 7.13.9 to 7.13.10

Bumps [@babel/preset-env](https://github.com/babel/babel/tree/HEAD/packages/babel-preset-env) from 7.13.9 to 7.13.10.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.13.10/packages/babel-preset-env)

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

Bump chai from 4.3.3 to 4.3.4

Bumps [chai](https://github.com/chaijs/chai) from 4.3.3 to 4.3.4.
- [Release notes](https://github.com/chaijs/chai/releases)
- [Changelog](https://github.com/chaijs/chai/blob/main/History.md)
- [Commits](https://github.com/chaijs/chai/compare/4.3.3...v4.3.4)

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

verify that cache entry is valid

Signed-off-by: Robin Appelman <robin@icewind.nl>

also implement for FailedCache and NullCache

Signed-off-by: Robin Appelman <robin@icewind.nl>

use returned file id for object store copy

Signed-off-by: Robin Appelman <robin@icewind.nl>

add ICopyFromCache trait to expose existing implementation

Signed-off-by: Robin Appelman <robin@icewind.nl>

Merge pull request #25722 from nextcloud/objectstore-copy-cross

apply object store copy optimization when 'cross storage' copy is wit…

Merge pull request #26049 from nextcloud/search-file-limit-results

limit constructing of result objects in file search

Merge pull request #26079 from nextcloud/bugfix/noid/chunk-phone-search

Chunk the array of phone numbers

Merge pull request #26080 from nextcloud/bugfix/noid/only-clear-known-users-when-we-had-at-least-one-phonebook-entry

Only clear known users when we had at least one phonebook entry

Only clear known users when we had at least one phonebook entry

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

Fix Test class name

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

Chunk the array of phone numbers

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

Merge pull request #26072 from nextcloud/s3-no-explicit-close

remove explicit fclose from S3->writeStream

Merge pull request #26078 from nextcloud-pr-bot/automated/noid/psalm-baseline-update

[Automated] Update psalm-baseline.xml

Update psalm baseline

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

[tx-robot] updated from transifex

Merge pull request #26067 from nextcloud/techdept/psalm/trashbin/execute

Cleanp execute usage in the trashbin

Merge pull request #26066 from nextcloud/techdept/psalm/type_error

Fix type error in CheckBackupCodes bg job

Merge pull request #26068 from nextcloud/bugfix/noid/fix-comments-value-update

Fix comments field value update

remove explicit fclose from S3->writeStream

streams get closed automatically when dropped, and in some cases the stream seems to be already closed by the S3 library, in which case trying to close it again will raise an error

Signed-off-by: Robin Appelman <robin@icewind.nl>

implement PagedProvider for file search

Signed-off-by: Robin Appelman <robin@icewind.nl>

Merge pull request #26064 from nextcloud/techdept/psalm/type_oauth_db

Add psalm annotation to the DB handlers of OAuth2

Cleanp execute usage in the trashbin

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Fix comments field value update

Fixes issue where the comment field value was not properly read out of
the rich content field.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Fix type error in CheckBackupCodes bg job

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Merge pull request #26057 from nextcloud/dependabot/npm_and_yarn/nextcloud/vue-3.7.2

limit constructing of result objects in file search

even thought we currently have no proper way of limiting the search itself, we can at least limit the construction of the result objects.

this saves about 40% of the time spend in the search request in my local testing

Signed-off-by: Robin Appelman <robin@icewind.nl>

Add psalm annotation to the DB handlers of OAuth2

* Psalm annotations
* To new executeUpdate

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Merge pull request #25569 from nextcloud/JakubOnderka-patch-1

file-upload: Correctly handle error responses for HTTP2

Fix SharingInput Multiselect

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Compile js

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Bump @nextcloud/vue from 3.2.0 to 3.7.2

Bumps [@nextcloud/vue](https://github.com/nextcloud/nextcloud-vue) from 3.2.0 to 3.7.2.
- [Release notes](https://github.com/nextcloud/nextcloud-vue/releases)
- [Changelog](https://github.com/nextcloud/nextcloud-vue/blob/master/CHANGELOG.md)
- [Commits](https://github.com/nextcloud/nextcloud-vue/compare/v3.2.0...v3.7.2)

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

Merge pull request #26019 from ste101/patch-1

Allow webp as background image

Merge pull request #26047 from nextcloud/JakubOnderka-patch-2

Fix warning in AppFetcher.php when appstore is disabled

Merge pull request #25911 from nextcloud/bugfix/noid/fix-share-password-saving

Only clear share password model when actually saved

Merge pull request #26032 from nextcloud/techdept/psalm/requesthandlerController_loggerinterface

Move requesthandler controller to LoggerInterface

Merge pull request #26050 from nextcloud/view-search-reuse-usermanager

reuse usermanager instance when processing search results

Only clear share password model when actually saved

Instead of always clearing the password in the share model, now only do
it when it was actually saved.

Fixes an issue where saving another field would clear the password too
early and prevent it to be saved when the dropdown closed.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>

Merge pull request #26015 from nextcloud/dependabot/composer/psr/container-1.1.1

Bump psr/container from 1.0.0 to 1.1.1

Merge pull request #26031 from nextcloud/feature/noid/allow-autocomplete-based-on-phone-sync

Allow autocomplete based on phone sync

[tx-robot] updated from transifex

Clean up the logic of the contacts store a bit

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

Rename some parameters and methods to make the API more clear

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

Bump version to run migration

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

Merge pull request #26048 from nextcloud/urlgenerator-cache-base-url

cache baseurl in url generator

Merge pull request #25940 from nextcloud/dependabot/npm_and_yarn/css-loader-5.1.1

Bump css-loader from 5.0.1 to 5.1.1

Bump psr/container from 1.0.0 to 1.1.1

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

Also clear the knownUser when changing via the settings endpoint

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

reuse usermanager instance when processing search results

saves some time when processing large search results

Signed-off-by: Robin Appelman <robin@icewind.nl>