From 90f41055bede0b5714066b50ffae8a579e549b69 Mon Sep 17 00:00:00 2001 From: grimsi <9295182+grimsi@users.noreply.github.com> Date: Wed, 17 Aug 2022 14:30:48 +0200 Subject: [PATCH] Fix bug when trying to determine file size for files (instead of directories) --- backend/pom.xml | 2 +- .../de/grimsi/gameyfin/mapper/GameMapper.java | 15 +++++++++++++-- frontend/pom.xml | 2 +- pom.xml | 2 +- 4 files changed, 16 insertions(+), 5 deletions(-) diff --git a/backend/pom.xml b/backend/pom.xml index 85270e6..b4e55e4 100644 --- a/backend/pom.xml +++ b/backend/pom.xml @@ -7,7 +7,7 @@ gameyfin de.grimsi - 1.1.2 + 1.1.3 gameyfin-backend diff --git a/backend/src/main/java/de/grimsi/gameyfin/mapper/GameMapper.java b/backend/src/main/java/de/grimsi/gameyfin/mapper/GameMapper.java index a61e068..fd63e13 100644 --- a/backend/src/main/java/de/grimsi/gameyfin/mapper/GameMapper.java +++ b/backend/src/main/java/de/grimsi/gameyfin/mapper/GameMapper.java @@ -104,8 +104,19 @@ public class GameMapper { stopWatch.start(); - // Some benchmarks I did have shown that trying to parallelize this process makes it slower instead of faster - long fileSize = FileUtils.sizeOfDirectory(path.toFile()); + long fileSize; + + if(Files.isDirectory(path)) { + // Some benchmarks I did have shown that trying to parallelize this process makes it slower instead of faster + fileSize = FileUtils.sizeOfDirectory(path.toFile()); + } else { + try{ + fileSize = Files.size(path); + } catch (IOException e) { + log.error("Error while calculating size of file '{}'.", path); + fileSize = -1L; + } + } stopWatch.stop(); diff --git a/frontend/pom.xml b/frontend/pom.xml index da6ec3d..b3961bd 100644 --- a/frontend/pom.xml +++ b/frontend/pom.xml @@ -5,7 +5,7 @@ gameyfin de.grimsi - 1.1.2 + 1.1.3 4.0.0 diff --git a/pom.xml b/pom.xml index 92cf638..5170e7e 100644 --- a/pom.xml +++ b/pom.xml @@ -5,7 +5,7 @@ de.grimsi gameyfin - 1.1.2 + 1.1.3 gameyfin gameyfin