Skip to content

Commit b3422db

Browse files
authored
Merge pull request #241 from TechnologyEnhancedLearning/Develop/Fixes/TD-6134-Issues-on-'My-account---Personal-details'-screen
TD-6134: My account personal details page issues
2 parents 11a544f + 1e88d5c commit b3422db

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

LearningHub.Nhs.UserApi.Repository/UserRoleUpgradeRepository.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public IQueryable<UserRoleUpgrade> GetByUserIdAsync(int userId)
3737
public IQueryable<UserRoleUpgrade> GetByEmailAddressAsync(string emailAddress, int userId)
3838
{
3939
return this.DbContext.Set<UserRoleUpgrade>()
40-
.Where(n => n.UserId == userId && n.EmailAddress == emailAddress && n.Deleted == false && n.UpgradeDate == null)
40+
.Where(n => n.UserId == userId && n.EmailAddress.ToLower() == emailAddress.ToLower() && n.Deleted == false && n.UpgradeDate == null)
4141
.AsNoTracking();
4242
}
4343
}

LearningHub.Nhs.UserApi.Services/ElfhUserService.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -980,10 +980,7 @@ public async Task UpdateMyAccountPersonalDetails(PersonalDetailsViewModel person
980980
user.FirstName = personalDetailsViewModel.FirstName;
981981
user.LastName = personalDetailsViewModel.LastName;
982982
user.PreferredName = personalDetailsViewModel.PreferredName;
983-
if (!string.IsNullOrEmpty(personalDetailsViewModel.SecondaryEmailAddress))
984-
{
985-
user.AltEmailAddress = personalDetailsViewModel.SecondaryEmailAddress;
986-
}
983+
user.AltEmailAddress = personalDetailsViewModel.SecondaryEmailAddress;
987984

988985
await this.elfhUserRepository.UpdateAsync(currentUserId, user);
989986
}

0 commit comments

Comments
 (0)