SyncEngine: Rename phase finishing slots

slotDiscoveryJobFinished -> slotDiscoveryFinished
slotFinished -> slotPropagationFinished

This should be clearer. Particular the
slotFinished -> finalize -> emit finished()
chain was confusing before.

Discovery: Fix aborting during discovery #6972

vfs suffix: Ignore server files or synced files with the suffix #6953

Discovery: Ensure selective sync lists are sorted #6958

vfs: Add vfs migration options to folder context menu

This allows enabling and disabling vfs.

To distinguish this operation from setting the root pin state, the
availability setting is adjusted as well to be similar to the
menu that shows in the shell extensions.

vfs: Allow retrieving of pin state paths and flags

Result: Add copy/move ctor/op=

It has a destructor and these operations make sense. Particularly the
move is important for code like:

Result<x, y> foo() { Result<x, y> v; return v; }

because the move-ctor will not autogenerate if x or y are not trivially
destructible.

vfs: Change per-folder message on vfs support

I'm unsure this note is all that useful. Remove entirely?

vfs: Allow (de-)hydrating the full sync folder

vfs: Don't assume suffix mode for old folders

That would break with old folders that use selective sync.

vfs: Disable selective sync if vfs support is available

Wizard: Ensure client cert doesn't get lost #6911

Wizard: Drop unused function

Sync: Display theme in debug log

sqlite: Set exclusive locking_mode to avoid WAL issues #6881

Can be overridden with OWNCLOUD_SQLITE_LOCKING_MODE

Fix lookup of system wide settings

sqlite: Update bundled version to 3.26.0

For OS X and Windows.

HttpCredentials: Do not re-enter the event loop

https://sentry.io/owncloud/desktop-win-and-mac/issues/777907931/
mention a crash in OCC::HttpCredentialsGui::showDialog
One possible explaination is that this is caused by re-entring the event loop.
So don't do that.

Chunking Move: Fix too many starting slashes in the destination header

Commit 6ca724f fixed it for the move case. But the upload MOVE did the same

Issue #6904

Gui: do not show the settings when opening a virtual file

Issue #6764

SocketAPI: Overhaul ui for 'available locally' vs 'online only'

For #6815

Db: Add wiping of pin state for subtrees

vfs: Remove newFilesAreVirtual - use root PinState instead

This unifies how to deal with pin states.

Also enable reading a folders direct pin state vs its effective pin
state.

Discovery: Handle the blacklistFiles from the server capabilities

Issue #434

Ideally one could add the blacklist to the exlucde reggexp, but this
is simpler

Progress: Virtual file creation needs no transfer progress #6933

Treat virtual file creation as a size-less action, similar to
propagating renames or deletions.

vfs: Show "new files are virtual" option only with vfs

If there's no vfs possibility, showing it makes little sense.

vfs: SocketAPI actions adjust pin state of directories #6815

Downloading a folder also sets its pin state; releasing a folder sets
its pin state.

vfs: Use PinState in sync algorithm #6815

New files are virtual if the file's pin state is OnlineOnly.

vfs: Introduce PinState db storage #6815

The idea is to allow folders (and later maybe files?) to be
- pinned to be available locally
- pinned to be online only
- inherit their pin from the parent

Where this pinning only controls the default for new files.
Subfolders may have a different pin state, and contained files
may be hydrated or dehydrated based on user actions.

This value is stored in a new 'flags' table. The idea is to store
data there that doesn't necessarily exist for each metadata entry.
The selective sync state could be migrated to this table.

Move test for issue #1329 from t1.pl to new test system

Discovery: Set right direction when restoring deleted discovery because it has modified files

(Catched by a faillure of t1.pl)

Port the exclude test to the QTest Framework

This is just a port to QtTest, I did not change the layout of the test.
I did search and replace to replace the assert with QCOMPARE/QVERIFY
I still call setup and setup_init like before (only explicitly, now)

Also ported the preformence tests to QBENCHMAK because windows don't have
gettimeofday.

Relates #6358

Test: Move the test for Utility::normalizeEtag to testutility.cpp

I just moved the text and did the minimum to port it to QtTest
Did not change hte layout of it.

Relates #6358

Remove check_csync_util

It's testing nothing.
Only that one can convert a csync instruction to a string.
But this is only used in debug anyway

Relates to #6358

Database: Change path for new dbs to .sync_* #5904

This is to avoid issues on OSX, where the ._ prefix has special meaning.

Originally (before 2.3.2) ._ was necessary to guarantee exclusion. But
since then the .sync_ prefix is excluded as well.

This does not affect existing database files.

Vfs: Have a static list of potential plugins for now

Fixes in-source builds and other cases where more non-plugin directories
are created in src/libsync/vfs.

Vfs: Ensure older versions gracefully ignore winvfs folders

Previously there'd likely be a mess if a 2.6 winvfs folder was attempted
to be used with a 2.5 client. Now the older clients will ignore these
folders.

Vfs plugins: Available plugins must load

A plugin that can't be loaded due to dependency issues should not be
considered as available.

