Merge pull request #481 from ederag/forbid-unparseable-take2

Forbid unparseable facts
projecthamster · Nov 23, 2019 · 556d31a · 556d31a
2 parents 660fc55 + c3da6a2
commit 556d31a
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/hamster/edit_activity.py b/src/hamster/edit_activity.py
@@ -356,6 +356,11 @@ def validate_fields(self):
             self.update_status(status="warning", markup=markup)
             return fact
 
+        roundtrip_fact = Fact.parse(fact.serialized())
+        if roundtrip_fact != fact:
+            self.update_status(status="wrong", markup="Fact could not be parsed back")
+            return None
+
         # nothing unusual
         self.update_status(status="looks good", markup="")
         return fact