Merge pull request #157 from SCCapstone/delete-pfp

delete pfp
SCCapstone · Mar 31, 2024 · e28ea5c · e28ea5c
2 parents d036f82 + 0d1b7fd
commit e28ea5c
Show file tree

Hide file tree

Showing 4 changed files with 68 additions and 14 deletions.
diff --git a/backend/app/account_settings_module/controller.py b/backend/app/account_settings_module/controller.py
@@ -152,3 +152,18 @@ def get_profile_picture_controller():
     except:
         return dict(error='Error Getting Profile Picture'), HTTPStatus.INTERNAL_SERVER_ERROR
 
+def delete_profile_picture_controller():
+    try:
+        # Get the profile picture for the user
+        profile = Profile.query.get(current_user.email)
+
+        if not profile:
+            return dict(error='Profile picture not found'), HTTPStatus.NOT_FOUND
+
+        # Delete the profile picture from the database
+        db.session.delete(profile)
+        db.session.commit()
+
+        return dict(mssg='Profile Picture Deleted Successfully!'), HTTPStatus.OK
+    except:
+        return dict(error='Error Deleting Profile Picture'), HTTPStatus.INTERNAL_SERVER_ERROR
diff --git a/backend/app/account_settings_module/routes.py b/backend/app/account_settings_module/routes.py
@@ -5,7 +5,8 @@
 from app.account_settings_module.controller import (
     update_password_controller,
     update_profile_picture_controller,
-    get_profile_picture_controller
+    get_profile_picture_controller,
+    delete_profile_picture_controller
 )
 
 @blueprint.route('/update_password', methods=['POST'])
@@ -21,4 +22,9 @@ def update_profile_picture():
 @blueprint.route('/get_profile_picture', methods=['GET'])
 @login_required
 def get_profile_picture():
-    return get_profile_picture_controller()
+    return get_profile_picture_controller()
+
+@blueprint.route('/delete_profile_picture', methods=['DELETE'])
+@login_required
+def delete_profile_picture():
+    return delete_profile_picture_controller()
diff --git a/frontend/app/src/config.js b/frontend/app/src/config.js
@@ -39,6 +39,7 @@ module.exports = global.config = {
 
     UPDATE_PROFILE_PICTURE_URL: BASE_URL + "update_profile_picture",
     GET_PROFILE_PICTURE_URL: BASE_URL + "get_profile_picture",
+    DELETE_PROFILE_PICTURE_URL: BASE_URL + "delete_profile_picture",
 
     USER_CREATION_URL: BASE_URL + "signup",
     MANUAL_USER_CREATION_URL: BASE_URL + "manual_signup",

diff --git a/frontend/app/src/pages/AccountSettings/AccountSettings.js b/frontend/app/src/pages/AccountSettings/AccountSettings.js
@@ -1,5 +1,5 @@
 import React, { useState, useEffect } from 'react';
-import { UPDATE_PASSWORD_URL, UPDATE_PROFILE_PICTURE_URL, DELETE_EVALS_URL, DELETE_ALL_GRANTS_URL, DELETE_ALL_PUBS_URL, DELETE_ALL_EXPENS_URL, GET_PROFILE_PICTURE_URL } from '../../config';
+import { UPDATE_PASSWORD_URL, UPDATE_PROFILE_PICTURE_URL, DELETE_EVALS_URL, DELETE_ALL_GRANTS_URL, DELETE_ALL_PUBS_URL, DELETE_ALL_EXPENS_URL, GET_PROFILE_PICTURE_URL, DELETE_PROFILE_PICTURE_URL } from '../../config';
 import { useAuthContext } from '../../hooks/useAuthContext'
 import { useCourseAnalyticsContext } from '../../hooks/useCourseAnalyticsContext';
 import { useStudentEvalsContext } from '../../hooks/useStudentEvalsContext';
@@ -52,6 +52,8 @@ const AccountSettings = () => {
     const [pictureFile, setPictureFile] = useState()
     const [pictureProcessing, setPictureProcessing] = useState(false)
 
+    const [deletePictureError, setDeletePictureError] = useState(null)
+
     const updatePassword = async (e) => {
         e.preventDefault()
 
@@ -159,6 +161,28 @@ const AccountSettings = () => {
       }
     }
 
+    const handleDeletePicture = async (e) => {
+      const alertResponse = window.confirm("Are you sure you want to delete all student evaluation data? This cannot be undone.");
+      if (!alertResponse) {
+        return
+      }
+      e.preventDefault()
+      const response = await fetch(DELETE_PROFILE_PICTURE_URL, {
+        method: 'DELETE',
+        credentials: 'include',
+        headers: {'Content-Type': 'application/json'}
+      })
+      const json = await response.json()
+      if (!response.ok) {
+        setDeletePictureError(json.error)
+      }
+      if (response.ok) {
+        setDeletePictureError(null)
+        userDispatch({type: 'SET_PROFILE_PICTURE_URL', payload: null})
+        navigate('/account-settings', { state: { mssg: 'Profile picture deleted successfully', status: 'ok' }})
+      }
+    }
+
     const deleteAllEvals = async (e) => {
       e.preventDefault()
       if (deleteEvalsConfirm !== confirmation_text) {
@@ -297,17 +321,6 @@ const AccountSettings = () => {
     return (
         <>
           <h1 className="accountSettingsPageHeader">Account Settings</h1>
-          <section className="updatePasswordCard">
-            <h1>Update Profile Picture</h1>
-            <label>Please ensure your image is square for the best results</label>
-            <form onSubmit={handleSubmitPicture} className="evalupload-form">
-                <input type="file" onChange={handleChangePicture} className="evalupload-form-input" />
-                <button type="submit" className="evalupload-form-button">Upload</button>
-                {pictureError && <div className="errorField">{ pictureError }</div>}
-                {pictureProcessing && <p>Processing...</p>}
-            </form>
-          </section>
-
 
           <section className="updatePasswordCard">
             <h1>Update Password</h1>
@@ -331,6 +344,25 @@ const AccountSettings = () => {
             </form>
           </section>
 
+          <section className="updatePasswordCard">
+            <h1>Update Profile Picture</h1>
+            <label>Please ensure your image is square for the best results</label>
+            <form onSubmit={handleSubmitPicture} className="evalupload-form">
+                <input type="file" onChange={handleChangePicture} className="evalupload-form-input" />
+                <button type="submit" className="evalupload-form-button">Upload</button>
+                {pictureError && <div className="errorField">{ pictureError }</div>}
+                {pictureProcessing && <p>Processing...</p>}
+            </form>
+          </section>
+
+          <section className="updatePasswordCard">
+            <h1>Delete Profile Picture</h1>
+            <form onSubmit={handleDeletePicture} className="evalupload-form">
+                <button type="submit" className="evalupload-form-button">Delete Profile Picture</button>
+                {deletePictureError && <div className="errorField">{ deletePictureError }</div>}
+            </form>
+          </section>
+
           <section className="updatePasswordCard">
             <h1>Delete All Grants</h1>
             <form className="updatePassword" onSubmit={ deleteAllGrants  }>