Mike Kienenberger said: > Nathan Bubna <[EMAIL PROTECTED]> wrote: > > that looks an awful lot like business logic > > you're slipping into the View. if you ask me, that's just asking for > trouble > > down the road. can't you move some of that logic into your > "currentAccount" > > I ran into this error while converting some old code. > I already knew that some refactoring would be required, and the code has > already been changed to the following immediately after reporting the bug, > which made the bug a moot point for me. > > #if ( ($currentAccount.hasAmountDue()) && > ($currentAccount.isAccountStatusActive()) && > ($currentAccount.hasScheduledPaymentList()) )
yeah, that's better. > But the bug for the original statement still remains. It's beyond "sucky." > It's internally inconsistent. huh? the bug (and a huge one at that) was in *your* template's syntax (the "BigDecimal" nonsense), not in velocity. my "sucky" comment referred only to the usefulness of the error message. i don't get how that is "internally inconsistent." please clarify. Nathan Bubna [EMAIL PROTECTED] --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
