rollback cache rename if trashbin move fails

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

fix moving files from external storage to object store trashbin

having the "cache rename" after the "storage move" caused the target
to get the fileid from the source file, without taking care that the object
is stored under the original file id.

By doing the "cache rename" first, we trigger the "update existing file"
logic while moving the file to the object store and the object gets stored for the
correct file id

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

Merge pull request #21963 from nextcloud/fix/sharing/respect_default_share_permissions

Respect default share permissions

Respect default share permissions

Else we'll always create new shares with the max permissions.

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

Merge pull request #21957 from nextcloud/techdebt/noid/bootstrap-testing

Use IBootstrap for the testing app

[tx-robot] updated from transifex

Use IBootstrap for the testing app

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

Merge pull request #21946 from nextcloud/fix/webauthn/windows_hello

Do not double encode the userid in webauthn login

Merge pull request #21949 from nextcloud/smb-3.2.6

update icewind/smb to 3.2.6

Merge pull request #21950 from nextcloud/bugfix/noid/wipe-translation

Properly fetch translation for remote wipe confirmation dialog

Do not double encode the userid in webauthn login

* Else login fails with webauthn devices that send the userid
  - Like windows hello

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

Compile assets

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

Merge pull request #21870 from nextcloud/fix/bootstrap-context-container-interfaces

Make the bootstrap context return ContainerInterface instances

Properly fetch translation for remote wipe confirmation dialog

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Merge pull request #21948 from nextcloud/bugfix/noid/accesibilityoca

Load accessibilityoca script

Merge pull request #21867 from nextcloud/dependageorg/npm_and_yarn/nextcloud/vue-2.2.1

Hide share link menu before copying the share link in acceptance tests

The (old) Firefox version used in the acceptance tests does not properly
render the share link menu. As the menu is taller than it should
sometimes it covers the copy link button, so it is not possible to click
it without hiding the share link menu. Moreover, in those cases the
share menu button is also covered by the share menu, so the menu needs
to be closed by pressing the "Esc" key.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Recompile JS assets

Signed-off-by: Georg Ehrke <developer@georgehrke.com>

Fix breaking changes of nc/vue update

Signed-off-by: Georg Ehrke <developer@georgehrke.com>

Bump @nextcloud/vue from 1.5.0 to 2.2.1

Signed-off-by: Georg Ehrke <developer@georgehrke.com>

update icewind/smb to 3.2.6

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

Load accessibilityoca script

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Merge pull request #21905 from nextcloud/dependabot/npm_and_yarn/babel/core-7.10.5

Bump @babel/core from 7.10.4 to 7.10.5

Bump @babel/core from 7.10.4 to 7.10.5

