even if local item has disappeared, record the result of a remote MOVE
when receiving the reply
enable better state tracking when syncing rename or move operations
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
orignally added by https://github.com/owncloud/client/pull/6632
most probably a too strong assumption on the behavior of the Nextcloud
server
better check real item metadata like fileid, size or modification time
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
NEW/NEW conflicts could sometime be ignored and replaced by update
metadata instructions
we stop doing this and handle them like any other conflicts
that would cause more download from the server
those conflicts would be solved automatically in case this is not a real
conflict but the client was missing the server reply with the updated
metadata
will enable more changes to improve MOVE detection from server side
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
should hopefully allows to understand when local file system discovery
is done or omitted
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
Turns out since we are forced to replace colons with hyphens we need to
replace them back to colons when we receive the domain identifier from
the file provider extension. If we do not then we get a null
accountState pointer in the socket controller and this causes a crash
Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
category of logs accessed via log context can be null
in such a case, strcmp will crash
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
the QWizard AeroStyle style is broken on windows dark mode, switch to
ModernStyle that is working fine
see upstream Qt bug: https://bugreports.qt.io/browse/QTBUG-123853
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
will log all deletions with the result of the discovery by the sync
enginre
should enable analyze even long time after such a delete occured
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
should decrease the cost of their usage when compared to preprocessor
conditional directives
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
seems we have an issue with Windows and QTimer instances used to detect
network timeout
workaround, find cause of https://github.com/nextcloud/desktop/issues/7184
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
to be able to modify a locked file using token lock, we must sent it
with webdav requests
this must not be sent if the file is locked but the lock type is not a
token lock
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
a file can be unlocked when:
* the lock is an user lock from the current user
* the lock is a token lock from the current desktop files client
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
will hopefuly make the qml engine search inside install dir within qml
sub folder
Close#7156
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
we import QtQuick module that also define Window component name
to avoid any ambiguity, just use a different name on our side
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
