Merge pull request #21208 from nextcloud/dependabot/composer/doctrine/dbal-2.10.2

Bump doctrine/dbal from 2.9.2 to 2.10.2

Merge pull request #21311 from nextcloud/techdebt/noid/make-share-order-reliable

Always sort shares in a reliable way

Merge pull request #21322 from nextcloud/techdebt/deprecate-iservercontainer-geteventdispatcher

Deprecate \OCP\IServerContainer::getEventDispatcher

Merge pull request #21318 from nextcloud/fix/noid/user-delete-clear-ldap-cache

clear LDAP cache after user deletion

Merge pull request #21326 from nextcloud/enh/noid/office-pattern-opendocument

Extends office documents pattern to match open documents.

Merge pull request #21332 from nextcloud/fix/ocp-since-checker

Use empty for the OCP @since checker

Merge pull request #21327 from nextcloud/techdebt/noid/certificates-js

Remove obsolete scripts for certificates ui

Merge pull request #21302 from nextcloud/techdebt/noid/adjust-to-parent-implementation

Adjust to parent implementation

Use empty for the OCP @since checker

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

[tx-robot] updated from transifex

Extends office documents pattern to match open documents.

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

Remove obsolete scripts

Leftover from https://github.com/nextcloud/server/pull/21224.

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

Deprecate \OCP\IServerContainer::getEventDispatcher

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

clear LDAP cache after user deletion

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

Always sort shares in a reliable way

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

Merge pull request #21058 from nextcloud/bug/set_serverBackground_custom_default

Set serverBackground if it is a custom background or default hub

Merge pull request #21198 from nextcloud/bugfix/19040

Only use background fade if nextcloud blue is set

Adjust to parent implementation

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

Merge pull request #21299 from nextcloud/fix/generic-event-wrapper-constructor-get-class

Fix missing parent constructor call and get_class usage in GenericEventWrapper

Set serverBackground if it is a custom background or default hub

Signed-off-by: GretaD <gretadoci@gmail.com>

Merge pull request #20272 from nextcloud/enh/sabre/gzip_propfind

Gzip the propfind response

Fix missing parent constructor call and get_class usage in GenericEventWrapper

* The parent constructor was not called
* `get_class` does not allow null values in php7.2+

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

Merge pull request #21278 from nextcloud/dependabot/npm_and_yarn/nextcloud/auth-1.3.0

Bump @nextcloud/auth from 1.2.3 to 1.3.0

Remove dropped " ON " when join has no condition

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

[tx-robot] updated from transifex

Merge pull request #21274 from nextcloud/bug/21268/paswordless-for-everyone

Enable passwordless for everyone not only admins.

Bump @nextcloud/auth from 1.2.3 to 1.3.0

