Always call flush() as getAllKeys() is broken

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

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

Merge pull request #30809 from nextcloud/dependabot/composer/build/integration/stable21/sabre/dav-4.3.1

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

Update sabre/dav requirement from 4.3.0 to 4.3.1 in /build/integration

Updates the requirements on [sabre/dav](https://github.com/sabre-io/dav) to permit the latest version.
- [Release notes](https://github.com/sabre-io/dav/releases)
- [Changelog](https://github.com/sabre-io/dav/blob/master/CHANGELOG.md)
- [Commits](https://github.com/sabre-io/dav/compare/4.3.0...4.3.1)

---
updated-dependencies:
- dependency-name: sabre/dav
  dependency-type: direct:development
...

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

[tx-robot] updated from transifex

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

Merge pull request #30787 from nextcloud/fix-backport/30470/stable21

[tx-robot] updated from transifex

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

Fix method for 21

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

Add hashed attribute column for indexed job existence check

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

Revert "[stable21] Hash job argument"

Merge pull request #30767 from nextcloud/release/21.0.8rc3

Merge pull request #30693 from nextcloud/backport/29923/stable21

Merge pull request #30660 from nextcloud/backport/29622/stable21

21.0.8 RC3

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

[tx-robot] updated from transifex

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

Merge pull request #30755 from nextcloud/backport/30738/stable21

Merge pull request #30749 from nextcloud/backport/30615/stable21

Create block-merge-freeze.yml

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

[tx-robot] updated from transifex

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

Update autoload

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

Improve installation pages

Signed-off-by: Christopher Ng <chrng8@gmail.com>
(cherry picked from commit 22768769c3f29f9952110f86a032e4bf3a4bf460)
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

Merge pull request #30737 from nextcloud/backport/30470/stable21

[stable21] Hash job argument

Add hashed attribute column for indexed job existence check

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

[tx-robot] updated from transifex

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

Merge pull request #30680 from nextcloud/release/21.0.8rc2

21.0.8 RC2

Merge pull request #30633 from nextcloud/backport/30358/stable21

[stable21] Reset job disabling timer on adding the job again

Merge pull request #30682 from nextcloud/backport/30531/stable21

[stable21] Optimize FileSystemTags workflow for groupfolder

Merge pull request #30686 from nextcloud/backport/30499/stable21

[tx-robot] updated from transifex

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

Merge pull request #30665 from nextcloud/backport/30530/stable21

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

files_external SMB: throw InvalidArgument when user is not set

Signed-off-by: Anderson Luiz Alves <alacn1@gmail.com>

Merge pull request #30677 from nextcloud/backport/stable21/30597

Remove inefficient fed share scanner

Remove scanAll which relies on the "shareinfo" endpoint that returns the
full cache tree.
The latter can become big for big shares and result in timeouts.
Furthermode, the full tree would be retrieved again for each and every
detected change which can become expensive quickly.

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

Merge pull request #30676 from nextcloud/backport/30275/stable21

Fix psalm issues

Signed-off-by: Carl Schwan <carl@carlschwan.eu>

Add helper method in Wrapper

Signed-off-by: Carl Schwan <carl@carlschwan.eu>

The storage is not static anymore

Don't call twice $cache->getId

Signed-off-by: Carl Schwan <carl@carlschwan.eu>

Optimize FileSystemTags workflow for groupfolder

In https://github.com/nextcloud/server/pull/28774 we disabled the
caching for the groupfolder application since it worked due to the fact
that in groupfolders, getFileIds could be called with the same $cacheId
and path for actually different groupfolders.

This revert this change and instead add the folderId from the
groupFolder to the cacheId. This solve the issue of the uniqueness of
the cacheId inside GroupFolder. Downside is that we introduce
groupfolder specific implementation inside the server repo.

The seconf optimization is to not consider paths starting with
__groupfolders in executeCheck. This is due to the fact that files in
the groupfolder application call two times executeCheck one time with
the url __groupfolder/<folderId>/<path> and the other time with <path>.
The first time will always return an empty systemTags array while the
second call will return the correct system tags.

Signed-off-by: Carl Schwan <carl@carlschwan.eu>

Merge pull request #30669 from nextcloud/backport/30363/stable21

[stable21] improve accessibility settings on mobile

21.0.8 RC2

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

Merge pull request #30667 from nextcloud/backport/30568/stable21

[stable21] Fix primary key change in user_ldap migration

Fix file picker not respecting hidden files settings

This will only respect the setting inside the file app. For other apps
we will either need to do an API call or add an input field with the
same idea to spare a blocking api call.

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
(cherry picked from commit 1fa58be1aa78ff6216b7a129925a7bb270ac9fe1)

Add version of disabled apps when available

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

address 2nd review

Signed-off-by: szaimen <szaimen@e.mail.de>

address review

Signed-off-by: szaimen <szaimen@e.mail.de>

improve accessibility settings on mobile

Signed-off-by: szaimen <szaimen@e.mail.de>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

Merge pull request #30642 from nextcloud/backport/30600/stable21

[stable21] Fix idn emails not working in shares

Fix linkToRoute

Remove useless indexes with duplicated names on backup table

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

Update autoloader

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

Fix user_ldap migration for long DNs support

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

Move duplicated code to a base class for group_mapping migrations

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Co-authored-by: Joas Schilling <213943+nickvergessen@users.noreply.github.com>

Split dropTable and createTable in two migrations

It is not possible to drop and create the same table in one migration

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

Fix primary key change in user_ldap migration

Use a backup table to copy the data, drop table and recreate it with
 correct primary key, then copy the data back and drop the backup table.

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

Fix wrong unified search link to folder

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

Merge pull request #30596 from nextcloud/fix/static_analysis

Implement multibucket shift for ObjectStore

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

[tx-robot] updated from transifex

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

Merge pull request #30599 from nextcloud/backport/29118/stable21

[stable21] add better index for finding unindexed paths

Fix static analysis on CI

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

Fix idn emails not working in shares

And add check before sending email that email address is valid

Fix #30595

Signed-off-by: Carl Schwan <carl@carlschwan.eu>

add better index for finding unindexed paths

for the following query 'SELECT "path" FROM "oc_filecache" WHERE ("storage" = $storage) AND ("size" < 0) ORDER BY "fileid" DESC LIMIT 1;'

currently the database will in some cases decide to priorize the sort by fileid over the filter when picking what index to use, resulting in a much slower query.

by creating an index that allows first sorting by fileid and also filter by storage and size this case will be greatly sped up

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

Merge pull request #30638 from nextcloud/backport/30620/stable21

[stable21] Properly abort uploads

Properly hide progress bar after error

Whenever an error occurs, also hide the progress bar.

The logic was also adjusted to properly detect uploads that are pending
deletion, in which case the progress bar can already be hidden.

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

Properly abort uploads

Add a new approach for flagging an upload as aborted because we can't
rely on the browser fully cancelling the request as we now seem to
receive an error response from the server instead of a jQuery "abort"
message.

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

[tx-robot] updated from transifex

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

Update lib/private/BackgroundJob/JobList.php

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

Merge pull request #30629 from nextcloud/backport/30593/stable21

[stable21] Trigger "changeDirectory" event on URL change

Reset job disabling timer on adding the job again

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

Trigger "changeDirectory" even on URL change

When using the browser back button or clicking on sections on the left
sidebar (like favorites), the "changeDirectory" jQuery event did not get
called, so apps like recommendations would not notice the directory
change.

This fixes the issue by also setting changeUrl to true when the file
list's directory got changed as a result from a URL change.

Added optional changedThroughUrl argument to make sure the event
recipient knows if the change was done through a URL change and make it
possible prevent a loop in the onDirectoryChange handler that actually
changes the URL when the origin was already from a URL change.

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

Merge pull request #30617 from nextcloud/backport/30609/stable21

[stable21] Fix RequestURL check for cli commands

Merge pull request #30601 from nextcloud/backport/30291/stable21

[stable21] Prevent loading images that would require too much memory.

Fix RequestURL check for cli commands

Fix https://github.com/nextcloud/files_automatedtagging/issues/526

Fix https://github.com/nextcloud/groupfolders/issues/1855

Signed-off-by: Carl Schwan <carl@carlschwan.eu>

[tx-robot] updated from transifex

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

Prevent loading images that would require too much memory.

For most image formats, the header specifies the width/height.
PHP allocates an image object from that size, even if the actual
image data is much smaller. This image object size is not limited
by the limit configured in PHP.

The memory limit can be configured through "config.php" setting
"preview_max_memory" and defaults to 128 MBytes which should be
enough for most images without filling up all memory.

Signed-off-by: Joachim Bauch <bauch@struktur.de>

Merge pull request #30590 from nextcloud/backport/30571/stable21

[stable21] fix: only use jquery once it is available

Merge pull request #30579 from nextcloud/backport/30565/stable21

[stable21] Fix fail when keys/files folder already exists

Merge pull request #30583 from nextcloud/bugfix/noid/npm-ci-for-karma/stable21

[stable21] Use npm ci when running JS tests

fix: only use jquery once it is available

publicpage.js is loaded very early and cannot rely on jquery being loaded already.

Move the use of `$` into the `DomContentLoaded` handler.

Signed-off-by: Azul <azul@riseup.net>

Fix fail when keys/files folder already exists

Fixes an issue with transfer ownership in move mode where the folder
"files_encryption/keys/files" already exists.

Instead of failing, its existence is checked before calling mkdir.

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

Update package-lock in build dir

It wasn't up to date with package.json

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

[tx-robot] updated from transifex

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

Recompile JS

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

Fix JS tests after test lib updates

Prevent XHR during load by checking window.TESTING.
Adjust some expected values.

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

Make chokidar happy with fsevents

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

Update node deps for JS tests

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

Use npm ci when running JS tests

This makes sure that the pinned versions are selected correctly.
Fixes the colors.js incident.

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

[tx-robot] updated from transifex

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

Merge pull request #30559 from nextcloud/backport/30468/stable21

[stable21] Handle LocalServerException when scanning external shares

Handle LocalServerException when scanning external shares

When remoteIsOwnCloud trows LocalServerException, the storage is
unavailable and instead of crashing the scanner, ignore the specific
storage.

Signed-off-by: Carl Schwan <carl@carlschwan.eu>

[tx-robot] updated from transifex

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

Merge pull request #30454 from nextcloud/backport/30420/stable21

[stable21] improve status modal

fix compiling

Signed-off-by: szaimen <szaimen@e.mail.de>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

[tx-robot] updated from transifex

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