From f1515439c5fd26a96817723db5d48b77baa82fb6 Mon Sep 17 00:00:00 2001
From: transtrike <transtrike@gmail.com>
Date: Tue, 26 Jan 2021 14:38:21 +0200
Subject: Working on Update User; Currently not updating user in UserRepo

---
 src/DevHive.Services/Services/UserService.cs | 62 ++++++++++++++++------------
 1 file changed, 35 insertions(+), 27 deletions(-)

(limited to 'src/DevHive.Services/Services/UserService.cs')

diff --git a/src/DevHive.Services/Services/UserService.cs b/src/DevHive.Services/Services/UserService.cs
index 960630e..0e3bf72 100644
--- a/src/DevHive.Services/Services/UserService.cs
+++ b/src/DevHive.Services/Services/UserService.cs
@@ -111,9 +111,7 @@ namespace DevHive.Services.Services
 
 			await this.ValidateUserCollections(updateUserServiceModel);
 
-			updateUserServiceModel = await this.PopulateUpdateModelWithIds(updateUserServiceModel);
-
-			User user = this._userMapper.Map<User>(updateUserServiceModel);
+			User user = await this.PopulateModel(updateUserServiceModel);
 
 			bool successful = await this._userRepository.EditAsync(updateUserServiceModel.Id, user);
 
@@ -298,49 +296,59 @@ namespace DevHive.Services.Services
 			return admin.Id;
 		}
 
-		private async Task<UpdateUserServiceModel> PopulateUpdateModelWithIds(UpdateUserServiceModel updateUserServiceModel)
+		private async Task<User> PopulateModel(UpdateUserServiceModel updateUserServiceModel)
 		{
-			/* Roles */
-			int roleCount = updateUserServiceModel.Roles.Count;
-			for (int i = 0; i < roleCount; i++)
+			User user = this._userMapper.Map<User>(updateUserServiceModel);
+
+			/* Fetch Roles and replace model's*/
+			HashSet<Role> roles = new();
+			int rolesCount = updateUserServiceModel.Roles.Count;
+			for (int i = 0; i < rolesCount; i++)
 			{
 				Role role = await this._roleRepository.GetByNameAsync(updateUserServiceModel.Roles.ElementAt(i).Name) ??
 					throw new ArgumentException("Invalid role name!");
 
-				updateUserServiceModel.Roles.ElementAt(i).Id = role.Id;
+				roles.Add(role);
 			}
+			user.Roles = roles;
 
-			/* Languages */
-			int langCount = updateUserServiceModel.Languages.Count;
-			for (int i = 0; i < langCount; i++)
+			/* Fetch Friends and replace model's*/
+			HashSet<User> friends = new();
+			int friendsCount = updateUserServiceModel.Friends.Count;
+			for (int i = 0; i < friendsCount; i++)
 			{
-				Language language = await this._languageRepository.GetByNameAsync(updateUserServiceModel.Languages.ElementAt(i).Name) ??
-					throw new ArgumentException("Invalid language name!");
+				User friend = await this._userRepository.GetByUsernameAsync(updateUserServiceModel.Friends.ElementAt(i).UserName) ??
+					throw new ArgumentException("Invalid friend's username!");
 
-				updateUserServiceModel.Languages.ElementAt(i).Id = language.Id;
+				friends.Add(friend);
 			}
+			user.Friends = friends;
 
-			/* Technologies */
-			int techCount = updateUserServiceModel.Technologies.Count;
-			for (int i = 0; i < techCount; i++)
+			/* Fetch Languages and replace model's*/
+			HashSet<Language> languages = new();
+			int languagesCount = updateUserServiceModel.Languages.Count;
+			for (int i = 0; i < languagesCount; i++)
 			{
-				Technology technology = await this._technologyRepository.GetByNameAsync(updateUserServiceModel.Technologies.ElementAt(i).Name) ??
-					throw new ArgumentException("Invalid technology name!");
+				Language language = await this._languageRepository.GetByNameAsync(updateUserServiceModel.Languages.ElementAt(i).Name) ??
+					throw new ArgumentException("Invalid language name!");
 
-				updateUserServiceModel.Technologies.ElementAt(i).Id = technology.Id;
+				languages.Add(language);
 			}
+			user.Languages = languages;
 
-			/* Friends */
-			int friendsCount = updateUserServiceModel.Friends.Count;
-			for (int i = 0; i < friendsCount; i++)
+			/* Fetch Technologies and replace model's*/
+			HashSet<Technology> technologies = new();
+			int technologiesCount = updateUserServiceModel.Technologies.Count;
+			for (int i = 0; i < technologiesCount; i++)
 			{
-				User friend = await this._userRepository.GetByUsernameAsync(updateUserServiceModel.Friends.ElementAt(i).UserName) ??
-					throw new ArgumentException("Invalid friend's username!");
+				Technology technology = await this._technologyRepository.GetByNameAsync(updateUserServiceModel.Technologies.ElementAt(i).Name) ??
+					throw new ArgumentException("Invalid technology name!");
 
-				updateUserServiceModel.Friends.ElementAt(i).Id = friend.Id;
+				technologies.Add(technology);
 			}
+			user.Technologies = technologies;
 
-			return updateUserServiceModel;
+			return user;
 		}
 		#endregion
 	}
-- 
cgit v1.2.3