GitLab

Also, always use the folder path from the fs::IFolder instance rather
than the user provided path

And use it to force a device cache refresh

The lack of it could cause a folder that is a mounpoint to be detected
as a folder of the containing device, instead of the device itself

This is assumed when composing file path

This is now only part of the test implementation, and allows us to
remove quite a bit of overhead

Instead of re-computing it dynamically while it's never going to change

The file wasn't added to the removable device

This is not portable, and doesn't work with subfolders, nor with
mountpoints.

As a result, the Media.Duplicate test is obsolete, since a filename
isn't unique in itself. Only the couple (mrl, folder_id) is UNIQUE.
However, UNIQUE doesn't apply when the folder_id foreign key is NULL
Uniqueness has been manually tested for a real folder/mrl couple

Relying on the fact that we only use a specific set of getters is not
future proof at all.
Also, this is using lazy loading instead of a fairly heavy overhead each
time we fetch all the folders

This is needed to instantiate a device in order to get its mountpoint

This will be used to remove the device mountpoint from full length path

This is IDevice's job now

There is no point in exposing directories to the outside world