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

New discovery algorithm

Some progress:
TestSyncEngine::testDirUpload

New discovery algorithm: Initial work.

SyncEngineTest testFileDownload is passing

Revert "Merge pull request #1454 from nextcloud/syncjournal-del-prio"

This reverts commit d9fd9cfef24ce500ebe812eaba38ce2418ab6ae0, reversing
changes made to 2dcf594fc6ca5e924f40c7ad557bc62984eaa415.

Rename ocsync library to ${APPLICATION_EXECUTABLE}_csync

Install libocsync to lib/ without subfolder.

Installing to lib/${APPLICATION_EXECUTABLE} has caused a bunch of
irritations in the past and subtle annoying to fix bugs. To avoid name
clashes with branded clients ${APPLICATION_EXECUTABLE} becomes now
part of the filename instead of the subfolder.

The concrete motivation to change this now is that on Windows there
is no RPATH and it's not possible to run owncloud directly from the
Craft Root folder, which is nice when you're developing on Windows.

It would have been possible to change this just for Windows but as
written earlier this has caused lots of issues and thus I think it's
a good idea to just stay consistent accross platforms when touching it.

nautilus shell integration: Fix when there are several branded client installed

It appears that several extension can be loaded at the same time, but their
classname for the extension need to be different, otherwise only the last
loaded one would be active.

Issue #6524

Wizard + OAuth: Make opening a new browser after clicking back works again

Issue #6574

When there is an error in the advanced page, OwncloudAdvancedSetupPage::updateStatus
(and others) call completeChanged(), which is connected to
QWizardPrivate::_q_updateButtonStates which will re-enable the back button from the
last page.

When the user click "back" and re-open the browser, the account's credentials
already have a oauth token set. So the call to the API to get a new token fails
because we use the previous token instead of using the client's secret_id.
Fix this with the HttpCredentials::DontAddCredentialsAttribute.

Now, this is still not working because the session cookies are confusing the
server.  So we'll clear the cookies when re-opening the browser

FindSql3.cmake: Add the input variable in the search path

Issue #6278

FolderStatusModel: Fix crash when there is an error while expanding folders

In FolderStatusModel::slotLscolFinishedWithError, the call to parentInfo->resetSubs
deleted the 'job' and the reply 'r' which we accessed later to get the error code.

Fix this problem twice by
 1) Get the error code before caling resetSubs
 2) in FolderStatusModel::SubFolderInfo::resetSubs, call deleteLater instead of delete

Regression introduced in commit d69936e0

Issue #6562

csync: refactor csync_s::error_string to avoid valgrind error

The problem here is that we were sometimes allocating the error_string with
qstrdup, which need to be released with delete[] and not free().

Simplify the code by using QString instead.

