Fixing Issue #436. OSArgument::valueAsBool was being called without f…

…irst checking OSArgument::hasValue. [#55122410]
NREL · Aug 13, 2013 · 63cf3e7 · 63cf3e7
1 parent bede2e3
commit 63cf3e7
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/openstudiocore/src/shared_gui_components/EditController.cpp b/openstudiocore/src/shared_gui_components/EditController.cpp
@@ -232,8 +232,6 @@ InputController::InputController(EditController * editController,const ruleset::
   {
     BoolInputView * boolInputView = new BoolInputView();
 
-    boolInputView->checkBox->setChecked(m_argument.valueAsBool());
-
     boolInputView->checkBox->setText(QString::fromStdString(m_argument.displayName()));
 
     if( m_argument.hasValue() )
@@ -244,6 +242,10 @@ InputController::InputController(EditController * editController,const ruleset::
     {
       boolInputView->checkBox->setChecked(m_argument.defaultValueAsBool());
     }
+    else {
+      boolInputView->checkBox->setChecked(false);
+      setValue(false);
+    }
 
     bool bingo = connect(boolInputView->checkBox,SIGNAL(clicked(bool)),this,SLOT(setValue(bool)));