Moves single constructor parameters to new lines.

Based on:
https://github.com/nextcloud/server/pull/38764#discussion_r1227630895

Signed-off-by: Faraz Samapoor <fsa@adlas.at>

 Uses PHP8's constructor property promotion.
 in core/Command/App, /Background, and /Broadcast classes.

Signed-off-by: Faraz Samapoor <fsa@adlas.at>

Merge pull request #38637 from fsamapoor/constructor_property_promotion_part2

[2/3] Refactors /core controllers using constructor property promotion.

Merge pull request #38868 from nextcloud/enh/noid/fix-mail-confusion

fix confusion around mail settings and improve layout a bit

Merge pull request #38878 from nextcloud/cleanup/controller-service-aliases

Cleanup controller service aliases

Merge pull request #38776 from nextcloud/fix/app-menu-do-not-invert-text

fix(core): Do not invert app menu text color

Compile JS assets

Signed-off-by: Jonas <jonas@freesources.org>

fix(core): Do not invert app menu text color

* Also fixes other cypress test
* Build assets

Signed-off-by: Philipp Hempel <Philipp.Hempel1@web.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

workflowengine: Stop using a service alias for controller

Signed-off-by: jld3103 <jld3103yt@gmail.com>

comments: Stop using a service alias for controller

Signed-off-by: jld3103 <jld3103yt@gmail.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 confusion around mail settings

Signed-off-by: Simon L <szaimen@e.mail.de>

Fix(l10n): Update translations from Transifex

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

Applies agreed-upon indentation convention to the changed controllers.

Based on https://github.com/nextcloud/server/pull/38636#discussion_r1218167753

Signed-off-by: Faraz Samapoor <f.samapoor@gmail.com>

Fixes psalm error.

Signed-off-by: Faraz Samapoor <f.samapoor@gmail.com>

Fixes php-cs-fixer error.

Signed-off-by: Faraz Samapoor <f.samapoor@gmail.com>

Refactors controllers by using PHP8's constructor property promotion.

Signed-off-by: Faraz Samapoor <f.samapoor@gmail.com>

Merge pull request #38843 from nextcloud/message-log-event-type

fix type in BeforeMessageLoggedEvent

Merge pull request #38823 from nextcloud/enh/noid/fix-icons

generate user themed favicon and touchicon

Merge pull request #38793 from Jenandra/fix-subadminsgroups-css-name

align subadmins css class name

Fix(l10n): Update translations from Transifex

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

generate user themed favicon and touchicon

Signed-off-by: Simon L <szaimen@e.mail.de>

Merge pull request #38841 from nextcloud/fix/code-ownders-vitor

fix: Removed invalid user from CODEOWNERS file

fix type in BeforeMessageLoggedEvent

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

fix: Removed invalid user from CODEOWNERS file

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

Merge pull request #38830 from nextcloud/fix/files-api-route-names

Fix files API route names

Merge pull request #38827 from nextcloud/automated/noid/master-update-psalm-baseline

[master] Update psalm-baseline.xml

duplicate css for subadmins to allow for both class names being used

Signed-off-by: Silke Suck <silke.suck@adacor.com>

align subadmins css class name with actually used class name subAdminsGroups to fix display issue when users are in many groups

Signed-off-by: Silke Suck <silke.suck@adacor.com>

Fix files API route names

Signed-off-by: jld3103 <jld3103yt@gmail.com>

Update psalm baseline

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

Fix(l10n): Update translations from Transifex

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

Merge pull request #38770 from nextcloud/fix/38398/oauth-encrypted-secret-length

Make oauth2 client secret column larger

add extra migration that sets the secret column length in case the previous step has run when it was setting it to 256

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

make oauth2 client secret column larger

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

add test that inserts a long oauth client secret

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

Merge pull request #36214 from nextcloud/techdebt/noid/define-parameters-of-federated-file-sharing

Specify the parameters of FederatedFileSharing controllers

Merge pull request #38742 from nextcloud/automated/noid/master-fix-npm-audit

[master] Fix npm audit

Specify the parameters of FederatedFileSharing controllers

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

chore(deps): fix npm audit

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

Merge pull request #38804 from nextcloud/fix/38798/show-button-only-one-time

make sure to show download button only one time

Merge pull request #38490 from nextcloud/log-dispatch-event

emit an event when a message is logged

