aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--app/blueprints/packages/screenshots.py4
-rw-r--r--app/tasks/importtasks.py3
2 files changed, 4 insertions, 3 deletions
diff --git a/app/blueprints/packages/screenshots.py b/app/blueprints/packages/screenshots.py
index 5aec9ae..cb21a56 100644
--- a/app/blueprints/packages/screenshots.py
+++ b/app/blueprints/packages/screenshots.py
@@ -29,13 +29,13 @@ from wtforms.validators import *
class CreateScreenshotForm(FlaskForm):
- title = StringField("Title/Caption", [Optional()])
+ title = StringField("Title/Caption", [Optional(), Length(-1, 100)])
fileUpload = FileField("File Upload", [InputRequired()])
submit = SubmitField("Save")
class EditScreenshotForm(FlaskForm):
- title = StringField("Title/Caption", [Optional()])
+ title = StringField("Title/Caption", [Optional(), Length(-1, 100)])
approved = BooleanField("Is Approved")
delete = BooleanField("Delete")
submit = SubmitField("Save")
diff --git a/app/tasks/importtasks.py b/app/tasks/importtasks.py
index e0ee044..b0542ee 100644
--- a/app/tasks/importtasks.py
+++ b/app/tasks/importtasks.py
@@ -226,10 +226,11 @@ def postReleaseCheckUpdate(self, release, path):
return tree
except MinetestCheckError as err:
+ db.session.rollback()
+
if "Fails validation" not in release.title:
release.title += " (Fails validation)"
- db.session.rollback()
release.task_id = self.request.id
release.approved = False
db.session.commit()