Mercurial > p > roundup > code
changeset 1196:ee204fb67798
better detection of unset required props
| author | Richard Jones <richard@users.sourceforge.net> |
|---|---|
| date | Tue, 24 Sep 2002 02:00:09 +0000 |
| parents | e0032f4ab334 |
| children | 7d1b69122133 |
| files | roundup/cgi/client.py |
| diffstat | 1 files changed, 17 insertions(+), 18 deletions(-) [+] |
line wrap: on
line diff
--- a/roundup/cgi/client.py Tue Sep 24 01:59:28 2002 +0000 +++ b/roundup/cgi/client.py Tue Sep 24 02:00:09 2002 +0000 @@ -1,4 +1,4 @@ -# $Id: client.py,v 1.40 2002-09-19 02:37:41 richard Exp $ +# $Id: client.py,v 1.41 2002-09-24 02:00:09 richard Exp $ __doc__ = """ WWW request handler (also used in the stand-alone server). @@ -1139,30 +1139,29 @@ if value: value = date.Date(form[key].value.strip()) else: - value = None + continue elif isinstance(proptype, hyperdb.Interval): if value: value = date.Interval(form[key].value.strip()) else: - value = None + continue elif isinstance(proptype, hyperdb.Link): # see if it's the "no selection" choice if value == '-1': - value = None - else: - # handle key values - link = proptype.classname - if not num_re.match(value): - try: - value = db.classes[link].lookup(value) - except KeyError: - raise ValueError, _('property "%(propname)s": ' - '%(value)s not a %(classname)s')%{'propname':key, - 'value': value, 'classname': link} - except TypeError, message: - raise ValueError, _('you may only enter ID values ' - 'for property "%(propname)s": %(message)s')%{ - 'propname':key, 'message': message} + continue + # handle key values + link = proptype.classname + if not num_re.match(value): + try: + value = db.classes[link].lookup(value) + except KeyError: + raise ValueError, _('property "%(propname)s": ' + '%(value)s not a %(classname)s')%{'propname':key, + 'value': value, 'classname': link} + except TypeError, message: + raise ValueError, _('you may only enter ID values ' + 'for property "%(propname)s": %(message)s')%{ + 'propname':key, 'message': message} elif isinstance(proptype, hyperdb.Multilink): if isinstance(value, type([])): # it's a list of MiniFieldStorages
