Get rid of warnings

SSWConsulting · Oct 24, 2023 · 9a5f751 · 9a5f751
1 parent 1ee76af
commit 9a5f751
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/src/TimesheetGPT.Core/Models/Email.cs b/src/TimesheetGPT.Core/Models/Email.cs
@@ -3,7 +3,7 @@ namespace TimesheetGPT.Core.Models;
 public class Email
 {
     public string? Subject { get; set; }
-    public string Body { get; set; }
-    public string To { get; set; }
+    public string? Body { get; set; }
+    public string? To { get; set; }
     public string? Id { get; set; }
 }
diff --git a/src/TimesheetGPT.Core/Plugins.cs b/src/TimesheetGPT.Core/Plugins.cs
@@ -10,7 +10,7 @@ namespace TimesheetGPT.Core;
 public class GraphPlugins(IGraphService graphService)
 {
     [SKFunction, Description("Get email body from Id")]
-    public async Task<string> GetEmailBody(string id)
+    public async Task<string?> GetEmailBody(string id)
     {
         return (await graphService.GetEmailBody(id)).Body;
     }

diff --git a/src/TimesheetGPT.Core/Services/GraphService.cs b/src/TimesheetGPT.Core/Services/GraphService.cs
@@ -41,8 +41,8 @@ public async Task<List<Email>> GetSentEmails(DateTime date, CancellationToken ca
             return new List<Email>(messages.Value.Select(m => new Email
             {
                 Subject = m.Subject,
-                Body = m.BodyPreview,
-                To = string.Join(", ", m.ToRecipients.Select(r => r.EmailAddress.Name).ToList()),
+                Body = m.BodyPreview ?? "",
+                To = string.Join(", ", m.ToRecipients?.Select(r => r.EmailAddress?.Name).ToList() ?? new List<string?>()),
                 Id = m.Id
             }));
         }
@@ -70,8 +70,8 @@ public async Task<List<Meeting>> GetMeetings(DateTime date, CancellationToken ca
         {
             return meetings.Value.Select(m => new Meeting
             {
-                Name = m.Subject,
-                Length = DateTime.Parse(m.End.DateTime) - DateTime.Parse(m.Start.DateTime),
+                Name = m.Subject ?? "",
+                Length = DateTime.Parse(m.End?.DateTime ?? string.Empty) - DateTime.Parse(m.Start?.DateTime ?? string.Empty),
                 Repeating = m.Type == EventType.Occurrence,
                 // Sender = m.EmailAddress.Address TODO: Why is Organizer and attendees null? permissions?
             }).ToList();
@@ -99,7 +99,7 @@ public async Task<List<TeamsCall>> GetTeamsCalls(DateTime date)
             {
                 return calls.Value.Select(m => new TeamsCall
                 {
-                    Attendees = m.Participants.Select(p => p.User.DisplayName).ToList(),
+                    Attendees = m.Participants?.Select(p => p.User?.DisplayName).ToList() ?? new List<string?>(),
                     Length = m.EndDateTime - m.StartDateTime ?? TimeSpan.Zero,
                 }).ToList();
             }
@@ -126,7 +126,7 @@ public async Task<Email> GetEmailBody(string id)
             return new Email
             {
                 Body = message.BodyPreview,
-                To = string.Join(", ", message.ToRecipients.Select(r => r.EmailAddress.Name).ToList())
+                To = string.Join(", ", (message.ToRecipients ?? new List<Recipient>()).Select(r => r.EmailAddress?.Name).ToList())
             };
         }
 
@@ -136,6 +136,6 @@ public async Task<Email> GetEmailBody(string id)
 
 public class TeamsCall
 {
-    public List<string> Attendees { get; set; }
+    public List<string?>? Attendees { get; set; }
     public TimeSpan Length { get; set; }
 }