gnucash maint: [aqbanking] Bug 797430: Avoid double init of gui

Christian Stimming cstim at code.gnucash.org
Sun Sep 29 15:58:58 EDT 2019


Updated	 via  https://github.com/Gnucash/gnucash/commit/82e9b14d (commit)
	from  https://github.com/Gnucash/gnucash/commit/bb4aa19a (commit)



commit 82e9b14dc5fde111821577cce546932fdb1e2418
Author: Christian Stimming <christian at cstimming.de>
Date:   Sun Sep 29 21:58:42 2019 +0200

    [aqbanking] Bug 797430: Avoid double init of gui
    
    Patch by Mike Pieper. Thanks!
    
    Explanation for why this is necessary: The previous (old) behaviour
    changed in 62f7aa66. Before this commit (or when AQBANKING6 is not
    set) it calls AB_JobSepaTransfer_new. For AQBANKING6 it calls
    AB_Transaction_new. AB_Transaction_new is not using gui. Therefore
    gui->state stays on INIT, while before it made several status changes
    (=> RUNNING, FINISHED, HIDDEN). On second call to gnc_GWEN_Gui_get it
    checks the state and finds it still in INIT which seems to be wrong.
    
    So the right fix is to either use gui in AB_Transaction_new or to
    remove first allocation, which is done here.



Summary of changes:
 gnucash/import-export/aqb/gnc-ab-transfer.c | 2 ++
 1 file changed, 2 insertions(+)



More information about the gnucash-patches mailing list