Verified Commit b4092fe7 authored by Kevin Morris's avatar Kevin Morris
Browse files

fix(fastapi): pass request type's name to Request*Notification

Previously, we passed the straight up request type instance from
SQLAlchemy and had a .title() function that was transparently
treating the instance the same as the instance's Name in terms
of's use of it.

This commit removes that transparent behavior; it was not actually

Signed-off-by: Kevin Morris's avatarKevin Morris <>
parent 3b28be17
Pipeline #12307 passed with stage
in 6 minutes and 27 seconds
......@@ -17,14 +17,7 @@ class RequestType(Base):
def name_display(self) -> str:
""" Return the Name column with its first char capitalized. """
name = self.Name
return name[0].upper() + name[1:]
def title(self) -> str:
return self.name_display()
def __getitem__(self, n: int) -> str:
return self.Name[n]
return self.Name.title()
DELETION_ID = db.query(RequestType, RequestType.Name == DELETION).first().ID
......@@ -697,18 +697,20 @@ async def pkgbase_request_post(request: Request, name: str,
now = int(datetime.utcnow().timestamp())
reqtype = db.query(models.RequestType).filter(
models.RequestType.Name == type).first()
conn = db.ConnectionExecutor(db.get_engine().raw_connection())
notify_ = None
with db.begin():
pkgreq = db.create(models.PackageRequest, RequestType=reqtype,
RequestTS=now, PackageBase=pkgbase,
pkgreq = db.create(models.PackageRequest,
MergeBaseName=merge_into, User=request.user,
Comments=comments, ClosureComment=str())
# Prepare notification object.
conn = db.ConnectionExecutor(db.get_engine().raw_connection())
notify_ = notify.RequestOpenNotification(
conn, request.user.ID, pkgreq.ID, reqtype,
conn, request.user.ID, pkgreq.ID, reqtype.Name,
pkgreq.PackageBase.ID, merge_into=merge_into or None)
# Send the notification now that we're out of the DB scope.
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment