Set proper root path for single file shares originating from other storages

[juliusknorr]

Fixes#16617

The main issue here seems to be that the SharedStorage assumes an non-empty result of getUnjailedPath in

server/apps/files_sharing/lib/SharedStorage.php

Line 249 in d5f19c1

if ($source = $this->getUnjailedPath($path)) {

which is not given for shares that originate in the root of a mountpoint (like single file federated shares).

I'm not 100% sure this is the right approach, so some input from @icewind1991 or @rullzer would be highly appreciated. 😉 Maybe we could even just obit the if statement checking for the getUnjailedPath not being empty?

[rullzer]

@icewind1991 I'd like your take on this.

[icewind1991]

remove the check for empty getUnjailedPath

[ChristophWurst]

remove the check for empty getUnjailedPath

Applied. Please have a look at the new diff as I didn't test this nor have a clue what I just changed 😉

patch changed fundamentally

[rullzer]

@juliushaertl kill or redo?

[rullzer]

let me rebase...

[MorrisJobke]

@icewind1991 🏓

[faily-bot]

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 32410: failure

mariadb10.4-php7.3

Show full log

There were 2 warnings:
1) Test\Files\ViewTest::testRenameFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static
2) Test\Files\ViewTest::testCopyFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static
--
There was 1 failure:
1) OCA\Files_Versions\Tests\VersioningTest::testRestoreMovedShare
File content has not changed
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'version 2'
+'version 1'
/drone/src/apps/files_versions/tests/VersioningTest.php:729

acceptance-app-files

• tests/acceptance/features/app-files.feature:262

Show full log

 Scenario: unmarking a file as favorite causes the file list to be sorted again # /drone/src/tests/acceptance/features/app-files.feature:262
Given I am logged in # LoginPageContext::iAmLoggedIn()
And I create a new folder named "A name alphabetically lower than welcome.txt" # FileListContext::iCreateANewFolderNamed()
And I see that "A name alphabetically lower than welcome.txt" precedes "welcome.txt" in the file list # FileListContext::iSeeThatPrecedesInTheFileList()
And I close the details view # FilesAppContext::iCloseTheDetailsView()
And I see that the details view is closed # FilesAppContext::iSeeThatTheDetailsViewIsClosed()
│ Details view in Files app visibility could not be got
│ Exception message: Element not found with xpath, (//html//*[@id="app-sidebar" or contains(@class, 'app-sidebar')])[1]
│ │ Unable to locate element: {"method":"xpath","selector":"(//html//*[@id=\"app-sidebar\" or contains(@class, 'app-sidebar')])[1]"}
│ For documentation on this error, please visit: http://seleniumhq.org/exceptions/no_such_element.html
│ Build info: version: '2.53.1', revision: 'a36b8b1', time: '2016-06-30 17:37:03'
│ System info: host: '5491c924bce5', ip: '192.168.208.2', os.name: 'Linux', os.arch: 'amd64', os.version: '4.15.0-76-generic', java.version: '1.8.0_91'
│ Driver info: driver.version: unknown
│ Trying again
│ And I mark "welcome.txt" as favorite # FileListContext::iMarkAsFavorite()
And I see that "welcome.txt" is marked as favorite # FileListContext::iSeeThatIsMarkedAsFavorite()
And I see that "welcome.txt" precedes "A name alphabetically lower than welcome.txt" in the file list # FileListContext::iSeeThatPrecedesInTheFileList()
When I unmark "welcome.txt" as favorite # FileListContext::iUnmarkAsFavorite()
Then I see that "welcome.txt" is not marked as favorite # FileListContext::iSeeThatIsNotMarkedAsFavorite()
Not favorited state icon for file welcome.txt in file list could not be found after 100 seconds (NoSuchElementException)
And I see that "A name alphabetically lower than welcome.txt" precedes "welcome.txt" in the file list # FileListContext::iSeeThatPrecedesInTheFileList()

[icewind1991]

fixed the test and removed the if statement that isn't relevant anyway

[rullzer]

@icewind1991 +1 then?