Fix class names

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

Fix last unit tests

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

Don't end the abstract class name with Test.php

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

Add comment

Properly handle missing READ permission

Fix duplicate name of class

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

Fix forbidden backslash DAV integration tests

Fix FutureFile MOVE to keep destination node

Sabre usually deletes the target node on MOVE before proceeding with the
actual move operation. This fix prevents this to happen in case the
source node is a FutureFile.

Accept moving FutureFile into a Directory

Moved unit tests from ObjectTree::move to Directory

Remove ObjectTree::move and let is use the IMoveTarget approach instead

This removes the duplicated code

Ported ObjectTree::move to IMoveTarget in new DAV endpoint

Added test cases from core 16825

Simplified new endpoint move test

Added case when final chunk move must not change file id

Added test about checking file id after a move

Merge pull request #4430 from nextcloud/refactor-auth

Improve readability of login logic

Merge pull request #4478 from nextcloud/improve_settings_help

adding some documentation hints and some smaller cleanup

Merge pull request #4467 from nextcloud/lists-for-html-mails

Add lists to HTML emails

Merge pull request #4061 from nextcloud/downstream-26407

Redis cluster support

Merge pull request #4500 from nextcloud/downstream-27118

Restrict proppatch to the proper nodes

Merge pull request #4416 from gvmura/avoid-duplicate-files-google

Avoid duplicate files in google with the same name

Merge pull request #4449 from stweil/mimetypes

Add mimetypes for jp2 and webp

Refactor auth methods

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

Merge pull request #4486 from nextcloud/fix-js-unit

Remove DOMPurify from srcFiles

Merge pull request #4489 from nextcloud/ldap-gidnumber-sprt-xuanwo

user_ldap: Add support for gidNumber

[tx-robot] updated from transifex

Merge pull request #3233 from nextcloud/contactsmenu

Contacts menu

Merge branch 'master' into contactsmenu

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

Fix test

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

Merge pull request #4499 from nextcloud/downstream-26984

Trigger change when a user is enabled/disabled

Merge pull request #4508 from nextcloud/fix/remove-js-debug-logging

Remove js debug logging

Merge pull request #4479 from nextcloud/auth-flow

Add new auth flow basics

Merge pull request #4495 from nextcloud/downstream-27638

Catch session already closed exception in destructor

Remove js debug logging

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

Fix emptycontent HTML and wording

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

Fix spinner position of Contacts menu

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

Use HTTP POST to retrieve the list of contacts

We do not want to have sensitive information in the URL and
therefore also not in the access log. Thus the GET request is
replaced by a POST request.

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

Update composer autoloader

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

Adjust entry unit test to newly added avatar property

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

Fix failing contacts menu js test data

When we test wheter action menus in the contacts menu close
when clicking other ones, we have to provide test data
that actually causes the view to render the menu.

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

Fix icon of second action

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

Use absolute URI for action icons

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

Don't show contacts an entry for themselves

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

Let apps register contact menu provider via info.xml

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

Use tooltop for contact's top action title

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

Show mail address in popover menu

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

Contacts menu

* load list of contacts from the server
* show last message of each contact

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

Fix auth flow background color and redirect view layout

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

AppToken to 72 chars

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

Do not remove the state token to early

we should check the stateToken before we remove it. Else the check will
always fail.

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

Correctly case the stateToken

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

Add new auth flow

This implements the basics for the new app-password based authentication flow for our clients.
The current implementation tries to keep it as simple as possible and works the following way:

1. Unauthenticated client opens `/index.php/login/flow`
2. User will be asked whether they want to grant access to the client
3. If accepted the user has the chance to do so using existing App Token or automatically generate an app password.

If the user chooses to use an existing app token then that one will simply be redirected to the `nc://` protocol handler.
While we can improve on that in the future, I think keeping this smaller at the moment has its advantages. Also, in the
near future we have to think about an automatic migration endpoint so there's that anyways :-)

