Re: gnucash master: Multiple changes pushed

2015-08-05 Thread Derek Atkins
Geert Janssens  writes:

> On Tuesday 04 August 2015 12:00:06 Derek Atkins wrote:
>> Geert,
>> 
>> This change seems like the wrong direction; the point of including the
>> book in the API was to reduce the need for global variables/storage.
>> I'd like to see GnuCash move more towards being able to have multiple
>> files open simultaneously, not further away from it.
>> 
>> I clearly missed the gnc:*company-name* change the preceeded this
>> change otherwise I'd probably have commented on that, too.
>> 
>> -derek
>> 
>
> Derek,
>
> That's a fair point. I have reverted most of the commit that removes the 
> book parameter and instead corrected gnc:company-info to require it 
> (gnc:company-info was introduced by John during his kvp rewrite).
>
> I occurred to me though that the issue is much more fundamental than 
> that. I haven't found a universal way to tie a report to a book. It'd 
> have expected the options database to be able to tell what book it 
> belongs to, but at present it doesn't.

Yes, this is a more fundamental issue, and one which we will have to
work through eventually.  But in the meantime we should plan for that.

> So for now I have had to resort to (gnc-get-current-book) for a number 
> of reports instead of being able to extract the real book from one or 
> the other parameter. This will have to be revisited if/when we will 
> actually implement a multiple files open interface.

Yeah, it's unfortunate that there is not a better way.  :(

> Geert

-derek

-- 
   Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory
   Member, MIT Student Information Processing Board  (SIPB)
   URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH
   warl...@mit.eduPGP key available
___
gnucash-devel mailing list
gnucash-devel@gnucash.org
https://lists.gnucash.org/mailman/listinfo/gnucash-devel


Re: gnucash master: Multiple changes pushed

2015-08-05 Thread John Ralls

> On Aug 5, 2015, at 6:05 PM, Derek Atkins  wrote:
> 
> Geert Janssens  writes:
> 
>> On Tuesday 04 August 2015 12:00:06 Derek Atkins wrote:
>>> Geert,
>>> 
>>> This change seems like the wrong direction; the point of including the
>>> book in the API was to reduce the need for global variables/storage.
>>> I'd like to see GnuCash move more towards being able to have multiple
>>> files open simultaneously, not further away from it.
>>> 
>>> I clearly missed the gnc:*company-name* change the preceeded this
>>> change otherwise I'd probably have commented on that, too.
>>> 
>>> -derek
>>> 
>> 
>> Derek,
>> 
>> That's a fair point. I have reverted most of the commit that removes the 
>> book parameter and instead corrected gnc:company-info to require it 
>> (gnc:company-info was introduced by John during his kvp rewrite).
>> 
>> I occurred to me though that the issue is much more fundamental than 
>> that. I haven't found a universal way to tie a report to a book. It'd 
>> have expected the options database to be able to tell what book it 
>> belongs to, but at present it doesn't.
> 

I’m a bit at two minds about this: On the one hand it’s clear that many report 
customizations apply to only one book. On the other I have trusts for my two 
daughters which are pretty much identical and which it makes sense to use the 
same customizations. On the third, the two accounts are so identical that I can 
run the report twice in one of them changing only the title, so it’s not 
actually necessary to use the report in both except that my aging brain would 
have to remember in which account the reports live.

Regards,
John Ralls



___
gnucash-devel mailing list
gnucash-devel@gnucash.org
https://lists.gnucash.org/mailman/listinfo/gnucash-devel