Extract updateSharePasswordIfNeeded function

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

Join if block to preceding if chain

If getShareType() returns "email" it can not also return "user", "group"
nor "link", so the if block can be added to the preceding if chain.

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

Add missing unit test for updateShare with email share

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

Add missing hook check

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

Fix double hashing of shared link passwords

The plain text password for a shared links was hashed and, then, the
hashed password was hashed again and set as the final password. Due to
this the password introduced in the "Authenticate" page for the shared
link was always a wrong password, and thus the file could not be
accessed.

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

Fix working icon not hidden when successfully setting a password

When a request to set the password of a shared link is sent a working
icon is shown. However, as there was no "success" callback, the icon was
never hidden again after successfully setting the password (it worked
fine if there was an error, though).

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

Add acceptance tests for sharing password protected links

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

Add system to share data between acceptance test steps

The data storage (the "notebook") is shared between all the actors, so
the data can be stored and retrieved between different steps by any
actor in the same scenario.

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

Merge pull request #4434 from nextcloud/add-rate-limiting-to-solve-challenge-controller

Add rate limit to TOTP solve challenge controller

Merge pull request #4401 from nextcloud/caldav-carddav-nc-owner-displayname

add owner-displayname property to calendars and addressbooks

Merge pull request #4444 from nextcloud/remove-unused-variables

Remove unused variables

[tx-robot] updated from transifex

add owner-displayname property to calendars and addressbooks

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

[tx-robot] updated from transifex

Remove unused variables

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

Merge pull request #4435 from nextcloud/allow-at-maximum-10-reset-mails-requests-per-5-minutes

Add at most 10 password reset requests per 5 minutes and IP range

Merge pull request #4432 from nextcloud/fix-grammar

Fix grammar

Add at most 10 password reset requests per 5 minutes and IP range

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>

Add rate limit to TOTP solve challenge controller

Fixes https://github.com/nextcloud/server/issues/2626

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>

[tx-robot] updated from transifex

Merge pull request #4431 from nextcloud/acceptance-macos

Run acceptance tests on macOS

Fix grammar

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

Run acceptance tests on macOS

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

Merge pull request #4208 from danxuliu/add-basic-acceptance-test-system

Add basic acceptance test system

Merge pull request #4396 from nextcloud/scan-non-existing

show error when trying to scan non existing path

Merge pull request #4428 from nextcloud/file-by-id-limit-user

limit the user when searching for a file by id if we know the user already

Merge pull request #4426 from nextcloud/update-dom-purify

Bump to DOMPurify 0.8.6

Merge pull request #4429 from nextcloud/proper-return-code-for-welcome-email

Use RESPOND_NOT_FOUND for non existing user

Use RESPOND_NOT_FOUND for non existing user

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

limit the user when searching for a file by id if we know the user already

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

Update core.js

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

Merge pull request #4422 from nextcloud/favorite-wording-detail

Wording detail fix for favorite emptycontent view

Merge pull request #4423 from nextcloud/reload-fav-view

properly reload favorites list

Bump to DOMPurify 0.8.6

Fixes https://github.com/nextcloud/server/issues/4424

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>

Make test passwords valid for the password_policy app

As requested by Morris Jobke, the passwords in the acceptance tests were
modified to make them valid both for a clean Nextcloud server and one
with the password_policy app enabled.

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

Move acceptance tests from build/acceptance to tests/acceptance

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

Exclude data-autotest from the files copied to the container

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

Add safety parameter

As the script modifies the Git repository a safety parameter was added
to prevent running it by mistake and messing with the local copy of the
repository.

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

Remove no longer needed Docker helper and its related Dockerfile

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

Change default configuration to use local helper

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

Run acceptance tests using the local helper instead of the Docker one

When run through "run.sh" the acceptance tests were executed in the same
system in which the script was called and they started and stopped the
Nextcloud server using Docker containers that provided real web servers.
For consistency now they use the same approach used when run through
Drone: the acceptance tests are run in a Docker container and they start
and stop the Nextcloud server directly using the PHP built-in web server.

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

Generalize names and descriptions

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

Use PHP built-in web server instead of Apache in Drone

Instead of running an additional Drone service with the Nextcloud server
now the Nextcloud server is run in the same Drone step as the acceptance
tests themselves using the PHP built-in web server.

Thanks to this, the Nextcloud server control is no longer needed, as the
acceptance tests can now directly reset, start and stop the Nextcloud
server. Also, the "nextcloudci/php7.0:php7.0-7" image provides
everything needed to run and manage the Nextcloud server (including the
Git command used to restore the directory to a saved state), so the
custom image is no longer needed either.

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

