also cc @tigermick @davivel

Also @icewind1991

Looking at how the tests fail I'm guessing this isn't going to be that simple :(

Yeah, that is dangerous territory... especially due to the unlogical and inconsistent way how the View is used.

In my case $root was /admin// and the two slashes caused the bug.

So far I'd suggest putting on gloves and only fixing the problem in the calling code only, remove the duplicate slash if needed.

@PVince81 @oparoz @georgehrke it's working for me ! My problem about thumbnails in external local storage in owncloud app is solved !
Thank you very much !

I'm not sure if fixing only the external storage when using the thumbnail API is the best approach here.
The problem comes down to this function:

https://github.com/owncloud/core/blob/ed7e009e056cb46b67aba8827efb5fdb3961de1c/lib/private/Files/Mount/MountPoint.php#L190L198

$path has too many slashes, therefore strlen($this->mountPoint) won't work properly.

cc @icewind1991

Too bad we don't have a "comparePath" function...

What about normalizing both $path and $this->mountPoint?

or use some ltrim($path, '/') magic before concat, if that's even possible

Would probably be sense to normalize the incoming path in getInternalPath iirc, the mountpoint is already normalized

okay, will try it :)

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.