From 755834607195e5216525c7865f4614f21b7311e1 Mon Sep 17 00:00:00 2001
From: rubenwardy <rw@rubenwardy.com>
Date: Wed, 23 Nov 2022 19:13:03 +0000
Subject: [PATCH] Don't allow users to recreate removed packages

---
 app/blueprints/packages/packages.py | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/app/blueprints/packages/packages.py b/app/blueprints/packages/packages.py
index ced9bcff..ba2d7a49 100644
--- a/app/blueprints/packages/packages.py
+++ b/app/blueprints/packages/packages.py
@@ -260,13 +260,14 @@ def handle_create_edit(package: typing.Optional[Package], form: PackageForm, aut
 		package = Package.query.filter_by(name=form["name"].data, author_id=author.id).first()
 		if package is not None:
 			if package.state == PackageState.DELETED:
-				package.review_thread_id = None
-				db.session.delete(package)
+				flash(
+					gettext("Package already exists, but is removed. Please contact ContentDB staff to restore the package"),
+					"danger")
 			else:
 				flash(Markup(
 					f"<a class='btn btn-sm btn-danger float-right' href='{package.getURL('packages.view')}'>View</a>" +
 					gettext("Package already exists")), "danger")
-				return None
+			return None
 
 		package = Package()
 		package.author = author