If the user chooses to use the regular login the following happens:

1. A session state token is written to the session
2. User is redirected to the login page
3. If successfully authenticated they will be redirected to a page redirecting to the POST controller
4. The POST controller will check if the CSRF token as well as the state token is correct, if yes the user will be redirected to the `nc://` protocol handler.

This approach is quite simple but also allows to be extended in the future. One could for example allow external websites to consume this authentication endpoint as well.

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

Merge pull request #4439 from nextcloud/appmenu-responsive

Responsive app menu

Add test code for new image mime types

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Add repair steps for new image mime types

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Add mimetypes for jp2 and webp

Those image formats can be processed by Tesseract, so they are needed
for improved Nextcloud OCR.

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Remove DOMPurify from srcFiles

It is already included via core.json

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

Move h2 and settings-hint together

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

Merge pull request #4494 from nextcloud/fix-jsunit

Fix JS unit failure because of Jasmine 2.6 and the phantomjs launcher

Wording fixes

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

Small layout fixes to settings hints

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

change the headline as suggested

one more hint

adding some documentation hints and some smaller cleanup

Fix loading spinner for new app menu

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

Fix adding/removing apps

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

Responsive app menu

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

Restrict proppatch to the proper nodes

Need to fetch the node earlier because cancelling from within the
handler is not possible. Well, it is but it prevents other node types
using the same property names to run because the failure marks the
property with status 403.

Audit log for enabling/disabling a user

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

Trigger change when a user is enabled/disabled

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

Catch session already closed exception in destructor

Merge pull request #4490 from GitHubUser4234/audit_log_group_creation_deletion

Audit group creation/deletion

Merge pull request #3531 from nextcloud/theming-scss

Theming using SCSS variables

Merge pull request #4485 from nextcloud/translation-checker

Check whether we can json decode the translations

Fix JS unit failure because of Jasmine 2.6 and the phantomjs launcher

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

Merge pull request #3195 from nextcloud/settings-apps-tabular

Make apps settings tabular

Fix usersInGroup retrieval

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

Loop over the apps directory and add the task

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

Allow meta info and icons

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

Merge pull request #4462 from danxuliu/fix-sharing-password-protected-link

Fix sharing a password protected link

Merge pull request #4484 from nextcloud/fix/logincontroller-redirect-url-failed-login

Fix remember redirect_url on failed login attempts

4190

Signed-off-by: Roger Szabo <roger.szabo@web.de>

fix method name due to changes in master

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

Merge branch 'master' of https://github.com/Xuanwo/server into Xuanwo-master2

Mock it

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

Logo and background URLs must be relative

They are used way to often for such a change

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

Fix theming tests

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

Use theming cachebuster for server resources

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

Implement scss variable injection by OC_Defaults

Signed-off-by: Julius Haertl <jus@bitgrid.net>
Add Scss variables to example theme and theming app

Signed-off-by: Julius Haertl <jus@bitgrid.net>
Use SCSSCacher to build theming css

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

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

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

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Inject SCSSCacher for easier testing

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

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

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
Fix tests to always use absolute urls for theming images

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

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

Fix invalid character

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

Check whether we can json decode the translations

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

Fix remember redirect_url on failed login attempts

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

user_ldap: Add support for gidNumber

This patch is based on the work of @dleeuw (https://github.com/dleeuw)
(See https://github.com/nextcloud/server/issues/2640#issuecomment-269615883 for more details).
The difference is user & group data will be written into cache to have
better performance, and functions splited from primaryGroupID series to
make them more readable.

Fixed https://github.com/nextcloud/server/issues/2640

Signed-off-by: Xuanwo <xuanwo@yunify.com>

[tx-robot] updated from transifex

Fix app icon being inverted for app store apps too

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

Show spinner until app store images loaded

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

Show placeholder for image-less apps in app management

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

Center spinner in apps management

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

Add installed category in AppSettingsControlerTest

Signed-off-by: Felix A. Epp <work@felixepp.de>