Advanced portfolio report

[Andrew Sackville-West]

On Sat, Dec 01, 2007 at 09:47:53AM -0500, Derek Atkins wrote:
> Andrew Sackville-West <ajswest at mindspring.com> writes:
> 
> >> I decided to take a look at this a bit, although I don't have time to 
> >> really do too much right now.  So far I've tried Andrew's but not 
> >> Morrison's.  To get it to work I had to change "_" to "-" in 
> >> ACCT_TYPE_ASSET and ACCT_TYPE_LIABILITY around line 390 in case anyone 
> >> else is wondering why it doesn't work as sent.
> >
> > ??
> 
> Andrew, you have the following code:
> 
> 			(if (and (or (split-account-type? s ACCT_TYPE_ASSET)
> 				     (split-account-type? s ACCT_TYPE_LIABILITY))
> 
> But these should be ACCT-TYPE-ASSET and ACCT-TYPE-LIABILITY.

yeah, I found that when I started playing with Mike's spinoff
issue. Somehow that code never got seen before, I guess.

>  Note
> that this wont match ACCT-TYPE-BANK, ACCT-TYPE-CASH, ACCT-TYPE-CREDIT,
> etc.

Interesting. Thanks for pointing this out. I'm not sure if that's my
code or not, but I'd seen that and always assumed (poor Uma Thurmond)
that the others were sub-types. 

Thanks

A
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : http://lists.gnucash.org/pipermail/gnucash-devel/attachments/20071202/5e1e68f6/attachment.bin