[prev] [thread] [next] [lurker] [Date index for 2003/10/28]
Author: richardc
Date: 2003-10-28 12:30:24 +0000 (Tue, 28 Oct 2003)
New Revision: 1468
Removed:
trunk/Siesta-Plugin-DeliveryAddress/
Log:
This code just doesn't do what it's meant to, and to do that would require more
than just one weak plugin.
Since this handgrenade was tossed in via irc, please pay note to this
transcript:
11:21 <@richardc> you know, that's a fucking retarded idea
11:22 <@muttley> because we'll get bounces and we won't know who they're coming
from?
11:23 <@richardc> yup, plus we're not verifying that those addresses are owned
by the user who signed them up
11:25 <@muttley> good point
11:25 <@muttley> so perhaps a member has may addresses relationship might work
11:26 <@muttley> s/may/many/
11:26 <@richardc> you mean like was planned?
11:27 <@richardc> it still needs extra shit to nominate which one is used for
which list
11:27 <@muttley> yeah
11:57 <@richardc> oh btw. that module wouldn't do what jody wants it to
11:58 <@richardc> that just munges the To: header, which isn't right
11:59 <@richardc> what he wants is for delivery to be to a different address
11:59 <@muttley> hmm
11:59 <@muttley> you are indeed, as ever, a source of truth
11:59 <@muttley> I'll remove it from svn
12:00 <@richardc> what it should do is change $user->email
12:00 <@richardc> so that when Siesta::Plugin::Send actaully sends it out, it
goes to the right place
12:01 <@muttley> doesn't that run the risk of permently changing the user's
email address?
12:01 <@richardc> (and just hope nothing commits the change)
12:01 <@richardc> yarr, indeed it does
So in summary, no, fuck no, no.
Next time, make change requests via the mailing list, or with a patch.
Generated at 13:57 on 01 Jul 2004 by mariachi 0.52