Revert "Improve layout for the public download page"

This reverts commit 30decc3ed1f6c15a799278274cf1a75776441b56.

Signed-off-by: Max <max@nextcloud.com>

[tx-robot] updated from transifex

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

Merge pull request #30736 from nextcloud/backport/30470/stable22

[stable22] 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 #30692 from nextcloud/backport/29923/stable22

[stable22] Fix users can't login external mount user entered credentials not set

Merge pull request #30679 from nextcloud/release/22.2.4rc2

22.2.4 RC2

Merge pull request #30690 from nextcloud/backport/30645/stable22

[stable22] New Files internal link GET param to avoid opening the file

Merge pull request #30687 from nextcloud/backport/30499/stable22

[tx-robot] updated from transifex

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

Merge pull request #30694 from nextcloud/backport/30631/stable22

[stable22] updateEncryptedVersion: cleanup on target if cache already got renamed

[tx-robot] updated from transifex

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

updateEncryptedVersion: cleanup on target if cache already got renamed

When moving a file to trash with encryption enabled, the cache gets
moved before the actual file. According to @icewind1991 this is in order
to not break object storage.

When moving a file from an unencrypted storage (e.g. a collectives
storage) to the encrypted trashbin storage, this causes errors, see

This commit fixes it by doing `updateEncryptedVersion()` on the target
cache entry *if* the source cache entry doesn't exist anymore, but the
corresponding target cache entry does exist already.

Fixes: #26544
Signed-off-by: Jonas Meurer <jonas@freesources.org>

[tx-robot] updated from transifex

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

Merge pull request #30697 from nextcloud/backport/30655/stable22

[stable22] Improve layout for the public download page

Add non-breaking space in the file size

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

Improve layout for the public download page

This centers the content inside the pages, while also not hardcoded the
size of the footer.

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

Merge pull request #30683 from nextcloud/backport/30531/stable22

[stable22] Optimize FileSystemTags workflow for groupfolder

files_external SMB: throw InvalidArgument when user is not set

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

[files] new internal link GET param to avoid setting 'openfile' on redirect

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

Merge pull request #30673 from nextcloud/backport/stable22/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 #30675 from nextcloud/backport/30275/stable22

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 #30670 from nextcloud/backport/30363/stable22

[stable22] improve accessibility settings on mobile

22.2.4 RC2

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

Merge pull request #30664 from nextcloud/backport/30530/stable22

[stable22] Fix wrong unified search link to folder

Add version of disabled apps when available

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

Merge pull request #30666 from nextcloud/backport/30568/stable22

[stable22] 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)

Merge pull request #30658 from nextcloud/backport/29622/stable22

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>

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>

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 #30644 from nextcloud/backport/30600/stable22

[stable22] Fix idn emails not working in shares

Merge pull request #30598 from nextcloud/backport/29118/stable22

[stable22] add better index for finding unindexed paths

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 #30637 from nextcloud/backport/30620/stable22

[stable22] 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>

Merge pull request #30628 from nextcloud/backport/30593/stable22

[stable22] Trigger "changeDirectory" event on URL change

Merge pull request #30622 from nextcloud/backport/30358/stable22

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

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 #30618 from nextcloud/backport/30609/stable22

[stable22] Fix RequestURL check for cli commands

Merge pull request #30610 from nextcloud/backport/30588/stable22

[stable22] Fix undefined/unset scope in account properties

Reset job disabling timer on adding the job again

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

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>

Fix test

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

Fix undefined/unset scope in account properties

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

Merge pull request #30603 from nextcloud/backport/30291/stable22

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

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 #30591 from nextcloud/backport/30571/stable22

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

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>

Merge pull request #30578 from nextcloud/backport/30565/stable22

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

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>

Merge pull request #30581 from nextcloud/bugfix/noid/npm-ci-for-karma/stable22

[stable22] Use npm ci when running JS tests

[tx-robot] updated from transifex

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

Make chokidar happy with fsevents

Signed-off-by: Vincent Petry <vincent@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>

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 #30558 from nextcloud/backport/30468/stable22

[stable22] 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>

[tx-robot] updated from transifex

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

Merge pull request #30529 from nextcloud/backport/30511/stable22

[stable22]  update CRL after revocation of sharerenamer.crt

Merge pull request #30526 from nextcloud/backport/30491/stable22

Merge pull request #30519 from nextcloud/release/22.2.4

Update crl

Signed-off-by: Vitor Mattos <vitor@php.rio>

Fix email verification

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

Merge pull request #30523 from nextcloud/backport/30492/stable22

[stable22] Fix passing on the parameter

Fix passing on the parameter

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

Merge pull request #30515 from nextcloud/backport/29958/stable22

[stable22] Remove iconv from dependencies and tests

Merge pull request #30487 from nextcloud/backport/29780/stable22

Merge pull request #30237 from nextcloud/backport/30200/stable22

Merge pull request #29962 from nextcloud/backport/29951/stable22

22.2.4 RC1

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

[tx-robot] updated from transifex

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

[stable22] Remove iconv from dependencies and tests

which is not used anymore since: https://github.com/nextcloud/server/pull/29470

Signed-off-by: MichaIng <micha@dietpi.com>

[tx-robot] updated from transifex

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

Merge pull request #29324 from nextcloud/backport/28541/stable22

[stable22] add a prefix index to filecache.path, attempt 2

[tx-robot] updated from transifex

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

Run migrations fully when reenabling an app

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