IntenseFinance/gnucash
3
0
Fork 0
mirror of https://github.com/Gnucash/gnucash.git synced 2025-02-25 18:55:30 -06:00
Code Issues Packages Projects Releases Wiki Activity

Bug #635439: Patch to autoset owner in the payment dialog

Browse Source 
Patch by Matthijs Kooijman <matthijs@stdin.nl>:

    In the payment dialog, set the owner when an invoice is selected

    Before, you would always need to select an owner manually. If you would
    just select an invoice (which was possible, since the invoice selection
    is not disabled when no owner has been selected yet), pressing OK would
    error out because no owner was selected.

    This patch makes sure that when an invoice is selected, but no owner is
    selected yet, the right owner is filled into the owner selection box
    automatically.

    This also happens when the wrong owner is selected, which can happen
    because the invoice autocompletion is not limited to invoices from the
    selected owner, but always completes all invoices. This could cause
    weird behaviour: Select an owner, then use completion to select an
    invoice by another owner and press ok. This would cause the amount for
    the invoice by the other owner to be charged against the first owner.
    This patch prevents that by replacing the first owner by the other owner
    in the dialog.

    This patch also changes gnc_invoice_set_owner to no longer reset the
    invoice selection to NULL. Instead, this is now handled by

git-svn-id: svn+ssh://svn.gnucash.org/repo/gnucash/trunk@19853 57a11ea4-9604-0410-9ed3-97b8803252fd
This commit is contained in:
Christian Stimming 2010-11-21 20:17:29 +00:00
parent 789f42e134
commit 8129860609
2 changed files with 20 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/business/business-gnome/business-gnome-utils.c
View File

@ -310,7 +310,6 @@ void gnc_invoice_set_owner (GtkWidget *widget, GncOwner *owner)
gncOwnerCopy(owner, &isi->owner);
isi->have_owner = TRUE;
gnc_general_search_set_selected(GNC_GENERAL_SEARCH(widget), NULL);
/* Reset the label */
gnc_invoice_select_search_set_label(isi);

27
src/business/business-gnome/dialog-payment.c
View File

@ -96,17 +96,29 @@ gnc_payment_window_close_handler (gpointer data)
gtk_widget_destroy (pw->dialog);
}
/* Forward declaration */
static void gnc_payment_set_owner (PaymentWindow *pw, GncOwner *owner);
static void
gnc_payment_dialog_invoice_changed(PaymentWindow *pw)
{
GNCLot *lot;
gnc_numeric val;
GncOwner *owner;
/* Set the payment amount in the dialog */
if (pw->invoice)
{
lot = gncInvoiceGetPostedLot (pw->invoice);
val = gnc_numeric_abs (gnc_lot_get_balance (lot));
/* If a invoice is selected without selecting an owner first, or
* if the owner for the selected invoice is different from the
* previously selected owner, make sure the right owner is
* selected. */
owner = gncInvoiceGetOwner (pw->invoice);
if (pw->owner.owner.undefined == NULL || pw->owner.owner.undefined != owner->owner.undefined) {
gnc_payment_set_owner (pw, gncInvoiceGetOwner (pw->invoice));
}
}
else
{
@ -124,18 +136,19 @@ gnc_payment_dialog_owner_changed(PaymentWindow *pw)
KvpValue* value;
KvpFrame* slots;
/* If the owner changed, the invoice selection is invalid */
pw->invoice = NULL;
/* The selected invoice has a different owner than the newly
* selected owner, reset it */
if (pw->invoice != NULL && !gncOwnerEqual(gncInvoiceGetOwner(pw->invoice), &pw->owner)) {
pw->invoice = NULL;
gnc_invoice_set_invoice(pw->invoice_choice, NULL);
}
/* Limit the invoice selection to this owner */
gnc_invoice_set_owner(pw->invoice_choice, &pw->owner);
/* note that set_owner implies ...set_invoice(...,NULL); */
/* in case we don't get the callback */
gnc_payment_dialog_invoice_changed(pw);
/* XXX: We should set the sensitive flag on the invoice_choice
* based on whether 'owner' is NULL or not...
*/
/* Now handle the account tree */
slots = gncOwnerGetSlots(&pw->owner);
if (slots)