Update system addressbook card only when there was a change based on a cached etag

Due to our old and new hook system the card dav backend listens to old and new hooks. This triggers this code multiple times and always causes an update. With this change we cache the etag during a request and only trigger the update if the etag has changed. This does not catches all not needed updates, but it does not need another round trip to the database and still covers most cases where multiple attributes are updated during one single request.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

[tx-robot] updated from transifex

Merge pull request #21460 from nextcloud/enhancement/systemtags-bootstrap

Move systemtags to the new bootstrap mechanism

Merge pull request #21639 from nextcloud/techdebt/noid/move-away-from-database-xml

Move away from database xml

Merge pull request #21714 from nextcloud/td/remove/getAppFolder

Remove old deprecated getAppFolder

Merge pull request #18867 from HolgerHees/patch-1

Update SecurityMiddleware.php

Remove old deprecated getAppFolder

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Update SecurityMiddleware.php

OC::$WEBROOT can be empty in case if your nextcloud installation has no url prefix. This will result in an empty Location Header.

in other areas OC::$WEBROOT is always used together with an /

Merge pull request #21717 from nextcloud/fix/tests/sharebymail

Fix sharebymail tests

Merge pull request #20729 from nextcloud/cleanup/noid/shared-tags-code

Tags.php and the old sharing mechanism

Merge pull request #21706 from nextcloud/bugfix/21698/add-a-clear-abuse-message

Add a clear message why you could end up there

Fix sharebymail tests

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Merge pull request #21713 from nextcloud/td/deprecate/register_rotues

Deprecate registerRoutes

Update class loaders

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

Move user_ldap to migrations

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

Move files_trashbin to migrations

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

Move files_external to migrations

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

Move federatedfilesharing to migrations

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

Move federation to migrations

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

Deprecate registerRoutes

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Cascading effect - more code that now is not used anymore

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Tags.php and the old sharing mechanism

The old sharing mechanism isn't working anymore, because it was replaced by Share 2.0. Also it was nowhere used so this removes the code paths and reduces complexity.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #21704 from nextcloud/fix/18895/fix-unit-tests

Fix unit tests from #18895

Merge pull request #21074 from jvsalo/shared-lock-multi-release

Fix releasing a shared lock multiple times

Merge pull request #21171 from nextcloud/enh/noid/tidy-up-group-ldap

tidy up Group_LDAP

Merge pull request #21222 from nextcloud/bugfix/18965/reduce-contacts-search-load

Reduce contacts search load

Merge pull request #21238 from nextcloud/techdebt/noid/deferrable-notification-apps

Allow notification apps to defer and flush the sending

Add a clear message why you could end up there

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

Fix unit tests from #18895

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #18895 from nextcloud/bugfix/noid/fix_birthday_calendar_color

Changes the Birthday calendar color to slightly brighter one

Merge pull request #21665 from nextcloud/debt/noid/job-list

Fix wrong phpdoc for execute method

Merge pull request #21692 from nextcloud/cleanup/sharebymail

Cleanup share by mail a bit

Merge pull request #21581 from nextcloud/3rdparty/symfony/4.4.10

Bump symfony deps to 4.4.10

Use formal type hints instead of informal PHPDoc

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Bump symfony deps to 4.4.10

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Cleanup share by mail a bit

* Moved to ned IBootstrap
* Register everything via the capabilities api (So clients can use it as
  well)
  - This applies to the enforcing passwords
* Updated the sharing js code to use it
* removed app.php
* removed unused settings now
* typehints
* strict typing

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>

[tx-robot] updated from transifex

Merge pull request #21264 from stweil/master

build: Fix some typos (most of them found by codespell)

Merge pull request #21636 from nextcloud/lock-exception-readable-path

add proper paths to locking exceptions

Merge pull request #21626 from nextcloud/enhancement/injectible-callables

Callable parameter injection

Merge pull request #21686 from nextcloud/dependabot/npm_and_yarn/babel/core-7.10.4

Bump @babel/core from 7.10.3 to 7.10.4

Merge pull request #21533 from nextcloud/techdebt/noid/dedicated-method-to-get-a-users-language

