Implemented DB population through the LIC service API

Controllers/HL7Controller.cs

@@ -0,0 +1,97 @@
+using Microsoft.AspNetCore.Mvc;
+using SimpleLIS.DTO;
+using SimpleLIS.Services;
+using SimpleLIS.Models;
+using AutoMapper;
+using System.Text.Json;
+
+namespace SimpleLIS.Controllers;
+
+[ApiController]
+[Route("api/hl7")]
+public class HL7Controller : ControllerBase
+{
+    private readonly MessageService _messageService;
+    private readonly PatientService _patientService;
+    private readonly IMapper _mapper;
+
+    public HL7Controller(
+        MessageService messageService, 
+        PatientService patientService, 
+        IMapper mapper)
+    {
+        _messageService = messageService;
+        _patientService = patientService;
+        _mapper = mapper;
+    }
+
+[HttpPost("process")]
+public async Task<IActionResult> ProcessHL7Message([FromBody] JsonElement payload)
+{
+    try
+    {
+        // Create Patient
+        var patient = new Patient
+        {
+            HL7PatientId = payload.GetProperty("hl7PatientId").GetString(),
+            LastName = payload.GetProperty("lastName").GetString(),
+            FirstName = payload.GetProperty("firstName").GetString(),
+            DateOfBirth = payload.GetProperty("dateOfBirth").GetDateTime(),
+            Gender = payload.GetProperty("gender").GetString(),
+            Messages = new List<Message>()
+        };
+
+        // Check for existing patient
+        var existingPatient = await _patientService.GetPatientByNameAndDOB(
+            patient.FirstName!, 
+            patient.LastName!, 
+            patient.DateOfBirth
+        );
+
+        patient = existingPatient ?? await _patientService.CreatePatientAsync(patient);
+
+        // Create Message
+        var message = new Message
+        {
+            ControlId = payload.GetProperty("controlId").GetString(),
+            MessageType = payload.GetProperty("messageType").GetString(),
+            SendingApp = payload.GetProperty("sendingApp").GetString(),
+            SendingFacility = payload.GetProperty("sendingFacility").GetString(),
+            ReceivingApp = payload.GetProperty("receivingApp").GetString(),
+            ReceivingFacility = payload.GetProperty("receivingFacility").GetString(),
+            Timestamp = payload.GetProperty("timestamp").GetDateTime(),
+            Version = payload.GetProperty("version").GetString(),
+            PatientId = patient.PatientId,
+            Patient = patient,
+            Observations = new List<Observation>()
+        };
+
+        // Create Observations
+        var observationsArray = payload.GetProperty("observations");
+        foreach (var obsElement in observationsArray.EnumerateArray())
+        {
+            var observation = new Observation
+            {
+                ObservationCode = obsElement.GetProperty("ObservationCode").GetString(),
+                ObservationValue = obsElement.GetProperty("ObservationValue").GetString(),
+                Units = obsElement.GetProperty("Units").GetString(),
+                AbnormalFlag = obsElement.GetProperty("AbnormalFlag").GetString(),
+                ResultStatus = obsElement.GetProperty("ResultStatus").GetString() ?? "F",
+                Message = message
+            };
+            message.Observations.Add(observation);
+        }
+
+        var createdMessage = await _messageService.CreateMessageAsync(message);
+        
+        return Ok(new { 
+            message = _mapper.Map<MessageDTO>(createdMessage),
+            patient = _mapper.Map<PatientDTO>(patient)
+        });
+    }
+    catch (Exception ex)
+    {
+        return BadRequest(new { error = ex.Message + (ex.InnerException?.Message ?? "") });
+    }
+}
+}

DTO/MessageDTO.cs

