gnucash maint: Multiple changes pushed

[Geert Janssens]

Updated	 via  https://github.com/Gnucash/gnucash/commit/1c5eb86d (commit)
	 via  https://github.com/Gnucash/gnucash/commit/9bec660f (commit)
	 via  https://github.com/Gnucash/gnucash/commit/164f4847 (commit)
	from  https://github.com/Gnucash/gnucash/commit/3991ccb9 (commit)



commit 1c5eb86d9023d0505ea86d78a57030c7e3ef463e
Author: Geert Janssens <geert at kobaltwit.be>
Date:   Sun Sep 23 21:52:33 2018 +0200

    Simplify GNCQueryView's data model
    
    It was managing a number of redudant parameters which were leaking
    memory all over the place. The information that was tracked could
    easily be extracted from the underlying GtkTreeView, GtkTreeModel
    or GtkTreeSelection when needed.

commit 9bec660fba4ed92d86df05cf94501c6e3b7c1030
Author: Geert Janssens <geert at kobaltwit.be>
Date:   Sun Sep 23 18:14:35 2018 +0200

    Revert "Fix memory leaks in GncQueryView"
    
    This reverts commit 5609b704c569975663d3bf3b85ea50965f602399.
    
    While it fixed the memory leaks it also caused gnucash to crash
    when trying to search for invoices. Will be redone differently
    in a follow-up commit.

commit 164f484718205d17499307cc804fc81dd574b607
Author: Geert Janssens <geert at kobaltwit.be>
Date:   Sun Sep 23 17:35:29 2018 +0200

    Open customer/vendor/employee report when double-clicking an the respective owner the cust/vend/empl overview page
    
    Before this action would open the cust/vend/empl edit window. However opening the report is a much
    more common use case so it makes sense to make that one default.



Summary of changes:
 gnucash/gnome-search/dialog-search.c       |  51 +++++------
 gnucash/gnome-utils/gnc-query-view.c       | 132 +++++++++++++----------------
 gnucash/gnome-utils/gnc-query-view.h       |   5 +-
 gnucash/gnome/gnc-plugin-page-owner-tree.c |   8 +-
 4 files changed, 83 insertions(+), 113 deletions(-)