Bumps [@babel/core](https://github.com/babel/babel/tree/HEAD/packages/babel-core) from 7.10.4 to 7.10.5.
- [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.5/packages/babel-core)

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

Merge pull request #21904 from nextcloud/dependabot/npm_and_yarn/nextcloud/axios-1.3.3

Bump @nextcloud/axios from 1.3.2 to 1.3.3

[tx-robot] updated from transifex

Merge pull request #21658 from nextcloud/trashbin-configurable-size

allow admin to configure the max trashbin size

Use the proper IAppContainer and IServerContainer type hints to know which code runs with which container

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

Adjust apps' code to use the ContainerInterface

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

Make the bootstrap context return ContainerInterface instances

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

Merge pull request #21941 from nextcloud/design/font-ubuntu-cantarell

Move Cantarell before Ubuntu in font stack to fix GNOME + Ubuntu setups

Move Cantarell before Ubuntu in font stack to fix GNOME + Ubuntu setups

Signed-off-by: Jan C. Borchardt <hey@jancborchardt.net>

Merge pull request #21934 from nextcloud/tests/noid/disable-fragile-comments-tests

Disable fragile comments tests

Merge pull request #21929 from nextcloud/bugfix/noid/encryption-transfer-user

Encryption is ready if master key is enabled

Disable fragile comments tests

Those tests are actual problems in our code. See https://github.com/nextcloud/server/pull/20462#issuecomment-613380650 for a summary by @danxuliu

Basically they are introduced by #18929 but are only revealed by #18764.

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

Encryption is ready if master key is enabled

Signed-off-by: Julius Härtl <jus@bitgrid.net>

[tx-robot] updated from transifex

Merge pull request #21864 from nextcloud/techdebt/files-versions-psr-container

Migrate files versions to the PSR container

Merge pull request #21865 from nextcloud/techdebt/settings-psr-container

Migrate settigns to the PSR container

Bump @nextcloud/axios from 1.3.2 to 1.3.3

Bumps [@nextcloud/axios](https://github.com/nextcloud/nextcloud-axios) from 1.3.2 to 1.3.3.
- [Release notes](https://github.com/nextcloud/nextcloud-axios/releases)
- [Changelog](https://github.com/nextcloud/nextcloud-axios/blob/master/CHANGELOG.md)
- [Commits](https://github.com/nextcloud/nextcloud-axios/compare/v1.3.2...v1.3.3)

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

Merge pull request #21875 from nextcloud/techdebt/deprecate-ilogger

Deprecate ILogger in favor of the PSR-3 logger

Merge pull request #21862 from nextcloud/techdebt/files-sharing-psr-container

Migrate files sharing to the PSR container

Merge pull request #21866 from nextcloud/techdebt/ldap-psr-container

Migrate LDAP to the PSR container

Merge pull request #21846 from nextcloud/dependabot/npm_and_yarn/p-limit-3.0.2

Bump p-limit from 2.3.0 to 3.0.2

Merge pull request #21850 from nextcloud/techdebt/noid/register-alternative-logins

Allow to register AlternativeLogin on RegistrationContext

Merge pull request #21921 from nextcloud/fix/use-DOMContentLoaded

Use DOMContentLoaded and fix sharebymail loading issue

Bump p-limit from 2.3.0 to 3.0.2

Bumps [p-limit](https://github.com/sindresorhus/p-limit) from 2.3.0 to 3.0.2.
- [Release notes](https://github.com/sindresorhus/p-limit/releases)
- [Commits](https://github.com/sindresorhus/p-limit/compare/v2.3.0...v3.0.2)

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

Use DOMContentLoaded and fix sharebymail loading issue

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

Merge pull request #21916 from nextcloud/tests/remove-ui-regression

Remove outdated ui-regression tests

Remove outdated ui-regression tests

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Merge pull request #21880 from nextcloud/webauthn/discourage_user_auth

Discourage webauthn user interaction

[tx-robot] updated from transifex

Merge pull request #21891 from nextcloud/techdebt/undeprecate-iappcontainer-iservercontainer

Undeprecate IAppContainer and IServerContainer

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #21887 from nextcloud/techdebt/deprecate-server-container-query

Deprecate the query mothod on the server container

Merge pull request #21889 from nextcloud/techdebt/noid/make-magic-strings-publicly-available

Make magic strings of ClientFlowLogin and v2 publicly available

Undeprecate IAppContainer and IServerContainer

With 4152216bd8cf9d49e6749d26bb8b491dd49b089b these two interfaces got
deprecated with the reasoning that we only need the base PSR interface.
However, there are cases where in Nextcloud you still want to have a
specific container (the one for the app vs the one for the server) when
you either have a container injected or query one from a container.

With a single interface that would not be possible. So it's probably
better if we leave the two interfaces, but only have them extend the PSR
interface. IContainer – with the custom methods – shall still be phased
out, but the two other sub interfaces can stay for tagging purposes.
Tagging means that no methods shall be added.

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

Make magic strings of ClientFlowLogin and v2 publicly available

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

Merge pull request #21859 from nextcloud/techdebt/dav-psr-container

Migrate DAV to the PSR container

Deprecate the query mothod on the server container

The interface method has already been deprecated, but if some code uses
the concrete type instead, the deprecation is not shown (by phpstorm),
so I think it's better to have this method tagged as well.

The "fix" for this deprecation is to simply use `get` instead of
`query`. Right now this will work 100% the same, but the goal is to slim
down the interface and only use what PSR-11 offers.

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

[tx-robot] updated from transifex

Merge pull request #21869 from nextcloud/techdebt/noid/add-constants-for-magic-strings

Add constants for the magic strings of template rendering

Discourage webauthn user interaction

Else people might have the feeling this is also doing 2FA. And since it
is only prefered it can be ignored and hacked around.

Once we have proper 2FA with webauthn in one go this probably needs to
be revisted.

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

Merge pull request #21873 from nextcloud/techdebt/deprecate-server-container-getters

Deprecate the server container getters

Merge pull request #21874 from nextcloud/feature/scoped-psr-logger

Add a scoped PSR logger for apps

Merge pull request #21855 from p5n/master

fix typo in revokeShare()

Migrate contacts interaction to the PSR-3 logger

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

Deprecate ILogger

The logger service was always intended to follow the PSR-3 interface.
It's time to embrace this and switch over to the "official" API,
hence this custom interface can be slowly phased out.

With Nextcloud 20 the logger also got support for
* App id filled out automatically
* Exceptions handling (as replacement for logException)

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

Add a scoped PSR logger for apps

Just like for ILogger we should have a version that has the app ID
pre-set for the context (unless overwritten) so that each log entry can
be traced back to the app that produced it.

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

Deprecate the server container getters

Service locators are an anti pattern. These getters just make it more
appealing to do the wrong thing. If you want to locate a service the bad
way, just use the `get` method on a container – it will do the same in
also one line of code.

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

Merge pull request #21863 from nextcloud/techdebt/files-trashbin-container-cleanup

Clean up container usage in files trashbin

Migrate LDAP to the PSR container

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

Migrate settigns to the PSR container

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

Migrate files versions to the PSR container

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

Migrate files sharing to the PSR container

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

Add constants for the magic strings of template rendering

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

Migrate DAV to the PSR container

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

Merge pull request #21809 from nextcloud/techdebt/di-container-cleanup

Use PSR container interface and deprecate our own abstraction

Allow to register AlternativeLogin on RegistrationContext

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

Use PSR container interface and deprecate our own abstraction

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

Merge pull request #21858 from nextcloud/dependabot/npm_and_yarn/build/lodash-4.17.19

Bump lodash from 4.17.14 to 4.17.19 in /build

Clean up container usage in files trashbin

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

Bump lodash from 4.17.14 to 4.17.19 in /build

Bumps [lodash](https://github.com/lodash/lodash) from 4.17.14 to 4.17.19.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](https://github.com/lodash/lodash/compare/4.17.14...4.17.19)

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

[tx-robot] updated from transifex

fix typo

Merge pull request #21853 from nextcloud/enh/noid/better-event-description

Better event description for BeforeTemplateRenderedEvent in files and files_sharing

Better event description for BeforeTemplateRenderedEvent in files and files_sharing

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

Merge pull request #21815 from nextcloud/enh/noid/sharing-additional-scripts-event

Add LoadAdditionalScriptsEvent for files_sharing

Merge pull request #21822 from nextcloud/enh/events/additionalscripts

Add real events to load additionalscripts

Merge pull request #21843 from nextcloud/debt/noid/less-verbose-translation-checker

Make translation-checker.php less verbose

Merge pull request #21844 from bladewing/bladewing-harden-against-mimetype-mismatch

Avoid substr() error when strpos returns false

Merge pull request #21728 from nextcloud/bugfix/noid/s3-external-mtime-cache

Reuse cache for directory mtime/size if filesystem changes can be ignored

Merge pull request #21837 from nextcloud/enh/noid/also-show-cs-diff

Run cs:check a second time to show diff

Merge pull request #21848 from nextcloud/techdebt/deprecate-internal-symfony-dispatcher-adapter

Deprecate the internal Symfony dispatcher adapter

Merge pull request #21849 from nextcloud/fix/noid/integration-tests

Fix integration test due to new dashboard app

Run cs:check a second time to show diff

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

Fix integration test due to new dashboard app

See #21346

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

Add real events to load additionalscripts

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

Deprecate the internal Symfony dispatcher adapter

This type is not known on the public API, yet we should make sure that
any usage of this class is noticed and migrated as soon as possible.

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