I'm looking at the patch now and I should have some feedback soon.
I wrote up a proposal for you here: http://cwiki.apache.org/confluence/display/ROLLER/Proposal+Email+Registration - Dave On 3/19/07, Dave <[EMAIL PROTECTED]> wrote:
Thanks Sedat. I'll hope to get to this today. - Dave On 3/18/07, sedat ciftci <[EMAIL PROTECTED]> wrote: > Hello again, > > I add a small but necessary code into the activateUser > method of UserNewAction.java for handling the user > account activation manually by site admin (site > administrator can manually activate the user account > by simply enabling the user from the 'User Admin' > menu). > > zip file that contains diff patch for current trunk: > http://jroller.com/resources/s/sedat/newRollerPropertyVer5.jar > > Note: I apologize for too many versions but I hope > this is the last version. > > Sedat Çiftçi > > --- sedat ciftci <[EMAIL PROTECTED]> wrote: > > > Hello, > > > > Here is the zip file that contains the patch to work > > with the current trunk: > > > http://www.jroller.com/resources/s/sedat/newRollerPropertyVer4.jar > > > > Note that this patch file also contains new added > > class (org.apache.roller.pojos.UserActivationData) > > > > Note about sandbox: > > > > I do not know anything about sandbox. I do not make > > any changes on the files related under the directory > > sandbox. > > > > Sedat Çiftçi > > > > --- Dave <[EMAIL PROTECTED]> wrote: > > > > > On 3/17/07, sedat ciftci <[EMAIL PROTECTED]> > > > wrote: > > > > Yes, you are right. I forget to include it. Here > > > is the updated zip file: > > > > > > > > > > http://www.jroller.com/resources/s/sedat/newRollerPropertyVer3.jar > > > > > > Sedat, > > > > > > I took a look at the patch this morning. > > > Unfortunately, it targets the > > > 3.0 code-base and enough has changed that most of > > > the changes cannot > > > be applied to trunk. > > > > > > Unfortunately, I don't have time right now to port > > > your patch from 3.0 > > > to trunk. But, if you are willing to update the > > > patch to work with the > > > current trunk, then I'm willing to evaluate it and > > > write-up a short > > > proposal so we can get it committed. > > > > > > - Dave > > > > > > > > > > > > > > ____________________________________________________________________________________ > > No need to miss a message. Get email on-the-go > > with Yahoo! Mail for Mobile. Get started. > > http://mobile.yahoo.com/mail > > > > > > > ____________________________________________________________________________________ > Don't get soaked. Take a quick peek at the forecast > with the Yahoo! Search weather shortcut. > http://tools.search.yahoo.com/shortcuts/#loc_weather >
