diff options
author | Julius Härtl <jus@bitgrid.net> | 2020-05-19 21:05:05 +0300 |
---|---|---|
committer | Julius Härtl <jus@bitgrid.net> | 2020-05-20 11:05:04 +0300 |
commit | 11ec635fd4cd95f0374587d1f04a87ddee060ef7 (patch) | |
tree | 8f54740d46af0735efad7b6447691e3779ccac36 | |
parent | a735402828005249e944208e7128cdcc93914c29 (diff) |
Return early when iterating over supported filenames
This fixes multiple files showing up on case-insensitive filenames as otherwise
the IFolder::get call creates multiple entries in the filecache for the same file
Signed-off-by: Julius Härtl <jus@bitgrid.net>
-rw-r--r-- | lib/Service/WorkspaceService.php | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/lib/Service/WorkspaceService.php b/lib/Service/WorkspaceService.php index 143d02631..ae5d57f60 100644 --- a/lib/Service/WorkspaceService.php +++ b/lib/Service/WorkspaceService.php @@ -24,17 +24,15 @@ class WorkspaceService { } public function getFile(Folder $folder) { - $file = null; foreach ($this->getSupportedFilenames() as $filename) { if ($folder->nodeExists($filename)) { try { - $file = $folder->get($filename); + return $folder->get($filename); } catch (NotFoundException $e) { } - continue; } } - return $file; + return null; } public function getSupportedFilenames() { |