From b28eeb26579715ca487477fdad84fd96e301f6e1 Mon Sep 17 00:00:00 2001 From: johnniang Date: Thu, 21 Nov 2019 23:28:31 +0800 Subject: [PATCH] Remove redundant try clause --- .../app/service/impl/BackupServiceImpl.java | 30 +++++++++---------- 1 file changed, 14 insertions(+), 16 deletions(-) diff --git a/src/main/java/run/halo/app/service/impl/BackupServiceImpl.java b/src/main/java/run/halo/app/service/impl/BackupServiceImpl.java index dd633eb4b..81b7d1e75 100644 --- a/src/main/java/run/halo/app/service/impl/BackupServiceImpl.java +++ b/src/main/java/run/halo/app/service/impl/BackupServiceImpl.java @@ -178,22 +178,20 @@ public class BackupServiceImpl implements BackupService { @Override public List listHaloBackups() { - try { - // Build backup dto - try (Stream subPathStream = Files.list(Paths.get(haloProperties.getBackupDir()))) { - return subPathStream - .filter(backupPath -> StringUtils.startsWithIgnoreCase(backupPath.getFileName().toString(), HaloConst.HALO_BACKUP_PREFIX)) - .map(this::buildBackupDto) - .sorted((leftBackup, rightBackup) -> { - // Sort the result - if (leftBackup.getUpdateTime() < rightBackup.getUpdateTime()) { - return 1; - } else if (leftBackup.getUpdateTime() > rightBackup.getUpdateTime()) { - return -1; - } - return 0; - }).collect(Collectors.toList()); - } + // Build backup dto + try (Stream subPathStream = Files.list(Paths.get(haloProperties.getBackupDir()))) { + return subPathStream + .filter(backupPath -> StringUtils.startsWithIgnoreCase(backupPath.getFileName().toString(), HaloConst.HALO_BACKUP_PREFIX)) + .map(this::buildBackupDto) + .sorted((leftBackup, rightBackup) -> { + // Sort the result + if (leftBackup.getUpdateTime() < rightBackup.getUpdateTime()) { + return 1; + } else if (leftBackup.getUpdateTime() > rightBackup.getUpdateTime()) { + return -1; + } + return 0; + }).collect(Collectors.toList()); } catch (IOException e) { throw new ServiceException("Failed to fetch backups", e); }