r19673 - gnucash/trunk/src/backend/sql - Bug 632166: Notify user when something goes wrong with a transaction save.

Geert Janssens janssens-geert at telenet.be
Tue Oct 19 05:22:59 EDT 2010


On Monday 18 October 2010, John Ralls wrote:
> Author: jralls
> Date: 2010-10-18 17:56:05 -0400 (Mon, 18 Oct 2010)
> New Revision: 19673
> Trac: http://svn.gnucash.org/trac/changeset/19673
> 
> Modified:
>    gnucash/trunk/src/backend/sql/Makefile.am
>    gnucash/trunk/src/backend/sql/gnc-transaction-sql.c
> Log:
> Bug 632166: Notify user when something goes wrong with a transaction save.
> 
> _______________________________________________
> gnucash-patches mailing list
> gnucash-patches at gnucash.org
> https://lists.gnucash.org/mailman/listinfo/gnucash-patches
> 
John,

This change is definitely an improvement to the silent data loss issue we had.

I see that your commit now introduces a gtk dependency in the sql backend 
while it was strictly gui independent before. I have postponed fixing bug 
630770 [1] because I wanted to avoid adding gui code to the backend. My main 
reason for keeping the databackends free of gui code is that other programs 
may use this code, programs that aren't necessarily using gtk. The first 
example is CuteCash, a second example are the python bindings. 

I'm not sure what the best course of action is here. Silent data loss is a 
severe flaw, so perhaps it's better to have the introduction of a gtk 
dependency now so we can continue, and clean it up properly in the 2.5 
development cycle. Or perhaps the majority of the devs doesn't consider this 
an issue in which case nothing has to be done.

Cleaning it up now would probably be challenging or too intrusive. When I was 
investigating a way to fix bug 630770 I came to the conclusion that backend 
errors are not reported back all the way into the GUI code. There were several 
in between layers that had to be altered for that.

If the choice is to clean it up later, we best keep a bug report open for 
this.

What do others think of this ?

Geert

[1] https://bugzilla.gnome.org/show_bug.cgi?id=630770


More information about the gnucash-devel mailing list