refactor(ShareApiController): Check for null and empty strings with empty()

Proactive measure to avoid warnings in higher php versions as well possible type errors

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

Merge pull request #48381 from nextcloud/fix/48012/fix-share-email-send-mail-share

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

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 #48874 from nextcloud/jtr/setup-checks-heading

fix(settings): Change "setup warnings" -> "setup checks" in Overview

Merge pull request #48863 from nextcloud/fix/noid/textprocessing-list-types

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

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>

fix(settings): Change "setup warnings" -> "setup checks" in Overview

Signed-off-by: Josh <josh.t.richards@gmail.com>

Merge pull request #48864 from nextcloud/fix/invalid-app-config

fix(settings): Do not use `null` on `string` parameter for sharing disclaimer

chore: Compile assets

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

fix(settings): Do not use `null` on `string` parameter for sharing disclaimer

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

Merge pull request #48852 from nextcloud/fix/app-store-markdown

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

Merge pull request #48867 from nextcloud/fix/apps/wrong-missing-casts

chore(assets): Recompile assets

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

Merge pull request #48854 from nextcloud/fix/proxy-app-screenshot

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>

fix(apps): Fix wrong or missing casts

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

Merge pull request #48853 from nextcloud/fix/app-store-groups

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

Merge pull request #48850 from nextcloud/fix/unified-search-ctrl-f

fix(unified-search): Close on second ctrl+f

Merge pull request #48851 from nextcloud/artonge/fix/uid_type

Fix $uid type in Encryption stream

Merge pull request #48837 from nextcloud/followup/47329/add-all-types-to-handling

fix(entity): Fix mapping of old/sub-types to actually supported datab…

Merge pull request #48839 from nextcloud/build/translation-checker-print-rtl-limited-characters

build: Print RTL limited characters in translation-checker

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>

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>

chore: Compile assets

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

fix(unified-search): Close on second ctrl+f

Also only add the event listener if short-cuts are not disabled for accessibility reasons.

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

fix: $uid type in Encryption stream

As explained by the comment.

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

build: Show the actual string

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

fix(entity): Fix mapping of old/sub-types to actually supported database types

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>

build: Print RTL limited characters in translation-checker

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

Merge pull request #48790 from nextcloud/refactor/apps/constructor-property-promotion

Merge pull request #48756 from nextcloud/build/autoloader/remove-noisy-changes

Merge pull request #48799 from nextcloud/fix/people-translation

fix(files_sharing): Add correct context to translation

Fix(l10n): Update translations from Transifex

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

Merge pull request #48728 from nextcloud/fix/nav-quota-new-design

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 #48583 from nextcloud/fix/overide-itip-broker

fix: override iTip Broker to fix several issues

fix: override iTip Broker to fix several issues

Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>

Merge pull request #48758 from nextcloud/feat/files/chunked-upload-config-capabilities

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>

fix(psalm): Fix some newly detected issues

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

refactor(apps): Use constructor property promotion when possible

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

feat(files): Expose chunked upload config via capabilities

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

Merge pull request #48809 from nextcloud/rakekniven-patch-1

chore(i18n): Improved grammar

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>

Merge pull request #48812 from nextcloud/bugfix/noid/fix-download-activity-parameters

fix(activity): Fix download activity parameters

Fix(l10n): Update translations from Transifex

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

fix(activity): Fix download activity parameters

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

chore(i18n): Improved grammar

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>

Merge pull request #48804 from nextcloud/dependabot/composer/vendor-bin/openapi-extractor/nextcloud/openapi-extractor-1.0.1

Merge pull request #48806 from nextcloud/dependabot/composer/vendor-bin/cs-fixer/nextcloud/coding-standard-1.3.2

Merge pull request #48775 from nextcloud/automated/noid/master-update-code-signing-crl

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

chore(deps): Bump nextcloud/coding-standard in /vendor-bin/cs-fixer

