fix category changes (#747)

# Description

This PR includes the following proposed change(s):

- fix category change, document changes and update contact

src/Spd.Manager.Licence/SecurityWorkerAppManager.cs

@@ -380,16 +380,18 @@ public async Task<WorkerLicenceCommandResponse> Handle(AnonymousWorkerLicenceApp
             else
                 await _licenceAppRepository.CommitLicenceApplicationAsync(createLicResponse.LicenceAppId, ApplicationStatusEnum.PaymentPending, ct);
 
-            return new WorkerLicenceCommandResponse() 
-            { 
-                LicenceAppId = createLicResponse.LicenceAppId, 
-                Cost = price.LicenceFees.FirstOrDefault()?.Amount 
+            return new WorkerLicenceCommandResponse()
+            {
+                LicenceAppId = createLicResponse.LicenceAppId,
+                Cost = price.LicenceFees.FirstOrDefault()?.Amount
             };
         }
         else
         {
             //update contact directly
-            await _contactRepository.ManageAsync(_mapper.Map<UpdateContactCmd>(request), ct);
+            UpdateContactCmd updateCmd = _mapper.Map<UpdateContactCmd>(request);
+            updateCmd.Id = originalApp.ContactId ?? Guid.Empty;
+            await _contactRepository.ManageAsync(updateCmd, ct);
             return new WorkerLicenceCommandResponse() { LicenceAppId = createLicResponse?.LicenceAppId, Cost = 0 };
         }
     }
@@ -456,9 +458,17 @@ private async Task<ChangeSpec> MakeChanges(LicenceApplicationResp originalApp, W
             newList.Sort();
             List<WorkerCategoryTypeCode> originalList = originalApp.CategoryCodes.Select(c => Enum.Parse<WorkerCategoryTypeCode>(c.ToString())).ToList();
             originalList.Sort();
-            if (newList.SequenceEqual(originalList)) changes.CategoriesChanged = true;
+            if (!newList.SequenceEqual(originalList)) changes.CategoriesChanged = true;
+        }
+
+        IEnumerable<LicAppFileInfo> items = await GetAllNewDocsInfo(newApp.DocumentKeyCodes, ct);
+        //if any new doc contains category document, we think categorieschanged.
+        if (!changes.CategoriesChanged)
+        {
+            changes.CategoriesChanged = items.Any(i => i.LicenceDocumentTypeCode.ToString().StartsWith("Category"));
         }
 
+
         //DogRestraintsChanged
         if (newApp.UseDogs != originalApp.UseDogs ||
             newApp.CarryAndUseRestraints != originalApp.CarryAndUseRestraints ||
@@ -469,8 +479,6 @@ private async Task<ChangeSpec> MakeChanges(LicenceApplicationResp originalApp, W
             changes.DogRestraintsChanged = true;
         }
 
-        IEnumerable<LicAppFileInfo> items = await GetAllNewDocsInfo(newApp.DocumentKeyCodes, ct);
-
         //PeaceOfficerStatusChanged: check if Hold a Position with Peace Officer Status changed, create task with high priority, assign to Licensing CS team
         PoliceOfficerRoleCode? originalRoleCode = originalApp.PoliceOfficerRoleCode == null ? null
             : Enum.Parse<PoliceOfficerRoleCode>(originalApp.PoliceOfficerRoleCode.ToString());