cmpilato@collab.net wrote:
>
> Blair Zajac <blair@orcaware.com> writes:
>
> > Can somebody with some Python fu check this? This sets the from address
> > to the revision author if no from_addr is set or it is set like this
> > from_addr =
> >
> > I'm not certain about the == None and == "" tests.
> >
> > Blair
> >
> >
> > Index: mailer.py
> > ===================================================================
> > --- mailer.py (revision 5592)
> > +++ mailer.py (working copy)
> > @@ -157,6 +157,10 @@
> > def start(self, group, params):
> > self.to_addr = self.cfg.get('to_addr', group, params)
> > self.from_addr = self.cfg.get('from_addr', group, params)
> > + if self.from_addr == None or self.from_addr == "":
> > + self.from_addr = self.repos.author
> > + if self.from_addr == None or self.from_addr == "":
> > + self.from_addr = 'no_author'
> > self.reply_to = self.cfg.get('reply_to', group, params)
>
> If I'm understanding the module docs correctly, the self.cfg.get()
> will throw a NoOptionError if the option doesn't exist at all.
> Presumably this is what you want to check for (as well as the ""
> case).
That doesn't appear to be the case. I just tried commenting out all
from_addr and there's no exception.
Blair
--
Blair Zajac <blair@orcaware.com>
Plots of your system's performance - http://www.orcaware.com/orca/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Apr 9 05:39:55 2003