Bumps [@nextcloud/auth](https://github.com/nextcloud/nextcloud-auth) from 1.2.3 to 1.3.0.
- [Release notes](https://github.com/nextcloud/nextcloud-auth/releases)
- [Changelog](https://github.com/nextcloud/nextcloud-auth/blob/master/CHANGELOG.md)
- [Commits](https://github.com/nextcloud/nextcloud-auth/compare/v1.2.3...v1.3.0)

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

Merge pull request #21224 from nextcloud/techdebt/noid/remove-unreachable-controller-and-route

Remove controller and routes which have no UI component anyway

[tx-robot] updated from transifex

Enable passwordless for everyone not only admins.

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

Allow for compression of the PROPFIND responses

For now it just gzip and only for propfind (the 99% case). Already saves
a bunch since the propfinds compress so nicely.

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

Remove old unit tests as well

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

Merge pull request #21253 from nextcloud/bugfix/noid/fix-psr4-namespace

Fix PSR-4 namespace

Merge pull request #21254 from nextcloud/td/js/remove/_capabilities

Actually remove _capabilities

Merge pull request #21257 from nextcloud/td/js/systemtags_linter

Fix linting on systemtags and use @nextcloud/router

[tx-robot] updated from transifex

Fix linting on systemtags and use @nextcloud/router

Makes the linter happy. And thus me a bit happy since it now is
complaining less.

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

Actually remove _capabilities

Leftover from a previous PR.

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

Fix PSR-4 namespace

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

Remove old files from autoloader

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

Merge pull request #21240 from nextcloud/enh/theming/js_to_initial_state

Move theming js to initial state

Merge pull request #21246 from nextcloud/enh/js/maintenance_no_oc_import

Do not import the full OC since we don't need it

Merge pull request #21197 from nextcloud/bugfix/noid/app-screenshots

Check if screenshot is available before showing it

Bump doctrine/dbal from 2.9.2 to 2.10.2

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

Move theming js to initial state

A bit more elegant. Plus it will allow us to also write a proper
@nextcloud/theming package. To make life easier down the line for all.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

Check if screenshot is available before showing it

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

Do not import the full OC since we don't need it

Import what we need. Saves off a bunch of bytes of the bundle (600
kilobytes to be exact)

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

Merge pull request #21225 from nextcloud/techdebt/humanfilesize-getcanonicallocale-package

Move humanFileSize and OC.getCanonicalLocale to npm packages

Merge pull request #21229 from nextcloud/fix-empty-event-reminder-notifications

Fix empty event UUID reminder notifications

Move humanFileSize and OC.getCanonicalLocale to npm packages

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

[tx-robot] updated from transifex

Merge pull request #20914 from nextcloud/enh/deprecated_event_log_usage

Log usage of a deprecated event instead of emitting

Merge pull request #21232 from nextcloud/bug/20427/use-OC-CLI-instead-of-PHP-SAPI

Use \OC::$CLI instead of PHP_SAPI

Merge pull request #21165 from nextcloud/dependabot/npm_and_yarn/babel/plugin-proposal-class-properties-7.10.1

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

Merge pull request #21191 from nextcloud/td/js/use_router

OC.routing is just a @nextcloud/routing proxy

Use \OC::$CLI instead of PHP_SAPI

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

Fix empty event UUID reminder notifications

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

OC.routing is just a @nextcloud/routing proxy

So lets kill the duplicate code!

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

Merge pull request #21199 from nextcloud/bugfix/noid/prevent-harder-sharing-your-root

Prevent harder to share your root

Remove controller and routes which have no UI component anyway

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

[tx-robot] updated from transifex

Merge pull request #20938 from nextcloud/fix/share-talk-verification

Fix the Talk verification

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

Bumps [@babel/plugin-proposal-class-properties](https://github.com/babel/babel/tree/HEAD/packages/babel-plugin-proposal-class-properties) from 7.8.3 to 7.10.1.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/master/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.10.1/packages/babel-plugin-proposal-class-properties)

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

Merge pull request #21168 from nextcloud/dependabot/npm_and_yarn/babel/preset-env-7.10.2

Bump @babel/preset-env from 7.9.6 to 7.10.2

Prevent harder to share your root

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

Bump @babel/preset-env from 7.9.6 to 7.10.2

Bumps [@babel/preset-env](https://github.com/babel/babel/tree/HEAD/packages/babel-preset-env) from 7.9.6 to 7.10.2.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/master/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.10.2/packages/babel-preset-env)

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

Merge pull request #21166 from nextcloud/dependabot/npm_and_yarn/babel/core-7.10.2

Bump @babel/core from 7.9.6 to 7.10.2

Merge pull request #21167 from nextcloud/dependabot/npm_and_yarn/vue-router-3.3.2

Bump vue-router from 3.2.0 to 3.3.2

Merge pull request #21091 from nextcloud/sftp-normalize-path

normalize sftp path in read and write stream

Merge pull request #21181 from nextcloud/fix/remote-share-remove-error-handling

Do not only catch Exceptions but any Throwable during rmt share delete

Only use background fade if nextcloud blue is set

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

[tx-robot] updated from transifex

Merge pull request #21190 from nextcloud/td/js/linter

Fix some linter warning

Fix some linter warning

Part of my ongoing battle against deprecation messages.

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

Bump vue-router from 3.2.0 to 3.3.2

Bumps [vue-router](https://github.com/vuejs/vue-router) from 3.2.0 to 3.3.2.
- [Release notes](https://github.com/vuejs/vue-router/releases)
- [Changelog](https://github.com/vuejs/vue-router/blob/dev/CHANGELOG.md)
- [Commits](https://github.com/vuejs/vue-router/compare/v3.2.0...v3.3.2)

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

Merge pull request #21164 from nextcloud/dependabot/npm_and_yarn/nextcloud/dialogs-1.3.1

Bump @nextcloud/dialogs from 1.3.0 to 1.3.1

Do not only catch Exceptions but any Throwable during rmt share delete

When a remote share is deleted, we notify the remove instance. This may
trigger various error conditions, which we want to catch in order to
avoid undeletable shares. The try-catch, however, did only capture
exceptions, so things like TypeErrors were not caught and caused the
process to fail hard.

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

[tx-robot] updated from transifex

Restore previous state when failing to change video verification state

When video verification can not be enabled or disabled the previous
state is set again in the JavaScript share object. This ensures that the
UI will not reflect a misleading state.

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

Fix the Talk verification

When enabling or disabling Talk verification in mail shares the server
expects also a new password to be set. As we always just update one
property at a time this means the Talk verification was impossible to
activate or deactivate. With this patch, we send the talk option AND the
new password. If there is no new password, the Talk option is disabled
(in mail shares; in link shares it is possible to enable or disable the
video verification without changing the password).

When we finally have descriptive text on ActionCheckbox'es we should
definitely add some explanatory text for the user. Right now this is as
good as it gets.

We'll have to backport to 18.

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

[tx-robot] updated from transifex

Bump @nextcloud/dialogs from 1.3.0 to 1.3.1

Bumps [@nextcloud/dialogs](https://github.com/nextcloud/nextcloud-dialogs) from 1.3.0 to 1.3.1.
- [Release notes](https://github.com/nextcloud/nextcloud-dialogs/releases)
- [Changelog](https://github.com/nextcloud/nextcloud-dialogs/blob/master/CHANGELOG.md)
- [Commits](https://github.com/nextcloud/nextcloud-dialogs/compare/v1.3.0...v1.3.1)

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

Bump @babel/core from 7.9.6 to 7.10.2

Bumps [@babel/core](https://github.com/babel/babel/tree/HEAD/packages/babel-core) from 7.9.6 to 7.10.2.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/master/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.10.2/packages/babel-core)

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

Merge pull request #21155 from nextcloud/td/move_to_capabilities_package

Move to @nextcloud/capabilities package

[tx-robot] updated from transifex

Merge pull request #21154 from nextcloud/fix/file_info_load_order

Fix the loading order of the js files_info and files_client

Merge pull request #21156 from nextcloud/do-not-clone-the-whole-talk-repository-for-integration-tests

Do not clone the whole Talk repository for integration tests

Do not clone the whole Talk repository for integration tests

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

Merge pull request #21141 from nextcloud/fix/require-ext-pdo-composer

List ext-pdo as hard requirement for composer

Move to @nextcloud/capabilities package

And kill of oc_capabilities and _oc_capabilities globals

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

Fix the loading order of the js files_info and files_client

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

Merge pull request #20970 from nextcloud/td/js/escape-html-global

Remove the escape-html global

Merge pull request #21143 from nextcloud/fix-password-changes-in-link-and-mail-shares

Fix password changes in link and mail shares

Remove the escape-html global

Already announced for 19 that it would go. So now it realy does go.

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

Merge pull request #21118 from nextcloud/td/move_systemtags

Move systemtags to a proper location

Merge pull request #21146 from nextcloud/fix-reference-to-wrong-class-name

Fix reference to wrong class name

[tx-robot] updated from transifex

Add integration tests for video verification

Enabling the "send password by Talk" property of shares require that
Talk is installed and enabled, so the Drone step that runs them has to
first clone the Talk repository.

When the integration tests are run on a local development instance,
however, it is not guaranteed that Talk is installed. Due to this the
"@Talk" tag was added, which ensures that any feature or scenario marked
with it will first check if Talk is installed and, if not, skip the
scenario (instead of failing).

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

Add integration tests for creating and updating a mail share

In most cases, when a mail share is created or updated an e-mail is sent
to the sharee, which is done by connecting to the SMTP server set in the
configuration. If the server can not be contacted then the creation or
update of the mail share fails.

To make possible to test mail shares without using a real SMTP server a
fake one has been added. The original script, which is MIT licensed, was
based on inetd, so it was slightly modified to run on its own.

In order to use it from the integration tests the "Given dummy mail
server is listening" step has to be called in the scenarios in which the
mail server is needed.

For now that is the only available step; things like checking the sent
mails, while possible (as the script can log the mails to certain file),
have not been added yet.

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