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()

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

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

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>

Add BeforeTemplateRenderedEvent for files_sharing

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

Merge pull request #21720 from nextcloud/search-min-length-feedback

provide feedback about minimum length needed to trigger search

Make translation-checker.php less verbose.

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

provide feedback about minimum length needed to trigger search

instead of telling users that there are no results, tell them that search hasn't been triggered yet

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

Merge pull request #21346 from nextcloud/enh/20930/dashboard

Dashboard API

Bump bundles

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

Merge pull request #21683 from nextcloud/dependabot/npm_and_yarn/babel/plugin-proposal-class-properties-7.10.4

Bump @babel/plugin-proposal-class-properties from 7.10.1 to 7.10.4

Extra white space to follow other ifs

Signed-off-by: lui87kw <lukas.ifflaender@uni-wuerzburg.de>

Avoid substr() error when strpos returns false

"Exception: substr() expects parameter 3 to be int, bool given" can occur on Line 378 $mimePart = substr($icon, 0, strpos($icon, '-'));
This happens, when '-' is not found and strpos returns false instead of an int.
When this occurs, e.g., Activity hangs.

Signed-off-by: lui87kw <lukas.ifflaender@uni-wuerzburg.de>

Include name in translatable greeting

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

Dashboard strict typing

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

Load more than one panel

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

Add load method for apps to bootstrap their panels

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

Update Dashboard app icon

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

Cleanup public api methods

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

Initialize panels early to make sure that scripts can be loaded

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

Deprecate panel register event right away but keep it for not breaking backward compatible apps

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

Move to lazy panel registration during registration context

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

Update autoloader

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

Separate icon from translatable text

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

Fix Dashboard layout, positioning and responsiveness

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

Modify dashboard welcome sentences

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

Add dashboard app

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

Deprecate old dashboard API

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

Add new dashboard public API

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

Bump @babel/plugin-proposal-class-properties from 7.10.1 to 7.10.4

Bumps [@babel/plugin-proposal-class-properties](https://github.com/babel/babel/tree/HEAD/packages/babel-plugin-proposal-class-properties) from 7.10.1 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-plugin-proposal-class-properties)

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

Merge pull request #21829 from nextcloud/bugfix/noid/fix-login-form-erors

Fix login form erors

Rebuild

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

Merge pull request #21839 from nextcloud/debt/noid/move-since-check-to-checkers

Move since check to checkers

[tx-robot] updated from transifex

Merge pull request #21838 from nextcloud/enh/dav/do_not_log_invalid_range

Do not log RequestedRangeNotSatisfiable exceptions in DAV

Move check for missing since tag to autotest-checkers.sh

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

Do not log RequestedRangeNotSatisfiable exceptions in DAV

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

Merge pull request #21816 from nextcloud/techdebt/noid/federatedfilesharing-to-ibootstrap

Use IBootstrap for the app federatedfilesharing

Merge pull request #21825 from nextcloud/fix/appframework/services

Fix AppFramework services

Merge pull request #21812 from nextcloud/feature/noid/app-bootstrap-without-app.php

Do not load app.php if Application implements IBootstrap

Merge pull request #21833 from nextcloud/docfix/noid/irequest_getHeader

Fix PHPDoc of IRequest::getHeader

Merge pull request #21828 from nextcloud/enhancement/lazy-bootstrap-registration-delegation

Delegate bootstrap registration lazily

Use IBootstrap for the app federatedfilesharing

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

Fix AppFramework services

* We can't just register an alias as the services need the appId to be
  injected. if we just register an alias this blows up since the main
  container doesn't have the appId.
* Moved the Authtokens over to show the PoC works

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

Do not load app.php if Application implements IBootstrap

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

Delegate bootstrap registration lazily

* Keep the registration context
* Expose the context object for other components
* Ensure registration is only run once

Search providers are migrated for demonstration.

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

Use generateUrl and imagePath instead of OC.*

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

Use space instead of tab inside translation

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

Merge pull request #21832 from nextcloud/bugfix/noid/source-maps-prod

Build source maps on production build again

Merge pull request #21827 from nextcloud/fix/psr3-logger-exception

Make the PSR-3 adapter exception-aware

Compile assets

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

Fix PHPDoc of IRequest::getHeader

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

Build source maps on production build again

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

Make the PSR-3 adapter exception-aware

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

Merge pull request #21824 from nextcloud/techdebt/noid/bootstrap-user_ldap

Use IBootstrap for the app user_ldap

Merge pull request #21357 from nextcloud/feature/21265/personal-share-notes

Add Editing for Link Share Labels

Merge pull request #21791 from nextcloud/bug/21790/update-guzzle

Add Guzzle's InvalidArgumentException

Use IBootstrap for the app user_ldap

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

[tx-robot] updated from transifex