[Wizard] Make vfs dialog blocking

Calling the callback after the receiver was deleted caused a crash
Fixes: #7709
Fixes: #7711

fixup test

Tests: add a couple of move tests

This was an attempt to reproduce #7722, but this actually does not
reproduce it

[SSL] Print warning only if something changed

[SSL] Properly restore user accepted certificats

[Sanity] Remove old ifdef

Fix saving of cookies

Fixes: #7700

Don't hardcode PLUGINDIR

Make PluginFactory virtual to silent -Wnon-virtual-dtor warning

This is not necessary, but it also shouldn't hurt.

SyncEngine: no need to use QAtomicInt

This was done because the propagator jobs where running in a thread a long
time ago, but this is no longer the case.

(Also QAtomicInt::load is marked as deprecated now)

ProxyAuthHandler Use QPointer instead of QWeakPointer

QWeakPointer::data is deprecated.
In this case we should use QPointer

Remove old test scripts

Fixes: 7679

[Core] Include more information about the OS in the user agent

Don't ignore file sync notification after an unlock

For a usual file sync event we check for actual changes in the local file,
after an unlock the local file might be unchanged so we need to sync it anyhow.

Fixes: owncloud/enterprise#3609

Prepend PLUGINDIR else its pointless

Use Q_ENUM on AuthType to get the name printed in the log

Add Actions to the Desktop file

Application: Allow to quit running instances by commandline

Cleanup

Disable http2 support for now

Issue: #7610

Don't show the "All files deleted" popup when unselecting everything with selective sync

Issue #7337

AccountSettings: limit the clickable region of the 'add folder' button

Issue #7326

Network Settings: Show a warning that proxy settings do not apply to localhost

Only show this if at least one account is detected to have an url that looks
like localhost, because this could otherwise be confusing

Issue #7169

Discovery: Allow more HTTP error code to be treated as ignored dir

The original code from csync was stopping at any error.
But we have been whitelisting soeme http error code one by one
to ignore the directory instead of aborting the sync.
However, as there are more requests to continue the sync in case
of error, just ignore most HTTP errors

Issue #7586

[libsync] Export verifyFileUnchanged

This fixes Windows builds that are broken since #7562 was merged

VFS: Do not overwrite existing files by placeholder

For issue #7557 and #7556

Note: this change the API of the VFS plugin, so the VFS plugin needs small
adaptations

[Gui] Set proper hdpi attributes

Discovery: fix TestSyncVirtualFiles::testExtraFilesLocalDehydrated on windows

On windows we do a test to know if we should change the case of the files,
but that conflict with the test that checks if the file was still there
when the filename is actually the same. Which can happen with virtual files
as they have two representation (the one with and without suffix).

Test: Fix testsyncvirtualfiles test

Like previous commit, the failure was caused by two conflicting commits

Fix build of tests.

Resulted from a conflict between two patches

Vfs: Lots of tests and corrections for suffix edge cases

Avoid or deal with problems that happen when suffixed files exist on the
server or suffix and non-suffixed files exist locally.

See #7350, #7261.

Discovery: Attempt to fix issue with windows VFS and new files

(or moved files)
As seen in the log of #7558, a conflict may be issued by mistake.

See investigation in https://github.com/owncloud/client/issues/7558#issuecomment-547385362

This hopefully fix #7558

Account Settings: change the color of info message from green to blue

To avoid confusion with the color of "success"

Issue #7403

Wizard: fix compilation warnings about unused variables

SyncEngine: Save a bit of memory by not keeping a set of all filename

This is only used for conflict files, so only save conflict files.
(The _seenFile was used for other things in 2.5, but not anymore)

When moving is allowed but deleting is not, do not restore moved items

Issue #7293

Restoration items should appear in the sync protocol

