Merge pull request #23395 from nextcloud/backport/23300/stable20

[stable20] Stop transfer of ownership between same users

Merge pull request #23403 from nextcloud/backport/23396/stable20

[stable20] VersioningTest.php:729 is unreliable and should be disabled

VersioningTest:729 is unreliable and should be disabled

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

Merge pull request #23281 from nextcloud/backport/23269/stable20

[stable20] Fix the user remove from group in UI

Stop transfer of ownership between same users

Stop the transfer of ownership between same user.

Signed-off-by: Sujith Haridasan <sujith.h@gmail.com>

Compile assets

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

Merge pull request #23388 from nextcloud/backport/23378/stable20

[stable20] Fix typo 'shared'

Merge pull request #23390 from nextcloud/backport/23379/stable20

[stable20] Expose CLOUD federation for local users in the recent addressbook

Drop X-NEXTCLOUD-UID

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

Expose CLOUD federation for local users as well

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

Fix code style

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

Fix typo 'shared'

Merge pull request #23383 from nextcloud/backport/23371/stable20

[stable20] Add Psalm type for the bootstrap registration context

Add Psalm type for the bootstrap registration context

This can potentially catch a few bugs with wrong class strings.

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

Merge pull request #23375 from nextcloud/backport/23310/stable20

[stable20] Add template typing to the QBMapper

Add template typing to the QBMapper

The QBMapper is kind of a generic type, though this concept does not
exist in php. Hence you have a lot of type coercion in subtypes (mappers
in the individual apps) because you suddenly don't expect an Entity[]
but your specific type.

Luckily Psalm lets us type those. Then in the subclass you can
psalm-implement the mapper with a concrete type and psalm will do all
the magic to ensure types are used correctly.

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

Merge pull request #23288 from nextcloud/backport/23275/stable20

[stable20] Fix dashboard statuses sort

fix dashboard statuses sort function

Signed-off-by: Julien Veyssier <eneiluj@posteo.net>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

Fix the user remove from group in UI

Fix the removal of user from the group in UI.

Signed-off-by: Sujith Haridasan <sujith.h@gmail.com>

Merge pull request #23271 from nextcloud/fix/sidebar-updateTabs

Fix sidebar updateTabs method

Merge pull request #23256 from nextcloud/backport/23184/stable20

[stable20] Make sure getUsersFavoritingObject can be run without a user

Fix sidebar updateTabs method

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

Make sure getUsersFavoritingObject can be run without a user

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

Merge pull request #23254 from nextcloud/backport/23252/stable20

[stable20] Use the correct l10n for activities

Use the correct l10n for activities

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

Merge pull request #23216 from nextcloud/backport/23183/stable20

[stable20] Don't influence toastify errors and always use white

Merge pull request #23235 from nextcloud/backport/22974/stable20

[stable20] Fix appid in translation

Fix appid in translation

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

Merge pull request #23213 from nextcloud/backport/22937/stable20

[stable20] Allow configuring the activity update interval of token

Merge pull request #23220 from nextcloud/backport/23215/stable20

[stable20] Only run the query to get the account data once

Merge pull request #23195 from nextcloud/backport/23175/stable20

[stable20] Fix legacy update notifications

Merge pull request #23170 from nextcloud/backport/22913/stable20

[stable20] add mount point to quota warning message

Merge pull request #23211 from nextcloud/backport/23096/stable20

[stable20] Also check the path based mimetype for flow rule checks

Only run the query to get the account data once

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

Don't influence toastify errors and always use white

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

Allow configuring the activity update interval of token

On some systems with a lot of users this creates a lot of extra DB
writes.
Being able to increase this interval helps there.

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

Merge pull request #23185 from nextcloud/backport/23109/stable20

[stable20] dont hold a transaction during the move to trash

Merge pull request #23207 from nextcloud/backport/23028/stable20

[stable20] Fix array to string conversion on event search

Make sure to check the mimetype on the path if a secure one could not be detemined though the content

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

Fix array to string conversion on event search

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

Merge pull request #23203 from nextcloud/backport/23182/stable20

[stable20] Fix the user email issue while creating a user

Fix the user email issue while creating a user

When the user is created, the provisioning api
was not adding the email address of the user
when provided if the `send email to new user`
is not set.

Signed-off-by: Sujith Haridasan <sujith.h@gmail.com>

Fix unit tests

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

Merge pull request #23192 from nextcloud/backport/23179/stable20

[stable20] Fix dashboard rendering if accessibility app is disabled

Fix dashboard rendering if accessibility app is disabled

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

Fix legacy update notifications

Signed-off-by: mrtnmtth <mrtn.mtth@gmx.de>

dont hold a transaction during the move to trash

because moving to trash can take a long time, keeping a transaction active for the duration can lead to issues

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

add mount point to quota warning message

makes it more clear to the user what the quota applies to

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

Merge pull request #23071 from nextcloud/version/20.0.0/final

20 final

Merge pull request #23143 from nextcloud/backport/23114/stable20

[stable20]  Show icon only with dnd status in the message

Show the full status and icon all the time

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Show icon only with dnd status in the message

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

Merge pull request #23141 from nextcloud/backport/23136/stable20

[stable20] Mark all compiled JS as binary

Merge pull request #23132 from nextcloud/backport/23130/stable20

