Fix windows build

Account Settings: fix progress being written in white when there are errors

Virtual files: Wipe selective sync settings when enabled

virtual files: show option only when branding allows it

Virtual files: Use theme to check for option availability

Account Settings: Add a context menu entry to enable or disable virtual files

Issue #6725

Upload: Do not error out if the server do not send the X-OC-MTime: accepted header

If the server does not set the mtime, it is not a big problem for the
synchronisation.

The test was used before so we could do a PROPPATCH for server that did not
support this header. But now that all server supports that we don't need to
to the check. (We do not do the PROPPATCH since we got rid of the neon
dependency)

Apparently, it may happen that some backend don't support setting mtime
and this can lead to this error.

https://github.com/owncloud/client/issues/6797

Do not select ownCloud in Finder after installation to fix #6781

OAuth2: Store 'Account::davUser' in the config, and use that user for connecting

We need to use the user id to check if we are connected to the right account.
These might be different from the HTTP Basic Auth login. (LDAP setups)

When the account was configured as an oauth2 account form the wisard, the
http_user was already set correctly to the user id. But when the server is
upgrading from basic auth to oauth2, we need to pick the right login.

Note that Account::davUser() already defaults to the HTTP user when none
is set, so this means the upgrade will be fine if this is not set in the
config.

Issues:
https://github.com/owncloud/oauth2/issues/109
https://github.com/owncloud/enterprise/issues/2781

Local discovery: always recurse within touched directory

If the file system watcher tells us a directory was modified, we should
recurse into it because it means it is probably a new directory

Issue #6804

PropagateUpload: Avoid crash due to cascading aborts

https://sentry.io/owncloud/desktop-win-and-mac/issues/698694072/activity/

Fix leaks in tests

As discovered by AddressSanitizer

Discovery: restructure processFileAnalyzeLocalInfo

Discovery: easy conditions first in processFileAnalyzeRemoteInfo

Removing two levels of indent that way

Discovery: Add comments

Discovery: make sure finished is not called twice, even in case of errors

Remove the backup deteciton code which was used for server < 9.1

Discovery: Remove stale DB entries

And test the Remove/Remove case.

This means we need to always query the database for all the entries.
This showed another small bug in the test in which sync item for virtual
files at the root could have a slash in front of them.

Discovery: Comments and visibility adjustments

Discovery: cleanups and comments

Discovery: move checkMovePermissions to its own function

Discovery: Adjust the instruction in case of resolved conflict

When resolving a conflict because the file was just updated on the server,
we write all the metadata on the database immediatly, so INSTRUCITON_NONE
is enough and UPDATE_METADATA is not needed

ProcessDirectoryJob: always set _childModified to true, regardless the direction

This was like that to handle the case of CSYNC_INSTRUCTION_TYPE_CHANGE, but just add
a condition in this location.

Discovery phase: refactor some code in DiscoveryPhase::findAndCancelDeletedJob

Less code duplication

New Discovery Algorithm: more cleanups

 - rename progress() to be more explicit
 - Make some more member of the discovery phase private

Fixup commit 835c9163374f42003aa2f7795ade3f4ff62c8877

The previous code considered the also HTTP 207 code without the
application/xml header to have this message.
httpCode 0 does not make much sense anyway.

This change the behavior to consider any 2xx without the xml header
to show this error message

New discovery algorithm: fixups

Adapt reviews from ckamm in https://github.com/owncloud/client/pull/6738#pullrequestreview-164623532

- SyncJournalFileRecord: initialize everything inline
- Add more comments
- And some ENFORCE

Move Result to its own header

New discovery algorithm: Parallel PROPFIND

New Discovery algorithm: Remove the sync cleanup phase

Since we do not recurse within some directories, many files are not seen.

The stale entry will cleanup by themself as the sync engine try to remove
the files that are already removed.
Should we need to actually do this cleanup, it should be dotected in the
discovery.

New discovery algorithm: Set the originalFile for ignored files

This is used to show the name in the UI

New Propagation algorithm: Fetch, and emit, the root etag

Remove the feature to concatenate etags as servers that don't
have a root etag are no longer suported

New Discovery Algorithm: Ge tthe size of new folders

Also add a test that this works properly

New Discovery algorithm: Split the process function even further

Move the finialization in its own function.
This allow to save a bit of code duplication.

Also change the order of the parameter in the constructor for consistency

New Discovery Algo: Refactor by splitting the processFile in two

New Discovery Algorithm: Handle rename of virtual files

Virtual files: Renames propagate #6718

Folder Wizard: Disable the 'choose what to sync' treeview if virtual files are selected

Don't allow to configure virtual files with a selective sync blacklist

AccountSetting: Fix Small Memoy leak

Fix tx.pl

The conflict filename changed

AccountSettings: Force sync should wipe the blacklist

Issue #6757

Virtual files: don't check for new big folders

https://github.com/owncloud/client/issues/6724#issuecomment-417368475

