Fix logic of upload cover image

charles7668 · Dec 11, 2024 · 0aea310 · 0aea310
1 parent ca12a6d
commit 0aea310
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 8 deletions.
diff --git a/GameManager/Components/Pages/components/Card.razor.cs b/GameManager/Components/Pages/components/Card.razor.cs
@@ -179,18 +179,19 @@ private async Task OnEdit()
                 return;
             try
             {
-                if ((resultModel.Cover == null && GameInfoParam.CoverPath != null)
+                if ((string.IsNullOrEmpty(resultModel.Cover) && !string.IsNullOrEmpty(GameInfoParam.CoverPath))
                     || (CoverIsLocalFile(resultModel.Cover) && !CoverIsLocalFile(GameInfoParam.CoverPath)))
                 {
                     await ConfigService.DeleteCoverImage(GameInfoParam.CoverPath);
                 }
-                else if (resultModel.Cover != null &&
-                         (!CoverIsLocalFile(GameInfoParam.CoverPath) || GameInfoParam.CoverPath == null) &&
+                else if (!string.IsNullOrEmpty(resultModel.Cover) &&
+                         (!CoverIsLocalFile(GameInfoParam.CoverPath) ||
+                          string.IsNullOrEmpty(GameInfoParam.CoverPath)) &&
                          CoverIsLocalFile(resultModel.Cover))
                 {
                     resultModel.Cover = await ConfigService.AddCoverImage(resultModel.Cover);
                 }
-                else if (resultModel.Cover != null &&
+                else if (!string.IsNullOrEmpty(resultModel.Cover) &&
                          CoverIsLocalFile(resultModel.Cover))
                 {
                     await ConfigService.ReplaceCoverImage(resultModel.Cover, GameInfoParam.CoverPath);

diff --git a/GameManager/Components/Pages/components/GameDetail.razor.cs b/GameManager/Components/Pages/components/GameDetail.razor.cs
@@ -195,18 +195,19 @@ private async Task OnEditGameInfo()
                 return;
             try
             {
-                if ((resultModel.Cover == null && _gameInfo.CoverPath != null)
+                if ((string.IsNullOrEmpty(resultModel.Cover) && !string.IsNullOrEmpty(_gameInfo.CoverPath))
                     || (CoverIsLocalFile(resultModel.Cover) && !CoverIsLocalFile(_gameInfo.CoverPath)))
                 {
                     await ConfigService.DeleteCoverImage(_gameInfo.CoverPath);
                 }
-                else if (resultModel.Cover != null &&
-                         (!CoverIsLocalFile(_gameInfo.CoverPath) || _gameInfo.CoverPath == null) &&
+                else if (!string.IsNullOrEmpty(resultModel.Cover) &&
+                         (!CoverIsLocalFile(_gameInfo.CoverPath) ||
+                          string.IsNullOrEmpty(_gameInfo.CoverPath)) &&
                          CoverIsLocalFile(resultModel.Cover))
                 {
                     resultModel.Cover = await ConfigService.AddCoverImage(resultModel.Cover);
                 }
-                else if (resultModel.Cover != null &&
+                else if (!string.IsNullOrEmpty(resultModel.Cover) &&
                          CoverIsLocalFile(resultModel.Cover))
                 {
                     await ConfigService.ReplaceCoverImage(resultModel.Cover, _gameInfo.CoverPath);