Doc: For conflicts, the local file is renamed #5521

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Remove unneeded Q_DECLARE_METATYPE

Skip TestSyncEngine::abortAfterFailedMkdir

Merge pull request #5518 from ckamm/asserts

Improve usage of asserts

CMake: Don't force-enable asserts #5429

Should be safer now that we've gone through asserts and assured that
one ones that should be fatal continue to exist in release builds
independently of QT_NO_DEBUG.

Double check usage of asserts #5429

A few are supposed to be fatal.

Performance: reserve() _remotePerm #5496

Folder: Change the wording for the massive deletion message

Issue #5503

Settings: Don't migrate settings on access error #5499 (#5523)

Previously, we'd try migrating from legacy settings if reading
the settings failed with an error. Now, we try again after a
couple of seconds and eventually give up.

AccountManager: Fix small leak in the account migration code

The QSettings object was not deleted in every possible code paths

CentOS: More build fixes

CentOS: Another attempt

[tx-robot] updated from transifex

Improve previous fix

After comment on https://github.com/owncloud/client/commit/78a798eef3eb8d04360c6d9542db55469f59cacd

CentOS: Another fix for old compiler version (2)

ChangeLog for 2.3 additions

Old Qt: Another build fix

CentOS: Another fix for old compiler version

CentOS: Fix build for old compiler version

Revert "Disable RTTI"

This reverts commit 85d3de15898385a06b2f34479e875c885e5df6be.
This breaks our old Qt on CentOS.

We can re-introduce this with #5470

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge remote-tracking branch 'origin/master' into 2.3

We can do that because the only changes that were in master but not in 2.3 were the
translations change and documentation change, and the support for the 'M' permission
which we want in 2.3.

Account Wizard: don't whitelist everything on the first sync if we still want to confirm big folder

Issue: https://github.com/owncloud/client/pull/5340#issuecomment-274564441

General Settings: Make sure to reload the settings when an account is added

The Size limit, or confirmation checkboxes might have changed.

We need to guard against saving if the control changes while we are loading

Issue: https://github.com/owncloud/client/pull/5340#issuecomment-274878023

Confirm External Storage: adjust the notification message

We need to forward the information that the folder is an external storage
for the notification message.

Issue: https://github.com/owncloud/client/pull/5340#issuecomment-274878023

Wizard: Increase the size

The advanced page has become quite complex and does not fit on the screen
anymore if the fonts are too big

FolderStatusModel: Different icon for external storages

Issue: https://github.com/owncloud/client/pull/5340#issuecomment-274564441

Discovery: Filter 'M' out of permissions for non root storage