@@ -3,31 +3,14 @@ namespace SimpleLIS.DTO;
 public class MessageDTO
 {
     public int MessageId { get; set; }
-#pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
+    public required string ControlId { get; set; }
-    public string ControlId { get; set; } // Unique identifier for the HL7 message
+    public required string MessageType { get; set; }
-#pragma warning restore CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
+    public string? SendingApp { get; set; }
-#pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
+    public string? SendingFacility { get; set; }
-    public string MessageType { get; set; } // Type of HL7 message (e.g., ORU^R01)
+    public string? ReceivingApp { get; set; }
-#pragma warning restore CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
+    public string? ReceivingFacility { get; set; }
-#pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
+    public DateTime Timestamp { get; set; }
-    public string SendingApp { get; set; } // Sending application
+    public string? Version { get; set; }
-#pragma warning restore CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
+    public int PatientId { get; set; }
-#pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
+    public required List<ObservationDTO> Observations { get; set; }
-    public string SendingFacility { get; set; } // Sending facility
-#pragma warning restore CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
-#pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
-    public string ReceivingApp { get; set; } // Receiving application
-#pragma warning restore CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
-#pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
-    public string ReceivingFacility { get; set; } // Receiving facility
-#pragma warning restore CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
-    public DateTime Timestamp { get; set; } // Message creation time
-#pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
-    public string Version { get; set; } // HL7 version
-#pragma warning restore CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
-
-    // Associated Observations
-#pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
-    public List<ObservationDTO> Observations { get; set; }
-#pragma warning restore CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
 }

DTO/ObservationDTO.cs

@@ -2,20 +2,11 @@ namespace SimpleLIS.DTO;
 
 public class ObservationDTO
 {
-#pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
+    public required string ObservationCode { get; set; }
-    public string ObservationCode { get; set; } // Code identifying the observation (e.g., WBC, HGB)
+    public required string ObservationValue { get; set; }
-#pragma warning restore CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
+    public string? Units { get; set; }
-#pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
+    public string? AbnormalFlag { get; set; }
-    public string ObservationValue { get; set; } // The observed value
+    public string? ResultStatus { get; set; }
-#pragma warning restore CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
-#pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
-    public string Units { get; set; } // Units of measurement (e.g., g/dL, %)
-#pragma warning restore CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
-#pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
-    public string AbnormalFlag { get; set; } // Indicates if the result is abnormal
-#pragma warning restore CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
-#pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
-    public string ResultStatus { get; set; } // Status of the result (e.g., F for final)
     public int ObservationId { get; internal set; }
-#pragma warning restore CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
+    public int MessageId { get; set; }
 }

DTO/PatientDTO.cs

@@ -1,14 +1,8 @@
 public class PatientDTO
 {
     public int PatientId { get; set; }
-#pragma warning disable CS8618
+    public string? LastName { get; set; }
-    public string LastName { get; set; }
+    public string? FirstName { get; set; }
-#pragma warning restore CS8618
-#pragma warning disable CS8618
-    public string FirstName { get; set; }
-#pragma warning restore CS8618
     public DateTime DateOfBirth { get; set; }
-#pragma warning disable CS8618
+    public string? Gender { get; set; }
-    public string Gender { get; set; }
-#pragma warning restore CS8618
 }

MessageProfile.cs

@@ -8,11 +8,15 @@ public class MappingProfile : Profile
 {
     public MappingProfile()
     {
-        CreateMap<MessageDTO, Message>();
+        CreateMap<Message, MessageDTO>()
-        CreateMap<Message, MessageDTO>();
+            .ForMember(dest => dest.Observations, opt => opt.MapFrom(src => src.Observations));
-        CreateMap<ObservationDTO, Observation>();
+        CreateMap<MessageDTO, Message>()
+            .ForMember(dest => dest.Patient, opt => opt.Ignore());
         CreateMap<Observation, ObservationDTO>();
-        CreateMap<PatientDTO, Patient>();
+        CreateMap<ObservationDTO, Observation>()
+            .ForMember(dest => dest.Message, opt => opt.Ignore());
         CreateMap<Patient, PatientDTO>();
+        CreateMap<PatientDTO, Patient>()
+            .ForMember(dest => dest.Messages, opt => opt.Ignore());
     }
 }

Migrations/20241212113523_Initial migrations.Designer.cs

