[Gnucash-changes] r13129 - gnucash/trunk/src/engine - Revert Chris'
change in r13128.
Derek Atkins
warlord at cvs.gnucash.org
Mon Feb 6 01:45:25 EST 2006
Author: warlord
Date: 2006-02-06 01:45:25 -0500 (Mon, 06 Feb 2006)
New Revision: 13129
Trac: http://svn.gnucash.org/trac/changeset/13129
Modified:
gnucash/trunk/src/engine/Transaction.c
Log:
Revert Chris' change in r13128.
Keep in the error check for db-transactional data storage
until the objects can be modified to support db-transactional
autocommit (by calling BeginEdit/CommitEdit internally instead
of checking that the caller did so). But revert for now to
make sure a caller has called BeginEdit in order to make
sure the data gets "saved" (assuming a SQLish backend).
Modified: gnucash/trunk/src/engine/Transaction.c
===================================================================
--- gnucash/trunk/src/engine/Transaction.c 2006-02-06 04:50:02 UTC (rev 13128)
+++ gnucash/trunk/src/engine/Transaction.c 2006-02-06 06:45:25 UTC (rev 13129)
@@ -1626,6 +1626,7 @@
gint fraction, old_fraction;
if (!trans || !curr || trans->common_currency == curr) return;
+ check_open (trans);
old_fraction = gnc_commodity_get_fraction (trans->common_currency);
trans->common_currency = curr;
@@ -1672,6 +1673,7 @@
xaccTransDestroy (Transaction *trans)
{
if (!trans) return;
+ check_open (trans);
if (xaccTransGetReadOnly (trans) &&
!qof_book_shutting_down(trans->inst.book)) return;
@@ -2139,6 +2141,8 @@
if (acc && !acc->inst.do_free && xaccTransGetReadOnly (trans))
return FALSE;
+ check_open (trans);
+
mark_split (split);
if (trans)
@@ -2177,6 +2181,7 @@
{
if (!trans || !split) return;
g_return_if_fail (trans->inst.book == split->book);
+ check_open (trans);
/* First, make sure that the split isn't already inserted
* elsewhere. If so, then remove it. */
@@ -2493,6 +2498,8 @@
static inline void
xaccTransSetDateInternal(Transaction *trans, Timespec *dadate, Timespec val)
{
+ check_open(trans);
+
PINFO ("addr=%p set date to %llu.%09ld %s",
trans, val.tv_sec, val.tv_nsec,
ctime (({time_t secs = (time_t) val.tv_sec; &secs;})));
@@ -2633,6 +2640,7 @@
{
char * tmp;
if (!trans || !xnum) return;
+ check_open (trans);
tmp = gnc_string_cache_insert((gpointer) xnum);
gnc_string_cache_remove(trans->num);
@@ -2652,6 +2660,7 @@
{
char * tmp;
if (!trans || !desc) return;
+ check_open (trans);
tmp = gnc_string_cache_insert((gpointer) desc);
gnc_string_cache_remove(trans->description);
@@ -2670,6 +2679,7 @@
xaccTransSetNotes (Transaction *trans, const char *notes)
{
if (!trans || !notes) return;
+ check_open (trans);
kvp_frame_set_str (trans->inst.kvp_data, trans_notes_str, notes);
}
More information about the gnucash-changes
mailing list