adapt tests for nc18

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

do not flip available state to unavailable, allow empty results

- the detection relies that the first, requested result is not empty
- it might be empty though – groups without members
- protect switching from available to unavailable
  - switching the other way around was also not envisaged either

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

LDAP: shortcut in reading nested group members when IN_CHAIN is available

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

Merge pull request #22184 from nextcloud/backport/21173/stable18

[stable18] Fix empty 'more' apps navigation after installing an app

Merge pull request #22191 from nextcloud/backport/22190/stable18

[stable18] Fix default log_rotate_size in config.sample.php

Fix default log_rotate_size in config.sample.php

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

Merge pull request #22179 from nextcloud/backport/22176/stable18

[stable18] Remove unexpected argument

Fix empty 'more' apps navigation after installing an app

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Merge pull request #22172 from nextcloud/backport/21728/stable18

[stable18] Reuse cache for directory mtime/size if filesystem changes can be ignored

Remove unexpected argument

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

[tx-robot] updated from transifex

Merge pull request #21982 from nextcloud/backport/21628/stable18

[stable18] fix moving files from external storage to object store trashbin

Reuse cache for directory mtime/size if filesystem changes can be ignored

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #22054 from nextcloud/backport/22015/stable18

[stable18] Correctly remove usergroup shares on removing group members

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

fix object store trashbin handling

object stores are "special" given how they interact with the cache on their own

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

dont update mimetype when moving to trash

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

fix renameFromStorage messing with folder mimetype

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

use exceptions for error signaling in writeStream

this remove the ambiguity when writing zero length files

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

rollback cache rename if trashbin move fails

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

fix moving files from external storage to object store trashbin

having the "cache rename" after the "storage move" caused the target
to get the fileid from the source file, without taking care that the object
is stored under the original file id.

By doing the "cache rename" first, we trigger the "update existing file"
logic while moving the file to the object store and the object gets stored for the
correct file id

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

Merge pull request #22071 from nextcloud/backport/22005/stable18

[stable18] Fix height to big for iPhone when using many apps

Fix height to big for iPhone when using many apps

Same story as https://github.com/nextcloud/server/pull/10276
I'm testing on iPhone without home button. These devices now have an even higher bottom bar.

[tx-robot] updated from transifex

Merge pull request #22051 from nextcloud/dependabot/npm_and_yarn/stable18/elliptic-6.5.3

Merge pull request #22058 from nextcloud/backport/21972/stable18

[stable18] Fix PHPUnit deprecation warnings

Remove deprecated test of internal attributes via assertAttributeEquals in constructor tests

I removed the tests completely because they just test that the constructor assigns the values to the internal properties. Nothing that should be cared about from the outside.

See https://github.com/sebastianbergmann/phpunit/issues/3339#issuecomment-428843322

It is seen as bad practice to test internal stuff of objects instead of the actual input and output of mathod calls.

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

Replace deprecated assertArraySubset with logic that does the same

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

Use assertEqualsCanonicalizing instead of deprecated assertEquals parameter

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

Remove deprecated test of internal attributes via assertAttributeEquals

See https://github.com/sebastianbergmann/phpunit/issues/3339#issuecomment-428843322

It is seen as bad practice to test internal stuff of objects instead of the actual input and output of mathod calls.

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

Use assertStringContainsString instead of assertContains on strings

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

Use old IServerContainer methods

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

Correctly remove usergroup shares on removing group members

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

[Security] Bump elliptic from 6.5.2 to 6.5.3

Bumps [elliptic](https://github.com/indutny/elliptic) from 6.5.2 to 6.5.3. **This update includes a security fix.**
- [Release notes](https://github.com/indutny/elliptic/releases)
- [Commits](https://github.com/indutny/elliptic/compare/v6.5.2...v6.5.3)

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

Merge pull request #21976 from nextcloud/backport/21658/stable18

[stable18] allow admin to configure the max trashbin size

Merge pull request #22035 from nextcloud/backport/21950/stable18

[stable18]  Properly fetch translation for remote wipe confirmation dialog

Properly fetch translation for remote wipe confirmation dialog

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #21992 from nextcloud/backport/21977/stable18

[stable18] Ignore whitespace in sharing by mail

[tx-robot] updated from transifex

Ignore whitespace in sharing by mail

Signed-off-by: onehappycat <one.happy.cat@gmx.com>

allow admin to configure the max trashbin size

this allows an admin to configure the max trashbin size instead of always relying on the users quota.

The trashbin size can be configured using the `occ trash:size` command and can be set both globally and per-user

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

Merge pull request #21979 from nextcloud/backport/21973/stable18

[stable18] Fix risky test in twofactor_backupcodes

[tx-robot] updated from transifex

Fix risky test in twofactor_backupcodes

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

Merge pull request #21807 from nextcloud/backport/21748/stable18

[stable18] Improve error reporting on sharing errors

Improve error reporting on sharing errors

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

Merge pull request #21961 from nextcloud/backport/21760/stable18

[stable18] Fix parsing of language code

Bump bundles

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

Fix parsing of language code

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #21936 from nextcloud/backport/21929/stable18

[stable18] Encryption is ready if master key is enabled

Encryption is ready if master key is enabled

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

[tx-robot] updated from transifex

Merge pull request #21877 from nextcloud/backport/21855/stable18

[stable18] fix typo in revokeShare()

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #20842 from nextcloud/backport/20832/stable18

[stable18] Do not filter id matching userId on non-user-share shares

Merge pull request #21841 from nextcloud/backport/21838/stable18

[stable18] Do not log RequestedRangeNotSatisfiable exceptions in DAV

Merge pull request #21783 from nextcloud/backport/21777/stable18

[stable18] lower minimum search length to 2 characters

fix typo

[tx-robot] updated from transifex

Merge pull request #21802 from nextcloud/version/18.0.7/final

18.0.7 final

[tx-robot] updated from transifex

Do not log RequestedRangeNotSatisfiable exceptions in DAV

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

Do not filter id matching userId on non-user-share shares

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

[tx-robot] updated from transifex

18.0.7 final

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

lower minimum search length to 2 characters

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

Merge pull request #21700 from nextcloud/version/18.0.7/RC1

18.0.7 RC1

Merge pull request #21781 from nextcloud/backport/21778/stable18

[stable18] Set the moment locale even earlier

Merge pull request #21773 from nextcloud/backport/21489/stable18

[stable18] Use the correct mountpoint to calculate

Set the moment locale even earlier

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

better cleanup in share tests

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

ensure home storage is initialized on first setup

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

ensure mounts are scanned during tests

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

Add more integration tests for resharing permissions

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Use the correct mountpoint to calculate

If we use the owners mount point this results in null. And then the rest
of the checks get called with null. Which doesn't work.

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

[tx-robot] updated from transifex

Merge pull request #21752 from nextcloud/backport/21706/stable18

[stable18] Add a clear message why you could end up there

Add a clear message why you could end up there

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

Merge pull request #21499 from lmamane/stable18-21285-oneliner

fix #21285 as oneliner

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #21575 from nextcloud/backport/21379/stable18

[stable18] Fix share permission checkboxes enabled when permissions can not be set