Re: Ping: Re: [patch] FIx loan druid when auto_decimal is enabled

2007-03-28 Thread Jerry Quinn (ibm)
Derek Atkins wrote: > > Well, I could (probably) apply the patch, but I'm not confortable > doing so because I don't understand the underlying code. > > >> Can you attach the patch to the bug, please? >> > > Definitely a good idea... Um, what bug# is it? > The bug is #343795. I attache

Re: Ping: Re: [patch] FIx loan druid when auto_decimal is enabled

2007-03-28 Thread Derek Atkins
"Josh Sled" <[EMAIL PROTECTED]> writes: >>> What else would you like me to do? >> >> Convince JSled to look at it? He knows that code better than I do. >> The patch looks okay to me, but I don't understand the code.. > > I'd be happy to, but I'm out of town on business this week, and am not > rea

Re: Ping: Re: [patch] FIx loan druid when auto_decimal is enabled

2007-03-27 Thread Josh Sled
On Tue, March 27, 2007 8:05 am, Derek Atkins wrote: > Jerry Quinn <[EMAIL PROTECTED]> writes: >> Derek Atkins wrote: >>> I haven't. I don't know if Josh has. Has anyone tested this patch? >>> Have you? Does it work? >> >> It works as expected for me when I test it, both with auto-decimal >> enab

Re: Ping: Re: [patch] FIx loan druid when auto_decimal is enabled

2007-03-27 Thread Derek Atkins
Jerry Quinn <[EMAIL PROTECTED]> writes: > Derek Atkins wrote: >> I haven't. I don't know if Josh has. Has anyone tested this patch? >> Have you? Does it work? > > It works as expected for me when I test it, both with auto-decimal enabled > and > disabled. > > What else would you like me to do

Re: Ping: Re: [patch] FIx loan druid when auto_decimal is enabled

2007-03-26 Thread Jerry Quinn
Derek Atkins wrote: > I haven't. I don't know if Josh has. Has anyone tested this patch? > Have you? Does it work? It works as expected for me when I test it, both with auto-decimal enabled and disabled. What else would you like me to do? Thanks, Jerry __

Re: Ping: Re: [patch] FIx loan druid when auto_decimal is enabled

2007-03-26 Thread Derek Atkins
I haven't. I don't know if Josh has. Has anyone tested this patch? Have you? Does it work? -derek Jerry Quinn <[EMAIL PROTECTED]> writes: > Hi, all. Has anyone had a chance to look at this patch yet? > > Thanks, > Jerry Quinn > > > Jerry Quinn wrote: >> Hi, folks. >> >> This patch fixes #34

Ping: Re: [patch] FIx loan druid when auto_decimal is enabled

2007-03-23 Thread Jerry Quinn
Hi, all. Has anyone had a chance to look at this patch yet? Thanks, Jerry Quinn Jerry Quinn wrote: > Hi, folks. > > This patch fixes #343795 for me. > > This patch makes the loan druid generate correct loan amortization numbers > when auto_decimal is turned on. The problem is that the formu