csync: apply strict QString handling

VFS: remove now unused parameter

Use the servers checksum type by default

Log state before specific assertion

Win: Use full Windows paths in file watcher and improve logging

Win: Move hresultToQString from vfs plugin to Utility::formatWinError

Remove more legacy C code

Tests: Port check_vio_ext to Windows

Remove custome string functions

Logging: Print enum before cast in SqlQuer::bindValue

Fix warning

warning C4573: the usage of 'QObject::disconnect' requires the compiler to capture 'this' but the current default capture mode does not allow it

Log: Only print 'Saved account settings' in debug mode

Don't log that there was no error...

Log: Improve log message [ info gui.folder ]: Folder sync result:  4...

by including the name of the folder and print the name of the result

Gui: Don't try to display the wizard during shutdown

Fixes: #7936

Remove dead code

Make ownCloud accassible on the Application object

Remove ifdef(Q_OS_X11) code, there never was such a define.

As noone complained since I don't think anyone will miss the code

A pedantic .desktop parser want's a list to end with ;

VFS: Tell the vfs plugin whether we have multiple accounts

This allows us to decide on the presentation of the account

UnitTests: Set a CookiesJar in FakeQNAM

CMake: Remove krazy2 support, we now use clazy

VFS: Prevent duplicated navigation panel icon

Issue: #7748

Log HTTP requests and responses

Issue: #7873

Beautify flags

Fix 3aeca58b316363dc12ad4b392750fc57e7a2965a

Fix  warning C4715: 'OCC::Utility::vfsCurrentAvailabilityText': not all control paths return a value

Fix 150a5b4d06600d140ddf05383fb74cda6ae6e941

Fix crash on settings migration

Fixes: #7878

Fix usage of QMessageBox

Fixes: #7874

Fix order of pluginkit commands and modernise

Use separate loggin category for upload v1 and ng

Wizard: Raise own window

The OAuth authentication brings the broweser to the front, once thats done the wizard continues.
But the wizard ist now most probably hidden behind the browser

VFS Dialog: Don't enable on close button clicked

Issue: #7710

Remove unused struct

Remvoe now unneeded SyncRunFileLog::directionToStr

Remvoe now unneeded SyncRunFileLog::instructionToStr

Cleanup moc_csync.cpp include

Rename csync_instructions_e -> SyncInsturctions

Remove now unneede csync_instruction_str

Make bit flags better readable

Use Q_ENUM_NS for PinState

Test: Disable parallelism to ensure serial execution

Cookies: Do set cookies in DetermineAuthTypeJob too

As we don't support cookie based authentication anymore we can provide cookies here.
This fixes issues with loadbalancers access policy managers.

Cookies: Don't override cookies with outdated values

This code was actually not breaking most cookie handling by accident.
As the raw cookies where not split properly we added cookies with values like
"key: val; key2 = val2; key3 = val3"
When the code was corrected we overwrote the newer values in the jar with
the old ones from a request.

SocketAPI: fix status after a failure to move

Issue #7759

Test: Fix testMovedWithError for vfs mode

Sync: Fix handling of virtual files in error state

Issue: #7799

Test: Add test for vfs failed move crash

[Gui] Implement raiseDialog on Windows

Issue: #7774

[SocketApi] Ensure listener still exists

Improve logging of issues during plugin loading

If the plugin could not be loaded the client calls qFatal
Make the loading warnings critical so they get printed before we crash

Don't insert items into the folder model if selectiveSync is not supported

This fixes an assertion in FolderStatusModel::SubFolderInfo::resetSubs
rowCount reported 0 but we actually had items in the model

Make sure all cases are handled

[Gui] Fix vfs dialog not showing on Windows

Remove unused defines

Use Q_ENUM_NS to pretty print csync enums

Fix potential null pointer access

https://sentry.io/organizations/owncloud/issues/1529161263/events/02509984b5ca42ffb3960d9c9e161414/?project=79001&statsPeriod=14d

Folder Wizard: warn when adding a folder which is the parent of a sync'ed folder

issue #7741

Fix expansion of tree view on newly added accounts

The change is based on 97ce20ac028660e6ae3dd0b98d4b487999d8768a

I removed a few lines of code there which are already part of fetchMore()

Fixes: #7336

[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