diff --git a/IRaCIS.Core.Application/Service/Allocation/VisitTaskService.cs b/IRaCIS.Core.Application/Service/Allocation/VisitTaskService.cs
index 14cc3b6da..354aa4cc2 100644
--- a/IRaCIS.Core.Application/Service/Allocation/VisitTaskService.cs
+++ b/IRaCIS.Core.Application/Service/Allocation/VisitTaskService.cs
@@ -93,7 +93,7 @@ namespace IRaCIS.Core.Application.Service.Allocation
                     {
                         await _subjectUserRepository.AddAsync(new SubjectUser() {TrialId=command.TrialId,  ArmEnum = doctorArm.ArmEnum, DoctorUserId = doctorArm.DoctorUserId, SubjectId = subjectId, AssignTime = DateTime.Now });
 
-                        await _visitTaskRepository.BatchUpdateNoTrackingAsync(t => t.SubjectId == subjectId && t.DoctorUserId == doctorArm.DoctorUserId && t.ArmEnum == doctorArm.ArmEnum && t.TrialId==command.TrialId, u => new VisitTask()
+                        await _visitTaskRepository.BatchUpdateNoTrackingAsync(t => t.SubjectId == subjectId && t.ArmEnum == doctorArm.ArmEnum && t.TrialId==command.TrialId && t.TaskAllocationState==TaskAllocationState.NotAllocate, u => new VisitTask()
                         {
                             AllocateTime = DateTime.Now,
                             DoctorUserId = doctorArm.DoctorUserId,
@@ -157,7 +157,7 @@ namespace IRaCIS.Core.Application.Service.Allocation
 
                 await _subjectUserRepository.DeleteFromQueryAsync(t => t.Id == command.Id);
 
-                await _visitTaskRepository.BatchUpdateNoTrackingAsync(t => t.SubjectId == command.SubjectId && t.DoctorUserId == command.DoctorUserId && t.ArmEnum == command.ArmEnum, u => new VisitTask()
+                await _visitTaskRepository.BatchUpdateNoTrackingAsync(t => t.SubjectId == command.SubjectId && t.DoctorUserId == command.DoctorUserId && t.ArmEnum == command.ArmEnum && t.ReadingTaskState!=ReadingTaskState.WaitReading, u => new VisitTask()
                 {
                     AllocateTime=null,
                     DoctorUserId=null,