r18064 - gnucash/trunk/src/gnome-utils - Fix bug #475806. Mismatch between return value (SCM_BOOL_F) and test (SCM_NULLP) to determine if

[Phil Longstaff]

Author: plongstaff
Date: 2009-05-01 18:54:53 -0400 (Fri, 01 May 2009)
New Revision: 18064
Trac: http://svn.gnucash.org/trac/changeset/18064

Modified:
   gnucash/trunk/src/gnome-utils/dialog-options.c
Log:
Fix bug #475806.  Mismatch between return value (SCM_BOOL_F) and test (SCM_NULLP) to determine if
budget is valid or not.  Changed test to "!= SCM_BOOL_F" to match the way the value is handled
for accounts.


Modified: gnucash/trunk/src/gnome-utils/dialog-options.c
===================================================================
--- gnucash/trunk/src/gnome-utils/dialog-options.c	2009-04-29 00:53:32 UTC (rev 18063)
+++ gnucash/trunk/src/gnome-utils/dialog-options.c	2009-05-01 22:54:53 UTC (rev 18064)
@@ -2525,7 +2525,8 @@
     GtkTreeModel *tm;
     GtkTreeIter iter;
 
-    if (!SCM_NULLP(value)) {
+//    if (!SCM_NULLP(value)) {
+    if (value != SCM_BOOL_F) {
         if (!SWIG_IsPointer(value))
             scm_misc_error("gnc_option_set_ui_value_budget",
                            "Option Value not a wcp.", value);