@@ -1,149 +0,0 @@
-// <auto-generated />
-using System;
-using Microsoft.EntityFrameworkCore;
-using Microsoft.EntityFrameworkCore.Infrastructure;
-using Microsoft.EntityFrameworkCore.Migrations;
-using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
-using SimpleLIS;
-
-#nullable disable
-
-namespace SimpleLIS.Migrations
-{
-    [DbContext(typeof(HL7DbContext))]
-    [Migration("20241212113523_Initial migrations")]
-    partial class Initialmigrations
-    {
-        /// <inheritdoc />
-        protected override void BuildTargetModel(ModelBuilder modelBuilder)
-        {
-#pragma warning disable 612, 618
-            modelBuilder.HasAnnotation("ProductVersion", "8.0.11");
-
-            modelBuilder.Entity("SimpleLIS.Models.Message", b =>
-                {
-                    b.Property<int>("MessageId")
-                        .ValueGeneratedOnAdd()
-                        .HasColumnType("INTEGER");
-
-                    b.Property<string>("ControlId")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.Property<string>("MessageType")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.Property<string>("ReceivingApp")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.Property<string>("ReceivingFacility")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.Property<string>("SendingApp")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.Property<string>("SendingFacility")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.Property<DateTime>("Timestamp")
-                        .HasColumnType("TEXT");
-
-                    b.Property<string>("Version")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.HasKey("MessageId");
-
-                    b.ToTable("Messages");
-                });
-
-            modelBuilder.Entity("SimpleLIS.Models.Observation", b =>
-                {
-                    b.Property<int>("ObservationId")
-                        .ValueGeneratedOnAdd()
-                        .HasColumnType("INTEGER");
-
-                    b.Property<string>("AbnormalFlag")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.Property<int>("MessageId")
-                        .HasColumnType("INTEGER");
-
-                    b.Property<string>("ObservationCode")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.Property<string>("ObservationValue")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.Property<string>("ResultStatus")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.Property<string>("Units")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.HasKey("ObservationId");
-
-                    b.HasIndex("MessageId");
-
-                    b.ToTable("Observations");
-                });
-
-            modelBuilder.Entity("SimpleLIS.Models.Patient", b =>
-                {
-                    b.Property<int>("PatientId")
-                        .ValueGeneratedOnAdd()
-                        .HasColumnType("INTEGER");
-
-                    b.Property<DateTime>("DateOfBirth")
-                        .HasColumnType("TEXT");
-
-                    b.Property<string>("FirstName")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.Property<string>("Gender")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.Property<string>("HL7PatientId")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.Property<string>("LastName")
-                        .IsRequired()
-                        .HasColumnType("TEXT");
-
-                    b.HasKey("PatientId");
-
-                    b.ToTable("Patients");
-                });
-
-            modelBuilder.Entity("SimpleLIS.Models.Observation", b =>
-                {
-                    b.HasOne("SimpleLIS.Models.Message", "Message")
-                        .WithMany("Observations")
-                        .HasForeignKey("MessageId")
-                        .OnDelete(DeleteBehavior.Cascade)
-                        .IsRequired();
-
-                    b.Navigation("Message");
-                });
-
-            modelBuilder.Entity("SimpleLIS.Models.Message", b =>
-                {
-                    b.Navigation("Observations");
-                });
-#pragma warning restore 612, 618
-        }
-    }
-}

Migrations/20241213124817_Patients got messages.cs

