mirror of
https://github.com/minetest/contentdb.git
synced 2025-01-09 22:47:36 +01:00
Fix crash when creating collection
This commit is contained in:
parent
eb81674f06
commit
aaee730ba5
@ -152,8 +152,9 @@ def handle_create_edit(collection: Collection, form: CollectionForm,
|
|||||||
|
|
||||||
if collection is None:
|
if collection is None:
|
||||||
collection = Collection()
|
collection = Collection()
|
||||||
collection.name = name
|
|
||||||
collection.author = author
|
collection.author = author
|
||||||
|
form.populate_obj(collection)
|
||||||
|
collection.name = name
|
||||||
db.session.add(collection)
|
db.session.add(collection)
|
||||||
|
|
||||||
if initial_package:
|
if initial_package:
|
||||||
@ -168,6 +169,9 @@ def handle_create_edit(collection: Collection, form: CollectionForm,
|
|||||||
collection.get_url("collections.view"), None)
|
collection.get_url("collections.view"), None)
|
||||||
|
|
||||||
else:
|
else:
|
||||||
|
form.populate_obj(collection)
|
||||||
|
collection.name = name
|
||||||
|
|
||||||
for i, package_id in enumerate(form.package_ids):
|
for i, package_id in enumerate(form.package_ids):
|
||||||
item = next((x for x in collection.items if str(x.package.id) == package_id.data), None)
|
item = next((x for x in collection.items if str(x.package.id) == package_id.data), None)
|
||||||
if item is None:
|
if item is None:
|
||||||
@ -179,7 +183,6 @@ def handle_create_edit(collection: Collection, form: CollectionForm,
|
|||||||
f"Edited collection {collection.author.username}/{collection.name}",
|
f"Edited collection {collection.author.username}/{collection.name}",
|
||||||
collection.get_url("collections.view"), None)
|
collection.get_url("collections.view"), None)
|
||||||
|
|
||||||
form.populate_obj(collection)
|
|
||||||
db.session.commit()
|
db.session.commit()
|
||||||
return redirect(collection.get_url("collections.view"))
|
return redirect(collection.get_url("collections.view"))
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user