[shell/windows] Fix the windows status push not working (#4784)

Since the windows implementation first does cache lookups using the
path string, directories need to be passed identically as through
RETRIEVE_FILE_STATUS.

Change the convention to never have a trailing slash for directories
in the protocol. This allows the convention to be applied without
having to access the disk (since we'd need to know if the path is
represented by a directory) and also matches the convention of the
rest of the sync engine. Individual file manager plugins are then
responsible of handling pushed paths as not ending with a trailing
slash.

This also:
- Moves the trailing slash removal logic from the SyncFileStatusTracker
  to the SocketApi class
- Remove the unneeded QString::normalized call in fileStatus, since
  this should already be done by the FolderWatcher and plugins

[tx-robot] updated from transifex

Dolphin plugin: fix build with branding

Make sure that the name of the _export.h file is the same no matter the plugin

[tx-robot] updated from transifex

NSIS: Properly uninstall both x64 and x86 DLLs #4762

Viel hilft viel ;-)

Qt patches: Update with information about Qt 5.6

Merge branch 'overlayIconFixes' into 2.2 (#4765)

[osx] Fix missing overlay icons on client startup

Since the statuses are cached and that we can't invalidate the cache,
sending NOP would need to be overwritten by the default OK status
once the client successfully connected. But instead of remembering
which files we NOPed, rather wait until we are ready to sync before
sending the REGISTER_PATH message to the socket API client. It will
also prevent the client from sending unnecessary RETRIEVE_FILE_STATUS
requests.

Also remove AccountState::canSync, since it does the same as
isConnected and syncing is not an account responsibility.

Avoid a SyncFileStatusTracker private overload with the same name

Having an overload as a private function in the same class makes the
code harder to follow. Rename the private fileStatus to syncFileItemStatus.

Simplify the root status logic

Go through fileStatus like other cases to make sure that all use
cases go through the same code path. This also makes sure to use
lookupProblem which will use lower_bound which is more efficient
for larger sets of sync problems.

This also fixes the issue with lookupProblem that prevented it to
properly match an empty pathToMatch, caused by the fact that the
problem map contains relative paths not starting with a slash.

Fix the root item sync status #4682

Make sure that we push the new status when the status of the SyncEngine
changed. SyncEngine::started comes a bit late, only when the propagation
starts, although it's better in this case since child folders will
only switch to Sync in aboutToPropagate.

Also fix an issue with SyncEngine::findSyncItem when using an empty
fileName; this would match and return the wrong item, even though
not currently happening with the code since fileStatus won't call
it with an empty fileName anymore.

Fix the sync status push for parent directories #4682

As before, we rely on metadata-update SyncFileItem entries for parent
directories to notify us that a directory contains files to propagate,
and to know when all children were propagated through its itemCompleted
signal.

Those metadata SyncFileItems however have a None direction and we need
to add a explicit directory check to show them as Sync.
This fix also handles new files as well as existing ones, so no need
to keep a separate logic for new files.

Fix nautilus and nemo plugin branding strings. (#4780)

* Fix nautilus and nemo plugin branding strings.

Fix for owncloud/enterprise#1314

* nautilus plugin: Add a comment why the line needs to stay unchanged.

[tx-robot] updated from transifex

Propagator: fix qt4 build

propagatedownload.cpp:712:35: error: 'seenLockedFile' is a protected member of 'OCC::OwncloudPropagator'

Signals are protected in Qt4 but public in Qt5, mark the class accessing it
as friend when compiling with Qt4

Re-Apply: Disambiguate socket API pathes on Windows with user name

Addresses #3411

ShareDialog: show the error message in red

Do the visual stuff from designer.
The previous code that was ment to change the color in red did not work
and changed it to gray instead.
Also I don't see why there should be a frame.

Issue #4773

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

LockWatcher: Keep an eye on Windows file locks (#4758)

When a conflict-rename or a temporary-rename fails, notify the
LockWatcher. It'll regularly check whether the file has become
accesible again. When it has, another sync is triggered.

owncloud/enterprise#1288

Add spinner during sharee search (#4764)

Fixed #4740

When searching for sharees we should display a loading spinner.

Merge remote-tracking branch 'origin/2.1' into 2.2

PropagateDownload: Throw an error if the file is empty while it should not have been (#4753)

If the downloaded file is empty but the PROPFIND previously announced it
should not have been empty, this might mean the file was somehow corrupted
because of a bug on the server and that we should therefore not accept
the file.

Normaly we accept a change between the actual size of the file and what we
got during discovery because the file might have been updated to a new version
inbetween. But after this patch we won't accept the file if it was replaced
by an empty file.

Will help for issue #4583
Also requested by IL for issue 548

[tx-robot] updated from transifex

Shibboleth: Load username from config for UI (#4751)

For https://github.com/owncloud/enterprise/issues/1034

Merge pull request #4756 from owncloud/revert-4738-fix_3411

Revert "Disambiguate socket API pathes on Windows with user name"

Revert "Disambiguate socket API pathes on Windows with user name"

Merge pull request #4738 from owncloud/fix_3411

Disambiguate socket API pathes on Windows with user name

Merge pull request #4748 from owncloud/win_stat_revisited

vio for windows: Use more graceful create file params.

Checksums: Skip uploads of .eml files only #4754

This is a workaround. A correct solution would still update file metadata
such as the mtime. See #4755.

Fix perl tests #4722

* The dav auth realm is "sabre/dav" for newer server versions
* Content checksums are enabled generally, so conflict detection
  is smarter

FolderWizard: Sort remote folders #4612

Selective sync: Sort folders in FolderStatusModel #4612

[tx-robot] updated from transifex

Protocol: Make timestamp column width fit the text #4721

Merge pull request #4739 from ckamm/trackdirty

Overlay icons: Track touched files #4730

Overlay icons: Track touched files #4730

This uses the file watcher to keep track of files that were modified
in order to assign them the blue icon.

This is transient state that's not persisted across restarts.

sqlite: Update to version 3.12.2

For OS X and Windows

ConnectionValidator: Also set short timeout for capabilities job

Default timeout was 5 min.

Vio for windows: Removed the overlap param again.

After discussion and studying docs again we agreed that
the parameter is not neccessary here as it only affects
the process that opens the handle. And we do not want to
do async stuff.

AccountSettings: context menu: don't rely on the alias

Don't rely on the alias to know weather we shuld show a context menu or not.
Use the classify function to know if it's a root folder instead

Workaround ubuntu 16.04 systemtray bug (#4693) (#4747)

vio for windows: Use more graceful create file params.

Remote folder selection: improve path typing #4745 #4746

* Scroll to the target typed path
* Show non-404 errors that were triggered by typing paths

[tx-robot] updated from transifex

Merge pull request #4741 from owncloud/fix-alias

Accountsettings: Generate an alias for new folders

Folder: Generate an alias for new folders

Before commit 1a51b6718a345f1e5f28dd1f4e0c04bdaeeb5357, the wizard was
making sure folder had an alias but this is no longer the case.
So generate still an unique alias.

Alias is not used in the UI any longer, it's just use for internal purposes.

For issue #4737

Merge pull request #4744 from owncloud/fixAliasesOnceMore2

UI: Fix account name/alias display oddities #4577

UI: Fix account name/alias display oddities #4577

Disambiguate socket API pathes on Windows with user name

Addresses #3411

Checksums: Use the first supported type if nothing is preferred

Merge pull request #4736 from ckamm/socketapi_nautilus_invalidate

Overlay icons: Fix invalidation with nautilus #3249

Overlay icons: Fix invalidation with nautilus #3249

This avoids an extra socketapi call after invalidate_extension_info
is called from a STATUS message.

Overlay icons: Fix duplicate icons in nautilus #3249

Share link: Consistent order of options owncloud/core#24122

(cherry picked from commit 8dc178a9f3091e92a18ca17a274487342312069c)

QTokenizer: fix Qt4 build

QSharedPointer::reset is new in Qt5

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #4704 from owncloud/doc_overlay_icons

Doc: Draft overlay icons

Merge pull request #4717 from owncloud/fix_webdav_property

Only 1 : between namespace and property

Only 1 : between namespace and property

[tx-robot] updated from transifex

Merge pull request #4715 from owncloud/translate-desktop-ini

Adding new resource for the desktop.ini file

Fix search & replace error in Dockerfile

Adding new resource for the desktop.ini file

Theme: missing virtual keyword

Quota: Add branding option for the base folder (#4714)

As discussed on issue ##4460
Having the quote to be queried on subfolder is wrong in the generic case,
so add a branding option to configure it.

This partially reverts commit ff4cdc3161ddbb74a0fe10f969043ff898fbb93a

Doc: Applied review feedback for overlay icon docs.

Merge pull request #4713 from owncloud/fix_win-sis

Fix sync to Windows deduplication enabled storages

[tx-robot] updated from transifex

Windows VIO: Remove unused variable.

local_vio_win: Fix handling files that have been deduplicated

as part of Windows SIS or Data Deduplication. Fixes #4056

Merge pull request #4689 from owncloud/overlays_handle_root

SyncFileStatusTracker: Add a method to handle the root folder of the sync.

Also, handle new files put in the sync in the statustracker logic.

SocketAPI: Code simplifications as asked for in the review.

SyncFileStatusTracker: Remember the state of new files.

In the before-propagate slot, new files that wait to be
pushed to the server are remembered in the _syncProblems
map. That way, the parents show a sync icon properly as
asked for in #4682.

After the item has been transfered properly, the item is
removed from the map again because success is the default.

owncloudcmd: Save the selective sync list on the first run

For issue #4690

In the first run, the journal might not exist, we should not cancel
setting the selective sync list

Merge pull request #4709 from owncloud/fix_4691

Use QTokenizer to properly parse netrc

Use QTokenizer to properly parse netrc

Addresses #4691

Update doc and Dockerfile for 2.2 builds

Merge pull request #4663 from 'ckamm/disabletransmissionchecksum'

Disable validation of transmission checksums by default

Checksums: remove the downloadChecksumValidationEnabled option

Added in previous commit from pull request #4663

As discussed, we do not need this option so no need to introduce
a new dependency on the config file in the sync engine

[tx-robot] updated from transifex

SocketAPI: Return NOP state if folder is diconnected or paused.

Later, we can send a specific state for that.

Nautilus plugin: Be more robust against new states from socketapi

Do not bail out if an unknown state comes from SocketApi

Nautilus plugin: Remove the bogus logging

Merge pull request #4708 from owncloud/aliasGui

UI: Remove alias from more places #4577 #4695

UI: Remove alias from more places #4577 #4695

[tx-robot] updated from transifex

a bit of editing on new section

Doc: Draft overlay icons

SocketAPI: Display a warning on the directory if there are errors

Errors with individual files within a directory now result in
a warning label on the parent directories.

Merge pull request #4696 from owncloud/issue4573_negative_content_length

Discovery: Be more explicit about files with unknown size #4573

Discovery: Be more explicit about files with unknown size #4573

Also related https://github.com/owncloud/core/issues/23468

Merge pull request #4700 from owncloud/fix_4409

Use expireDate if returned by the OCS Share API

Use expireDate if returned by the OCS Share API

If an app modifies the expiration date (for example the password policy
app) then on more recent versions of the server we will get the share
object back REST style. We should use that info!

Fixes #4409

[tx-robot] updated from transifex

Merge pull request #4687 from owncloud/fix_share-permissions-namespace

The share-permissions namespace was moved