@@ -1,50 +0,0 @@
-using Microsoft.EntityFrameworkCore.Migrations;
-
-#nullable disable
-
-namespace SimpleLIS.Migrations
-{
-    /// <inheritdoc />
-    public partial class Patientsgotmessages : Migration
-    {
-        /// <inheritdoc />
-        protected override void Up(MigrationBuilder migrationBuilder)
-        {
-            migrationBuilder.AddColumn<int>(
-                name: "PatientId",
-                table: "Messages",
-                type: "INTEGER",
-                nullable: false,
-                defaultValue: 0);
-
-            migrationBuilder.CreateIndex(
-                name: "IX_Messages_PatientId",
-                table: "Messages",
-                column: "PatientId");
-
-            migrationBuilder.AddForeignKey(
-                name: "FK_Messages_Patients_PatientId",
-                table: "Messages",
-                column: "PatientId",
-                principalTable: "Patients",
-                principalColumn: "PatientId",
-                onDelete: ReferentialAction.Cascade);
-        }
-
-        /// <inheritdoc />
-        protected override void Down(MigrationBuilder migrationBuilder)
-        {
-            migrationBuilder.DropForeignKey(
-                name: "FK_Messages_Patients_PatientId",
-                table: "Messages");
-
-            migrationBuilder.DropIndex(
-                name: "IX_Messages_PatientId",
-                table: "Messages");
-
-            migrationBuilder.DropColumn(
-                name: "PatientId",
-                table: "Messages");
-        }
-    }
-}

Migrations/20241214065908_Initial Migrations.Designer.cs

