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

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