Merge pull request #38803 from nextcloud/deps/nc

Bump @nextcloud/auth @nextcloud/axios @nextcloud/calendar-availability-vue @nextcloud/dialogs @nextcloud/files and @nextcloud/router

make sure to show download button only one time

Signed-off-by: Simon L <szaimen@e.mail.de>

Bump @nextcloud/auth @nextcloud/axios @nextcloud/calendar-availability-vue @nextcloud/dialogs @nextcloud/files and @nextcloud/router

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

Fix(l10n): Update translations from Transifex

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

Merge pull request #38794 from bensmrs/master

Enable version features again with S3 versioning

emit an event when a message is logged

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

Enable version features again with S3 versioning

Signed-off-by: Benjamin Somers <bsomers@resel.fr>

chore: don't copy additional configuration files to integration test container

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

Merge pull request #38745 from nextcloud/feature/ocs-xml-stdclass

Allow stdClass in XML responses

Merge pull request #38777 from nextcloud/fix/carddav/system-address-book-nullable-groupmanager

fix(carddav): Make SystemAddressBook::__construct $groupManager argument nullable

Merge pull request #38292 from dfuchss/feature/configurabe_timeout

Add config variable for curl timeout

Allow stdClass in XML responses

Signed-off-by: jld3103 <jld3103yt@gmail.com>

Fix deprecated method call

Signed-off-by: Dominik Fuchß <develop@fuchss.org>

Changes after code review.

Signed-off-by: Dominik Fuchß <develop@fuchss.org>

Add config variable for curl timeout

Add the config variable for curl calls ("remote_curl_timeout"). E.g., needed for nextcloud federation.

Signed-off-by: Dominik Fuchß <develop@fuchss.org>

Merge pull request #38781 from nextcloud/ci/noid/pin-setup-php

Merge pull request #38787 from nextcloud/dependabot/github_actions/actions/setup-node-3

Merge pull request #38786 from nextcloud/dependabot/github_actions/peter-evans/create-or-update-comment-3

Merge pull request #38788 from nextcloud/dependabot/github_actions/actions/upload-artifact-3

Merge pull request #38785 from nextcloud/dependabot/github_actions/skjnldsv/read-package-engines-version-actions-2.2

chore(deps): Bump skjnldsv/read-package-engines-version-actions from 1.1 to 2.2

Fix(l10n): Update translations from Transifex

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

chore(deps): Bump actions/upload-artifact from 2 to 3

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 2 to 3.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v2...0b7f8abb1508181956e8e162db84b466c27e18ce)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-major
...

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

chore(deps): Bump actions/setup-node from 2 to 3

