Doing some cleanup of all the Warnings that litter the build - Part 6 (…

…FritzAndFriends#190)

Co-authored-by: Jeffrey T. Fritz <csharpfritz@users.noreply.github.com>

src/TagzApp.Common/IMessagingService.cs

@@ -1,5 +1,4 @@
 using Microsoft.Extensions.Hosting;
-using TagzApp.Common.Models;
 
 namespace TagzApp.Web.Services;
 
@@ -12,8 +11,8 @@ public interface IMessagingService : IHostedService
 	Task<Content?> GetContentByIds(string provider, string providerId);
 
 	Task<IEnumerable<Content>> GetExistingContentForTag(string tag);
-
-	Task<IEnumerable<(Content, ModerationAction?)>> GetContentByTagForModeration(string tag);
+	// TODO: Needs checking if a non-nullable ModerationAction makes problems!
+	Task<IEnumerable<(Content, ModerationAction)>> GetContentByTagForModeration(string tag);
 
 	Task<IEnumerable<Content>> GetApprovedContentByTag(string tag);
 

src/TagzApp.Common/InMemoryContentMessaging.cs

@@ -1,5 +1,4 @@
 using System.Collections.Concurrent;
-using TagzApp.Common.Models;
 
 namespace TagzApp.Common;
 
@@ -74,9 +73,11 @@ public void StartProviders(IEnumerable<ISocialMediaProvider> providers, Cancella
 		_ProviderTasks.Clear();
 		foreach (var providerItem in providers)
 		{
-			_ProviderTasks.Add(Task.Factory.StartNew(async (object state) =>
+			_ProviderTasks.Add(Task.Factory.StartNew(async (object? state) =>
 			{
-				var provider = (ISocialMediaProvider)state;
+				var provider = state as ISocialMediaProvider;
+
+				if (provider is null) return;
 
 				var lastQueryTime = DateTimeOffset.UtcNow.AddHours(-1);
 

src/TagzApp.Web/Services/InMemoryMessagingService.cs

@@ -1,8 +1,6 @@
-using Microsoft.AspNetCore.SignalR;
 using System.Collections.Concurrent;
-using TagzApp.Common.Models;
+
 using TagzApp.Communication;
-using TagzApp.Web.Hubs;
 
 namespace TagzApp.Web.Services;