When an item is downloaded because it is restored, it shall be shown in the
sync protocol.
(It is also going to be shown in the not synchronized for a short while, but
that's fine)

Fix Upload of large (> 2GiB) files

Issue #7506

This is a regression introduced by the delta sync feature (as the chunk offset
changed from being the chunk number to be the byte offset, it needs to be a
qint64 now)

Tests: Fail if the initial sync fails

Discovery: Distinguish readdir and closedir errors

Tests: introduce ItemCompletedSpy to avoid a bit of code duplication

Fix warning about serverJob not being used

And fix a FIXME in the same time

Propagate dir: Never write the etag on remote mkdir #7481

It must always only be written once all children are successfully
propagated.

Ensure the url is complete before we validate it

Fixes: #6722

Checksums: Explicitly close file before reporting result

To ensure it's no longer open when the finished signal fires.

Test: Disable local discovery parallelism in permission test

Adding parallelism broke the test because it depended on the order of
discovery.

Discovery: Change local job signal signatures

To make it more explicit that data a copy of the data is transfered
between threads. (though Qt will make a copy of the arguments anyway)

Discovery: local job shouldn't be parented

Since it'll be deleted by the thread pool.

Don't store pointer to local job

There were crashes in the QPointer assignment. Possibly the thread pool
is done with the job and deletes it before the assignment is done.

Discovery: List local directories from thread #7456 #7439

Reconcile: Sort already during discovery #7445

ConnectionValidator: increase timeout to 57s #7456

When the gui thread blocks for several seconds it's possible for the
ConnectionValidator to timeout and decide that the account is
unreachable. It will then terminate all sync runs.

Increasing the timeout makes this less likely to happen. The tradeoff is
that real disconnects will not be detected as quickly.

This does not address the root cause but makes the symptom less likely
to appear.

Propagator: Delay job execution a bit #7439

Discovery: Remove level of indent around moves

No code changes.

Discovery: If a move is forbidden, restore the source

Previously the source was deleted (or attempted to be deleted), even if
the new location was not acceptable for upload. This could make data
unavilable on the server.

For #7410

Propagation: Fix delete-before-rename bug #7441

By introducing a PropagateRootDirectory job that explicitly
separates the directory deletion jobs from all the other jobs.

Note that this means that if there are errors in subJobs the
dirDeletionJobs won't get executed.

Propagator: Make sure we schedule only one job #7439

To not starve the event loop.
(There is still ~= 3 jobs running at the same time)

PropagateDownload: Don't try to open readonly temporaries

This situation could arrise when receiving a read-only share and the
temporary rename failed for some reason.

See #7419

Change Windows virtual files description to tech preview

Don't fatal on "Storage temporarily unavailable"

This is an unreliable workaround. The real fix will need to be deferred
to another release.

For #5088

OwncloudWizard: Mark vfs as tech preview instead of experimental

CMake VFS: Enable us to provided vfs plugins from an external directory

SocketApi: Avoid unneccessary conversions

SocketApi: cleanup debug output

Add ASYNC_ASSERT_ICON_IS_EQUAL command to SocketApi

Make findWidget more powerful

Add Q_PROPERTYs for gui testing

Switch GUI Testing SocketApi to Utf8

Fix build with recent Clang (on Linux)

Add GUI testing SocketApi extension

Don't use mutable lambdas as they hide where we modify work on a copy

AccountSettings: Use switch, case to ensure we handle all cases

Checksums: Add back QIODevice api

Because the winvfs plugin needs it. But be more careful about the
device's lifetime this time.

Remove maxLogLines config option

It's no longer used.

For owncloud/docs#1365

Download: Don't trigger too many concurrent hash computations

Previously the job would only become "active" when the downloads
started. That meant that arbitrarily many hash computations could be
queued at the same time.

Since the the file was opened during future creation this could lead to
a "too many open files" problem if there were lots of new-new conflicts.

To change this:
- Make PropagateDownload become active when computing a hash
  asynchronously.
- Make the computation future open the file only once it gets run. This
  will make it less likely for this problem to occur even if thousands
  of these futures are queued.

For #7372

Fix 'unused parameter name' warnings

Checksums: Fix crash due to threading issue

The checksum computation thread was potentially using a QFile that was
deleted in the gui thread.

For #7368

AccountSettings: Fetch subitems after wiping them

That helps avoid empty lists after account creation #7336

SyncEngine: Don't duplicate fatal errors

Previously fatal error texts were duplicated: Once they entered the
SyncResult via the SyncFileItem and once via syncError().

syncError is intended for folder-wide sync issues that are not pinned
to particular files. Thus that duplicated path is removed.

For #5088

Discovery: Parse etag to be consistent with RequestEtagJob

This avoids unnecessary sync runs.

For #7345

Vfs: Require local discovery after disabling vfs

Without it local files aren't guaranteed to be downloaded #6936.

Vfs: Remove old db record when dehydrating via rename

For #7338

Vfs: Make move detection work with virtual files #7001

Previously a checksum computation could be done on a suffix-placeholder
file, making discovery believe that no move took place.

Vfs: Ensure pins change with (de-)hydration

Previously an implicit hydration of a file in an online-only folder
would not change the pin state and cause a dehydration on the next
sync.

Vfs: Add hook to allow update-metadata for unchanged files

Allows winvfs to convert files to placeholders when vfs is enabled. This
is required to mark files as in-sync #7329.

Vfs: Improve strings for availability states

Vfs: Preserve pin state on hydration

For #7322 and #7323

Vfs: Retain existing data when enabling vfs #7302

Previously all local data was deleted because the root folder was marked
as OnlineOnly.

Add missing OWNCLOUDSYNC_EXPORT

Vfs: Make test pass with different suffix #7279

VfsSuffix: Wipe stale pin states #7273

Previously the pin states of deleted files stayed in the 'flags'
database and could be inadvertently reused when a new file with the same
name appeared. Now they are deleted.

To make this work right, the meaning of the 'path' column in the 'flags'
table was changed: Previously it never had the .owncloud file suffix.
Now it's the same as in metadata.path.

This takes the safe parts from #7274 for inclusion in 2.6. The more
elaborate database schema changes (why use 'path' the join the two
tables in the first place?) shall go into master.

RequestEtagJob: Consistently parse etags #7271

Previously RequestEtagJob did return the etag verbatim (including extra
quotes) while the db had the parsed form. That caused the etag
comparison during discovery move detection to always fail. The test
didn't catch it because the etags there didn't have quotes.

Now:
- RequestEtagJob will parse the etag, leading to a consistent format
- Tests have etags with quotes, detecting the problem

UploadDevice: Fix windows issues #7264

- Close the UploadDevice to close the QFile after the PUT job is done.
  This allows winvfs to get an oplock on the file later.

- Don't rely on QFile::fileName() to be valid after
  openAndSeekFileSharedRead() was called. The way it is openend on
  Windows makes it have an empty filename.

Vfs: Move pin state if files move #7250

Previously renames of items didn't carry the pin state with them.

Vfs: Don't let new local files start out unpinned #7250

If one adds a new file to an online-only folder the previous behavior
was to upload the file in one sync and dehydrate it in the next. Now
these new files get set to Unspecified pin state, making them retain
their data.

OAuth2: Better error logging

This does not fix a bug, just was found while spotting a bug that was no bug.
For https://github.com/owncloud/enterprise/issues/2951

Upload: Read file chunks gradually #7226

Instead of all at once, to reduce peak memory use.

Changing UploadDevice in this way requires keeping the file open for the
duration of the upload. It also means changes to open(), seek(), close()
to ensure that uses of the device work right when a request needs to
be resent.

Http2: Resend requests on ContentReSend error #7174

Since Qt does not yet transparently resend HTTP2 requests in some cases
we do it manually.

The test showed a problem where the initial non-200 reply would close
the target temporary file and the follow-up request couldn't store any
data. Removing that close() call is safe because there also is a
_saveBodyToFile flag that guards writes to the target file.

Discovery: Do not abort the sync in case of error 404 (or 500)

Issue: #7199

SyncEngine: Fix renaming a single file cause the "delete all file" popup

Possibly a regression, since the new discovery discovers rist the renamed
files as removed

Issue #7204