fix(appinfo): Make sure screenshot, author and category are always arrays

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

fix(appsmanagement): Fix exception on generating preview url for screenshots

Some installed apps meant for older server versions might unexpectedly
offer up screenshot values in a non-string format (e.g. health). Avoid
an exception by checking first if the first app screenshot is indeed a
string and otherwise we take the value of the parameter

Signed-off-by: Claudio Cambra <developer@claudiocambra.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 #48898 from nextcloud/fix/files--list-header-button-title

fix(files): add title for files list table header button

chore(assets): Recompile assets

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>

fix(files): add title for files list table header button

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>

Merge pull request #48895 from nextcloud/jtr/chore-bug-report-logs

chore: Clarify log entry requirements in bug report form

chore: Clarify log entry requirements in bug report form

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

Merge pull request #48892 from nextcloud/update-stale-bot-configuration

chore: don't remove needs info label when commenting on a stale issue

Merge pull request #48884 from nextcloud/bugfix/noid/allow-to-force-db-throttler

feat(bruteforce): Allow forcing the database throttler

Merge pull request #48625 from nextcloud/fix/files--handle-empty-view-with-error

fix(files): handle empty view with error

chore: don't remove needs info label when commenting on a stale issue

1) Removing the "needs info" label when someone comments on a stale issue seems incorrect.
2) By default, the stale action removes the "stale" label upon commenting. However, the labels-to-remove-when-unstale option triggers an additional API request to remove the "stale" label, which was already removed in the previous API call.

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

chore(assets): Recompile assets

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>

refactor(files): migrate to @nextcloud/sharing

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>

fix(files): handle empty view with error

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>

chore(files): migrate davUtils to TS

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>

Fix(l10n): Update translations from Transifex

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

Merge pull request #48882 from nextcloud/safety-net-null-check

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

feat(bruteforce): Allow forcing the database throttler

Using the database is most likely worse for performance, but makes investigating
issues a lot easier as it's possible to look directly at the table to see all
logged remote addresses and actions.

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

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>