Bumps [actions/setup-node](https://github.com/actions/setup-node) from 2 to 3.
- [Release notes](https://github.com/actions/setup-node/releases)
- [Commits](https://github.com/actions/setup-node/compare/v2...64ed1c7eab4cce3362f8c340dee64e5eaeef8f7c)

---
updated-dependencies:
- dependency-name: actions/setup-node
  dependency-type: direct:production
  update-type: version-update:semver-major
...

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

chore(deps): Bump peter-evans/create-or-update-comment from 2 to 3

Bumps [peter-evans/create-or-update-comment](https://github.com/peter-evans/create-or-update-comment) from 2 to 3.
- [Release notes](https://github.com/peter-evans/create-or-update-comment/releases)
- [Commits](https://github.com/peter-evans/create-or-update-comment/compare/v2...c6c9a1a66007646a28c153e2a8580a5bad27bcfa)

---
updated-dependencies:
- dependency-name: peter-evans/create-or-update-comment
  dependency-type: direct:production
  update-type: version-update:semver-major
...

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

chore(deps): Bump skjnldsv/read-package-engines-version-actions

Bumps [skjnldsv/read-package-engines-version-actions](https://github.com/skjnldsv/read-package-engines-version-actions) from 1.1 to 2.2.
- [Release notes](https://github.com/skjnldsv/read-package-engines-version-actions/releases)
- [Commits](https://github.com/skjnldsv/read-package-engines-version-actions/compare/v1.1...8205673bab74a63eb9b8093402fd9e0e018663a1)

---
updated-dependencies:
- dependency-name: skjnldsv/read-package-engines-version-actions
  dependency-type: direct:production
  update-type: version-update:semver-major
...

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

Merge pull request #38737 from kurtmckee/add-dependabot-for-actions

Add a Dependabot configuration to autoupdate GitHub action versions

ci: pin setup-php to avoid a regression

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

fix(carddav): Make SystemAddressBook::__construct $groupManager argument nullable

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

Merge pull request #38642 from nextcloud/chore/appframework/drop-emptycontentsecuritypolicy-allowinlinescript

chore(appframework)!: Drop \OCP\AppFramework\Http\EmptyContentSecurityPolicy::allowInlineScript

Merge pull request #38591 from nextcloud/fix/caching/avoid-haskey-get

fix(caching): Avoid checking existence before fetching

Merge pull request #38735 from nextcloud/enh/noid/update-disabled-updater-wording

allow to specify upgrade.cli-upgrade-link in order to link to the correct documentation

Merge pull request #38636 from fsamapoor/constructor_property_promotion_part1

[1/3] Refactors /core controllers using constructor property promotion.

chore(appframework)!: Drop \OCP\AppFramework\Http\EmptyContentSecurityPolicy::allowInlineScript

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

Merge pull request #38725 from nextcloud/dropMetaRobots

Drop meta robots header

Merge pull request #38619 from fsamapoor/replace_strpos_calls_in_files_external_app

Refactors "strpos" calls in /apps/files_external

fix(caching): Avoid checking existence before fetching

The cache might expire between checking for key existence and fetching
the value. In this rare case the code continues with a null value when
it doesn't expect one.

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

Merge pull request #38543 from fsamapoor/use_null_coalescing_operator

Uses "Null Coalescing Operator" to improve code readability.

Fix(l10n): Update translations from Transifex

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

Merge pull request #38734 from nextcloud/dependabot/npm_and_yarn/nextcloud/event-bus-3.1.0

chore(deps): Bump @nextcloud/event-bus from 3.0.2 to 3.1.0

chore(deps): Bump @nextcloud/event-bus from 3.0.2 to 3.1.0

Bumps [@nextcloud/event-bus](https://github.com/nextcloud/nextcloud-event-bus) from 3.0.2 to 3.1.0.
- [Release notes](https://github.com/nextcloud/nextcloud-event-bus/releases)
- [Changelog](https://github.com/nextcloud/nextcloud-event-bus/blob/master/CHANGELOG.md)
- [Commits](https://github.com/nextcloud/nextcloud-event-bus/compare/v3.0.2...v3.1.0)

---
updated-dependencies:
- dependency-name: "@nextcloud/event-bus"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

Fix(l10n): Update translations from Transifex

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

Add a Dependabot configuration to autoupdate GitHub action versions

Signed-off-by: Kurt McKee <contactme@kurtmckee.org>

allow to specify upgrade.cli-upgrade-link in order to link to the correct documentation

Signed-off-by: Simon L <szaimen@e.mail.de>

Merge pull request #38733 from nextcloud/dependabot/composer/build/integration/dms/phpunit-arraysubset-asserts-0.5.0

chore(deps-dev): Bump dms/phpunit-arraysubset-asserts

Bumps [dms/phpunit-arraysubset-asserts](https://github.com/rdohms/phpunit-arraysubset-asserts) from 0.4.0 to 0.5.0.
- [Release notes](https://github.com/rdohms/phpunit-arraysubset-asserts/releases)
- [Commits](https://github.com/rdohms/phpunit-arraysubset-asserts/compare/v0.4.0...v0.5.0)

---
updated-dependencies:
- dependency-name: dms/phpunit-arraysubset-asserts
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

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

Fix(l10n): Update translations from Transifex

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

Merge pull request #38226 from nextcloud/file-object-utils

add utility command for object store objects

add utility command for object store objects

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

Drop meta robots tag

Revert mistake

Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>

Merge pull request #38645 from nextcloud/enh/noid/link-ref-provider-local-image-uri

avoid requests to opengraph image if no host detected

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

Merge pull request #38703 from nextcloud/fix/trashbin-types-for-32bits

Merge pull request #38715 from nextcloud/deps/nextcloud/vue-7.12.0

chore(deps): Bump @nextcloud/vue from 7.11.6 to 7.12.0

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

Merge pull request #38710 from nextcloud/enh/noid/fix-issue-template

fix wrong link in issue template

Correctly react to bad configuration

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

fix false link in issue template

Signed-off-by: Simon L <szaimen@e.mail.de>