Made ownerid nulalble for voucher

src/MAVN.Service.SmartVouchers.Domain/Models/Voucher.cs

@@ -10,7 +10,7 @@ public class Voucher
         public string ValidationCodeHash { get; set; }
         public Guid CampaignId { get; set; }
         public VoucherStatus Status { get; set; }
-        public Guid OwnerId { get; set; }
+        public Guid? OwnerId { get; set; }
         public DateTime? PurchaseDate { get; set; }
         public DateTime? RedemptionDate { get; set; }
     }

src/MAVN.Service.SmartVouchers.Domain/Repositories/IVouchersRepository.cs

@@ -9,7 +9,7 @@ namespace MAVN.Service.SmartVouchers.Domain.Repositories
     public interface IVouchersRepository
     {
         Task<long> CreateAsync(Voucher voucher);
-        Task ReserveAsync(Voucher voucher);
+        Task ReserveAsync(Voucher voucher, Guid ownerId);
         Task CancelReservationAsync(Voucher voucher);
         Task UpdateAsync(Voucher voucher, string validationCode = null);
         Task<VoucherWithValidation> GetWithValidationByShortCodeAsync(string shortCode);

src/MAVN.Service.SmartVouchers.DomainServices/VouchersService.cs

@@ -92,9 +92,7 @@ public async Task<VoucherReservationResult> ReserveVoucherAsync(Guid voucherCamp
                     try
                     {
                         voucher = vouchers.FirstOrDefault();
-                        voucher.OwnerId = ownerId;
-                        voucher.Status = VoucherStatus.Reserved;
-                        await _vouchersRepository.ReserveAsync(voucher);
+                        await _vouchersRepository.ReserveAsync(voucher, ownerId);
                     }
                     catch (Exception e)
                     {
@@ -282,7 +280,6 @@ private string GenerateShortCodeFromId(long voucherId)
                 return ProcessingVoucherError.VoucherNotFound;
             }
 
-            voucher.Status = VoucherStatus.InStock;
             await _vouchersRepository.CancelReservationAsync(voucher);
 
             await _redisLocksService.ReleaseLockAsync(voucher.ShortCode, voucher.ShortCode);

src/MAVN.Service.SmartVouchers.MsSqlRepositories/Entities/VoucherEntity.cs

@@ -28,7 +28,7 @@ public class VoucherEntity
         public VoucherStatus Status { get; set; }
 
         [Column("owner_id")]
-        public Guid OwnerId { get; set; }
+        public Guid? OwnerId { get; set; }
 
         [Column("purchase_date")]
         public DateTime? PurchaseDate { get; set; }

src/MAVN.Service.SmartVouchers.MsSqlRepositories/Migrations/20200429095915_MakeOwnerNullable.cs

@@ -0,0 +1,31 @@
+using System;
+using Microsoft.EntityFrameworkCore.Migrations;
+
+namespace MAVN.Service.SmartVouchers.MsSqlRepositories.Migrations
+{
+    public partial class MakeOwnerNullable : Migration
+    {
+        protected override void Up(MigrationBuilder migrationBuilder)
+        {
+            migrationBuilder.AlterColumn<Guid>(
+                name: "owner_id",
+                schema: "smart_vouchers",
+                table: "voucher",
+                nullable: true,
+                oldClrType: typeof(Guid),
+                oldType: "uniqueidentifier");
+        }
+
+        protected override void Down(MigrationBuilder migrationBuilder)
+        {
+            migrationBuilder.AlterColumn<Guid>(
+                name: "owner_id",
+                schema: "smart_vouchers",
+                table: "voucher",
+                type: "uniqueidentifier",
+                nullable: false,
+                oldClrType: typeof(Guid),
+                oldNullable: true);
+        }
+    }
+}

src/MAVN.Service.SmartVouchers.MsSqlRepositories/Migrations/SmartVouchersContextModelSnapshot.cs

@@ -142,7 +142,7 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                         .HasColumnName("campaign_id")
                         .HasColumnType("uniqueidentifier");
 
-                    b.Property<Guid>("OwnerId")
+                    b.Property<Guid?>("OwnerId")
                         .HasColumnName("owner_id")
                         .HasColumnType("uniqueidentifier");
 

src/MAVN.Service.SmartVouchers.MsSqlRepositories/Repositories/VouchersRepository.cs

@@ -43,12 +43,15 @@ public async Task<long> CreateAsync(Voucher voucher)
             }
         }
 
-        public async Task ReserveAsync(Voucher voucher)
+        public async Task ReserveAsync(Voucher voucher, Guid ownerId)
         {
             var entity = _mapper.Map<VoucherEntity>(voucher);
 
             using (var context = _contextFactory.CreateDataContext())
             {
+                entity.OwnerId = ownerId;
+                entity.Status = VoucherStatus.Reserved;
+
                 context.Vouchers.Update(entity);
 
                 var campaign = await context.VoucherCampaigns.FindAsync(entity.CampaignId);
@@ -65,6 +68,9 @@ public async Task CancelReservationAsync(Voucher voucher)
 
             using (var context = _contextFactory.CreateDataContext())
             {
+                entity.Status = VoucherStatus.InStock;
+                entity.OwnerId = null;
+
                 context.Vouchers.Update(entity);
 
                 var campaign = await context.VoucherCampaigns.FindAsync(entity.CampaignId);