[tx-robot] updated from transifex

Folder: Keep files option after aboutToRemoveAllFiles should not clear selective sync list (#5531)

We were removing the wholme journal db when the user wanted to keep all files,
But that would also remove the selective sync lists.
We should only remove the metadata table.

Issue #5484

Update windows docker file to use 2.3 OBS repo

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

[tx-robot] updated from transifex

Merge remote-tracking branch 'origin/2.3'

Disable tests uncompatible with Qt4 for that build

Fix the UNIT_TESTING build on Windows

- Put all tests in the bin directory so that DLLs can be loaded
- Add missing exports
- Skip tests that use code depending on zlib
- The "GMT" timezone is named differently, use the int constructor instead

5 tests are still failing, it's not really worth fixing at the moment
since no developper is currently using Windows as its main platform.

Fix FolderManTest on macOS

On macOS /var is a symlink to /private/var and we have to make sure that we
use the canonical path before and after it enters the code to make sure we
compare paths correctly.

Fix ExcludedFilesTest on macOS

Use the same logic to find sync-exclude as check_csync_exclude.c

Fix FileSystemTest on macOS

Fix FolderWatcherTest on macOS and Windows

- We need to use a QGuiApplication on macOS or else we don't get notifications
- Switch to use QSignalSpy rather than lists and sleeps
- Use system() for all modifications since we pass kFSEventStreamCreateFlagIgnoreSelf
- Keep using the local process on Windows since it catches its own events

Fix UtilityTest on macOS

Folder: remove unused variable

[tx-robot] updated from transifex

Modify Jenkinsfile for 2.3

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Remove unneeded Q_DECLARE_METATYPE

Fix TestSyncEngine::abortAfterFailedMkdir

It could be possible that _firstJob is marked as finished if
aborted before its parent PropagateDirectory was marked as finished,
allowing a posted scheduleNextJob call to schedule the child job
in-between.

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