Skip to content

Commit 40a0fb5

Browse files
fix email conflict bug
1 parent d3eaa29 commit 40a0fb5

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

app/server/routes/admin.py

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,11 @@
33
from fastapi.security import HTTPBasicCredentials
44
from passlib.context import CryptContext
55

6-
from app.server.database.database import admin_collection
6+
from server.database.database import admin_collection
77
#from app.server.auth.admin import validate_login
8-
from app.server.auth.jwt_handler import signJWT
9-
from app.server.database.database import add_admin
10-
from app.server.models.admin import AdminModel
8+
from server.auth.jwt_handler import signJWT
9+
from server.database.database import add_admin
10+
from server.models.admin import AdminModel
1111

1212
router = APIRouter()
1313

@@ -37,9 +37,10 @@ async def admin_login(admin_credentials: HTTPBasicCredentials = Body(...)):
3737

3838
@router.post("/")
3939
async def admin_signup(admin: AdminModel = Body(...)):
40-
admin = await admin_collection.find_one({"email": admin.email}, {"_id": 0})
41-
if(admin):
40+
admin_exists = await admin_collection.find_one({"email": admin.email}, {"_id": 0})
41+
if(admin_exists):
4242
return "Email already exists"
43+
4344
admin.password = hash_helper.encrypt(admin.password)
4445
new_admin = await add_admin(jsonable_encoder(admin))
4546
return new_admin

0 commit comments

Comments
 (0)