From 34746c6ebfac59e7bce654ba28c4458d5f6aec8f Mon Sep 17 00:00:00 2001
From: Dave Peticolas <dave@krondo.com>
Date: Tue, 22 Aug 2000 08:26:32 +0000
Subject: [PATCH] Fix a rebalancing bug.

git-svn-id: svn+ssh://svn.gnucash.org/repo/gnucash/trunk@2690 57a11ea4-9604-0410-9ed3-97b8803252fd
---
 ChangeLog                |  3 +++
 src/engine/Transaction.c | 10 +++++-----
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 3681959d3c..a39e054206 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2000-08-22  Dave Peticolas  <dave@krondo.com>
 
+	* src/engine/Transaction.c (xaccSplitSetBaseValue): check to
+	make sure the security is not "" before allowing price changes.
+
 	* src/gnome/window-register.c (gnc_register_check_close): use
 	xaccSRCancelCursorTransChanges to cancel any pending changes.
 
diff --git a/src/engine/Transaction.c b/src/engine/Transaction.c
index bc07687282..c5630ccf9b 100644
--- a/src/engine/Transaction.c
+++ b/src/engine/Transaction.c
@@ -674,12 +674,12 @@ xaccSplitSetBaseValue (Split *s, double value, const char * base_currency)
       }
    }
 
-   if(s->acc &&
-      s->acc->security &&
-      safe_strcmp(s->acc->security, s->acc->currency) &&
-      !DEQEPS(s->damount, 0.0, ZERO_THRESH_VALUE)) {
+   if (s->acc &&
+       s->acc->security &&
+       *s->acc->security &&
+       safe_strcmp(s->acc->security, s->acc->currency) &&
+       !DEQEPS(s->damount, 0.0, ZERO_THRESH_VALUE))
      adjust_price = 1;
-   }
 
    /* The value of a split depends on the currency we express the
     * value in.  This may or may not require a divide.