@@ -11,8 +11,8 @@ using SimpleLIS;
 namespace SimpleLIS.Migrations
 {
     [DbContext(typeof(HL7DbContext))]
-    [Migration("20241213124817_Patients got messages")]
+    [Migration("20241214065908_Initial Migrations")]
-    partial class Patientsgotmessages
+    partial class InitialMigrations
     {
         /// <inheritdoc />
         protected override void BuildTargetModel(ModelBuilder modelBuilder)
@@ -38,26 +38,21 @@ namespace SimpleLIS.Migrations
                         .HasColumnType("INTEGER");
 
                     b.Property<string>("ReceivingApp")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<string>("ReceivingFacility")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<string>("SendingApp")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<string>("SendingFacility")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<DateTime>("Timestamp")
                         .HasColumnType("TEXT");
 
                     b.Property<string>("Version")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.HasKey("MessageId");
@@ -74,7 +69,6 @@ namespace SimpleLIS.Migrations
                         .HasColumnType("INTEGER");
 
                     b.Property<string>("AbnormalFlag")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<int>("MessageId")
@@ -93,7 +87,6 @@ namespace SimpleLIS.Migrations
                         .HasColumnType("TEXT");
 
                     b.Property<string>("Units")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.HasKey("ObservationId");
@@ -113,19 +106,15 @@ namespace SimpleLIS.Migrations
                         .HasColumnType("TEXT");
 
                     b.Property<string>("FirstName")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<string>("Gender")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<string>("HL7PatientId")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<string>("LastName")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.HasKey("PatientId");

Migrations/20241214065908_Initial Migrations.cs

@@ -6,11 +6,28 @@ using Microsoft.EntityFrameworkCore.Migrations;
 namespace SimpleLIS.Migrations
 {
     /// <inheritdoc />
-    public partial class Initialmigrations : Migration
+    public partial class InitialMigrations : Migration
     {
         /// <inheritdoc />
         protected override void Up(MigrationBuilder migrationBuilder)
         {
+            migrationBuilder.CreateTable(
+                name: "Patients",
+                columns: table => new
+                {
+                    PatientId = table.Column<int>(type: "INTEGER", nullable: false)
+                        .Annotation("Sqlite:Autoincrement", true),
+                    HL7PatientId = table.Column<string>(type: "TEXT", nullable: true),
+                    LastName = table.Column<string>(type: "TEXT", nullable: true),
+                    FirstName = table.Column<string>(type: "TEXT", nullable: true),
+                    DateOfBirth = table.Column<DateTime>(type: "TEXT", nullable: false),
+                    Gender = table.Column<string>(type: "TEXT", nullable: true)
+                },
+                constraints: table =>
+                {
+                    table.PrimaryKey("PK_Patients", x => x.PatientId);
+                });
+
             migrationBuilder.CreateTable(
                 name: "Messages",
                 columns: table => new
@@ -19,33 +36,23 @@ namespace SimpleLIS.Migrations
                         .Annotation("Sqlite:Autoincrement", true),
                     ControlId = table.Column<string>(type: "TEXT", nullable: false),
                     MessageType = table.Column<string>(type: "TEXT", nullable: false),
-                    SendingApp = table.Column<string>(type: "TEXT", nullable: false),
+                    SendingApp = table.Column<string>(type: "TEXT", nullable: true),
-                    SendingFacility = table.Column<string>(type: "TEXT", nullable: false),
+                    SendingFacility = table.Column<string>(type: "TEXT", nullable: true),
-                    ReceivingApp = table.Column<string>(type: "TEXT", nullable: false),
+                    ReceivingApp = table.Column<string>(type: "TEXT", nullable: true),
-                    ReceivingFacility = table.Column<string>(type: "TEXT", nullable: false),
+                    ReceivingFacility = table.Column<string>(type: "TEXT", nullable: true),
                     Timestamp = table.Column<DateTime>(type: "TEXT", nullable: false),
-                    Version = table.Column<string>(type: "TEXT", nullable: false)
+                    Version = table.Column<string>(type: "TEXT", nullable: true),
+                    PatientId = table.Column<int>(type: "INTEGER", nullable: false)
                 },
                 constraints: table =>
                 {
                     table.PrimaryKey("PK_Messages", x => x.MessageId);
-                });
+                    table.ForeignKey(
-
+                        name: "FK_Messages_Patients_PatientId",
-            migrationBuilder.CreateTable(
+                        column: x => x.PatientId,
-                name: "Patients",
+                        principalTable: "Patients",
-                columns: table => new
+                        principalColumn: "PatientId",
-                {
+                        onDelete: ReferentialAction.Cascade);
-                    PatientId = table.Column<int>(type: "INTEGER", nullable: false)
-                        .Annotation("Sqlite:Autoincrement", true),
-                    HL7PatientId = table.Column<string>(type: "TEXT", nullable: false),
-                    LastName = table.Column<string>(type: "TEXT", nullable: false),
-                    FirstName = table.Column<string>(type: "TEXT", nullable: false),
-                    DateOfBirth = table.Column<DateTime>(type: "TEXT", nullable: false),
-                    Gender = table.Column<string>(type: "TEXT", nullable: false)
-                },
-                constraints: table =>
-                {
-                    table.PrimaryKey("PK_Patients", x => x.PatientId);
                 });
 
             migrationBuilder.CreateTable(
@@ -57,8 +64,8 @@ namespace SimpleLIS.Migrations
                     MessageId = table.Column<int>(type: "INTEGER", nullable: false),
                     ObservationCode = table.Column<string>(type: "TEXT", nullable: false),
                     ObservationValue = table.Column<string>(type: "TEXT", nullable: false),
-                    Units = table.Column<string>(type: "TEXT", nullable: false),
+                    Units = table.Column<string>(type: "TEXT", nullable: true),
-                    AbnormalFlag = table.Column<string>(type: "TEXT", nullable: false),
+                    AbnormalFlag = table.Column<string>(type: "TEXT", nullable: true),
                     ResultStatus = table.Column<string>(type: "TEXT", nullable: false)
                 },
                 constraints: table =>
@@ -72,6 +79,11 @@ namespace SimpleLIS.Migrations
                         onDelete: ReferentialAction.Cascade);
                 });
 
+            migrationBuilder.CreateIndex(
+                name: "IX_Messages_PatientId",
+                table: "Messages",
+                column: "PatientId");
+
             migrationBuilder.CreateIndex(
                 name: "IX_Observations_MessageId",
                 table: "Observations",
@@ -85,10 +97,10 @@ namespace SimpleLIS.Migrations
                 name: "Observations");
 
             migrationBuilder.DropTable(
-                name: "Patients");
+                name: "Messages");
 
             migrationBuilder.DropTable(
-                name: "Messages");
+                name: "Patients");
         }
     }
 }

Migrations/HL7DbContextModelSnapshot.cs

