From 08479c1134f24ac3605668b76b771b05ebd4a7a4 Mon Sep 17 00:00:00 2001
From: Luke Else <mail@luke-else.co.uk>
Date: Fri, 5 Jan 2024 14:11:46 +0000
Subject: [PATCH] Removed the ability for duplicate accounts being able to be
 made

---
 controllers/database/database.py | 4 ++++
 controllers/web/endpoints.py     | 7 +++++++
 2 files changed, 11 insertions(+)

diff --git a/controllers/database/database.py b/controllers/database/database.py
index bfa910e..69a120c 100644
--- a/controllers/database/database.py
+++ b/controllers/database/database.py
@@ -23,6 +23,10 @@ class DatabaseController(ABC):
             setattr(obj, attr, value)
         return obj
     
+    """
+        Set of CRUD methods to allow for Data manipulation on the backend
+    """
+
     @abstractmethod
     def create(self):
         pass
diff --git a/controllers/web/endpoints.py b/controllers/web/endpoints.py
index 4585b36..f30035f 100644
--- a/controllers/web/endpoints.py
+++ b/controllers/web/endpoints.py
@@ -49,6 +49,12 @@ def display_signup(error: str = None):
 @blueprint.post('/signup')
 def signup():
     database = UserController()
+
+    # User already exists
+    if database.read(request.form['username']) != None:
+        error = "User, " + request.form['username'] + " already exists"
+        return display_signup(error)
+
     database.create(Customer(
         0,
         request.form['username'],
@@ -64,6 +70,7 @@ def signup():
     return redirect("/login", code=307)
 
 
+### SIGN OUT FUNCTIONALITY
 # Function responsible for handling logouts from the site
 @blueprint.route('/logout')
 def logout():