From afb1f9570dd2059074f045204f44dcd130ee6f51 Mon Sep 17 00:00:00 2001 From: Bill Thornton Date: Tue, 3 Jun 2025 17:13:38 -0400 Subject: [PATCH] Add guard against dividing by zero when calculating used percentage Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> --- .../features/storage/components/StorageListItem.tsx | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/src/apps/dashboard/features/storage/components/StorageListItem.tsx b/src/apps/dashboard/features/storage/components/StorageListItem.tsx index 5d63b6446..b93696f39 100644 --- a/src/apps/dashboard/features/storage/components/StorageListItem.tsx +++ b/src/apps/dashboard/features/storage/components/StorageListItem.tsx @@ -22,8 +22,14 @@ interface StorageListItemProps { const calculateUsed = (folder?: FolderStorageDto) => { if (typeof folder?.UsedSpace === 'undefined') return 0; if (typeof folder.FreeSpace === 'undefined') return 100; - - return folder.UsedSpace / (folder.FreeSpace + folder.UsedSpace) * 100; + + const totalSpace = folder.FreeSpace + folder.UsedSpace; + if (totalSpace === 0) return 0; + + // Ensure we don't have negative values + const usedSpace = Math.max(0, folder.UsedSpace); + + return Math.min(100, (usedSpace / totalSpace) * 100); }; const getStatusColor = (percent: number) => {