@@ -35,26 +35,21 @@ namespace SimpleLIS.Migrations
                         .HasColumnType("INTEGER");
 
                     b.Property<string>("ReceivingApp")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<string>("ReceivingFacility")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<string>("SendingApp")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<string>("SendingFacility")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<DateTime>("Timestamp")
                         .HasColumnType("TEXT");
 
                     b.Property<string>("Version")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.HasKey("MessageId");
@@ -71,7 +66,6 @@ namespace SimpleLIS.Migrations
                         .HasColumnType("INTEGER");
 
                     b.Property<string>("AbnormalFlag")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<int>("MessageId")
@@ -90,7 +84,6 @@ namespace SimpleLIS.Migrations
                         .HasColumnType("TEXT");
 
                     b.Property<string>("Units")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.HasKey("ObservationId");
@@ -110,19 +103,15 @@ namespace SimpleLIS.Migrations
                         .HasColumnType("TEXT");
 
                     b.Property<string>("FirstName")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<string>("Gender")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<string>("HL7PatientId")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.Property<string>("LastName")
-                        .IsRequired()
                         .HasColumnType("TEXT");
 
                     b.HasKey("PatientId");

Models/Message.cs

@@ -3,36 +3,15 @@ namespace SimpleLIS.Models;
 public class Message
 {
     public int MessageId { get; set; }
-#pragma warning disable CS8618
+    public required string ControlId { get; set; }
-    public string ControlId { get; set; }
+    public required string MessageType { get; set; }
-#pragma warning restore CS8618
+    public string? SendingApp { get; set; }
-#pragma warning disable CS8618
+    public string? SendingFacility { get; set; }
-    public string MessageType { get; set; }
+    public string? ReceivingApp { get; set; }
-#pragma warning restore CS8618
+    public string? ReceivingFacility { get; set; }
-#pragma warning disable CS8618
-    public string SendingApp { get; set; }
-#pragma warning restore CS8618
-#pragma warning disable CS8618
-    public string SendingFacility { get; set; }
-#pragma warning restore CS8618
-#pragma warning disable CS8618
-    public string ReceivingApp { get; set; }
-#pragma warning restore CS8618
-#pragma warning disable CS8618
-    public string ReceivingFacility { get; set; }
-#pragma warning restore CS8618
     public DateTime Timestamp { get; set; }
-#pragma warning disable CS8618
+    public string? Version { get; set; }
-    public string Version { get; set; }
+    public required ICollection<Observation> Observations { get; set; }
-#pragma warning restore CS8618
-
-    // Navigation Property for Observations
-#pragma warning disable CS8618
-    public ICollection<Observation> Observations { get; set; }
-#pragma warning restore CS8618
-
     public int PatientId { get; set; }
-#pragma warning disable CS8618
+    public required Patient Patient { get; set; }
-    public Patient Patient { get; set; }
-#pragma warning restore CS8618
 }

Models/Observation.cs

@@ -3,25 +3,11 @@ namespace SimpleLIS.Models;
 public class Observation
 {
     public int ObservationId { get; set; }
-    public int MessageId { get; set; } // Foreign Key
+    public int MessageId { get; set; }
-#pragma warning disable CS8618
+    public required string ObservationCode { get; set; }
-    public string ObservationCode { get; set; }
+    public required string ObservationValue { get; set; }
-#pragma warning restore CS8618
+    public string? Units { get; set; }
-#pragma warning disable CS8618
+    public string? AbnormalFlag { get; set; }
-    public string ObservationValue { get; set; }
+    public required string ResultStatus { get; set; }
-#pragma warning restore CS8618
+    public required Message Message { get; set; }
-#pragma warning disable CS8618
-    public string Units { get; set; }
-#pragma warning restore CS8618
-#pragma warning disable CS8618
-    public string AbnormalFlag { get; set; }
-#pragma warning restore CS8618
-#pragma warning disable CS8618
-    public string ResultStatus { get; set; }
-#pragma warning restore CS8618
-
-    // Navigation Property
-#pragma warning disable CS8618
-    public Message Message { get; set; }
-#pragma warning restore CS8618
 }

