MoveMessageToTrash() - Remove null execution path

abhinavminhas · Jul 27, 2024 · 277cd59 · 277cd59
1 parent 97e87e5
commit 277cd59
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/GmailAPIHelper/GmailHelper.cs b/GmailAPIHelper/GmailHelper.cs
@@ -646,12 +646,17 @@ public static bool MoveMessageToTrash(this GmailService gmailService, string que
             }
             if (messages.Count > 0)
             {
+                var isMoved = false;
                 var latestMessage = messages.OrderByDescending(item => item.InternalDate).FirstOrDefault();
-                var moveToTrashRequest = service.Users.Messages.Trash(userId, latestMessage.Id);
-                moveToTrashRequest.Execute();
+                if (latestMessage != null)
+                {
+                    var moveToTrashRequest = service.Users.Messages.Trash(userId, latestMessage.Id);
+                    moveToTrashRequest.Execute();
+                    isMoved = true;
+                }
                 if (disposeGmailService)
                     service.DisposeGmailService();
-                return true;
+                return isMoved;
             }
             if (disposeGmailService)
                 service.DisposeGmailService();