Yet another cleanup
Fix PHPdoc in apps/files_sharing

using scrutinizer patch
Fix PHPdoc in OC\Share\Share and OCP\Share
Permit personal mount points to be used for sharing

An issue existed where `readData` used `OCP\User::getUser()` to get the user
for personal mount points, which worked in all situations apart from when a
personal mount point was used for sharing, so the return from `getUser()` is
not the user that owns the share. As such, any personal mount points would not
work correctly when shared.

`readData` and `writeData` have been changed from using a `$isPersonal`
boolean to using a `$user` string|null. `$isPersonal = false` can now be
written as `$user = NULL` (or left out in the case of `readData`), and
`$isPersonal = true` can be written as `$user = OCP\User::getUser()`.
Merge branch 'master' into type-hinting-sharing

Conflicts:
	apps/files_sharing/lib/share/file.php
	apps/files_sharing/tests/api.php
	lib/private/share/share.php
Some more params are optional
Better documentation for isValidSource function
Correct return type for getFilePath
Typo
fix folder icon (file had a big background rectangle before)
Use strict type comparison

We certainly don't want to have type juggling on that.
Use 'boolean' as type instead of 'bool'
[tx-robot] updated from transifex
Merge pull request #8221 from owncloud/type-hinting-sharing

Type hinting fixes for sharing
Fix sharing of ext storage mount points

When sharing an ext storage mount point, it will now use the name of the
mount point instead of an empty string for the target path.
Added unit test for sharing ext storage mount points
Merge pull request #8293 from owncloud/files_external_fix_readData

Fix issue with personal mount points and sharing
Merge pull request #8493 from owncloud/we-certainly-don't-want-type-juggling-on-that

Use strict type comparison

* owncloud/we-certainly-don't-want-type-juggling-on-that:
  Use strict type comparison
Fix getPathById for Oracle

Added extra code to handle the case of Oracle which saves empty strings
as null values.
Memcached hasKey should test for success, the get can fail for other reasons.

One of the other failures is no running server.