Allow users to delete profile photos

marvac · Jan 24, 2019 · 3509a2b · 3509a2b
1 parent 6d632ab
commit 3509a2b
Show file tree

Hide file tree

Showing 4 changed files with 58 additions and 1 deletion.
diff --git a/Client/src/app/components/members/photo-editor/photo-editor.component.html b/Client/src/app/components/members/photo-editor/photo-editor.component.html
@@ -8,7 +8,7 @@
               [ngClass]="photo.isMain ? 'btn-success active' : 'btn-secondary'"
               [disabled]="photo.isMain">Main</button>
 
-      <button type="button" class="btn btn-xs btn-danger"><i class="fa fa-trash-o"></i></button>
+      <button type="button" class="btn btn-xs btn-danger" (click)="deletePhoto(photo.id)" [disabled]="photo.isMain"><i class="fa fa-trash-o"></i></button>
     </div>
   </div>
 </div>

diff --git a/Client/src/app/components/members/photo-editor/photo-editor.component.ts b/Client/src/app/components/members/photo-editor/photo-editor.component.ts
@@ -74,4 +74,17 @@ export class PhotoEditorComponent implements OnInit {
       })
   }
 
+  deletePhoto(photoId: number) {
+    this.alertify.confirm("Are you sure you want to delete this photo?", () => {
+      const userId = this.authService.decodedToken.nameid;
+      this.userService.deletePhoto(userId, photoId)
+        .subscribe(() => {
+          const index = this.photos.findIndex(p => p.id == photoId);
+          this.photos.splice(index, 1);
+          this.alertify.success("Photo deleted");
+        }, error => {
+          this.alertify.error(error);
+        })
+    });
+  }
 }
diff --git a/Client/src/app/services/user.service.ts b/Client/src/app/services/user.service.ts
@@ -27,4 +27,8 @@ export class UserService {
   setMainPhoto(userId: number, photoId: number) {
     return this.http.post(`${this.usersUrl}${userId}/photos/${photoId}/setMain`, {});
   }
+
+  deletePhoto(userId: number, photoId: number) {
+    return this.http.delete(`${this.usersUrl}${userId}/photos/${photoId}`);
+  }
 }
diff --git a/Controllers/PhotosController.cs b/Controllers/PhotosController.cs
@@ -141,5 +141,45 @@ public async Task<ActionResult> SetMainPhoto(int userId, int photoId)
 
             return BadRequest("Something went wrong");
         }
+
+        [HttpDelete("{photoId}")]
+        public async Task<IActionResult> DeletePhoto(int userId, int photoId)
+        {
+            int id = int.Parse(User.FindFirst(ClaimTypes.NameIdentifier).Value);
+
+            if (id != userId)
+            {
+                return Unauthorized();
+            }
+
+            var user = await _repo.GetUser(userId);
+
+            if (!user.Photos.Any(x => x.Id == photoId))
+            {
+                return Unauthorized();
+            }
+
+            var photo = await _repo.GetPhoto(photoId);
+
+            if (photo.IsMain)
+            {
+                return BadRequest("Cannot delete main photo");
+            }
+
+            if (!string.IsNullOrWhiteSpace(photo.PublicId))
+            {
+                DeletionParams deletionParams = new DeletionParams(photo.PublicId);
+                var deleteResult = _cloudAccount.Destroy(deletionParams);
+            }
+
+            _repo.Delete(photo);
+
+            if (await _repo.SaveChangesAsync())
+            {
+                return Ok();
+            }
+
+            return BadRequest("Something went wrong");
+        }
     }
 }