Models/Patient.cs

@@ -3,21 +3,10 @@ namespace SimpleLIS.Models;
 public class Patient
 {
     public int PatientId { get; set; }
-#pragma warning disable CS8618
+    public string? HL7PatientId { get; set; }
-    public string HL7PatientId { get; set; }
+    public string? LastName { get; set; }
-#pragma warning restore CS8618
+    public string? FirstName { get; set; }
-#pragma warning disable CS8618
-    public string LastName { get; set; }
-#pragma warning restore CS8618
-#pragma warning disable CS8618
-    public string FirstName { get; set; }
-#pragma warning restore CS8618
     public DateTime DateOfBirth { get; set; }
-#pragma warning disable CS8618
+    public string? Gender { get; set; }
-    public string Gender { get; set; }
+    public required ICollection<Message> Messages { get; set; }
-#pragma warning restore CS8618
-
-#pragma warning disable CS8618
-    public ICollection<Message> Messages { get; set; }
-#pragma warning restore CS8618
 }

Services/MessageService.cs

@@ -19,11 +19,11 @@ public class MessageService
         return message;
     }
 
-    public async Task<Message> GetMessageByIdAsync(int id)
+    public async Task<Message?> GetMessageByIdAsync(int id)
     {
-#pragma warning disable CS8603
+        return await _context.Messages
-        return await _context.Messages.Include(m => m.Observations).FirstOrDefaultAsync(m => m.MessageId == id);
+            .Include(m => m.Observations)
-#pragma warning restore CS8603
+            .FirstOrDefaultAsync(m => m.MessageId == id);
     }
 
     public async Task<IEnumerable<Message>> ListMessagesAsync()

Services/ObservationService.cs

@@ -19,16 +19,16 @@ public class ObservationService
         return observation;
     }
 
-    public async Task<Observation> GetObservationByIdAsync(int id)
+    public async Task<Observation?> GetObservationByIdAsync(int id)
     {
-#pragma warning disable CS8603
+        return await _context.Observations
-        return await _context.Observations.FirstOrDefaultAsync(o => o.ObservationId == id);
+            .Include(o => o.Message)
-#pragma warning restore CS8603
+            .FirstOrDefaultAsync(o => o.ObservationId == id);
     }
 
     public async Task<IEnumerable<Observation>> ListObservationsAsync()
     {
-        return await _context.Observations.ToListAsync();
+        return await _context.Observations.Include(o => o.Message).ToListAsync();
     }
 
     public async Task<Observation> UpdateObservationAsync(Observation observation)

Services/PatientService.cs

@@ -12,7 +12,6 @@ public class PatientService
         _context = context;
     }
 
-
     public async Task<Patient> CreatePatientAsync(Patient patient)
     {
         _context.Patients.Add(patient);
@@ -20,11 +19,11 @@ public class PatientService
         return patient;
     }
 
-    public async Task<Patient> GetPatientByIdAsync(int id)
+    public async Task<Patient?> GetPatientByIdAsync(int id)
     {
-#pragma warning disable CS8603
+        return await _context.Patients
-        return await _context.Patients.FirstOrDefaultAsync(p => p.PatientId == id);
+            .Include(p => p.Messages)
-#pragma warning restore CS8603
+            .FirstOrDefaultAsync(p => p.PatientId == id);
     }
 
     public async Task<IEnumerable<Patient>> ListPatientsAsync()
@@ -53,4 +52,14 @@ public class PatientService
     {
         return await _context.Patients.AnyAsync(p => p.PatientId == id);
     }
+    public async Task<Patient?> GetPatientByNameAndDOB(string firstName, string lastName, DateTime dateOfBirth)
+    {
+        return await _context.Patients
+            .FirstOrDefaultAsync(p => 
+                p.FirstName == firstName && 
+                p.LastName == lastName && 
+                p.DateOfBirth == dateOfBirth);
+    }
+
+
 }