fixing edit methods in repos

4 files changed, 41 insertions, 67 deletions

diff --git a/src/DevHive.Data/Repositories/LanguageRepository.cs b/src/DevHive.Data/Repositories/LanguageRepository.cs
index 808fa9a..59c88a6 100644
--- a/src/DevHive.Data/Repositories/LanguageRepository.cs
+++ b/src/DevHive.Data/Repositories/LanguageRepository.cs
@@ -17,7 +17,6 @@ namespace DevHive.Data.Repositories
 		}
 
 		#region Create
-
 		public async Task<bool> AddAsync(Language entity)
 		{
 			await this._context.Languages
@@ -28,7 +27,6 @@ namespace DevHive.Data.Repositories
 		#endregion
 
 		#region Read
-
 		public async Task<Language> GetByIdAsync(Guid id)
 		{
 			return await this._context.Languages
@@ -58,19 +56,15 @@ namespace DevHive.Data.Repositories
 		#endregion
 
 		#region Delete
-
 		public async Task<bool> DeleteAsync(Language entity)
 		{
-			this._context
-				.Set<Language>()
-				.Remove(entity);
+			this._context.Languages.Remove(entity);
 
 			return await this.SaveChangesAsync(this._context);
 		}
 		#endregion
 
 		#region Validations
-
 		public async Task<bool> DoesLanguageNameExistAsync(string languageName)
 		{
 			return await this._context.Languages
diff --git a/src/DevHive.Data/Repositories/PostRepository.cs b/src/DevHive.Data/Repositories/PostRepository.cs
index db2c4af..71602e7 100644
--- a/src/DevHive.Data/Repositories/PostRepository.cs
+++ b/src/DevHive.Data/Repositories/PostRepository.cs
@@ -16,11 +16,10 @@ namespace DevHive.Data.Repositories
 			this._context = context;
 		}
 
-		//Create
+		#region Create
 		public async Task<bool> AddAsync(Post post)
 		{
-			await this._context
-				.Set<Post>()
+			await this._context.Posts
 				.AddAsync(post);
 
 			return await this.SaveChangesAsync(this._context);
@@ -28,18 +27,17 @@ namespace DevHive.Data.Repositories
 
 		public async Task<bool> AddCommentAsync(Comment entity)
 		{
-			await this._context
-				.Set<Comment>()
+			await this._context.Comments
 				.AddAsync(entity);
 
 			return await this.SaveChangesAsync(this._context);
 		}
+		#endregion
 
-		//Read
+		#region Read
 		public async Task<Post> GetByIdAsync(Guid id)
 		{
-			return await this._context
-				.Set<Post>()
+			return await this._context.Posts
 				.FindAsync(id);
 		}
 
@@ -52,8 +50,7 @@ namespace DevHive.Data.Repositories
 
 		public async Task<Comment> GetCommentByIdAsync(Guid id)
 		{
-			return await this._context
-				.Set<Comment>()
+			return await this._context.Comments
 				.FindAsync(id);
 		}
 
@@ -63,12 +60,12 @@ namespace DevHive.Data.Repositories
 				.FirstOrDefaultAsync(p => p.IssuerId == issuerId &&
 					p.TimeCreated == timeCreated);
 		}
+		#endregion
 
-		//Update
+		#region Update
 		public async Task<bool> EditAsync(Post newPost)
 		{
-			this._context
-			.Set<Post>()
+			this._context.Posts
 			.Update(newPost);
 
 			return await this.SaveChangesAsync(this._context);
@@ -76,18 +73,17 @@ namespace DevHive.Data.Repositories
 
 		public async Task<bool> EditCommentAsync(Comment newEntity)
 		{
-			this._context
-				.Set<Comment>()
+			this._context.Comments
 				.Update(newEntity);
 
 			return await this.SaveChangesAsync(this._context);
 		}
+		#endregion
 
-		//Delete
+		#region Delete
 		public async Task<bool> DeleteAsync(Post post)
 		{
-			this._context
-				.Set<Post>()
+			this._context.Posts
 				.Remove(post);
 
 			return await this.SaveChangesAsync(this._context);
@@ -95,27 +91,24 @@ namespace DevHive.Data.Repositories
 
 		public async Task<bool> DeleteCommentAsync(Comment entity)
 		{
-			this._context
-				.Set<Comment>()
+			this._context.Comments
 				.Remove(entity);
 
 			return await this.SaveChangesAsync(this._context);
 		}
+		#endregion
 
 		#region Validations
-
 		public async Task<bool> DoesPostExist(Guid postId)
 		{
-			return await this._context
-				.Set<Post>()
+			return await this._context.Posts
 				.AsNoTracking()
 				.AnyAsync(r => r.Id == postId);
 		}
 
 		public async Task<bool> DoesCommentExist(Guid id)
 		{
-			return await this._context
-				.Set<Comment>()
+			return await this._context.Comments
 				.AsNoTracking()
 				.AnyAsync(r => r.Id == id);
 		}
diff --git a/src/DevHive.Data/Repositories/RoleRepository.cs b/src/DevHive.Data/Repositories/RoleRepository.cs
index 684fbd7..25549bf 100644
--- a/src/DevHive.Data/Repositories/RoleRepository.cs
+++ b/src/DevHive.Data/Repositories/RoleRepository.cs
@@ -1,6 +1,5 @@
 using System;
 using System.Threading.Tasks;
-using DevHive.Common.Models.Misc;
 using DevHive.Data.Interfaces.Repositories;
 using DevHive.Data.Models;
 using Microsoft.EntityFrameworkCore;
@@ -16,32 +15,31 @@ namespace DevHive.Data.Repositories
 			this._context = context;
 		}
 
-		//Create
+		#region Create
 		public async Task<bool> AddAsync(Role entity)
 		{
-			await this._context
-				.Set<Role>()
+			await this._context.Roles
 				.AddAsync(entity);
 
 			return await this.SaveChangesAsync(this._context);
 		}
+		#endregion
 
-		//Read
+		#region Read
 		public async Task<Role> GetByIdAsync(Guid id)
 		{
-			return await this._context
-				.Set<Role>()
+			return await this._context.Roles
 				.FindAsync(id);
 		}
 
 		public async Task<Role> GetByNameAsync(string name)
 		{
-			return await this._context
-				.Set<Role>()
+			return await this._context.Roles
 				.FirstOrDefaultAsync(x => x.Name == name);
 		}
+		#endregion
 
-		//Update
+		#region Update
 		public async Task<bool> EditAsync(Role newEntity)
 		{
 			Role role = await this.GetByIdAsync(newEntity.Id);
@@ -53,31 +51,32 @@ namespace DevHive.Data.Repositories
 
 			return await this.SaveChangesAsync(this._context);
 		}
+		#endregion
 
-		//Delete
+		#region Delete
 		public async Task<bool> DeleteAsync(Role entity)
 		{
-			this._context
-				.Set<Role>()
+			this._context.Roles
 				.Remove(entity);
 
 			return await this.SaveChangesAsync(this._context);
 		}
+		#endregion
 
+		#region Validations
 		public async Task<bool> DoesNameExist(string name)
 		{
-			return await this._context
-				.Set<Role>()
+			return await this._context.Roles
 				.AsNoTracking()
 				.AnyAsync(r => r.Name == name);
 		}
 
 		public async Task<bool> DoesRoleExist(Guid id)
 		{
-			return await this._context
-				.Set<Role>()
+			return await this._context.Roles
 				.AsNoTracking()
 				.AnyAsync(r => r.Id == id);
 		}
+		#endregion
 	}
 }
diff --git a/src/DevHive.Data/Repositories/TechnologyRepository.cs b/src/DevHive.Data/Repositories/TechnologyRepository.cs
index 6853893..35ee567 100644
--- a/src/DevHive.Data/Repositories/TechnologyRepository.cs
+++ b/src/DevHive.Data/Repositories/TechnologyRepository.cs
@@ -18,11 +18,9 @@ namespace DevHive.Data.Repositories
 		}
 
 		#region Create
