Age | Commit message (Collapse) | Author |
|
Completed was emitted before the database entry was created in PropagateDirectory::slotSubJobsFinished.
|
|
Fixes: #9208, #9133
|
|
|
|
|
|
Fixes: #9016
|
|
Fixes: #9014
|
|
|
|
We now call convertToPlacholder in vfs->updateMetaData.
We now always go through the propagator to prevent code duplication.
|
|
|
|
Prevent duplicates and guarantee sorting
|
|
|
|
* Replaced obsolete foreach-loops with for-loops
* Added a copy of queries into SqlDatabase::close
* Used const reference to avoid unneeded copying
* Fixed Qt containers possible detach within for-loop
* Removed unneeded copies before for-loops
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
not when it was processed by the client, to determine the quality of the connection.
|
|
|
|
Fixes: #8170
|
|
|
|
|
|
warning C4002: too many arguments for function-like macro invocation 'ENFORCE_1'
|
|
|
|
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)
|
|
|
|
selective sync
Issue #7337
|
|
For issue #7557 and #7556
Note: this change the API of the VFS plugin, so the VFS plugin needs small
adaptations
|
|
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)
|
|
|
|
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
|
|
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
|
|
Allows winvfs to convert files to placeholders when vfs is enabled. This
is required to mark files as in-sync #7329.
|
|
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.
|
|
Possibly a regression, since the new discovery discovers rist the renamed
files as removed
Issue #7204
|
|
The previous patch ensured that the sqlite temporaries weren't deleted
and recreated for every sync run, but there was still time between
client startup and the first sync run where they would have the
"needs-sync" icon.
|
|
The db-close operation is likely a leftover from when the SyncEngine
owned its own db connection and serves no purpose anymore.
Closing the db causes the removal of the temporary wal and shm files.
These files are recreated when the db is opened again, which happens
almost immediately.
This is a problem for winvfs because the delete-recreate step wipes the
exclusion state on these files just after the sync is done. That meant
that the db temporaries permanently had a "needs sync" icon marker shown
in the explorer.
Avoiding reopening the db also reduces the number of log messages per
sync.
|
|
Previously if one set the instruction to ERROR while forgetting to set
an error status, it'd propagate as FileIgnored. Now the default is
NormalError for INSTRUCTION_ERROR and FileIgnored for
INSTRUCTION_IGNORE.
|
|
As far as I'm aware local discovery can be skipped on folders that are
selective-sync blacklisted, so a local discovery is required when an
entry is removed from the blacklist.
Also rename
avoidReadFromDbOnNextSync() -> schedulePathForRemoteDiscovery()
since the old name might also imply it's not read from db in the local
discovery - which is not the case. Use Folder::
schedulePathForLocalDiscovery() for that.
|
|
Sizes are always qint64, not unsigned.
TransferIds are always uint.
|
|
- unspecified and inherited are different
- move enum to header in common/
- access through Vfs instead of directly in Journal
|
|
Possibly the behavior should actually change and the function should
de-placeholder all items, not just dehydrated ones.
|
|
This allows SyncFileStatusTracker to also know about these. After all
its information is used to provide icons for them too.
|
|
Since 'placeholder' just means that it's an item of the special type
that the vfs plugin can deal with - no matter whether hydrated or
dehydrated - all done items should become placeholders. Even
directories.
Now every file that passes through updateMetadata() will be converted to
a placeholder if necessary.
|