refactor(dav): various modernizations of CardDAV SyncService

Signed-off-by: Thomas Citharel <tcit@tcit.fr>

feat(dav): expose Nextcloud groups to Contact's system addressbook contacts

And move event listeners related to SyncService to proper ones
Note: Changes to user system addressbook cards are now always done in a
QueuedJob

Closes #38432

Signed-off-by: Thomas Citharel <tcit@tcit.fr>

Merge pull request #41133 from nextcloud/bugfix/noid/dont-duplicate-notification-warning

Don't duplicate notification warning

Merge pull request #41120 from nextcloud/imaginaryInvalidStream

Don't duplicate notification warning

Ref https://github.com/nextcloud/notifications/pull/1699

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

Merge pull request #41123 from nextcloud/bugfix/noid/get-storage-before-updating

fix(external): Get storage before update it

fix(external): Get storage before update it

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

Fix(l10n): Update translations from Transifex

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

Don't pass invalid streams to Imaginary

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

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

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

Merge pull request #41093 from nextcloud/fix-files-list-popper

chore(assets): Recompile assets

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

fix(files): disable containment in header

Signed-off-by: Varun Patil <varunpatil@ucla.edu>

Merge pull request #41108 from nextcloud/fix/40708-remove_tab_index_from_navigation_and_app_content

Remove unneeded `tabindex="0"` from sidebar and app content

Remove unneeded `tabindex="0"` from sidebar and app content

Signed-off-by: julia.kirschenheuter <julia.kirschenheuter@nextcloud.com>

Merge pull request #41110 from nextcloud/fix/login-cypress-test

fix(cypress): Fix selector for the user menu button

fix(cypress): Fix selector for the user menu button

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

Merge pull request #41089 from nextcloud/fix/workflow-empty-content

fix(workflowengine): Add an empty content when no flows are installed or configured

Merge pull request #41021 from nextcloud/fix/move-tests-from-drone-to-cypress

fix(tests): Move leftover acceptance tests for users from drone to Cypress

fix(settings): Stablize user list cypress tests

* Use common `data-testid` to identify elements rather than to depend on internal classes or properties
* Force clean the state for the user tests
* Move leftover acceptance tests for users from drone to cypress

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

Merge pull request #41080 from nextcloud/fix/40682-Add_more_visibility_on_background_buttons

Change styles on active, hover and focus state for background buttons

fix(build): phpDocumentor usage

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

fix: use default php version

Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>

Fix(l10n): Update translations from Transifex

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

Merge pull request #41094 from nextcloud/fix/indirect-safe-dep

fix(dav): Remove usage of indirect thecodingmachine/safe dependency

Merge pull request #41046 from nextcloud/fix/recently-contacted-duplicates

Fix: duplicating contacts with the Recently contacted feature

fix(dav): Remove usage of indirect thecodingmachine/safe dependency

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

Merge pull request #41062 from nextcloud/artonge/fix/return_fals_if_touch_fails

Return correct status if touch failed

Merge pull request #39044 from nextcloud/more-empty-mount-checking

Fix root mounts not being setup in some cases

fix(workflowengine): Add an empty content when no flows are installed or configured

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

Merge pull request #41068 from nextcloud/fix/37092/user-status--html-validation

fix(user_status): separate user menu and dashboard status layout for HTML validation

Merge pull request #41035 from nextcloud/fix/37092/apps-list--html-validation

fix(settings): apps list html validation and loading icon

Fix: duplicating contacts with the Recently contacted feature

Signed-off-by: hamza221 <hamzamahjoubi221@gmail.com>

Merge pull request #41061 from nextcloud/bugfix/noid/dont-use-indirect-dependency

fix(3rdparty): Don't use indirect dependency "Safe/" for functions

chore(assets): Recompile assets

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

fix(settings): apps list loading classes

- fix setting loading classes
- also removes invalid HTML attributes, there is no such props in `NcContent`

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

