[Gnucash-changes] r13316 - gnucash/trunk - Allow nested begin/commit in xaccTransSetTxnType(). Fixes #331771.

[Derek Atkins]

Author: warlord
Date: 2006-02-19 23:40:06 -0500 (Sun, 19 Feb 2006)
New Revision: 13316
Trac: http://svn.gnucash.org/trac/changeset/13316

Modified:
   gnucash/trunk/ChangeLog
   gnucash/trunk/src/engine/Transaction.c
Log:
	  Allow nested begin/commit in xaccTransSetTxnType().  Fixes #331771.
	  The old code wouldn't let you set the value if you were already
	  within a BeginEdit/CommitEdit block.



Modified: gnucash/trunk/ChangeLog
===================================================================
--- gnucash/trunk/ChangeLog	2006-02-20 03:19:17 UTC (rev 13315)
+++ gnucash/trunk/ChangeLog	2006-02-20 04:40:06 UTC (rev 13316)
@@ -1,3 +1,10 @@
+2006-02-19  Derek Atkins  <derek at ihtfp.com>
+
+	* src/engine/Transaction.c:
+	  Allow nested begin/commit in xaccTransSetTxnType().  Fixes #331771.
+	  The old code wouldn't let you set the value if you were already
+	  within a BeginEdit/CommitEdit block.
+
 2006-02-19  Chris Lyttle  <chris at wilddev.net>
 
 	* NEWS: Added some text about the release.

Modified: gnucash/trunk/src/engine/Transaction.c
===================================================================
--- gnucash/trunk/src/engine/Transaction.c	2006-02-20 03:19:17 UTC (rev 13315)
+++ gnucash/trunk/src/engine/Transaction.c	2006-02-20 04:40:06 UTC (rev 13316)
@@ -2606,7 +2606,7 @@
 {
   char s[2] = {type, '\0'};
   g_return_if_fail(trans);
-  if (!qof_begin_edit(&trans->inst)) return;
+  qof_begin_edit(&trans->inst);
   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);