Skip to content

Commit

Permalink
Bug 797084 - Provide an option to leave edited transactions reconciled
Browse files Browse the repository at this point in the history
Instead of making this optional, changes to transaction fields in
case the transaction has reconciled splits will now only issue a
warning. If the user continues anyway with changes to the transaction
field the reconcile state of splits won't be affected.
Changing protected split fields will continue to unreconcile that split.
  • Loading branch information
gjanssens committed Mar 28, 2020
1 parent 5b18405 commit fdd4798
Showing 1 changed file with 2 additions and 11 deletions.
13 changes: 2 additions & 11 deletions gnucash/register/ledger-core/split-register-model.c
Original file line number Diff line number Diff line change
Expand Up @@ -2194,9 +2194,8 @@ gnc_split_register_confirm (VirtualLocation virt_loc, gpointer user_data)
}
title = _("Change transaction containing a reconciled split?");
message_format =
_("The transaction you are about to change is protected because it contains reconciled splits in the following accounts:\n%s"
"\n\nIf you continue editing this transaction all reconciled splits will be unreconciled. "
"This might make future reconciliation difficult! Continue with this change?");
_("The transaction you are about to change is contains reconciled splits in the following accounts:\n%s"
"\n\nAre you sure you want to continue with this change ?");

message = g_strdup_printf (message_format, acc_list);
g_free (acc_list);
Expand Down Expand Up @@ -2250,14 +2249,6 @@ gnc_split_register_confirm (VirtualLocation virt_loc, gpointer user_data)
}
}

if (protected_trans_cell)
{
if (reg->unrecn_splits != NULL)
g_list_free (reg->unrecn_splits);

reg->unrecn_splits = g_list_copy (xaccTransGetSplitList (trans));
}

PINFO ("Unreconcile split list length is %d", g_list_length(reg->unrecn_splits));
info->change_confirmed = TRUE;
}
Expand Down

0 comments on commit fdd4798

Please sign in to comment.