Virtual files: Don't show selective sync

Issue #6724

Socket API: add an option to replace existing files with virtual files

Issue #6726

Update the minimum supported version

We want to warn if the server version is not supported and did not get
appropriate QA for this client version.

https://github.com/owncloud/enterprise/issues/2687

CMakeLists.txt: Log the Qt version in compile

Conflicts: Rename BasePath to InitialBasePath #6709

Checksum: Add support for SHA256 and SHA3

In case, some day, the server also supports it

Move the checksum related routine from FileSystem to checksum, where they belong

Fix a compilation error on windows

New Discovery Algo: readability improvements

As proposed by ckamm on #6738

New discovery algo: data finger print

New Disco Algo: Handle spurious virtual files

Port commit 0b9049e6ff01401883184d29d22dd4a30c092da8
for issue #6718

Fix test link with older Qt

Conflict handling: add the OC-ConflictBasePath header

Issue #6702

Account Settings: Add a text for folder using vierual files

Issue #6723

Virtual Files: Set the mtime of the virtual file to that of the server

Note: When the server mtime is modified, the mtime of the file is not updated

Issue #6727

Virtual Files Mime Type: use "vnd." prefix instead of "x-"

As reported in https://github.com/owncloud/client/issues/6717#issuecomment-413703567

Virtual files: Renaming to virtual doesn't delete data #6718

Unfortunately to do this, the local update phase must write to the
database, creating a new side-effect and order dependency (local update
must run before remote update).

SyncEngine: Fix renames making hierarchy inversion

Issue #6694

RemotePermissions: Fix empty vs null

Fixes two bugs that appeared since the introduction of the struct:
- when reading permissions from the journal, null ("") was read as
  empty-not-null
- when reading permissinos from the server, empty ("") was read as null

Addresses #4608

FolderStatusModel: Do not abort applying selective sync if one folder has an error

Issue #6675

SyncEngine: Fix the "direction" of the "all file delted" message when the server is reset

Using the direction of the "first" item is not enough as it might be a directory
which is a UPDATE_META_DATA,  or there can have been a few changes locally as well.

As reported on https://github.com/owncloud/client/issues/6317#issuecomment-412163113

Do not require server replies to contain an mtime

The check was added for #6317 in commit
13eb64584f5f96647ced39dcd3252860ebec5a37.

We did see missing mtimes in replies in tests with live servers though.
Possibly those were old incomplete responses cached in the stat cache?

New discovery algorithm: Call FolderDiscovered

New Discovery algorithm: Refactor a bit the way the signal are emited

Fix TestDownload: _ignore_hidden_files was not properly initialized

New Discovery Phase: fix a few FIXME

Some FIXME were already fixed.

New discovery algorithm: handle symlinks

New discovery algo: Handle Database error

Skip failing test

These tests are failling because now we consider that any files that ends
with .owncloud is a virtual file, regardless what the DB say.

New discovery phase: read the direct download URL and Cookie

More cleanup of csync remains

New Discovery Algo: Permsission check

Remove csync_update and csync_reconcile

I guess some other csync utilities can also be remove dnow but
that will be for later

Remove the check_csync_update test

since csync_update is going away

New Discovery Algorithm: handle server errors

Backport 64014dd374dc81cef24898b51830703a527e923f

Added test that checks what happens when there is an error in the remote discovery

(Many of the expected error string are left empty because the current
error message is not insterresting

Convert p7.pl to a C++ test

This is just a translation of test/scripts/txpl/t7.pl to C++ using the test
framework.

New Discovery Algo: Support the DatabaseAndFilesystem mode for local discovery

New Disco algortihm: Fix TestChunkingNG::connectionDroppedBeforeEtagRecieved

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

New disco algo: Fix TestSyncFileStatusTracker

New discovery algorithm: More work on virtual files

New Disco algorithm: Type change (file to dir)

New discovery algorithm: Virtual files

The commented tests lines were implementation details

New discovery algorithm: Fix directory deletion

TestAllFilesDeleted passes

New Discovery algorithm

Some error handling. In particular for the case where there is a conflict
between files and directories.

SyncEngineTest and SyncMoveTest passes

New Discovery algorithm: Handle of move within a moved directory

New discovery algoritmh: more on Renames

New Discovery algorithm: Check that the original file is still on the server while renaming

New disco algorithm: Fix some moving

Fix TestSyncMove::testSelectiveSyncMovedFolder

New discovery algorithm: Local rename

New discovery algo: Remote move

TestSyncMove::testRemoteChangeInMovedFolder

Handle Encoding Problems

TestSyncEngine now passes

Ignore Hidden Files

Port the invalid napme regexp

TestSyncEngine::testInvalidFilenameRegex

More progress

TestSyncEngine::testFakeConflict / TestSyncEngine::testSyncFileItemProperties

New discovery algo

Make TestSyncEngine::testSelectiveSyncBug pass

New discovery: TestSyncEngine::testEmlLocalChecksum