Add suppress warning for ordinal string comparison

equinor · Nov 10, 2023 · fc3f891 · fc3f891
1 parent 00b4554
commit fc3f891
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 27 deletions.
diff --git a/backend/api/Database/Context/FlotillaDbContext.cs b/backend/api/Database/Context/FlotillaDbContext.cs
@@ -102,31 +102,6 @@ protected override void ConfigureConventions(ModelConfigurationBuilder configura
             configurationBuilder.Properties(typeof(Enum)).HaveConversion<string>();
         }
 
-        // SQLite does not have proper support for DateTimeOffset via Entity Framework Core, see the limitations
-        // here: https://docs.microsoft.com/en-us/ef/core/providers/sqlite/limitations#query-limitations
-        // To work around this, when the Sqlite database provider is used, all model properties of type DateTimeOffset
-        // use the DateTimeOffsetToBinaryConverter
-        // Based on: https://github.com/aspnet/EntityFrameworkCore/issues/10784#issuecomment-415769754
-        // This only supports millisecond precision, but should be sufficient for most use cases.
-        private static void AddConverterForDateTimeOffsets<TOwnerEntity, TDependentEntity>(
-            ref OwnedNavigationBuilder<TOwnerEntity, TDependentEntity> entity
-        )
-            where TOwnerEntity : class
-            where TDependentEntity : class
-        {
-            var properties = entity.OwnedEntityType.ClrType
-                .GetProperties()
-                .Where(
-                    p =>
-                        p.PropertyType == typeof(DateTimeOffset)
-                        || p.PropertyType == typeof(DateTimeOffset?)
-                );
-            foreach (var property in properties)
-            {
-                entity.Property(property.Name).HasConversion(new DateTimeOffsetToBinaryConverter());
-            }
-        }
-
         private static void AddConverterForDateTimeOffsets<T>(ref EntityTypeBuilder<T> entity)
             where T : class
         {

diff --git a/backend/api/Services/InspectionService.cs b/backend/api/Services/InspectionService.cs
@@ -1,4 +1,5 @@
-using Api.Database.Context;
+using System.Diagnostics.CodeAnalysis;
+using Api.Database.Context;
 using Api.Database.Models;
 using Api.Services.Models;
 using Api.Utilities;
@@ -10,6 +11,11 @@ public interface IInspectionService
         public Task<Inspection> UpdateInspectionStatus(string isarStepId, IsarStepStatus isarStepStatus);
     }
 
+    [SuppressMessage(
+        "Globalization",
+        "CA1309:Use ordinal StringComparison",
+        Justification = "EF Core refrains from translating string comparison overloads to SQL"
+    )]
     public class InspectionService : IInspectionService
     {
         private readonly FlotillaDbContext _context;

diff --git a/backend/api/Services/MissionTaskService.cs b/backend/api/Services/MissionTaskService.cs
@@ -1,4 +1,5 @@
-using Api.Database.Context;
+using System.Diagnostics.CodeAnalysis;
+using Api.Database.Context;
 using Api.Database.Models;
 using Api.Services.Models;
 using Api.Utilities;
@@ -10,6 +11,11 @@ public interface IMissionTaskService
         public Task<MissionTask> UpdateMissionTaskStatus(string isarTaskId, IsarTaskStatus isarTaskStatus);
     }
 
+    [SuppressMessage(
+        "Globalization",
+        "CA1309:Use ordinal StringComparison",
+        Justification = "EF Core refrains from translating string comparison overloads to SQL"
+    )]
     public class MissionTaskService : IMissionTaskService
     {
         private readonly FlotillaDbContext _context;