SyncEngine: Ensure that the paths passed to the discovery ends with slashes

This was making the tx.pl test fail

Propagator: Don't abort sync on error 503

Only do it when it is actually a maintenance mode

Issues #5088, #5859, https://github.com/owncloud/enterprise/issues/2637

Allow to open log window via command line to already running client

Issue: #4098

Fix installation of vfs suffix plugin

VFS: Plugin needs to be installed, otherwise it can't be loaded

Fix compilation warning

src/libsync/propagatorjobs.cpp:63:10: warning: lambda capture 'this' is not used [-Wunused-lambda-capture]
        [this, &deleted](const QString &path, bool isDir) {
         ^~~~~

vfs: Make Vfs ctor explicit

vfs: Separate vfs availability from new-files-virtual

This helps support 2.5 settings where there are virtual files in the
tree but new files aren't created virtual.

It's also a prelude for #6815

There's currently no way of
- upgrading vfs plugins (a silent suffix->winvfs upgrade is attempted
  once only, when moving to master)
- disabling vfs capabilities outright

vfs: Don't always load plugins, check metadata

vfs: ensure backwards-compatible settings value is always set

vfs: Ensure SyncOptions::_vfs is never null

- Create a VfsOff derived class
- Make it a shared pointer shared with Folder::_vfs

vfs: Switch order of deletion and journal removal in dehydration

To be more crash-resilient.

vfs: Update outdated comment

vfs: Ensure local discovery is done on dehydration request

Folder settings: Keep "usePlaceholders" to be backwards compatible

vfs: Be more careful about Vfs instance ownership

vfs: Remove VfsSuffixPrivate

vfs: Document ignored return value.

Plugin: Drop PluginLoader

Plugin: style fixes

Checksums: Make file ownership more explicit

vfs: Improve modeFromString() signature

Generalize Result<> class, add Optional<>

To make it nicer to use outside of HTTP results.

vfs: Add API docs

Remove unused include directories

vfs: Fix dealing with missing vfs instance

Maybe there should just be a VfsOff instance to make this less error
prone.

Fix export of vfs functions

Cleanup CMake (output)

vfs: Add check for presence of win plugin

vfs: Make switching vfs on/off work again

vfs: Fix plugin decision in wizards, sanitize loading

vfs: Reenable local metadata updating of vfs files

vfs: Make some behaviors suffix-vfs specific

vfs: Allow remote renames to propagate as such

vfs: Allow folders without vfs

Fix big-folder detection when vfs is disabled

vfs: Implement dehydration tagging

vfs: Fix suffix detection and handling

winvfs: Better error reporting

- fix download and validation error reporting
- add tests

Fixes after rebase to master

- use vfs suffix in ProcessDirectoryJob
- fix include vfs.h
- fix local vio passing vfs
- fix checksum computation
- vfs mode use
- mingw lambda compile issue

winvfs: initial work

Done by ckamm and dschmidt

FolderMan: Remove assumption of unique running sync

Checksums: Work on QIODevice*s

Needed for cfapi where we want to feed data through a custom device
which retrieves data from the windows api.

IgnoreListEditor: increase a bit the size

Should Help for issue #6641

Account: Fix crash when deleting an account

We should not have request parented to the account, otherwise we might get
a loop on the deletion order.

Issue #6893

LockWatcher: Add chrono header

cjhash: Fix build fix

See 17a4055e8d658f24b1fc47c44fd2daf66c7d5dbf

cjhash: use Q_FALLTHROUGH

This fixes the warning
   warning: unknown option after ‘#pragma GCC diagnostic’ kind [-Wpragmas]

Issue #6872

Excludes: switch all the path handling to QString

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

CMakeLists.txt: increment sqlite version

3.9 is required for custom index to work

csync_vio_locale: use QString for the path in opendir

So we don't have to convert to utf8 and back again

Remove csync_misc: The code no longer use fnmatch

Exclude: do everything with QString wiuthout converting to char*

ProcessDirectoryJob::process: optimize so there is only one map

PropagateUpload:  Avoid many allocations by using QByteArrayLiteral

Optimisation: Add a cache SyncJournalDb::mapChecksumType

No need to do two sql query for something that's always the same and
there are very few checksum types

Sync: optimize by removing setFileRecordMetadata

Inh most case we already have a record from before, so avoid doing a useless
lookup in the database.
In owncloudpropagator.cpp, directories do not have a checksum so no need
to call a function that preserves it

Test System: Optimisations so the benchmark is significant

OwnSql: Speedup by avoiding allocating an error string on success

owncloudcmd: fix compiliation with TOKEN_AUTH_ONLY

libsync: Fix build error with TOKEN_AUTH_ONLY

When enabling TOKEN_AUTH_ONLY, the code path using QPainter is disabled.
So we also don't need the includes.
This header is not available for Remarkable.

Disable stack protector features when cross compiling

When cross-compiling this for remarkable using the poky toolchain
this results in linker errors with stack protector: libssp_nonshared.a
not found