From 8c952c08d0606f3e093eb6a409ce2bf12ea7e9b0 Mon Sep 17 00:00:00 2001
From: rr- <rr-@sakuya.pl>
Date: Sat, 11 Jun 2016 11:15:47 +0200
Subject: [PATCH] server/general: report bad field names to user

---
 server/szurubooru/func/util.py | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/server/szurubooru/func/util.py b/server/szurubooru/func/util.py
index 908c760..b413710 100644
--- a/server/szurubooru/func/util.py
+++ b/server/szurubooru/func/util.py
@@ -4,7 +4,7 @@ import hashlib
 import re
 import tempfile
 from contextlib import contextmanager
-from szurubooru.errors import ValidationError
+from szurubooru import errors
 
 def get_serialization_options(ctx):
     return ctx.get_param_as_list('_fields', required=False, default=None)
@@ -20,7 +20,8 @@ def serialize_entity(entity, field_factories, options):
             factory = field_factories[key]
             ret[key] = factory()
         except KeyError:
-            pass
+            raise errors.ValidationError('Invalid key: %r. Valid keys: %r.' % (
+                key, list(sorted(field_factories.keys()))))
     return ret
 
 @contextmanager
@@ -70,7 +71,7 @@ def parse_time_range(value, timezone=datetime.timezone(datetime.timedelta())):
 
     value = value.lower()
     if not value:
-        raise ValidationError('Empty date format.')
+        raise errors.ValidationError('Empty date format.')
 
     if value == 'today':
         now = datetime.datetime.now(tz=timezone)
@@ -110,7 +111,7 @@ def parse_time_range(value, timezone=datetime.timezone(datetime.timedelta())):
             datetime.datetime(year, month, day),
             datetime.datetime(year, month, day + 1) - one_second)
 
-    raise ValidationError('Invalid date format: %r.' % value)
+    raise errors.ValidationError('Invalid date format: %r.' % value)
 
 def icase_unique(source):
     target = []