-
 		public async Task<bool> AddAsync(Technology entity)
 		{
-			await this._context
-				.Set<Technology>()
+			await this._context.Technologies
 				.AddAsync(entity);
 
 			return await this.SaveChangesAsync(this._context);
@@ -30,11 +28,9 @@ namespace DevHive.Data.Repositories
 		#endregion
 
 		#region Read
-
 		public async Task<Technology> GetByIdAsync(Guid id)
 		{
-			return await this._context
-				.Set<Technology>()
+			return await this._context.Technologies
 				.FindAsync(id);
 		}
 		public async Task<Technology> GetByNameAsync(string technologyName)
@@ -46,24 +42,18 @@ namespace DevHive.Data.Repositories
 		#endregion
 
 		#region Edit
-		public async Task<bool> EditAsync(Technology entity)
+		public async Task<bool> EditAsync(Technology newEntity)
 		{
-			Technology technology = await this._context.Technologies
-				.FirstOrDefaultAsync(x => x.Id == entity.Id);
-
-			this._context.Update(technology);
-			this._context.Entry(entity).CurrentValues.SetValues(entity);
+			this._context.Technologies.Update(newEntity);
 
 			return await this.SaveChangesAsync(this._context);
 		}
 		#endregion
 
 		#region Delete
-
 		public async Task<bool> DeleteAsync(Technology entity)
 		{
-			this._context
-				.Set<Technology>()
+			this._context.Technologies
 				.Remove(entity);
 
 			return await this.SaveChangesAsync(this._context);
@@ -71,11 +61,9 @@ namespace DevHive.Data.Repositories
 		#endregion
 
 		#region Validations
-
 		public async Task<bool> DoesTechnologyNameExistAsync(string technologyName)
 		{
-			return await this._context
-				.Set<Technology>()
+			return await this._context.Technologies
 				.AsNoTracking()
 				.AnyAsync(r => r.Name == technologyName);
 		}