gnucash maint: Fix gtk accelerator mixups by explicitly disabling actions that are not relevant

Geert Janssens gjanssens at code.gnucash.org
Mon Sep 24 11:15:20 EDT 2018


Updated	 via  https://github.com/Gnucash/gnucash/commit/f89a40e3 (commit)
	from  https://github.com/Gnucash/gnucash/commit/26a17987 (commit)



commit f89a40e3ebf7ffae2f59b352d5d2d2cbb00f1ab1
Author: Geert Janssens <geert at kobaltwit.be>
Date:   Mon Sep 24 17:15:11 2018 +0200

    Fix gtk accelerator mixups by explicitly disabling actions that are not relevant
    
    The issue I found was with File->Print and its variants for various page types.
    They all use the same '<primary>p' as global accelerator. It was possible to
    have gnucash invoke the 'Print Report...' action instead of the 'Print Invoice...'
    action when hitting <primary>p on an invoice page. If no report page was
    open at this point, gnucash would crash. If a report was open, gnucash
    would invoke the print action on that report instead of opening an
    invoice report for the invoice. This looks like a bug internally in
    the gtkaction handling or our improper use of it. However I could
    work around it by disabling action groups before unmerging them.



Summary of changes:
 gnucash/gnome-utils/gnc-plugin-page.c | 2 ++
 1 file changed, 2 insertions(+)



More information about the gnucash-patches mailing list