Merge pull request #24 from Team-Kaleidoscope/feature/profile_picture_implementation

- Removed InsertProfilePicture from Service layer (unneeded)
- ReadProfilePicture endpoint implemented

4 files changed, 131 insertions, 30 deletions

diff --git a/src/Services/DevHive.Services/Interfaces/IProfilePictureService.cs b/src/Services/DevHive.Services/Interfaces/IProfilePictureService.cs
new file mode 100644
index 0000000..edf2775
--- /dev/null
+++ b/src/Services/DevHive.Services/Interfaces/IProfilePictureService.cs
@@ -0,0 +1,30 @@
+using System;
+using System.Threading.Tasks;
+using DevHive.Services.Models.ProfilePicture;
+
+namespace DevHive.Services.Interfaces
+{
+	public interface IProfilePictureService
+	{
+		/// <summary>
+		/// Get a profile picture by it's Guid
+		/// </summary>
+		/// <param name="id">Profile picture's Guid</param>
+		/// <returns>The profile picture's URL in the cloud</returns>
+		Task<string> GetProfilePictureById(Guid id);
+
+		/// <summary>
+		/// Uploads the given picture and assigns it's link to the user in the database
+		/// </summary>
+		/// <param name="profilePictureServiceModel">Contains User's Guid and the new picture to be updated</param>
+		/// <returns>The new profile picture's URL in the cloud</returns>
+		Task<string> UpdateProfilePicture(ProfilePictureServiceModel profilePictureServiceModel);
+
+		/// <summary>
+		/// Delete a profile picture from the cloud and the database
+		/// </summary>
+		/// <param name="id">The profile picture's Guid</param>
+		/// <returns>True if the picture is deleted, false otherwise</returns>
+		Task<bool> DeleteProfilePicture(Guid id);
+	}
+}
diff --git a/src/Services/DevHive.Services/Interfaces/IUserService.cs b/src/Services/DevHive.Services/Interfaces/IUserService.cs
index a55f9dd..da07507 100644
--- a/src/Services/DevHive.Services/Interfaces/IUserService.cs
+++ b/src/Services/DevHive.Services/Interfaces/IUserService.cs
@@ -45,14 +45,6 @@ namespace DevHive.Services.Interfaces
 		Task<UserServiceModel> UpdateUser(UpdateUserServiceModel updateUserServiceModel);
 
 		/// <summary>
-		/// Uploads the given picture and assigns it's link to the user in the database
-		/// Requires authenticated user
-		/// </summary>
-		/// <param name="updateProfilePictureServiceModel">Contains User's Guid and the new picture to be updated</param>
-		/// <returns>The new picture's URL</returns>
-		Task<ProfilePictureServiceModel> UpdateProfilePicture(UpdateProfilePictureServiceModel updateProfilePictureServiceModel);
-
-		/// <summary>
 		/// Deletes a user from the database and removes his data entirely
 		/// Requires authenticated user
 		/// </summary>
