ChunkingNG: Use the 'If' header

As discussed in https://github.com/owncloud/core/pull/26368

Merge remote-tracking branch 'origin/master' into chunking-ng

Progress: Reset lastCompletedItem #5256

Otherwise progress listeners think it's still the last-completed
item when the next sync starts. This lead to spurious entries in
the "Recent Changes" list.

Force a remote discovery when upgrading to this version #5242

A remote discovery will fix up any lingering problems caused by #5190.

Remove stale commented code

These functions don't even exist anymore.

SyncEngine: Permission error in subfolder of a folder that cannnot be added is a SoftError

Issue #5059

[tx-robot] updated from transifex

OS X: Use monochrome tray icon by default (#5229)

Propagator: Also upload more in parallel (#5230)

Fix for https://github.com/owncloud/client/issues/4986#issuecomment-227071801

This is before we have bundling at some point.

Add a note that the source repositories need to be enabled. (#5239)

* Add a note that the source repositories need to be enabled.

* Fix typo, is => are.

QProgressIndicator: update copy from upstream

Update from commit 05ce8a23cdc12e825532dc6de06c267fb8d48b4f from
https://github.com/dragotin/QProgressIndicator
Which itself is forked from commit e5ba0fd09bfd43b067ee3646d70b294c7efcb558 from
upstream, with additional license header.

It was relicensed to MIT according to
https://github.com/mojocorp/QProgressIndicator/commit/14bb9d10e29407175fdf4b9f03ed9cc7bb124f93

Relates to issues #5180 and #5184

Fixup previous commit

Thanks @ogoffart for spotting.

Auto Start: Only set on first account setup, not all later ones

sqlite: Update bundled version to 3.14.2

For OS X and Windows.

SyncEngine: Fix renaming a folder should keep the selective sync state

Issue #5224

Two problems:

 - In the discovery phase, we need to check the selective sync entries of
   the source path in case of renames.

 - When the rename is done, we need to actually update the black list in the
   database.

TestFolderMan: don't pollute user's settings

AccountManager: do not load invalid account

Some tests (such as FolderManTest) can polute the config file with invalid
accounts.

(That's because most of the code, (even in libsync) always instentiate a ConfigFile)

Remove usused variable

SyncJournalDB: check the return value of the prepare statements in checkConnect

Relates to #5199

Theme: Fix compiler warning

Relates #5227

AccoutWizard: Change the set policy of a radio button

In an attempt to make sure the button next to it don't get too small.

Issue #5209

OwnSQL: avoid crashing in SqlQuery::reset_and_clear_bindings

The crash reporter shows a lot of crashes in sqlite3_clear_bindings
which seems to indicate that _stmt is null. We should guard against
a null value in order to avoid crashing.

This should only happen if the prepare call fails. We don't usually
check the return value of the prepare call, but if _stmt is null, the
exec call should return false, not true. We check the result of the
exec call, so this should then abort the sync with an error, rather
than crashing.

Tray: Fix crash on startup for non-OSX oses

Doc: Remove mention of "F5 to reset database" #5130

Improve macOS SysTray/StatusBar monochrome icon (#5227)

- Use a white icon if the context menu is visible.
- Enable `QIcon::setIsMask` if compiled on Qt >= 5.6 to allow automatic
  macOS color handling.
- No changes if the colored icons are used.

503: Also parse other server error message (#5228)

From https://github.com/owncloud/core/pull/26145

[tx-robot] updated from transifex

Package: Fix nautilus plugin name collision when branded

As suggested by @olifre in #5183

ShareDialog: Remove context help button

Ticket #5203

Merge remote-tracking branch 'origin/2.2'

[tx-robot] updated from transifex

VERSION.cmake: We're at 2.2.5

2.2.4: Set release date

Wizard: Do the system proxy lookup in a thread (#5160)

This is analogous to the code in ConnectionValidator.

Improve the "new big folder" UI #5202 (#5204)

Instead of using the regular selective-sync UI (where it's unclear what
the "Cancel" button would even mean in this context), provide a
different set of buttons that allow the user to quickly synchronize
all pending big folders, none of them, or perform manual changes
as usual.

Tray: Workaround collection (#5179)

Tray: Workaround collection

* QDBus workaround for Qt 5.5.0 only, there were reports of the tray
  working fine with 5.5.1. #5164
* OWNCLOUD_FORCE_QDBUS_TRAY_WORKAROUND to force the workaround on an off
* OWNCLOUD_TRAY_UPDATE_WHILE_VISIBLE to enable or disable updating of
  the menu while it's visible - disable by default due to problems on OSX and Xubuntu.
* Track the visibility of the tray menu with aboutToShow/aboutToHide
  only on OSX - the aboutToHide signal doesn't trigger reliably on linux
* Refactor such that setupContextMenu is different from updateContextMenu
* Don't use on-demand updating of the tray menu when the qdbus workaround
  is active, instead to occasional (30s) updates of the tray menu.

Missing Inc -> GmbH  (#5181)

Warning if mailto url fails #5142

Fix Windows build with WITH_UNIT_TESTING

owncloudcmd: fix --silent

We ignored csync log, but we also need to silent Qt debug output.
We need to ignore it at the very begining because there might be
qDebug also in account creation.

Issue #5196

Add log messages for LockWatcher use #5190

SyncEngine: Fix renaming of folder when file are changed (#5195)

Two bugs:
 - The change filed are not considered as move, they are re-downloaded
   but the old file was not removed from the database. The change in
   owncloudpropagator.cpp takes care of removing the old entries.

 - Next sync would then remove the file in the server in the old folder
   This was not a problem until we start reusing the sync engine, and
   that the _renamedFolders map is not cleared. We were before deleting
   a non-existing file. But now we delete the actual file.

Also improve the tests to be able to do move on the server.
This include support for file id.

Issue #5192

[tx-robot] updated from transifex

Merge pull request #5170 from owncloud/add_capabilities_owncloudcmd

add server capabilities to owncloudcmd

Propagator: properly send the success flag (#5189)

In case of the root directory, it may happen that the _item
is empty and the _item->_status is NoStatus.  But we still need to report
the proper success or error of the whole propagation. We should really
use _hasError for that. However, _hasError is also defined to NoStatus
if there was no error, so in that case we need to set Success.

This fixes the problem in which the data-fingerprint is not saved on the
database because the SyncEngine think that the sync failed. (Issue #5185)

ConnectionValidator: properly handle error in status.php (#5188)

We wwer enot connecting to the right signal from the check server
job, and therefore we were not catching the condition in which the
json was invalid. We would then never terminate the ConnectionValidator job.

Note that instanceNotFound is also emited if there is a network error.

The log looked like this:

10:25:51.247 OCC::CheckServerJob::finished: status.php from server is not valid JSON!
10:25:51.248 OCC::CheckServerJob::finished: status.php returns:  QMap()   QNetworkReply::NetworkError(NoError)  Reply:  QNetworkReplyHttpImpl(0x2b6a790)
10:25:51.248 OCC::CheckServerJob::finished: No proper answer on  QUrl("http://localhost/~owncloud/status.php")
10:26:23.235 OCC::AccountState::checkConnectivity: ConnectionValidator already running, ignoring "owncloud@localhost"
10:26:55.235 OCC::AccountState::checkConnectivity: ConnectionValidator already running, ignoring "owncloud@localhost"
[...]

Merge pull request #5182 from jturcotte/msvcBuild

Fix the MSVC build

Fix unit testing build warning on Linux

The compiler complained about the PathComponent reference on the lambdas comes from the anonymous namespace.

Fix the MSVC build

- Replace functions that are provided by MinGW with a Win32-based
  implementation
- Explicitly export needed symbols from ocsync.dll
- Rename share.h to sharemanager.h since the name clashes with one
  of the Windows headers and get included from there
- Remove the timestamp from the fallback csync stderr logging, it's
  not used since we always provide a log callback

Chunking-NG: add some headers when uploading chunks

These are not understood by owncloud yet, but were requested for CernBox

OC-Total-Length in the MKCOL: The full lenght of the file
OC-Chunk-Offset in the PUT: The offset within the file in which this chunk belongs
OC-Checksum in the MOVE: The transission checksum

Chunking-NG: The MOVE will return the code 204 if the file was already there

Chunking-NG: use OC-If-Destination-Match instread of If-Match

For the MOVE command, because If-Match in Webdav relates to the source, not
the destination

Chunking-NG: Fix destination URL

ConnectionValidator: fetch the account name.

This is needed for the new webdav path used by the new chunking.
The user might not be the same as the one used to connect

Folder/Watcher: Fix up #5174 #5153 (#5186)

It was surprising to have a broken symlink return empty for
canonical path.

Improvements to file recall

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

ChangeLog: 2.2.4

Dolphin Plugin: Use the Application name for the socket path (#5172)

do not hardcode.

This should fix #5165
(cherry picked from commit 2d110540ee5a67d2c014fcf4258840d3aaf538ac)

SyncEngine: Fix renaming of folder when file are changed (#5195)

Two bugs:
 - The change filed are not considered as move, they are re-downloaded
   but the old file was not removed from the database. The change in
   owncloudpropagator.cpp takes care of removing the old entries.

 - Next sync would then remove the file in the server in the old folder
   This was not a problem until we start reusing the sync engine, and
   that the _renamedFolders map is not cleared. We were before deleting
   a non-existing file. But now we delete the actual file.

Also improve the tests to be able to do move on the server.
This include support for file id.

Issue #5192
(cherry picked from commit 85b8ab178e21cf4dcaf792f68c15e55f6e0ee3c2)

[tx-robot] updated from transifex

ConnectionValidator: properly handle error in status.php (#5188)

We wwer enot connecting to the right signal from the check server
job, and therefore we were not catching the condition in which the
json was invalid. We would then never terminate the ConnectionValidator job.

Note that instanceNotFound is also emited if there is a network error.

The log looked like this:

10:25:51.247 OCC::CheckServerJob::finished: status.php from server is not valid JSON!
10:25:51.248 OCC::CheckServerJob::finished: status.php returns:  QMap()   QNetworkReply::NetworkError(NoError)  Reply:  QNetworkReplyHttpImpl(0x2b6a790)
10:25:51.248 OCC::CheckServerJob::finished: No proper answer on  QUrl("http://localhost/~owncloud/status.php")
10:26:23.235 OCC::AccountState::checkConnectivity: ConnectionValidator already running, ignoring "owncloud@localhost"
10:26:55.235 OCC::AccountState::checkConnectivity: ConnectionValidator already running, ignoring "owncloud@localhost"
[...]
(cherry picked from commit ff701bd473242b417e8ef9b987d4593e1495020c)

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Revert "FolderWatcher: Act on relative paths #5116 (#5153)"

This is not sufficient as it is not working for the Socket API.
Next commit will fix it in another layer.

Also, not ignoring paths that are not inside the folder is wrong
as it might still happen if the name has a different casing

This reverts commit d5a481f13240b50ff3993184152d3d4145ebfde0.

(cherry picked from commit 904cd46f757bf329b719bd815394ff986be99aaf)

OS X: Try to fix tray menu not updating correctly #5158 (#5177)

I even had this issue long before 2.3 that submenus suddenly disappeared.
Now we refrain from updating while visible.

Excludes: Consider files in hidden folders excluded #5163 (#5173)

Previously, we only checked the hiddenness of the target file and
ignored the hiddenness of the containing folders. This lead to
undesired behavior when people synced their home folders and there
was a folder watcher notification for a non-hidden file in one of
the hidden folders.

I'm not fully sure why, but sometimes notifications for .foo/bar were
already ignored, but notifications for .foo/bar/car were not. This may
be because of how we set up the FolderWatchers on Linux.

The new behavior is to check all path components for hiddenness, up
until the base path (but excluding the base path, so using a hidden
folder as a sync folder will work).

Merge pull request #5174 from owncloud/fix5116_2

Better fix for #5116

Folder: Cannonicalize the path

This is important because we compare the paths from the file system watcher if it
starts with this path.
Same in the SocketAPI where we need to use cannonical paths in the REGISTER_PATH command,
as the plugin themself will do this comparison.

Issue #5116

Revert "FolderWatcher: Act on relative paths #5116 (#5153)"

This is not sufficient as it is not working for the Socket API.
Next commit will fix it in another layer.

Also, not ignoring paths that are not inside the folder is wrong
as it might still happen if the name has a different casing

This reverts commit d5a481f13240b50ff3993184152d3d4145ebfde0.

add server capabilities to owncloudcmd

Dolphin Plugin: Use the Application name for the socket path (#5172)

do not hardcode.

This should fix #5165

FolderWatcher: Act on relative paths #5116 (#5153)

(cherry picked from commit d5a481f13240b50ff3993184152d3d4145ebfde0)

Set discovery thread priority to low #5017

(cherry picked from commit bd7ec196443202501d76f12db5f243e1efb7cf57)

ExcludeFiles: Fix when the folder casing is not the same in the settings and in the FS

If the folder has different case in the settings and in the FS, we should
not ignore all the files. This is important for the files system watcher.

(cherry picked from commit 98268d102fb343c103cbb4fb28dedd390f3c9ecf)

CleanupPollsJob: Fix possible leak

Missing deleteLater when the CleanupPollsJob aborts.
This is only a problem if the SyncEngine is kept alive a long time. Which is
usually not the case in the configuration where poll jobs are used.

(cherry picked from commit 34650248985914523f0684c597f0c366b703f8d7)

Recall: Copy instead of move recalled file #5150

That was an unintentional change in
2662203fb7277d5d1a57cef556279f8cd3b974f5

Also expand the test case to cover this.

(cherry picked from commit af9c4d0e2fe1a42e1faf14779ca5ef1088bd1314)

Selective Sync: Fix request loop and show error in view (#5154)

I got into a situation where the model would endlessly request the directory
contents from the server because we did not notice yet that the server
is actually in maintenance mode while we were expanding the tree view when
changing the tab to the account or when just expanding it by clicking.
(cherry picked from commit 524220d0907713408298a36df18bd04635ce5563)

ShareLink: Ensure the password line edit is enabled #5117

(cherry picked from commit b541fea793a1249d6833474765e2b7a6574e492c)

Update owner to be ownCloud GmbH

Fixes #5079

(cherry picked from commit a83c5598e850730c277f365e38a2348b825351a9)

Switch target version of this branch to 2.2.4

FolderWatcher: Act on relative paths #5116 (#5153)

Allow to disable Shiboleth to build without QtWebkit (#5166)

ExcludeFiles: Fix when the folder casing is not the same in the settings and in the FS

If the folder has different case in the settings and in the FS, we should
not ignore all the files. This is important for the files system watcher.

FolderStatusModel:  Fix beginInsertRow/endInsertRow

We need to do the change between the begin and the end call so the selection
don't get broken

Chunking-NG: Some fixup after feedback from the pull request #5102

Selective Sync: Fix request loop and show error in view (#5154)

I got into a situation where the model would endlessly request the directory
contents from the server because we did not notice yet that the server
is actually in maintenance mode while we were expanding the tree view when
changing the tab to the account or when just expanding it by clicking.

Recall: Copy instead of move recalled file #5150

That was an unintentional change in
2662203fb7277d5d1a57cef556279f8cd3b974f5

Also expand the test case to cover this.

[tx-robot] updated from transifex

Show the rename target in the client log file. (#5149)

[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

Chunking-NG: use new dav URL for the move

Chunking-NG: Qt4 compile