```
==7230== Mismatched free() / delete / delete []
==7230==    at 0x4C2E10B: free (vg_replace_malloc.c:530)
==7230==    by 0x57C2321: csync_s::reinitialize() (csync.cpp:247)
==7230==    by 0x548130F: OCC::SyncEngine::finalize(bool) (syncengine.cpp:1212)
==7230==    by 0x5481223: OCC::SyncEngine::handleSyncError(csync_s*, char const*) (syncengine.cpp:746)
==7230==    by 0x5483E78: OCC::SyncEngine::slotDiscoveryJobFinished(int) (syncengine.cpp:965)
==7230==    by 0x5495E34: QtPrivate::FunctorCall<QtPrivate::IndexesList<0>, QtPrivate::List<int>, void, void (OCC::SyncEngine::*)(int)>::call(void (OCC::SyncEngine::*)(int), OCC::SyncEngine*, void**) (qobjectdefs_impl.h:134)
==7230==    by 0x5495D92: void QtPrivate::FunctionPointer<void (OCC::SyncEngine::*)(int)>::call<QtPrivate::List<int>, void>(void (OCC::SyncEngine::*)(int), OCC::SyncEngine*, void**) (qobjectdefs_impl.h:167)
==7230==    by 0x5495CB5: QtPrivate::QSlotObject<void (OCC::SyncEngine::*)(int), QtPrivate::List<int>, void>::impl(int, QtPrivate::QSlotObjectBase*, QObject*, void**, bool*) (qobjectdefs_impl.h:396)
==7230==    by 0xA9BF2E1: QObject::event(QEvent*) (in /usr/lib/libQt5Core.so.5.11.0)
==7230==    by 0x64BE983: QApplicationPrivate::notify_helper(QObject*, QEvent*) (in /usr/lib/libQt5Widgets.so.5.11.0)
==7230==    by 0x64C625A: QApplication::notify(QObject*, QEvent*) (in /usr/lib/libQt5Widgets.so.5.11.0)
==7230==    by 0xA994BC8: QCoreApplication::notifyInternal2(QObject*, QEvent*) (in /usr/lib/libQt5Core.so.5.11.0)
==7230==  Address 0x225b2640 is 0 bytes inside a block of size 50 alloc'd
==7230==    at 0x4C2DC6F: operator new[](unsigned long) (vg_replace_malloc.c:423)
==7230==    by 0xA7E8FC8: qstrdup(char const*) (in /usr/lib/libQt5Core.so.5.11.0)
==7230==    by 0x53F5750: OCC::DiscoveryJob::remote_vio_opendir_hook(char const*, void*) (discoveryphase.cpp:666)
==7230==    by 0x57E1278: csync_vio_opendir(csync_s*, char const*) (csync_vio.cpp:39)
==7230==    by 0x57D718F: csync_ftw(csync_s*, char const*, int (*)(csync_s*, std::unique_ptr<csync_file_stat_s, std::default_delete<csync_file_stat_s> >), unsigned int) (csync_update.cpp:674)
==7230==    by 0x57C1B05: csync_update(csync_s*) (csync.cpp:109)
==7230==    by 0x53F5BCC: OCC::DiscoveryJob::start() (discoveryphase.cpp:718)
==7230==    by 0x54B8F74: OCC::DiscoveryJob::qt_static_metacall(QObject*, QMetaObject::Call, int, void**) (moc_discoveryphase.cpp:494)
==7230==    by 0xA9BF2E1: QObject::event(QEvent*) (in /usr/lib/libQt5Core.so.5.11.0)
==7230==    by 0x64BE983: QApplicationPrivate::notify_helper(QObject*, QEvent*) (in /usr/lib/libQt5Widgets.so.5.11.0)
==7230==    by 0x64C625A: QApplication::notify(QObject*, QEvent*) (in /usr/lib/libQt5Widgets.so.5.11.0)
==7230==    by 0xA994BC8: QCoreApplication::notifyInternal2(QObject*, QEvent*) (in /usr/lib/libQt5Core.so.5.11.0)
==7230==
```

PropagateDownload: Don't discard the body of error message

We want to keep the body so we can get the message from it
(Issue #6459)

TestDownload::testErrorMessage did not fail because the FakeErrorReply
did not emit readyRead and did not implement bytesAvailable.

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

.desktop file: the MimeType entry must ends with a semi colon

Also remove some comments

Virtual Files: Allow to download a folder recursively from the socket API

Issue: #6466

Virtual files: Wipe virtual after download completes, not before

Otherwise a interrupted or unsuccessful download would mean that the
download-intend was forgotten. The next sync would reestablish the
virtual file instead.

Rename SocketApi::FileData::accountRelativePath to serverRelativePath

As discussed in issue #6552

Share dialog: use the original name and not the virtual file name

When sharing a virtual file, we should actually use the original file name
not the virtual file name

Issue: #6461

Virtual files: Only remove virtual file once on download

With thanks to @ogoffart for spotting the problem.

Settings migration: Preserve future settings where possible

See discussion in #6506

Config: Add version flags to accounts and folders

Also, if there is too-new configuration, backup the file, show a
warning message asking the user whether it's ok to discard the
configuration from the future.

See #6504

Rename Placeholders to Virtual Files in code #6531

SyncRunFileLog: Add the requestid

https://github.com/owncloud/client/pull/6427#issuecomment-383879509

Blacklist: remember the X-Request-ID

Issue #6420
Store the X-Request-ID in the SyncFileItem and also in the blacklist.
Note that for consistency reason, the X-Request-ID is also in the
SyncFileItem if the request succeeds.

Currently there is no UI to access it, but it can be queried with sql
commands

Wizard: Add a "(recommended)" to the main option #6470

Rename "placeholder" feature

Placeholder: Stop adding ignore pattern

Because we can't make older clients preserve the version directive that
was attached to it.

See #6504 and #6498

Placeholder: Don't contain "stub"

Placeholders: Save to key that's invisible to <2.5 clients #6504

FolderWizard: fix crash when experimental feature are disabled

The _placeholderCheckBox only exists if the experimental features are enabled

Found via the crash reporter
https://sentry.io/owncloud/desktop-win-and-mac/issues/556407777/

Correct incorrect reference to OSX Menu bar

This fixes https://github.com/owncloud/documentation/issues/4109.

Share Dialog: Disable workaround for macOS too #6185

CMake: Add a default placeholder_suffix

To make themes work that don't define it explicitly.