[shell/windows] Fix the windows status push not working (#4784)
authorJocelyn Turcotte <jturcotte@woboq.com>
Fri, 6 May 2016 10:32:01 +0000 (12:32 +0200)
committerJocelyn Turcotte <jturcotte@woboq.com>
Fri, 6 May 2016 10:32:01 +0000 (12:32 +0200)
commit727e73d6401737a6b6814dab49095d7a722cc821
tree77d661dbdfae34457acddc4a004918f584f74217
parentd7804d8df388e9ddfcda26b28434694668ffb119
[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
src/gui/socketapi.cpp
src/libsync/syncfilestatustracker.cpp
src/libsync/syncfilestatustracker.h