diff --git a/src/Services/DevHive.Services/Services/ProfilePictureService.cs b/src/Services/DevHive.Services/Services/ProfilePictureService.cs
new file mode 100644
index 0000000..1de2114
--- /dev/null
+++ b/src/Services/DevHive.Services/Services/ProfilePictureService.cs
@@ -0,0 +1,101 @@
+using System;
+using System.Collections.Generic;
+using System.Threading.Tasks;
+using DevHive.Data.Interfaces;
+using DevHive.Data.Models;
+using DevHive.Services.Interfaces;
+using DevHive.Services.Models.ProfilePicture;
+using Microsoft.AspNetCore.Http;
+
+namespace DevHive.Services.Services
+{
+	public class ProfilePictureService : IProfilePictureService
+	{
+		private readonly IUserRepository _userRepository;
+		private readonly IProfilePictureRepository _profilePictureRepository;
+		private readonly ICloudService _cloudinaryService;
+
+		public ProfilePictureService(IUserRepository userRepository, IProfilePictureRepository profilePictureRepository, ICloudService cloudinaryService)
+		{
+			this._userRepository = userRepository;
+			this._profilePictureRepository = profilePictureRepository;
+			this._cloudinaryService = cloudinaryService;
+		}
+
+		public async Task<string> GetProfilePictureById(Guid id)
+		{
+			return (await this._profilePictureRepository.GetByIdAsync(id)).PictureURL;
+		}
+
+		public async Task<string> UpdateProfilePicture(ProfilePictureServiceModel profilePictureServiceModel)
+		{
+			ValidateProfPic(profilePictureServiceModel.ProfilePictureFormFile);
+			await ValidateUserExistsAsync(profilePictureServiceModel.UserId);
+
+			User user = await this._userRepository.GetByIdAsync(profilePictureServiceModel.UserId);
+			// if (user.ProfilePicture.PictureURL != ProfilePicture.DefaultURL)
+			if (user.ProfilePicture.Id != Guid.Empty)
+			{
+				List<string> file = new() { user.ProfilePicture.PictureURL };
+				bool removed = await this._cloudinaryService.RemoveFilesFromCloud(file);
+
+				if (!removed)
+					throw new ArgumentException("Cannot delete old picture");
+			}
+
+			return await SaveProfilePictureInDatabase(profilePictureServiceModel);
+		}
+
+		public async Task<bool> DeleteProfilePicture(Guid id)
+		{
+			ProfilePicture profilePic = await this._profilePictureRepository.GetByIdAsync(id) ??
+				throw new ArgumentException("Such picture doesn't exist!");
+
+			bool removedFromDb = await this._profilePictureRepository.DeleteAsync(profilePic);
+			if (!removedFromDb)
+				throw new ArgumentException("Cannot delete picture from database!");
+
+			List<string> file = new() { profilePic.PictureURL };
+			bool removedFromCloud = await this._cloudinaryService.RemoveFilesFromCloud(file);
+			if (!removedFromCloud)
+				throw new ArgumentException("Cannot delete picture from cloud!");
+
+			return true;
+		}
+
+		private async Task<string> SaveProfilePictureInDatabase(ProfilePictureServiceModel profilePictureServiceModel)
+		{
+			List<IFormFile> file = new() { profilePictureServiceModel.ProfilePictureFormFile };
+			string picUrl = (await this._cloudinaryService.UploadFilesToCloud(file))[0];
+			ProfilePicture profilePic = new() { PictureURL = picUrl };
+
+			User user = await this._userRepository.GetByIdAsync(profilePictureServiceModel.UserId);
+			profilePic.UserId = user.Id;
+			profilePic.User = user;
+
+			bool success = await this._profilePictureRepository.AddAsync(profilePic);
+			if (!success)
+				throw new ArgumentException("Unable to upload picture!");
+
+			user.ProfilePicture = profilePic;
+			bool userProfilePicAlter = await this._userRepository.EditAsync(user.Id, user);
+
+			if (!userProfilePicAlter)
+				throw new ArgumentException("Unable to alter user's profile picture");
+
+			return picUrl;
+		}
+
+		private static void ValidateProfPic(IFormFile profilePictureFormFile)
+		{
+			if (profilePictureFormFile.Length == 0)
+				throw new ArgumentException("Picture cannot be null");
+		}
+
+		private async Task ValidateUserExistsAsync(Guid userId)
+		{
+			if (!await this._userRepository.DoesUserExistAsync(userId))
+				throw new ArgumentException("User does not exist!");
+		}
+	}
+}
diff --git a/src/Services/DevHive.Services/Services/UserService.cs b/src/Services/DevHive.Services/Services/UserService.cs
index 4f74b06..d487de4 100644
--- a/src/Services/DevHive.Services/Services/UserService.cs
+++ b/src/Services/DevHive.Services/Services/UserService.cs
@@ -119,28 +119,6 @@ namespace DevHive.Services.Services
 			User newUser = await this._userRepository.GetByIdAsync(user.Id);
 			return this._userMapper.Map<UserServiceModel>(newUser);
 		}
-
-		public async Task<ProfilePictureServiceModel> UpdateProfilePicture(UpdateProfilePictureServiceModel updateProfilePictureServiceModel)
-		{
-			User user = await this._userRepository.GetByIdAsync(updateProfilePictureServiceModel.UserId);
-
-			if (!string.IsNullOrEmpty(user.ProfilePicture.PictureURL))
-			{
-				bool success = await _cloudService.RemoveFilesFromCloud(new List<string> { user.ProfilePicture.PictureURL });
-				if (!success)
-					throw new InvalidCastException("Could not delete old profile picture!");
-			}
-
-			string fileUrl = (await this._cloudService.UploadFilesToCloud(new List<IFormFile> { updateProfilePictureServiceModel.Picture }))[0] ??
-				throw new ArgumentException("Unable to upload profile picture to cloud");
-
-			bool successful = await this._userRepository.UpdateProfilePicture(updateProfilePictureServiceModel.UserId, fileUrl);
-
-			if (!successful)
-				throw new InvalidOperationException("Unable to change profile picture!");
-
-			return new ProfilePictureServiceModel() { ProfilePictureURL = fileUrl };
-		}
 		#endregion
 
 		#region Delete