From 51124ec80b2cbac408699ba5622b18c40b960924 Mon Sep 17 00:00:00 2001 From: John Donaldson Date: Thu, 15 Aug 2019 00:07:22 +0100 Subject: [PATCH] Do better checking for fields that may be empty or unset --- devel/validate.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/devel/validate.py b/devel/validate.py index 404d196..d73dd89 100644 --- a/devel/validate.py +++ b/devel/validate.py @@ -71,7 +71,9 @@ class MothValidator: def check_fields(self, puzzle): """Check if the puzzle has the requested fields""" for field in self.required_fields: - if not hasattr(puzzle, field): + if not hasattr(puzzle, field) or \ + getattr(puzzle,field) is None or \ + getattr(puzzle,field) == "": raise MothValidationError("Missing field %s" % (field,)) @staticmethod