test-book-merge update

[Derek Atkins]

Thanks.  Applied.

-derek

Neil Williams <linux at codehelp.co.uk> writes:

> I fixed a separate bug in qof_book_merge and this is the patch for the test 
> case. I've added a note to explain:
>
>  /* In this test rule_loop, any lines beginning with do_test() can be removed
>  from a working rule_loop routine. It would be wise to still use some of the
>  more obvious checks, e.g. that an entity or rule exists before querying the 
>  parameters.
>  
>  Take particular care with MERGE_NEW - targetEnt is always NULL until the
>  Commit.
>  Do not attempt to use param_getfcn on targetEnt in the loop called by 
>  qof_book_mergeRuleForeach(rule_loop, MERGE_NEW);
>
> This is because the target book cannot be altered until the Commit, so because 
> this entity is new, it doesn't have a target in the book yet - the target 
> entity in the rule is null. At Commit, a new entity is created using the 
> QofObject create: call which initialises it to the defaults. qof_book_merge 
> then updates the new entity with the import data.
>
> For some reason, the first message came back with a bad signature - it also 
> used the wrong date in the filename. Trying again.
>
> (The two are the same patch, just renamed the file.)
>
> -- 
>
> Neil Williams
> =============
> http://www.codehelp.co.uk/
> http://www.dclug.org.uk/
> http://www.isbn.org.uk/
> http://sourceforge.net/projects/isbnsearch/
>
> http://www.biglumber.com/x/web?qs=0x8801094A28BCB3E3
>
> _______________________________________________
> gnucash-patches mailing list
> gnucash-patches at gnucash.org
> https://lists.gnucash.org/mailman/listinfo/gnucash-patches

-- 
       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