Remove FIXME in syncengine
Rename _dir to _direction

(to avoid confusion with directory)
Prefer the use of _isDirectory over  item._type == SyncFileItem::Directory
Remove FIXME

This has been refactored since the comment was added and is no longer a hack
Remove a FIXME

The commit message say that this line should be necessary to show
the information when the setting dialog is open,  but this is working
now via the signals
Remove FIXME and TODO

They are outdated