fix(settings): apps list html validation

- Replace invalid `width="100%"` attribute
- Add empty required `alt`

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

chore(assets): Recompile assets

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

fix(user_status): separate user menu and dashboard status layout

- Remove href="#" from a button for HTML validation
- Use @nextcloud/vue NcButton instead of custom button in Dashboard

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

Merge pull request #41079 from nextcloud/bugfix/noid/allow-updating-from-27.1

Change styles on active, hover and focus state for background buttons

Signed-off-by: julia.kirschenheuter <julia.kirschenheuter@nextcloud.com>

fix(update): Allow updating from 27.1 to 28

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

Merge pull request #41033 from nextcloud/chore/migrate-login-feature

chore(tests): Migrate login acceptance tests from behat to Cypress

Merge pull request #40938 from nextcloud/fix/a11y/change-scope-actions

fix(settings): Account property scope actions

Fix(l10n): Update translations from Transifex

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

chore: compile assets

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

fix(settings): Styles for scope button

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

fix(settings): Name for scope button

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

fix(settings): Remove aria-label on scope actions

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

Merge pull request #41070 from nextcloud/deps/nextcloud/vue-8.0.0-beta.9

chore: Bump @nextcloud/vue to v8-beta.9

fix(tests): Update dav snapshot

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

chore: compile assets

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

fix: Share select search globally icon

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

chore: Bump @nextcloud/vue to v8-beta.9

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

Merge pull request #40482 from nextcloud/fix/workflowengine_fileaccess

Merge pull request #41064 from nextcloud/fix/40705-_increase_contrast_on_Device_and_Last_activity

Remove opacity on table header

enh(IMountManager): Add method to get MountPoint from CachedMountInfo

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

fix(isLegitimatedForUserId): Setup mountpoints to check file access

This fixes workflows on groupfolders, as it will consider access to
files in groupfolders.

It also fixes false positives where access to files was limited by other
means not taken into account before, e.g. access control.

For postDelete events, check for permissions of the parent folder
instead, as the file itself no longer exists.

Fixes: nextcloud/flow_notifications#71
Signed-off-by: Jonas <jonas@freesources.org>

Merge pull request #40495 from nextcloud/fix/master-name-in-share-cache

Merge pull request #41063 from nextcloud/fix/37092/dashboard-html-validation

Merge pull request #40980 from nextcloud/fix/37092/remove-unnecessary-roles

Merge pull request #41017 from nextcloud/fix/move-token-iprovider-to-ocp

Move IToken and IProvider::getToken to OCP

Merge pull request #41034 from nextcloud/fix/37092/users--html-validation

fix(settings): users page html validation

Remove opacity on table header

Signed-off-by: julia.kirschenheuter <julia.kirschenheuter@nextcloud.com>

chore(assets): Recompile assets

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

fix(dashboard): use span instead of div for icon to html

- div is not valid in h2
- div is not valid in label

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

chore(tests): Migrate login acceptance tests from behat to Cypress

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

Allow passing null to PublicKeyToken::setScope, fixes tests

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

Return correct status if touch failed

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

fix(3rdparty): Don't use indirect dependency "Safe/" for functions

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

Improve docblock annotations for tokens and their exceptions

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

Use more precise typing for setScope method parameter

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

Merge pull request #41055 from nextcloud/feat/noid/profile-manager-ocp

feat(profile): Add public interface for profile manager so apps can check config

Merge pull request #41045 from nextcloud/fix/openapi/provisioning_api/quota-fields

fix(provisioning_api): Fix quota fields in OpenAPI

fix(profile): Directly migrate to OCP constants

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

feat(profile): Add public interface for profile manager so apps can check config

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

Update autoloaders

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

Make sure that OC interfaces returns OC interfaces for backward compatibility

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

fix(provisioning_api): Fix quota fields in OpenAPI

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

Move Exceptions used in OCP to OCP

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

