Merge pull request #21198 from nextcloud/bugfix/19040

Only use background fade if nextcloud blue is set

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

Fix missing parent constructor call and get_class usage in GenericEventWrapper

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

[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

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>

Generalize integration test steps to download last share

Note that the "last link share can be downloaded" step was kept as it
tests the "url" property specific of link shares.

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

Remove unused variables

The step names were adjusted accordingly.

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

Fix disabling send password by Talk without new password in mail shares

When "send password by Talk" was disabled in a mail share it was
possible to keep the same password as before, as it does not pose any
security issue (unlike keeping it when "send password by Talk" is
enabled, as in that case the password was already disclosed by mail).

However, if a mail share is updated but the password is not set again
only the hashed password will be available. In that case it would not
make sense to send the password by mail, so now the password must be
changed when disabling "send password by Talk".

Note that, even if explicitly setting the same password again along with
the "send password by Talk" property would work, this was also prevented
for simplicity.

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

Fix reference to wrong class name

\OCP\IShare does not exist; the right name is \OCP\Share\IShare,
although it is already imported as IShare.

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

Merge pull request #21140 from nextcloud/enhancement/oc-get-all-apps-return-type-doc

Document \OC_App::getAllApps return type

Fix enabling send password by Talk with same password in mail shares

When "send password by Talk" is enabled in a mail share a new password
must be also set. However, when the passwords of the original and the
new share were compared it was not taken into account that the original
password is now hashed, while the new one is not (unless no new password
was sent, in which case the password of the original share was set in
the new share by the controller, but that was already prevented due to
both passwords being literally the same), so it was possible to set the
same password again.

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

Fix enabling send password by Talk with empty password in link shares

When "send password by Talk" is enabled in a link share now a non empty
password is enforced.

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

Fix creating a mail share with a password

When a mail share was created with a password the given password was not
hashed, so it was not possible to open the share with that password.
Moreover, if passwords were enforced the given password was ignored and
a new one was set (although in this case it was hashed so it worked as
expected). Now the given password is properly hashed and not overriden.

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

Extend mail shares unit tests to check the password and mail template

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

Move the systemtags to core/src

* Move to the handlebar loader from webpack

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

List ext-pdo as hard requirement for composer

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

Document \OC_App::getAllApps return type

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

Merge pull request #20597 from nextcloud/enh/lazy_node_events

Add lazy events for the Node API

Merge pull request #21124 from nextcloud/feature/20612/add-capability-for-file-comments

Add a capability when file comments are enabled

Add a capability when file comments are enabled

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

Add lazy events for the Node API

Right now if you want to get events via the Node API you have to have a
real instance of the Root. Which in turns sets up the whole FS.

We should make sure this is done lazy. Else enabling the preview
generator for example makes you setup the whole FS on each and every
authenticated call.

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

[tx-robot] updated from transifex

Merge pull request #21122 from nextcloud/enhancement/auth-token-exception-reporting

Improve traces of invalid token exceptions

Merge pull request #21123 from nextcloud/fix/noid/fix-retrieval-of-groups

simplify getGroups, fixing wrong chunking logic

Merge pull request #21112 from nextcloud/techdebt/noid/remove-db-prefix

Remove db prefix option from cli install command