Fix usages of afterMethod -> afterMethod:*

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

[tx-robot] updated from transifex

Merge pull request #20561 from nextcloud/revert-20555-dependabot/npm_and_yarn/build/karma-coverage-2.0.2

Revert "Bump karma-coverage from 2.0.1 to 2.0.2 in /build"

Merge pull request #19214 from nextcloud/enh/preview_folder_tree

Move to subfolders for preview files

Revert "Bump karma-coverage from 2.0.1 to 2.0.2 in /build"

Merge pull request #20555 from nextcloud/dependabot/npm_and_yarn/build/karma-coverage-2.0.2

Bump karma-coverage from 2.0.1 to 2.0.2 in /build

Merge pull request #20560 from nextcloud/dependabot/npm_and_yarn/style-loader-1.1.4

Bump style-loader from 1.1.3 to 1.1.4

Move to subfolders for preview files

Else the number of files can grow very large very quickly in the preview
folder. Esp on large systems.

This generates the md5 of the fileid. And then creates folders of the
first 7 charts. In that folder is then a folder with the fileid. And
inside there are the previews.

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

[tx-robot] updated from transifex

Bump style-loader from 1.1.3 to 1.1.4

Bumps [style-loader](https://github.com/webpack-contrib/style-loader) from 1.1.3 to 1.1.4.
- [Release notes](https://github.com/webpack-contrib/style-loader/releases)
- [Changelog](https://github.com/webpack-contrib/style-loader/blob/master/CHANGELOG.md)
- [Commits](https://github.com/webpack-contrib/style-loader/compare/v1.1.3...v1.1.4)

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

Bump karma-coverage from 2.0.1 to 2.0.2 in /build

Bumps [karma-coverage](https://github.com/karma-runner/karma-coverage) from 2.0.1 to 2.0.2.
- [Release notes](https://github.com/karma-runner/karma-coverage/releases)
- [Changelog](https://github.com/karma-runner/karma-coverage/blob/master/CHANGELOG.md)
- [Commits](https://github.com/karma-runner/karma-coverage/compare/v2.0.1...v2.0.2)

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

Merge pull request #20449 from liamjack/fix/19066/empty_file_versions

Array offset error due to empty file versions array

Merge pull request #20540 from nextcloud/fix/mail-test

Fix mailer test

Added trailing space

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

[tx-robot] updated from transifex

Merge pull request #20537 from nextcloud/external-tests-allow-local-access

allow local remote access in external storage ci

Fix mailer test

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

allow local remote access in external storage ci

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

Merge pull request #19901 from nextcloud/bugfix/noid/vcard-photo-handling

Improved vcard photo handling

Merge pull request #20529 from nextcloud/fix/Card-multiple

Update list of multiple properties

Merge pull request #20201 from rmanibus/issue_20130

display file list in flexbox

Merge pull request #20447 from liamjack/fix/13287/welcome_email

Increase max-height on button in welcome email template

Update list of multiple properties

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

Merge pull request #20381 from nextcloud/fix/allow_edit_admin_user

Allow to edit admin user

Fix tests

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

Fix breadcrumb in move or copy file dialog

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

[tx-robot] updated from transifex

Merge pull request #20523 from nextcloud/enh/noid/rename-updatenotification

Rename updatenotifications webpack module to updatenotification

Rename updatenotifications webpack module to updatenotification

The module build name for `updatenotification` is actually
`updatenotifications` which can be confusing.

It is the only module with a name that is
different from the app's name.

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

Allow to edit admin user

Signed-off-by: GretaD <gretadoci@gmail.com>
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

Merge pull request #20505 from nextcloud/fix/noid/system-creds

do not advertise nulled userId for for systemwide credentials

Merge pull request #20511 from nextcloud/fix/cs-fixer-ldap

PHP-CS-Fixer green

Merge pull request #16946 from exussum12/patch-1

Increase max read

Merge pull request #20246 from nextcloud/fix/untranslated_mail_footer

Provide the proper language to the mailer

PHP-CS-Fixer green

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

Provide the proper language to the mailer

Else we can't properly translate the footer in the recipients e-mail
language.

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

Merge pull request #20506 from nextcloud/version/19/beta4

19 beta 4

Merge pull request #20509 from nextcloud/revert-20050-bugfix/noid/mysql-equal-trims-shock

Revert "Make sure MySQL is not saying 'this' = 'this ' is true"

[tx-robot] updated from transifex

Revert "Make sure MySQL is not saying 'this' = 'this ' is true"

19 beta 4

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

Merge pull request #20503 from nextcloud/fix/noid/dav-catch-throwable

DAV's exception logger should deal with any Throwable

Merge pull request #20386 from nextcloud/version/19/beta3

19 beta 3

Merge pull request #20501 from nextcloud/fix/noid/run-cs-fix

Run cs:fix and fix HashWrapper code

Update View.php

fix credentialsManager documentation and ensure userId to be used as string

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

Merge pull request #20502 from nextcloud/techdebt/cs-check-diff

Print diff in cs:check command

add DB tests for credentials manager

these are actually expected to FAIL, because NULL as a userid is not
allowed in the schema, but documented to be used on the source

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

Update Contact Size after removing photo

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

Don't populate the PHOTO property when it's not an image

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

Always transform the etag and read the data

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

Use table variable everywhere

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

DAV's exception logger should deal with any Throwable

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

Merge pull request #20037 from nextcloud/fix/19127/deprecated-ldap_control_paged_result_1

do not use deprecated API for LDAP paged results

Print diff in cs:check command

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

Run cs:fix and fix HashWrapper code

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

Merge pull request #20500 from nextcloud/update-editor-config

Update editor config for .yml files

Merge pull request #19540 from jlutran/fixKeystoneV3Auth

Fix issues with Keystone auth v3 in files_external app

Update editor config for .yml files

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

Merge pull request #20481 from ACTom/add-mindmap-mimetype

Add mindmap mimetype.

Merge pull request #20050 from nextcloud/bugfix/noid/mysql-equal-trims-shock

Make sure MySQL is not saying 'this' = 'this ' is true

Merge pull request #20138 from nextcloud/bugfix/noid/make-remote-checking-more-generic

Make remote checking more generic

Merge pull request #20401 from nextcloud/fix/login-sso-redirct

Fix absolute redirect

Merge pull request #19191 from laurisb/patch-1

Improve Opcache detection

Merge pull request #20361 from nextcloud/write-stream-close-on-exception

Close the streams in `writeStream` even when there is an exception

Fix issues with Keystone auth v3 in files_external app

Signed-off-by: Julien Lutran <julien.lutran@corp.ovh.com>

Merge pull request #19351 from nextcloud/dav-upload-hash

Return hashes of uploaded content for dav uploads

Make sure MySQL is not saying 'this' = 'this ' is true

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

Merge pull request #20476 from nextcloud/techdebt/php-cs-check-action

Add php-cs check action

Allow local remotes in integration tests

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

Fix unit tests

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

[tx-robot] updated from transifex

Show a hint for the php-cs fix when the check fails

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

Merge pull request #20474 from nextcloud/update-crl

Update root.crl due to revocation of spgverein.crt

Merge pull request #19936 from nextcloud/bugfix/noid/handle-unset-owner-in-sharing

Handle unset owner in sharing

Add mindmap mimetype.

Signed-off-by: ACTom <i@actom.me>

Merge pull request #20181 from nextcloud/bugfix/noid/dont-break-on-path-null

Also bail out when the path is actually null

Also disallow ; in remote urls

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

Check all remotes for local access

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

Fix php-cs formatting

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

Add php-cs check action

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

Merge pull request #20472 from nextcloud/fix/19848/make-sure-destination-is-not-a-directory

Verify that destination is not a directory.

Update root.crl due to revocation of spgverein.crt

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

Merge pull request #20470 from nextcloud/fix/noid/karma-coverage

Fix jsunit tests

use serverControls directly with LDAP calls, fixes 19127

- adapters for PHP API version to Support PHP < 7.3
- switch to pass only one base per search
- cookie logic is moved from Access to API adapters

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

Verify that destination is not a directory.

Otherwise file_put_contents will fail later.

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

Fix jsunit tests

JSUnit tests are failing for some reason. It seems to be related to a new karma-coverage version published a few hours ago.

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

Merge pull request #20469 from nextcloud/fix/sidebar/sharing/password-error-message

Properly display share error messages

Properly display share error messages

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

[tx-robot] updated from transifex

Merge pull request #20444 from nextcloud/dependabot/npm_and_yarn/url-loader-4.1.0

Bump url-loader from 4.0.0 to 4.1.0

Merge pull request #19852 from nextcloud/dav/provide-dav-setting-for-default-calendar

Provide dav setting for user's default calendar

Merge pull request #20460 from nextcloud/fix-app-not-listed-in-provisioning-integration-test

Fix app not listed in provisioning integration test

Bump url-loader from 4.0.0 to 4.1.0

Bumps [url-loader](https://github.com/webpack-contrib/url-loader) from 4.0.0 to 4.1.0.
- [Release notes](https://github.com/webpack-contrib/url-loader/releases)
- [Changelog](https://github.com/webpack-contrib/url-loader/blob/master/CHANGELOG.md)
- [Commits](https://github.com/webpack-contrib/url-loader/compare/v4.0.0...v4.1.0)

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

Merge pull request #20218 from nextcloud/dependabot/npm_and_yarn/marked-0.8.2

Bump marked from 0.8.1 to 0.8.2

Bump marked from 0.8.1 to 0.8.2

Bumps [marked](https://github.com/markedjs/marked) from 0.8.1 to 0.8.2.
- [Release notes](https://github.com/markedjs/marked/releases)
- [Commits](https://github.com/markedjs/marked/compare/v0.8.1...v0.8.2)

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

Merge pull request #20445 from nextcloud/dependabot/npm_and_yarn/nextcloud/paths-1.1.2

Bump @nextcloud/paths from 1.1.1 to 1.1.2

Fix app not listed in provisioning integration test

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

Only delete the default calendar setting when the default calendar
itself is deleted

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

Provide dav setting for user's default calendar

And add tests to handle schedule-default-calendar-URL

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