gnucash master: Multiple changes pushed

Derek Atkins warlord at MIT.EDU
Wed Aug 5 13:05:00 EDT 2015


Geert Janssens <geert.gnucash at kobaltwit.be> 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
       warlord at MIT.EDU                        PGP key available


More information about the gnucash-devel mailing list