add check id > 0 before assign
This commit is contained in:
@@ -71,7 +71,8 @@ namespace CarParkValidationAPI.Repository
|
||||
|
||||
private AdminUser FillAdminUser(AdminUserDto dto, AdminUser model)
|
||||
{
|
||||
model.UserID = dto.Id;
|
||||
if (dto.Id > 0)
|
||||
model.UserID = dto.Id;
|
||||
model.Name = dto.Name;
|
||||
model.RoleType = dto.RoleType;
|
||||
model.StafflinkNo = dto.StafflinkNo;
|
||||
|
||||
@@ -50,7 +50,8 @@ namespace CarParkValidationAPI.Repository
|
||||
model.ConcessionDesc = dto.Description.Trim();
|
||||
if (!string.IsNullOrEmpty(dto.Display))
|
||||
model.ConcessionDisplay = dto.Display.Trim();
|
||||
model.ConcessionTypeID = dto.Id;
|
||||
if (dto.Id > 0)
|
||||
model.ConcessionTypeID = dto.Id;
|
||||
return model;
|
||||
}
|
||||
public async Task<ResultModel<List<ConcessionTypeDto>>> LoadConcessionTypeAsync(bool onlyActive)
|
||||
|
||||
@@ -75,7 +75,8 @@ namespace CarParkValidationAPI.Repository
|
||||
model.ConcessionExpiry = dto.ConcessionExpiry;
|
||||
model.ConcessionHolder = dto.ConcessionHolder.Trim();
|
||||
model.ConcessionTypeID = dto.ConcessionTypeId;
|
||||
model.ValidationID = dto.Id;
|
||||
if (dto.Id > 0)
|
||||
model.ValidationID = dto.Id;
|
||||
model.StafflinkNo = dto.StafflinkNo;
|
||||
model.StaffFirstName = dto.StaffFirstName.Trim();
|
||||
model.StaffLastName = dto.StaffLastName.Trim();
|
||||
|
||||
@@ -47,7 +47,8 @@ namespace CarParkValidationAPI.Repository
|
||||
model.Description = item.Description.Trim();
|
||||
if (!string.IsNullOrEmpty(item.Display))
|
||||
model.DisplayName = item.Display.Trim();
|
||||
model.ValidationPointID = item.Id;
|
||||
if (item.Id > 0)
|
||||
model.ValidationPointID = item.Id;
|
||||
return model;
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user