Mercurial > p > roundup > code
changeset 1224:e58a59b2a290
reversion - allow unsetting of Links again
| author | Richard Jones <richard@users.sourceforge.net> |
|---|---|
| date | Thu, 26 Sep 2002 03:21:03 +0000 |
| parents | 3232e3f11a23 |
| children | eae6a5eb62e0 |
| files | roundup/cgi/client.py |
| diffstat | 1 files changed, 16 insertions(+), 15 deletions(-) [+] |
line wrap: on
line diff
--- a/roundup/cgi/client.py Thu Sep 26 03:04:42 2002 +0000 +++ b/roundup/cgi/client.py Thu Sep 26 03:21:03 2002 +0000 @@ -1,4 +1,4 @@ -# $Id: client.py,v 1.44 2002-09-25 05:27:07 richard Exp $ +# $Id: client.py,v 1.45 2002-09-26 03:21:03 richard Exp $ __doc__ = """ WWW request handler (also used in the stand-alone server). @@ -1133,20 +1133,21 @@ elif isinstance(proptype, hyperdb.Link): # see if it's the "no selection" choice if value == '-1': - 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} + 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} elif isinstance(proptype, hyperdb.Multilink): if isinstance(value, type([])): # it's a list of MiniFieldStorages