Techdebt/noid/dedicated method to get a users language

Bump @babel/core from 7.10.3 to 7.10.4

Bumps [@babel/core](https://github.com/babel/babel/tree/HEAD/packages/babel-core) from 7.10.3 to 7.10.4.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.10.4/packages/babel-core)

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

Merge pull request #21685 from nextcloud/dependabot/npm_and_yarn/babel/preset-env-7.10.4

Bump @babel/preset-env from 7.10.3 to 7.10.4

Merge pull request #21682 from nextcloud/dependabot/npm_and_yarn/eslint-plugin-import-2.22.0

Bump eslint-plugin-import from 2.21.2 to 2.22.0

[tx-robot] updated from transifex

Bump @babel/preset-env from 7.10.3 to 7.10.4

Bumps [@babel/preset-env](https://github.com/babel/babel/tree/HEAD/packages/babel-preset-env) from 7.10.3 to 7.10.4.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.10.4/packages/babel-preset-env)

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

Bump eslint-plugin-import from 2.21.2 to 2.22.0

Bumps [eslint-plugin-import](https://github.com/benmosher/eslint-plugin-import) from 2.21.2 to 2.22.0.
- [Release notes](https://github.com/benmosher/eslint-plugin-import/releases)
- [Changelog](https://github.com/benmosher/eslint-plugin-import/blob/master/CHANGELOG.md)
- [Commits](https://github.com/benmosher/eslint-plugin-import/compare/v2.21.2...v2.22.0)

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

Allow notification apps to defer and flush the sending

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

Use \OCP\IContainer::injectFn to boot the admin_audit app

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

Use \OCP\IContainer::injectFn to boot the Accessibility app

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

Use \OCP\IContainer::injectFn to boot the DAV app

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

Callable parameter injection

This is like what we have to DI and classes, but for callables.

The motivating factor is to get rid of *service locators* in the `boot`
method of apps as a new pattern is about to emerge where we have lots of
`query` calls on the app or server container in order to fetch some
services.

With this little helper it's possible to call another (public) method
and magically have everything injected.

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

Merge pull request #21280 from nextcloud/dependabot/npm_and_yarn/nextcloud/router-1.1.0

Bump @nextcloud/router from 1.0.2 to 1.1.0

Use the new method everywhere

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

Fix wrong phpdoc for execute method

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

Merge pull request #21662 from nextcloud/fix/s3-bucket-compat-static-call

Fix static method call for s3 bucket compat check

Merge pull request #20198 from nextcloud/enh/enable-codecov

Send coverage to codecov again

Merge pull request #21439 from nextcloud/feature/noid/move-autocomplete-filter-event-to-new-dispatcher-and-class

Move AutoComplete::filterResults to new event dispatcher and GenericE…

Merge pull request #21431 from nextcloud/search-filter-invalid-results

Filter out search results that have invalid encoding

Fix static method call for s3 bucket compat check

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

Bump @nextcloud/router from 1.0.2 to 1.1.0

Bumps [@nextcloud/router](https://github.com/nextcloud/nextcloud-router) from 1.0.2 to 1.1.0.
- [Release notes](https://github.com/nextcloud/nextcloud-router/releases)
- [Changelog](https://github.com/nextcloud/nextcloud-router/blob/master/CHANGELOG.md)
- [Commits](https://github.com/nextcloud/nextcloud-router/commits)

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

[tx-robot] updated from transifex

Merge pull request #21644 from nextcloud/bugfix/noid/typo-in-variable-name

Fix typo in variable name and make check more accurate

Merge pull request #21591 from nextcloud/debt/noid/set-path-for-ocm-permissions

Pass $path to ocmPermissions2ncPermissions

Merge pull request #21648 from nextcloud/bug/21647/tar-header

Add missing TarHeader.php

Merge pull request #21360 from nextcloud/root-storage-prevent-scanning

prevent the root storage from accidentally scanning user folders

Merge pull request #21547 from nextcloud/enhancement/workflowengine-bootstrap

Migrate WFE to the new bootstrap mechanism

Merge pull request #21580 from therealklanni/master

Add audio formats to htaccess

Merge pull request #21590 from nextcloud/debt/noid/unused-method

Remove unused method.

Add missing TarHeader.php

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

Merge pull request #21657 from nextcloud/fix/no_undefined_in_template

Check if debugMode is defined before using it

Check if debugMode is defined before using it

Else the sabre error page (where the var is not defined) will throw
warnings all the time.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Merge pull request #21599 from nextcloud/debt/noid/type-to-types

Replace TYPE with TYPES

Merge pull request #21653 from nextcloud/bugfix/noid/fix-ipv6-remote-addresses-from-x-forwarded-for-header

Fix IPv6 remote addresses from X_FORWARDED_FOR headers before validating

Fix IPv6 remote addresses from X_FORWARDED_FOR headers before validating

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

Merge pull request #21597 from nextcloud/techdebt/noid/return-value-for-commands

Add return value to all commands

Fix typo in variable name and make check more accurate

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

Add a dedicated method to get the language for another user

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

Move AutoComplete::filterResults to new event dispatcher and GenericEvent

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

Merge pull request #19646 from firlevapz/bugfix/19400/oauth2-login-opens-new-browser-window

Change OAuth2 redirect link to relative link

[tx-robot] updated from transifex

Replace TYPE with TYPES

As TYPE::* is deprecated.

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

Hello codecov :rocket:

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

Merge pull request #21604 from nextcloud/dependabot/npm_and_yarn/vue-loader-15.9.3

Bump vue-loader from 15.9.2 to 15.9.3

Merge pull request #21588 from nextcloud/bug/21214/method-signature

Update presign method to match with interface again.

Merge pull request #21589 from nextcloud/debt/noid/wrong-exception-call

Set retry = false for forbidden exception.

Merge pull request #21603 from nextcloud/dependabot/npm_and_yarn/mochapack-2.0.2

Bump mochapack from 2.0.0 to 2.0.2

Merge pull request #21592 from nextcloud/debt/noid/drop-parallel-lint

Remove unused jakub-onderka/php-parallel-lint

Merge pull request #21596 from nextcloud/debt/noid/share-return-mismatch

Replace Share with IShare and add missing return

Merge pull request #21632 from nextcloud/techdebt/app-php-deprecation-log

Log usages of the deprecated app.php file

Merge pull request #21630 from nextcloud/fix/21627/close-x

Make closing dialogs callback with false

Merge pull request #21606 from nextcloud/dependabot/npm_and_yarn/exports-loader-1.1.0

Bump exports-loader from 1.0.1 to 1.1.0

add proper paths to locking exceptions

while some code paths do wrap the "raw" locking exception into one with a proper path, not all of them do
by adding the proper path to the original exception we ensure that we always have the usefull information in out logs

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

Make closing dialogs callback with false

Signed-off-by: Gary Kim <gary@garykim.dev>

Merge pull request #21605 from nextcloud/dependabot/npm_and_yarn/dompurify-2.0.12

Bump dompurify from 2.0.11 to 2.0.12

Bump dompurify from 2.0.11 to 2.0.12

Bumps [dompurify](https://github.com/cure53/DOMPurify) from 2.0.11 to 2.0.12.
- [Release notes](https://github.com/cure53/DOMPurify/releases)
- [Commits](https://github.com/cure53/DOMPurify/compare/2.0.11...2.0.12)

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

Merge pull request #21602 from nextcloud/dependabot/npm_and_yarn/css-vars-ponyfill-2.3.2

Bump css-vars-ponyfill from 2.3.1 to 2.3.2

Log usages of the deprecated app.php file

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

Bump css-vars-ponyfill from 2.3.1 to 2.3.2

Bumps [css-vars-ponyfill](https://github.com/jhildenbiddle/css-vars-ponyfill) from 2.3.1 to 2.3.2.
- [Release notes](https://github.com/jhildenbiddle/css-vars-ponyfill/releases)
- [Changelog](https://github.com/jhildenbiddle/css-vars-ponyfill/blob/master/CHANGELOG.md)
- [Commits](https://github.com/jhildenbiddle/css-vars-ponyfill/compare/v2.3.1...v2.3.2)

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