Bumps [nextcloud/coding-standard](https://github.com/nextcloud/coding-standard) from 1.3.1 to 1.3.2.
- [Release notes](https://github.com/nextcloud/coding-standard/releases)
- [Changelog](https://github.com/nextcloud/coding-standard/blob/master/CHANGELOG.md)
- [Commits](https://github.com/nextcloud/coding-standard/compare/v1.3.1...v1.3.2)

---
updated-dependencies:
- dependency-name: nextcloud/coding-standard
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

chore(deps): Bump nextcloud/openapi-extractor

Bumps [nextcloud/openapi-extractor](https://github.com/nextcloud-releases/openapi-extractor) from 1.0.0 to 1.0.1.
- [Changelog](https://github.com/nextcloud-releases/openapi-extractor/blob/main/CHANGELOG.md)
- [Commits](https://github.com/nextcloud-releases/openapi-extractor/compare/v1.0.0...v1.0.1)

---
updated-dependencies:
- dependency-name: nextcloud/openapi-extractor
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

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 #48787 from nextcloud/ci/noid/enable-required-php-extensions

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

Merge pull request #47329 from nextcloud/feat/add-datetime-qbmapper-support

feat(AppFramework): Add full support for date / time / datetime columns

Merge pull request #48547 from nextcloud/feat/row_format_check

Merge pull request #48765 from nextcloud/dbQueriesExecStmt2

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

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

Merge pull request #48418 from nextcloud/config-carddav-sync-request-timeout

feat: configurable request timeout for carddav sync

Merge pull request #48760 from nextcloud/feat/noid/support-email-mentions

feat(comments): Support mentioning emails

fix(comments): Add test for email mention

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

Fix(l10n): Update translations from Transifex

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

chore: Use constant for default value

Co-authored-by: Thomas Citharel <nextcloud@tcit.fr>
Signed-off-by: Pablo Zmdl <57864086+pabzm@users.noreply.github.com>

feat: configurable request timeout for carddav sync

Big federated setups may need a longer timeout, which they now can
configure.

Signed-off-by: Pablo Zmdl <pablo@nextcloud.com>
Co-authored-by: Josh <josh.t.richards@gmail.com>

chore(db): Apply query prepared statements

Fix: psalm

fix: bad file

fix: bug

chore: add batch

chore: add batch

chore: add batch

fix: psalm

fix: Adjust Entity types

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

fix: Prevent breaking change in IQueryBuilder

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

fix: Adjust parameter type usage and add SQLite support

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

feat(AppFramework): Add full support for date / time / datetime columns

This adds support for all Doctrine supported types, for the column types only the immutable variants needed to be added.
But especially those types are the important ones, as our **Entity** class works by detecting changes through setters.
Meaning if it is mutable, changes like `$entity->date->modfiy()` can not be detected, so the immutable types make more sense here.

Similar the parameter types needed to be added.

`Enity` and `QBMapper` needed to be adjusted so they support (auto map) those types, required when insert or update an entity.

Also added more tests, especially to make sure the mapper really serializes the values correctly.

Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de>
Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

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>

Merge pull request #48724 from nextcloud/dbQueriesExecStmt

chore(db): Apply query prepared statements

chore(db): Correctly apply query types

fix: psalm

fix: error

fix: add batch

fix: fatal error

fix: add batch

chore: add batch

chore: add batch

fix: psalm

fix: typo

fix: psalm

fix: return bool

fix: revert Manager

build(autoloader): Remove noisy changes when regenerating

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

Merge pull request #48743 from nextcloud/fix/45717/hide-last-modified-for-shipped-apps

Merge pull request #48734 from nextcloud/fix/45884/accept-notification

fix: get rid of denied notification when accept

Merge pull request #48373 from nextcloud/fix/788/add-password-confirmation-required-to-user-storage-create

fix: add PasswordConfirmationRequired to create user storages endpoint

Merge branch 'master' into fix/788/add-password-confirmation-required-to-user-storage-create

Fix(l10n): Update translations from Transifex

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

Merge pull request #46124 from nextcloud/fix/ownership-transfer-source-user-files

fix: Fix "Unknown path" error when source user `files` folder has not been initialized

refactor: Inject IRootFolder

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

fix: Fix "Unknown path" error when source user `files` folder has not been initialized

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

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

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

Merge pull request #48735 from nextcloud/fix/eslint-warning

refactor(code-style): Adjust JS code to our code style

chore(assets): Recompile assets

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

fix(code-style): Adjust JS code to our code style

This resolves 68 ESLint warnings about invalid code style.

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

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: Add frontend code for password confirmation

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