diff options
| author | rubenwardy <rw@rubenwardy.com> | 2020-08-19 13:06:21 +0100 |
|---|---|---|
| committer | rubenwardy <rw@rubenwardy.com> | 2020-08-19 13:06:21 +0100 |
| commit | 934d5817376a072cf97e69d4dfa925cdbd466464 (patch) | |
| tree | 6d8e9759e5277886e707b8637e0ce5c386f8cb37 | |
| parent | e85d1755f058e661e451f0ab77401f820d0c836e (diff) | |
| download | cheatdb-934d5817376a072cf97e69d4dfa925cdbd466464.tar.xz | |
Fix screenshot form not validating length
| -rw-r--r-- | app/blueprints/packages/screenshots.py | 4 | ||||
| -rw-r--r-- | app/tasks/importtasks.py | 3 |
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() |