loan/mortgage repayment via sched xactions: feedback request

[Josh Sled]

On Wed, Aug 14, 2002 at 06:04:13PM -0400, Derek Atkins wrote:

| A bunch of issues (this is rather long)...

Excellent!  I've listed the Bug#'s below; add yourself to the CC list if
you like...

| 1) I tried the druid with an empty account-tree, and I got a bunch of
|    warnings:

I never tried this; I should have.  I'll check it out. [Bug#90779]

| 2) The Interest Rate on the first page does not give enough decimal
|    places.  I wanted to type in "6.375" but it doesn't let me.  Worse,
|    it appears to be rounding my entry to the nearest .05% which is
|    "poor".

Noted.  My mistake.  Will fix soon. [Bug#90780]

| 3) It would be nice if the "periods remaining" defaulted to the whole

Yup. [Bug#90785]

| 4) on page 2: If I click on "... pay "Taxes"?", it auto-selects the
|    "via Escrow account"..  If I deselect and select again, it does not
|    do this.  Appears to be an initialization issue.  Same thing for
|    Insurance, and PMI, but "Other Expense" does not appear to be
|    affected.

That's true.  I have a static structure [array] from which I init
the druid's structure of what's going on in the druid... and the Other
Expense has a different default value [FALSE] than the others ... so it's
exactly that.  You're saying if the user doesn't change the selection,
it should have the same default behavior?  That seems pretty reasonable.

| 5) Why is there a "finish" on the "We're about to do the following"
|    page?

Because I only figured out how to DTRT for those controls in Druids after
I committed that. ;)  In fact, "finish" will appear on the last page,
and the FinishPage will go away.

[Bug#90781]

| 6) I clicked on "finish" here and received the following:
| 
| ** CRITICAL **: file SX-ttinfo.c: line 340 (gnc_ttsplitinfo_set_account): assertion `ttsi && acc' failed.
| 
| ** CRITICAL **: file SX-ttinfo.c: line 340 (gnc_ttsplitinfo_set_account): assertion `ttsi && acc' failed.
| 
| ** CRITICAL **: file SX-ttinfo.c: line 340 (gnc_ttsplitinfo_set_account): assertion `ttsi && acc' failed.
| Warning: xaccSchedXactionSetRemOccu...(): The number remaining is greater than the total occurrences

Cool...

So, I'm going to guess that since you didn't have any accounts in the tree,
you didn't select any principal/interest/escrow/&c. accounts in the Druid
... which explains the assertion failures...  I guess the Druid should
enforce that _some_ valid account is selected... [Bug#90784].

For the ...SetRemOccur, do you remember what you set the loan length and
remaining-length values to?

...jsled