The sync engine rely on the 'M' in premission to ask for confirmation
(As requested in issue #5340)
But we only want to ask the premission for the 'root' of the mounting point and not
for every subfolders within it.
So we change the discovery phase in a way that it does not keep the 'M' for
children within the external storage.

Wizard: Add options to ask confirmation for external storage

Added two checkboxes in the Account Wizard in the advanced page to change the first options.
Also added a checkbox in the general settings to ask for confirmation for external storages.

Theme options allow to hide the checkboxes in the wizard.

As described in issue #5340

Discovery: Add branding option to disable default sync of 'M' directories

Issue #5331 and https://github.com/owncloud/enterprise/issues/1594

[tx-robot] updated from transifex

Remove PropagateDirectory::_runningNow

This variable isn't used anymore except in a stray Q_ASSERT.

Clear SyncEngine::_remotePerms after the sync

Also make reuse the SyncFileItem::_remotePerm QByteArray allocation
instead of redoing the copy from the csync char*.

Pack boolean members in PropagateUploadFileCommon #5457

Don't use the StopWatch for each job in production builds #5456

This accounted for more than 10% of the peak memory usage while only
providing value through the debugging log.

Remove SyncFileItem::_requestDuration #5456

This will save a QElapsedTimer in each job that was used only for
.owncloudsync.log (for which the request duration doesn't bring much
value).

Don't reallocate a QByteArray for each contentChecksumType call #5481

Remove duplicate SyncFileItem entries for the log

The accuracy of that log isn't as important as the few bytes those
fields take as hostage for the whole sync.

Don't create legacy 'folders/' subdirectory

Cookies: Use different DB for different accounts (#5490)

This is a follow up to #5469

Delete finished propagation jobs in PropagateDirectory #5269 (#5400)

[tx-robot] updated from transifex

Release SyncFileItem objects with their job

We now delete subjobs as their propagation is complete. This allows us
to also release the item by making sure that nothing else is holding a
reference to it.

Remove the stored SyncFileItemVector from SyncEngine and SyncResult
and instead gather the needed info progressively as each itemCompleted
signal is emitted.

This frees some holes on the heap as propagation goes, allowing many
memory allocations without the need of requesting more virtual memory
from the OS, preventing the memory usage from increasingly growing.

Pass the SyncFileItem as SyncFileItemPtr in itemCompleted

This will allow us to keep a reference on the items in connected slots.

Don't pass the PropagatorJob in itemCompleted

This was to catch duplicate emissions for PropagateDirectory but we
don't emit this signal anymore from there.
This fixes a warning about PropagatorJob not being a registered metatype.

This reverts commit fe42c1a818c3b1ccadb2e9557971b62a350573b4.

Tests: Add a large sync benchmark

This simulates a ~50k files sync that can be used to measure memory
usage without having to wait for a server.

Bring back the error message if the sync-exclude.lst file is missing

We unconditionnally add the file on startup, but don't check for errors.
During sync we check for errors, but only try loading the file if it
exists.

Merge remote-tracking branch 'origin/selective-sync-big-folder' into 2.3

Merge pull request #5480 from owncloud/chunking-ng-cleanup

Chunking ng cleanup

[tx-robot] updated from transifex

Add extra note about deleting .csync_journal.db

This is added as per the recommendation from @guruz in #5252.

add note about deleting .csync_journal.db for "CSync unknown error"

Shib: Use different keychain entry per account #5469

Previously shib multiaccount didn't work at all because the
session cookie was stored in the same keychain entry.

Folders: Adjust ui when account is disconnected #5477

Previously if you paused/unpaused a folder for a disconnected account
they would prepare to sync and thus display the 'Waiting...' text. With
this change, folders that can't possibly sync don't show text like
this.

When account connectivity changes, all unpaused folders will be
scheduled anyway.

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

ChunkingNG: add '0' padding on the filename (#5476)

The server sorts the chunk by name alphabetically. So if we want to keep
the chunk in order, we need to add a few '0' in front of the chunk name

Update issue_template.md

ChunkingNG: Add some tests

 - Test that we recover correctly if the chunks were removed on the server.
 - Test changing the file localy while uploading.

ChunkingNG: remove stale chunks when cleaning the uploadInfo table

Stale chunks might be there because a file was removed or would just not
be uploaded, for any reason.
We just start the DeleteJob but we don't care if it success or not.

Relates to https://github.com/owncloud/core/issues/26981

One of the test is testing the case where the file is modified on the server
during the upload. So this test the precondition failed error.
The FakeGetReply logic was modified because resizing a 150MB big QByteArray
by increment of 16k just did not scale when downloading a big file.

ChunkingNG: delete stale chunks if the file was changed locally

Relates to https://github.com/owncloud/core/issues/26981

We do not track the success or error of the DeleteJob because it does not
matter. If it fails, it might be because the chunks were already removed.
If not, the chunks will be stale, but the server must anyway do a few
cleanup from time to time because we do not always remove the chunks

[tx-robot] updated from transifex

[tx-robot] updated from transifex

StatusTracker: Emit OK for the last child before parents (#5467)

The current logic tried to avoid a DB lookup just to fetch whether
the file is shared or not since that info is already in the
SyncFileItem. The implementation would however need to decrease the
sync count for itself (and parents) before emitting the new status,
thus emitting the OK status for parents before that last child that
ended the propagation for that folder.

Change the implementation to achieve what we want: give the
possibility to decSyncCount to use a pre-fetched sharing state while
still doing the emission for all involved files. This ensures that
the leaf file also gets its status emitted before its parents.

Issue #4797

Reference to build Qt5 with qtwebkit support in the docs #5129 (#5471)

This addresses the issue only on OS X. It should be stated though, that qtwebkit is not part of the qt5core lib and therefore a requirement for the projectto build in the rest of the OSS

shell/Windows: Update binaries to build 44

Update the MacOS automatic update instructions

As requested in #5431, the automatic update instructions for MacOS X
have been simplified; removing the older, incorrect instructions.

Fix several typos in the client documentation

shell/Windows: Avoid memory allocations and copies when querying icons

IsMemberOf is called for every file (in the ownCloud directory or not) and
with every instance of OCOverlay (we have 5) when displaying a list of
files in Explorer.

Refactor the code to avoid copying the list of watched directories, as
well as creating a wstring from a PWWSTR for files outside watched
directories.

Also change some calls of begin_with to use isDescendantOf since it
properly handles parent paths not ending with a backslash, which could
lead to SocketAPI queries for sibling folders with a name that starts with
a watched folder name.

shell/Windows: Fix the view not being updated on StateError

Since StateError == 0, if this was the status used when the path
isn't in the map already, the view would not be updated since the
new state would be the same as the default-constructed state in the
map. Fix by explicitly inserting in that case, this also avoid aving
to do two lookups in the map when a path already has an entry.

shell/Windows: Remove the UPDATE_VIEW message handling

The client now tries to only push STATUS messages to connections
unless they previously requested it with a RETRIEVE_FILE_STATUS,
but this means that we now have to make sure that a new connection
will rerequest every icon that the user will see on files from that
point on.

To that end, we now send a SHChangeNotify for every file in the cache
when we lose the connection (or on UNREGISTER_PATH) to make sure that
what's on the screen matches what's in the cache. We also remove the
_oldCache logic that made this more difficult to enforce.

The client has been able to reliably push status updates on macOS
for a few versions now, and we don't need it on Windows either. The
_oldCache mechanism was to avoid sending to many update requests
when receiving an UPDATE_VIEW.

Also fixes #4766

shell/Windows: Convert tabs to space

It was a mix of both and the rest of the code uses spaces.

shell/Windows: Remove unused files

The real OCContextMenu.cpp is in the OCContextMenu folder.

shell/macOS: Make sure the root folder's cache is cleared on disconnect

The SocketAPI now only sends status pushes for paths requested by the
shell. We have to make sure that Finder doesn't just show the entry
from its cache after a reconnect.

shell/macOS: Remove backslashes from the extension display name

Tell cmake not to escape shell strings adding backslashes before
spaces by passing the VERBATIM optiton. Also add the quotes to
whole command line arguments to prevent quotes from appearing on the
XCode side, because of the VERBATIM option.

This only affects themed clients since the default theme doesn't
have space in its APPLICATION_NAME.

SocketAPI: Only push status for files requested by the shell #5361

We currently push the SYNC status for all files that will be propagated,
and then the OK status when those files are propagated.
On top of this, we send those statuses to all clients connected, even
if the socket is kept open by an application that only needed to show
a file open dialog. On macOS we're also using an NSConnection which
means that we have to wait for the RPC call to return from the
extension, which makes bulk status changes possibly heavy.

Reduce the time spent needlessly sending status pushes by limiting
them to files requested through that socket since it connected.
To limit the data to store, only remember the parent directory of
files requested, and store those in a bloom filter.

Note that this adds a requirement to shell extensions: they should
make sure that the status cache only contains entries that have been
requested through the socket API. In other words, the status cache
must be empty when each socket client connects to the socket API.
Otherwise the cached icon type will be shown to the user, and the
SocketAPI won't push new status for that file if it didn't receive
a RETRIEVE_FILE_STATUS.

SocketApi::slotReadSocket: Small optimizations

- Use the looked-up method index also for the invocation
- Do the method name concatenation already on QByteArray since we'll
  convert anyway
- Use staticMetaObject instead of metaObject()

SocketAPI: Remove unecessary null-checks

Those methods are private and there is only one call-site, we can
be sure that this parameter won't be null already.

Sync Engine: Specify what chars are not syncable in the error message #1733 (#5449)

Fix more compilation errors with Qt 4

Ammand last commit which was a fixup to the change from pull request #5468
for issue #5453

Fix compilation with Qt 4

Fixup to the change from pull request #5468 for issue #5453

[tx-robot] updated from transifex

NSIS: Adjust to 5.6.2 image

Folder: Remove unused variable

Disable RTTI

Shrinks owncloud binary by 24 KB and libowncloudsync by 14 KB.
I don't know if it has influence on memory usage or runtime speed though.

Was worth a try.