fix(ShareAPI): Send mails for mail shares by default

It looks like, the frontend it needs to provide the `sendMail` param
for the backend to decide wether mails would be sent.

Our UI does not have that at the moment so it should default to sending
emails always for mail shares.

Not exactly sure how this was handled earlier but this is a good starting point.

Resolves : https://github.com/nextcloud/server/issues/48012

Signed-off-by: fenn-cs <fenn25.fn@gmail.com>

Merge pull request #48872 from nextcloud/backport/48852/stable30

[stable30] fix(app-store): Correctly render Markdown in app description

chore(assets): Recompile assets

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

Merge pull request #48877 from nextcloud/backport/48863/stable30

[stable30] [TextProcessing] Fix: Also list types that are available in task processing

fix(app-store): Correctly render Markdown in app description

This seems to be broken by an update because the renderer now passes an object instead of multiple arguments to the render functions.

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

feat(textprocessing): factorize taskProcessingCompatibleTaskTypes

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

fix(textprocessing): also list types that are available in task processing

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

Fix(l10n): Update translations from Transifex

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

Merge pull request #48836 from nextcloud/backport/48799/stable30

[stable30] fix(files_sharing): Add correct context to translation

chore(assets): Recompile assets

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

fix(files_sharing): Add correct context to translation

This string is extracted in the `files_sharing` app so it needs to be accessed there and not on `files`.

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

Merge pull request #48834 from nextcloud/backport/48728/stable30

[stable30] fix(files): Adjust NavigationQuota for Nextcloud 30 design

chore(assets): Recompile assets

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

fix(files): Adjust NavigationQuota for Nextcloud 30 design

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

Merge pull request #48866 from nextcloud/backport/48853/stable30

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

Merge pull request #48868 from nextcloud/backport/48854/stable30

[stable30] fix(app-store): Also proxy images of locally installed apps

fix(app-store): Also proxy images of locally installed apps

Before this only app store apps got their screenshots proxied,
but this will cause locally installed apps to not be correctly shown on the app-store.

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

Merge pull request #48748 from nextcloud/backport/48373/stable30

[stable30] fix: add PasswordConfirmationRequired to create user storages endpoint

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>

fix: Add frontend code for password confirmation

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

fix: add PasswordConfirmationRequired to update and delete method

Signed-off-by: yemkareems <yemkareems@gmail.com>

fix: add PasswordConfirmationRequired to create user storages endpoint

Signed-off-by: yemkareems <yemkareems@gmail.com>

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>