From 02a62532dab5ad77a90c08dc66fc58ececcbb268 Mon Sep 17 00:00:00 2001 From: Kevin Morris Date: Mon, 13 Dec 2021 16:34:17 -0800 Subject: [PATCH] fix(python): fix difference parsing of comaintainers Signed-off-by: Kevin Morris --- aurweb/routers/packages.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/aurweb/routers/packages.py b/aurweb/routers/packages.py index eb4cab74..76f82ffb 100644 --- a/aurweb/routers/packages.py +++ b/aurweb/routers/packages.py @@ -512,10 +512,10 @@ async def package_base_comaintainers_post( return RedirectResponse(f"/pkgbase/{name}", status_code=HTTPStatus.SEE_OTHER) - users = set(users.split("\n")) - users.remove(str()) # Remove any empty strings from the set. + users = {e.strip() for e in users.split("\n") if bool(e.strip())} records = {c.User.Username for c in pkgbase.comaintainers} + logger.debug(f"RemoveComaintainers: {records.difference(users)}") pkgutil.remove_comaintainers(pkgbase, records.difference(users)) # Default priority (lowest value; most preferred). @@ -533,6 +533,7 @@ async def package_base_comaintainers_post( if last_priority: priority = last_priority.Priority + 1 + logger.debug(f"AddComaintainers: {users.difference(records)}") error = pkgutil.add_comaintainers(request, pkgbase, priority, users.difference(records)) if error: