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

Add CMake option to disable GUI build

Get DatabaseErrorTest to pass

The E2EE code path would get the engine to go wrong in case of db error.
It's just better to have a failing upload or failing mkdir later in those
cases.

Emitting signals from a ctor is a bad idea anyway

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

Test that the sync behave well if there are errors while reading the database

Discovery: fix double emission of finished in case of error

Discovery: Fix downloading files when database is used for local discovery

This also fix the currently failling LockedFilesTest

Remove csync.cpp: It's only dead code

Add Ctrl-L as log window shortcut

F12 is taken on OSX and there's no other way of showing it.

Test: Add test for locked file tracking and propagation

Folder: Treat file unlock similar to external change #6822

For consistent handling of incoming notifications.

Propagator: Add assert against duplicate done() calls

RemoteDiscoveryTest: Fix after the merge from 2.5

The new discovery's message is slightly different

Test: Check for folder in error message #6826

gitignore: XCode userdata

Discovery: Fix renaming on windows

buf.type is ItemFileSkip because csync_vio_local_stat does not set this field

New Discovery: Fix trailing slash causing failure on windows

Folder: update the folder version

Since the new index would crash old version of the client, we need to
upgrade folder version so they do not load in the client 2.5.0

Settings: Make FoldersWithPlaceholders group sticky

If virtual files are disabled on a folder it might still have db entries
or local virtual files that would confuse older client versions.

owncloudcmd: Read the server version from the capabilities

Issue: #6846

owncloudcmd: fetch the dav user

This is required to get the new endpoint working when the server
uses ldap or that the dav user is not the same as the login.

Issue #6830