Replace downloaded Selenium server with Docker container

Instead of downloading the Selenium server and requiring a specific
Firefox version to be installed in the system now the Selenium server is
run using one of the official Selenium Docker images, which provides
both the Selenium server and the appropriate version of Firefox.

Moreover, as it is run inside the Docker container, the web browser is
now run in headless mode; however, if needed, it can still be viewed
through VNC.

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

properly reload favorites list

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

Wording detail fix for favorite emptycontent view

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

Merge pull request #4418 from nextcloud/fix_3380

No need to log guzzle exception

No need to log guzzle exception

When getting/requesting a shared secret there is no need to log the full
exception when an unexpected status code is returned since this won't
really tell us anything as the bad stuff happened on a remote server

Fixes #3380

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

Merge pull request #4406 from nextcloud/fix-unit-test-problems

Fix unit test problems

Merge pull request #4408 from nextcloud/accesslist_ng

Move systemtags and comments to new accesslist functions

[tx-robot] updated from transifex

Merge pull request #4412 from individual-it/master

deleted forgotten code in test

Merge pull request #4409 from nextcloud/socialharing_mail

Allow social sharing to specify if a new window is opened

Merge pull request #4404 from Ardinis/master

Remove json handling for files_external:config

Merge pull request #3595 from nextcloud/dav-sharing-changes

Restrict DAV share handling to the owner only

Fix unit tests

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

deleted forgotten code in test

how embarrassing, left code from before I've done the loop around the test
Signed-off-by: Artur Neumann <info@individual-it.net>

Merge pull request #4411 from nextcloud/fix-it

Remove accidentially commited module

Remove accidentially commited module

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

Merge pull request #4402 from nextcloud/fix-clickable-area

Fix clickable area in share permissions menu

Merge pull request #4303 from nextcloud/enforce-password-mailshare

allow admin to enforce password on mail shares

fix typo

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

Fix casing

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

fix unit tests

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

respect password policy for auto generated passwords

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

improve mail text

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

create activity if a password was send by mail

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

allow to set a password for shares which where created without a password before the admin started to enforce the password

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

templates no longer needed

allow admin to enforce password on mail shares

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

Allow social sharing to specify if a new window is opened

For example mail shares should not open a new window because it looks
weird.

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

Move systemtags and comments to new accesslist functions

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

Fix invalid Dav test

PHP Warning:  Declaration of PublicCalendarTest::testPrivateClassification()
should be compatible with CalendarTest::testPrivateClassification($expectedChildren, $isShared)
in apps/dav/tests/unit/CalDAV/PublicCalendarTest.php on line 29

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

Make sure we use a new encryption module all the time

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

check for existence before we start the db transaction

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

Don't assume the admin didn't configure Opcache correctly...

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

Fix theming tests

Trying to configure method "shouldReplaceIcons" which cannot be configured
because it does not exist, has not been specified, is final, or is static

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

Use the correct class

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

Import some classes

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

Use the correct Dummy and Backend class

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

Kill dead code

> No tests found in class "Test\Share\MailNotificationsTest".

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

Fix warning with undefined method

Trying to configure method "getRemember" which cannot be configured
because it does not exist, has not been specified, is final, or is
static

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

Don't use deprecated getMock() anymore

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

Restoring the error handler within the error handler causes unexpected results

See http://php.net/manual/en/function.restore-error-handler.php#120879
for more information.

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

Delete the correct config value

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

Only loop over Test cases

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

reviewed

Merge pull request #4398 from nextcloud/fix_accesslistcode

Get proper accesslist for userFolder

Merge pull request #4212 from individual-it/master

validate file name before uploading in upload only folder

Fix remaining "PHP Inspection" warnings

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

Add public access modifier to all methods

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

Restrict share handling to the owner only

Otherwise group members can remove the share for the complete group,
remove edit permissions and even single user shares for other users.

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

Get proper accesslist for userFolder

If the accesslist is requested for a users root folder we should
properly construct the path

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

Merge pull request #4381 from nextcloud/2954_take_2

Fix group settings routes and fix route regression

[tx-robot] updated from transifex

Merge pull request #4384 from nextcloud/renderHtml-fix

Rename renderHTML to renderHtml

Fix clickable area in share permissions menu

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

Add PHPDoc and handle exception in ScanAppData as well

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

Rename renderHTML to renderHtml

* fixes #4383
* improves consistency

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

Merge pull request #4387 from nextcloud/fix-little-glitch

Fix font-weight of settings button