[stable20] Move online status into modal

Mark all compiled JS as binary

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

Bump @nextcloud/vue to v2.6.8

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

Fix subline hint

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

Show the subline

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Set status is also there now

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

Update @nextcloud/vue to 2.6.7

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

Status: Add subline for Invisible to explain it properly

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

Enable scrollbar for too long content, same way as in Dashboard customize

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

Move online status into modal

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

Merge pull request #23089 from nextcloud/backport/23074/stable20

[stable20] Do not match sharees on an empty email address

Merge pull request #23115 from nextcloud/backport/23108/stable20

[stable20] Reset the user status when clearing the custom message

Merge pull request #23121 from nextcloud/backport/23112/stable20

[stable20] Increase the timeout of statuses

Increase the timeout of statuses

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

Merge pull request #23116 from nextcloud/backport/23113/stable20

[stable20] Change wording from 'custom status' to 'status message'

Change wording from 'custom status' to 'status message'

Signed-off-by: Jan C. Borchardt <hey@jancborchardt.net>
Signed-off-by: Gary Kim <gary@garykim.dev>

Reset the user status when clearing the custom message

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

Merge pull request #23098 from nextcloud/backport/22999/stable20

[stable20] Adjust scroll container height to make it a proper boundary element for actions

Merge pull request #23095 from nextcloud/backport/23043/stable20

[stable20] Avoid crash when unauthenticated users make weather-related requests

Adjust scroll container height to make it a proper boundary element for actions

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

Merge pull request #23092 from nextcloud/backport/23083/stable20

[stable20] Generate exception to log on php errors

avoid crash when unauthenticated users make weather-related requests, mention it in UI

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

Merge pull request #23085 from nextcloud/backport/23013/stable20

[stable20] Show federation and email results also with exact user match unless c…

Merge pull request #23084 from nextcloud/backport/22983/stable20

[stable20] Sync all users to the system addresssbook

Generate exception to log on php errors

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

Do not match sharees on an empty email address

When asking for sharees we compare not only UID and displayname but also
the email address. And if that matches we return the sharee as an exact
match. This logic had a flaw as in that it also matched the empty string
to users with no email address.

This is most noticeable when you disable sharee enumeration and open the
ownership transfer dialog. It suggested other users of the instance
before. This has stopped now.

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

Show federation and email results also with exact user match unless containing @

Before when you have a user "smith" and a federated user "smith@example.com"
you could see the federation result with "smit" but not with "smith" anymore.
With most LDAP configurations and local backend setups this is disturbing and
causes issues.
The idea of not showing the email and federation on a matching user was with:
Local user registered with "smith@example.com" user id and having that same
email / cloud id in your contacts addressbook. So we now only hide those
"side results" when the search does contain an @

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

Sync all users to the system addresssbook

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

Merge pull request #23072 from nextcloud/backport/23051/stable20

[stable20] Fix app text going too far down on hover/focus

Merge pull request #23029 from nextcloud/backport/23024/stable20

[stable20] Add occ command to set theming values

Fix app text going too far down on hover/focus

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

Merge pull request #23032 from nextcloud/backport/23015/stable20

[stable20] Log slow dashboard widgets

Merge pull request #23048 from nextcloud/backport/23034/stable20

[stable20] Fix numeric folders throwing on markDirty

20 final

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

Fix numeric folders throwing on markDirty

TypeError: strpos() expects parameter 1 to be string, int given

The problem is that in cacheNode() we strip of any slashes, so
a folder "0/" will be trimmed to "0" and be used as an array key.
Since PHP automatically casts numeric array keys to integers,
you afterwards get $nodePath as int(0). Since it's now a number,
the strpos() function does not accept it anymore. Simply casting
$nodePath to a string again in the foreach solves the issue

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

Merge pull request #23030 from nextcloud/backport/22948/stable20

[stable20] Add more integration tests for "files:transfer-ownership" command

Merge pull request #23038 from nextcloud/fix-running-video-verification-integration-tests-in-drone-in-stable20

[stable20] Fix running video verification integration tests in Drone

Fix running video verification integration tests in Drone in stable20

In order to run the video verification integration tests the Talk app
needs to be cloned in a branch compatible with the server.

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

Log a warning if a "lazy" initial state loads longer than 1 second

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

Log an error if a dashboard widget loads longer than 1 second

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

Add integration tests to check that only the given path is transferred

Until recently (it was fixed in ac2999a26a) when a path was transferred
other shares with the target user were removed, so a test was added to
ensure that it does not happen again.

Besides that a test to ensure that other files with the target user are
not transferred was added too (it did not fail before, but seemed
convenient to have that covered too :-) ).

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

Add integration tests for transferring files of a user with a risky name

The files:transfer-ownership performs a sanitization of users with
"risky" display names (including characters like "\" or "/").

In order to allow (escaped) double quotes in the display name the
regular expression used in the "user XXX with displayname YYY exists"
step had to be adjusted.

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

Add integration test for transferring the path of a single file

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

Fix tests

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

Add occ command to set theming values

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

Merge pull request #23000 from nextcloud/version/20.0.0/RC2

20 RC2

Merge pull request #23026 from nextcloud/backport/23014/stable20

[stable20] Make 'Reasons to use Nextcloud' button translatable, fix #22977