[Gnucash-changes] r13830 - gnucash/trunk/src/engine - Internalize the transactional semantics for all Transaction setters.

[Chris Shoemaker]

Author: chris
Date: 2006-04-22 01:14:48 -0400 (Sat, 22 Apr 2006)
New Revision: 13830
Trac: http://svn.gnucash.org/trac/changeset/13830

Modified:
   gnucash/trunk/src/engine/Transaction.c
Log:
   Internalize the transactional semantics for all Transaction setters.
   This is what I actually intended in r13130, but we want the stuff that
   xaccTransCommitEdit provides over-and-above what qof_commit_edit provides.


Modified: gnucash/trunk/src/engine/Transaction.c
===================================================================
--- gnucash/trunk/src/engine/Transaction.c	2006-04-22 05:09:17 UTC (rev 13829)
+++ gnucash/trunk/src/engine/Transaction.c	2006-04-22 05:14:48 UTC (rev 13830)
@@ -801,7 +801,7 @@
   gint fraction, old_fraction;
 
   if (!trans || !curr || trans->common_currency == curr) return;
-  qof_begin_edit(QOF_INSTANCE(trans));
+  xaccTransBeginEdit(trans);
 
   old_fraction = gnc_commodity_get_fraction (trans->common_currency);
   trans->common_currency = curr;
@@ -814,7 +814,7 @@
 
   qof_instance_set_dirty(QOF_INSTANCE(trans));
   mark_trans(trans);  /* Dirty balance of every account in trans */
-  qof_commit_edit(QOF_INSTANCE(trans));
+  xaccTransCommitEdit(trans);
 }
 
 /********************************************************************\
@@ -1251,7 +1251,7 @@
 static inline void
 xaccTransSetDateInternal(Transaction *trans, Timespec *dadate, Timespec val)
 {
-    qof_begin_edit(QOF_INSTANCE(trans));
+    xaccTransBeginEdit(trans);
 
     PINFO ("addr=%p set date to %" G_GUINT64_FORMAT ".%09ld %s",
            trans, val.tv_sec, val.tv_nsec, 
@@ -1260,7 +1260,7 @@
     *dadate = val;
     qof_instance_set_dirty(QOF_INSTANCE(trans));
     mark_trans(trans);
-    qof_commit_edit(QOF_INSTANCE(trans));
+    xaccTransCommitEdit(trans);
 
    /* Because the date has changed, we need to make sure that each of
     * the splits is properly ordered in each of their accounts. We
@@ -1344,10 +1344,10 @@
 xaccTransSetDateDueTS (Transaction *trans, const Timespec *ts)
 {
    if (!trans || !ts) return;
-   qof_begin_edit(QOF_INSTANCE(trans));
+   xaccTransBeginEdit(trans);
    kvp_frame_set_timespec (trans->inst.kvp_data, TRANS_DATE_DUE_KVP, *ts);
    qof_instance_set_dirty(QOF_INSTANCE(trans));
-   qof_commit_edit(QOF_INSTANCE(trans));
+   xaccTransCommitEdit(trans);
 }
 
 void
@@ -1355,20 +1355,20 @@
 {
   char s[2] = {type, '\0'};
   g_return_if_fail(trans);
-  qof_begin_edit(&trans->inst);
+  xaccTransBeginEdit(trans);
   kvp_frame_set_str (trans->inst.kvp_data, TRANS_TXN_TYPE_KVP, s);
   qof_instance_set_dirty(QOF_INSTANCE(trans));
-  qof_commit_edit(&trans->inst);
+  xaccTransCommitEdit(trans);
 }
 
 void xaccTransClearReadOnly (Transaction *trans)
 {
     if (trans) {
-        qof_begin_edit(QOF_INSTANCE(trans));
+        xaccTransBeginEdit(trans);
         kvp_frame_set_slot_path (trans->inst.kvp_data, NULL, 
                                  TRANS_READ_ONLY_REASON, NULL);
         qof_instance_set_dirty(QOF_INSTANCE(trans));
-        qof_commit_edit(QOF_INSTANCE(trans));
+        xaccTransCommitEdit(trans);
     }
 }
 
@@ -1376,11 +1376,11 @@
 xaccTransSetReadOnly (Transaction *trans, const char *reason)
 {
     if (trans && reason) {
-        qof_begin_edit(QOF_INSTANCE(trans));
+        xaccTransBeginEdit(trans);
         kvp_frame_set_str (trans->inst.kvp_data, 
                            TRANS_READ_ONLY_REASON, reason);
         qof_instance_set_dirty(QOF_INSTANCE(trans));
-        qof_commit_edit(QOF_INSTANCE(trans));
+        xaccTransCommitEdit(trans);
     }
 }
 
@@ -1401,11 +1401,11 @@
 xaccTransSetNum (Transaction *trans, const char *xnum)
 {
    if (!trans || !xnum) return;
-   qof_begin_edit(QOF_INSTANCE(trans));
+   xaccTransBeginEdit(trans);
 
    CACHE_REPLACE(trans->num, xnum);
    qof_instance_set_dirty(QOF_INSTANCE(trans));
-   qof_commit_edit(QOF_INSTANCE(trans));
+   xaccTransCommitEdit(trans);
 }
 
 static void
@@ -1420,11 +1420,11 @@
 xaccTransSetDescription (Transaction *trans, const char *desc)
 {
    if (!trans || !desc) return;
-   qof_begin_edit(QOF_INSTANCE(trans));
+   xaccTransBeginEdit(trans);
 
    CACHE_REPLACE(trans->description, desc);
    qof_instance_set_dirty(QOF_INSTANCE(trans));
-   qof_commit_edit(QOF_INSTANCE(trans));
+   xaccTransCommitEdit(trans);
 }
 
 static void
@@ -1439,11 +1439,11 @@
 xaccTransSetNotes (Transaction *trans, const char *notes)
 {
   if (!trans || !notes) return;
-  qof_begin_edit(QOF_INSTANCE(trans));
+  xaccTransBeginEdit(trans);
 
   kvp_frame_set_str (trans->inst.kvp_data, trans_notes_str, notes);
   qof_instance_set_dirty(QOF_INSTANCE(trans));
-  qof_commit_edit(QOF_INSTANCE(trans));
+  xaccTransCommitEdit(trans);
 }
 
 /********************************************************************\