fix(app-store): Ensure the `groups` property is always an array

If the value was a string, like a single group, then `json_decode` will also yield only a string.
So in this case we ensure the property is always an array with that value.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Fix(l10n): Update translations from Transifex

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

Fix(l10n): Update translations from Transifex

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

Fix(l10n): Update translations from Transifex

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

Merge pull request #48830 from nextcloud/backport/48603/stable30

[stable30] fix(logger): Remove more parameters of other methods

Fix(l10n): Update translations from Transifex

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

fix(logger): Remove more parameters of other methods

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

Merge pull request #48550 from nextcloud/backport/48540/stable30

[stable30] fix(filesexternal): Remove unneeded 3rdparty library use

Merge pull request #48731 from nextcloud/backport/48425/stable30

[stable30] fix(files): Ensure renaming state is correctly reset

chore(assets): Recompile assets

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

fix(files): Ensure renaming state is correctly reset

Problem: Is a node is renamed and the new name is out of the current
visible list of nodes the component will be recycled, this means
the props will change, so when the `onRename` functions is about to reset
the state the `this.source` will point to a different node.

To fix this, but also to separate business logic from visual representation,
the logic is moved into the renaming store and the component is only
responsible for rendering.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #48824 from nextcloud/backport/48787/stable30

[stable30] ci: Update list of installed PHP extensions (as some are not default …

Merge pull request #48730 from nextcloud/backport/48696/stable30

[stable30] fix(files_sharing): Add proper user facing messages on success

Merge pull request #48763 from nextcloud/dependabot/composer/stable30/symfony-6.4

[stable30] build: Bump symfony/* to 6.4

test(settings): Correctly only return bool for Symfony Input::hasParameterOption calls

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

fix(message): Fix logic of Mail test

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

fix: Ensure we only pass string to writeln()

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

test: Fix wrongly mocked method

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

test: Fix mock creation

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

ci: Fix psalm by typing the QuestionHelper

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

ci: Update psalm-baseline

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

fix(command): Make method compatible with upstream class (requires 3rdparty bump)

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

fix(command): Make method compatible with upstream class (backportable)

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

build: Bump symfony/* to 6.4

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

Merge pull request #48710 from nextcloud/backport/48682/stable30

ci: Update list of installed PHP extensions (as some are not default enabled in new images)

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

Fix(l10n): Update translations from Transifex

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

Merge pull request #48818 from nextcloud/backport/48812/stable30

[stable30] fix(activity): Fix download activity parameters

fix(activity): Fix download activity parameters

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

Fix(l10n): Update translations from Transifex

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

Merge pull request #48778 from nextcloud/automated/noid/stable30-update-code-signing-crl

[stable30] fix(security): Update code signing revocation list

fix(security): Update code signing revocation list

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

Fix(l10n): Update translations from Transifex

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

Merge pull request #48754 from nextcloud/backport/48743/stable30

[stable30] fix(appstore): Hide last modified information for shipped apps

fix(appstore): Hide last modified information for shipped apps

Signed-off-by: Marcel Müller <marcel-mueller@gmx.de>

Merge pull request #48785 from nextcloud/backport/48760/stable30

[stable30] feat(comments): Support mentioning emails

fix(comments): Add test for email mention

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

fix(comments): Document supported types and provide psalm typing

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

feat(comments): Support mentioning emails

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

Fix(l10n): Update translations from Transifex

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

Merge pull request #48240 from nextcloud/backport/47905/stable30

[stable30] fix(files): Ensure children are removed from folder and not duplicated

Merge pull request #48703 from nextcloud/backport/48673/stable30

[stable30] fix(share): Return empty string if no label is set

chore(assets): Recompile assets

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

fix(files_sharing): Add proper user facing messages on success

1. Use proper translated strings
2. Fix invalid use of translation message on API result
3. Log all errors

Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de>
Co-authored-by: Pytal <24800714+Pytal@users.noreply.github.com>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

test(files): Add tests for path handling

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #48751 from nextcloud/backport/48734/stable30

fix(files): Ensure children are removed from folder and not duplicated

* Resolves https://github.com/nextcloud/server/issues/47904

We need to make sure that we only add one source (unique!) once as a child,
this is ensured by simply use a native `Set`.
Also we need to remove children on from folders when the `files:node:deleted`
event is emitted.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #48458 from nextcloud/backport/47910/stable30

[stable30] fix(files): Add more visual move / copy notification

chore(assets): Recompile assets

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

fix: add notification icon for transfer ownership

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>

fix: do not duplicate existing entity

- this reverts 1e8048abee1745bab648dba5bf96f956c718e4e3

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>

fix: dismiss notification only after transfer bg job created

- do not create 'denied' notification if bg job exists

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>

fix(files): Add more visual move / copy notification

* Resolves: https://github.com/nextcloud/server/issues/46645

This adds loading toast notification while the move operation is running.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #47878 from nextcloud/backport/47848/stable30

Merge pull request #48589 from nextcloud/backport/47896/stable30

[stable30] fix: Make user removal more resilient

chore(assets): Recompile assets

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

fix(files): Make `openMenu` watcher null safe

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

fix(files): Reset context menu position on close

* Resolves https://github.com/nextcloud/server/issues/46934

Ensure that after a right-click the context menu position is resetted,
so that pressing the "actions"-menu button it is displayed on the correct location.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #48757 from nextcloud/release/30.0.1

30.0.1

build(hub): 30.0.1

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

Fix(l10n): Update translations from Transifex

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

Fix(l10n): Update translations from Transifex

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

fix(AppConfig): Add external JWT private key to sensitive keys

Signed-off-by: provokateurin <kate@provokateurin.de>

Fix(l10n): Update translations from Transifex

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

revert: "fix(files_sharing): Make share labels nullable"

This reverts commit 01c4fa3ba8a7311da331e6a2b70fc866b0a9b39c.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

fix(share): Return empty string if no label is set

* Resolves: https://github.com/nextcloud/server/issues/48629

While the database supports NULL, the typing has always said it only returns *string*.
So to not break any apps that might trust the typings we should return `''` if the database is set to `NULL`.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Fix(l10n): Update translations from Transifex

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

Fix(l10n): Update translations from Transifex

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

Fix(l10n): Update translations from Transifex

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

Fix(l10n): Update translations from Transifex

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

Merge pull request #48644 from nextcloud/release/30.0.1_rc2

30.0.1 RC2

build(hub): 30.0.1 RC2

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

Merge pull request #48630 from nextcloud/backport/48628/stable30

[stable30] fix(dav): Public WebDAV endpoint should allow `GET` requests

Fix(l10n): Update translations from Transifex

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

fix(dav): Public WebDAV endpoint should allow `GET` requests

`GET` should be allowed even without Ajax header to allow downloading files,
or show files in the viewer. All other requests could be guarded, but this should not.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #48553 from nextcloud/backport/48543/stable30

[stable30] fix(theming): Fix layout of themes

Fix(l10n): Update translations from Transifex

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

fix(UserTrait): Fix backend initialization

Signed-off-by: provokateurin <kate@provokateurin.de>

fix: Skip users that still exist in backend

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

fix: Make user removal more resilient

Currently there is a problem if an exception is thrown in `User::delete`,
because at that point the user is already removed from the backend,
but not all data is deleted.

There is no way to recover from this state, as the user is gone no information is available anymore.
This means the data is still available on the server but can not removed by any API anymore.

The solution here is to first set a flag and backup the user home,
this can be used to recover failed user deletions in a way the delete can be re-tried.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #48615 from nextcloud/backport/45765/stable30

[stable30] fix(settings): Use axios directly for health check to preven URL sanitizing

chore(assets): Recompile assets

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

fix(settings): Use axios directly for health check to preven URL sanitizing

When using Apache with `DirectorySlash` it will respond with 301 and the URL with trailing slash.
But when using traefik as the reverse proxy it can not rewrite redirects, this leads to the problem
that the Apache response is using HTTP in the redirect but the real server (traefik) is only listening on HTTPS.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #48558 from nextcloud/backport/48522/stable30

[stable30] fix(QueryBuilder): Account for aliases in output columns

Merge pull request #48551 from nextcloud/backport/48548/stable30

[stable30] fix(admin_audit): Listen to the right events

Fix(l10n): Update translations from Transifex

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

Merge pull request #48586 from nextcloud/backport/48584/stable30

[stable30] fix(config): Mark more app configs sensitive

fix(config): Mark more app configs sensitive

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

Fix(l10n): Update translations from Transifex

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

Fix(l10n): Update translations from Transifex

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

Fix(l10n): Update translations from Transifex

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

Fix(l10n): Update translations from Transifex

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

chore(assets): Recompile assets

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

fix(theming): Fix layout of themes

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

fix(admin_audit): Listen to the right events

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

fix(QueryBuilder): Account for aliases in output columns

Signed-off-by: provokateurin <kate@provokateurin.de>

fix(filesexternal): Remove unneeded 3rdparty library use

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

Fix(l10n): Update translations from Transifex

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

Merge pull request #48529 from nextcloud/backport/48519/stable30

[stable30] fix(dav): don't crash subscription on invalid calendar object

Merge pull request #48500 from nextcloud/release/30.0.1_rc1