Mercurial Repository: p/roundup/code: roundup/mailgw.py comparison

comparison roundup/mailgw.py @ 537:ad6dbc21a570

[SF#503340] creating issue with [asignedto=p.ohly]

author	Richard Jones <richard@users.sourceforge.net>
date	Tue, 15 Jan 2002 00:12:40 +0000
parents	dce4c75bef5a
children	d32977eb8bd4

comparison

equal deleted inserted replaced

-:8298fea69bc3
+:ad6dbc21a570
 set() method to add the message to the item's spool; in the second case we
 are calling the create() method to create a new node). If an auditor raises
 an exception, the original message is bounced back to the sender with the
 explanatory message given in the exception.
-$Id: mailgw.py,v 1.51 2002-01-14 02:20:15 richard Exp $
+$Id: mailgw.py,v 1.52 2002-01-15 00:12:40 richard Exp $
 '''
 import string, re, os, mimetools, cStringIO, smtplib, socket, binascii, quopri
 import time, random
 # handle the args
 args = m.group('args')
 if args:
 for prop in string.split(args, ';'):
+# extract the property name and value
 try:
 key, value = prop.split('=')
 except ValueError, message:
 raise MailUsageError, '''
 Subject argument list not of form [arg=value,value,...;arg=value,value...]
 (specific exception message was "%s")
 Subject was: "%s"
 '''%(message, subject)
+# ensure it's a valid property name
 key = key.strip()
 try:
 proptype =  properties[key]
 except KeyError:
 raise MailUsageError, '''
 Subject argument list refers to an invalid property: "%s"
 Subject was: "%s"
 '''%(key, subject)
+# convert the string value to a real property value
 if isinstance(proptype, hyperdb.String):
 props[key] = value.strip()
 if isinstance(proptype, hyperdb.Password):
 props[key] = password.Password(value.strip())
 elif isinstance(proptype, hyperdb.Date):
 Subject was: "%s"
 '''%(key, message, subject)
 elif isinstance(proptype, hyperdb.Link):
 link = self.db.classes[proptype.classname]
 propkey = link.labelprop(default_to_id=1)
-try:
+props[key] = value
-props[key] = link.get(value.strip(), propkey)
-except:
-props[key] = link.lookup(value.strip())
 elif isinstance(proptype, hyperdb.Multilink):
-link = self.db.classes[proptype.classname]
+# get the linked class
-propkey = link.labelprop(default_to_id=1)
+linkcl = self.db.classes[proptype.classname]
-l = [x.strip() for x in value.split(',')]
+propkey = linkcl.labelprop(default_to_id=1)
-for item in l:
+for item in value.split(','):
-try:
+item = item.split()
-v = link.get(item, propkey)
-except:
-v = link.lookup(item)
 if props.has_key(key):
-props[key].append(v)
+props[key].append(item)
 else:
-props[key] = [v]
+props[key] = [item]
 #
 # handle the users
 #
 props['nosy'].append(author)
 n[author] = 1
 # add assignedto to the nosy list
 if properties.has_key('assignedto') and props.has_key('assignedto'):
-try:
+assignedto = props['assignedto']
-assignedto = self.db.user.lookup(props['assignedto'])
+if not re.match('^\d+$', assignedto):
-except KeyError:
+try:
-raise MailUsageError, '''
+assignedto = self.db.user.lookup(assignedto)
+except KeyError:
+raise MailUsageError, '''
 There was a problem with the message you sent:
 Assignedto user '%s' doesn't exist
 '''%props['assignedto']
 if not n.has_key(assignedto):
 props['nosy'].append(assignedto)
 l.append(section)
 return summary, '\n\n'.join(l)
 #
 # $Log: not supported by cvs2svn $
+# Revision 1.51  2002/01/14 02:20:15  richard
+#  . changed all config accesses so they access either the instance or the
+#    config attriubute on the db. This means that all config is obtained from
+#    instance_config instead of the mish-mash of classes. This will make
+#    switching to a ConfigParser setup easier too, I hope.
+#
+# At a minimum, this makes migration a _little_ easier (a lot easier in the
+# 0.5.0 switch, I hope!)
+#
 # Revision 1.50  2002/01/11 22:59:01  richard
 #  . #502342 ] pipe interface
 #
 # Revision 1.49  2002/01/10 06:19:18  richard
 # followup lines directly after a quoted section were being eaten.

Mercurial > p > roundup > code

comparison roundup/mailgw.py @ 537:ad6dbc21a570