Adapt baseline to filename change

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

Align PublicKeyToken with interface changes

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

Merge pull request #40933 from nextcloud/check-datemax-enforcement-pubshare

Check expiry enforcement for all share types

Merge pull request #40966 from nextcloud/jr-security-policy-expansion

Add "no public GH Issues please" request, past advisories link, bounty mention, scope link to security.md

Fix(l10n): Update translations from Transifex

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

Merge pull request #41039 from nextcloud/fix/run-cypress-always-parallel

feat(cypress): Always run cypress CI in parallel

Do not treat `isDefaultExpireDateEnforced` as fallback

`isDefaultExpireDateEnforced` and its corresponding `defaultExpirationDate`
is currently treated as the enforcement fallback when share type enforcements are not
set.

However, `isDefaultExpireDateEnforced` and `defaultExpirationDate` are actually more like
`isDefaultPublicExpireDateEnforced` and `defaultPublicExpirationDate` and therefore only applies
to public shares.

It might be ideal to rename this variables all the way from the backend config to the way we use
them in the frontend code.

Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

Fix has expiration date logic

Current expiration date errorneously assumes that `defaultExpirationDate`
applies to all kinds of shares. But it only really applies to public shares despite
its name.

This commit, fixes that by paring expiration dates with the correct share types during
new share initialization and simplifying the `hasExpirationDate` (check) property.

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

Check expiry enforcement for all share types

We should check that a maximum expiry date has been enforced for
all shares and NOT JUST FOR internal shares before enforcing a UI max date,
like in commit 9757e680e2f643fe19432255c4142e96fbc88df5

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

Merge pull request #41024 from nextcloud/fix/follow-up-app-order

Merge pull request #41032 from nextcloud/dependabot/npm_and_yarn/typescript-5.2.2

chore(deps-dev): Bump typescript from 5.1.6 to 5.2.2

Fix(l10n): Update translations from Transifex

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

Merge pull request #41019 from nextcloud/fix-files-vl-h

fix(files): correct item height

fix(files): correct item height

1. grid: typo? The icon size is 160 - 44 + (44 / 2) = 138px
2. non-grid: border is already included in height because the tr
   is a border-box (made this explicit to be safe)

Signed-off-by: Varun Patil <varunpatil@ucla.edu>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

Merge pull request #41029 from nextcloud/dependabot/npm_and_yarn/nextcloud/calendar-availability-vue-2.0.0-beta.3

chore(deps): Bump @nextcloud/calendar-availability-vue from 2.0.0-beta.2 to 2.0.0-beta.3

Merge pull request #41028 from nextcloud/dependabot/npm_and_yarn/types/dockerode-3.3.21

chore(deps-dev): Bump @types/dockerode from 3.3.20 to 3.3.21

chore(deps): Bump @nextcloud/calendar-availability-vue

Bumps [@nextcloud/calendar-availability-vue](https://github.com/nextcloud/calendar-availability-vue) from 2.0.0-beta.2 to 2.0.0-beta.3.
- [Changelog](https://github.com/nextcloud/calendar-availability-vue/blob/main/CHANGELOG.md)
- [Commits](https://github.com/nextcloud/calendar-availability-vue/compare/v2.0.0-beta.2...v2.0.0-beta.3)

---
updated-dependencies:
- dependency-name: "@nextcloud/calendar-availability-vue"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

Merge pull request #41027 from nextcloud/dependabot/github_actions/actions/checkout-4.1.1

chore(deps): Bump actions/checkout from 4.1.0 to 4.1.1

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

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

Merge pull request #41030 from nextcloud/dependabot/npm_and_yarn/eslint-plugin-cypress-2.15.1

chore(deps-dev): Bump eslint-plugin-cypress from 2.14.0 to 2.15.1

Merge pull request #41036 from nextcloud/skjnldsv-patch-1

fix(l10n): invalid translation of a number