Issue #2606: Use default value only if it is defined

diff --git a/src/ModuleBase/ModuleBase_WidgetDoubleValue.cpp b/src/ModuleBase/ModuleBase_WidgetDoubleValue.cpp
index 7afe07635587a531917dc165b48b90b7f56b12ad..0f521b50aa97c82aac3a6f4b5f7cecfc26843eef 100644 (file)
--- a/src/ModuleBase/ModuleBase_WidgetDoubleValue.cpp
+++ b/src/ModuleBase/ModuleBase_WidgetDoubleValue.cpp
@@ -56,7 +56,7 @@
 
 ModuleBase_WidgetDoubleValue::ModuleBase_WidgetDoubleValue(QWidget* theParent,
                                                            const Config_WidgetAPI* theData)
-    : ModuleBase_ModelWidget(theParent, theData)
+    : ModuleBase_ModelWidget(theParent, theData), myHasDefault(false)
 {
   QFormLayout* aControlLay = new QFormLayout(this);
   ModuleBase_Tools::adjustMargins(aControlLay);
@@ -101,11 +101,9 @@ ModuleBase_WidgetDoubleValue::ModuleBase_WidgetDoubleValue(QWidget* theParent,
     mySpinBox->setSingleStep(aStepVal);
   }
 
-  myDefaultVal = QString::fromStdString(getDefaultValue()).toDouble(&isOk);
-  if (isOk)
+  myDefaultVal = QString::fromStdString(getDefaultValue()).toDouble(&myHasDefault);
+  if (myHasDefault)
     mySpinBox->setValue(myDefaultVal);
-  else
-    myDefaultVal = 0;
 
   QString aTTip = translate(theData->widgetTooltip());
   mySpinBox->setToolTip(aTTip);
@@ -218,7 +216,7 @@ bool ModuleBase_WidgetDoubleValue::restoreValueCustom()
       aRef->setExpressionError("");
       aRef->setExpressionInvalid(false);
     }
-    if (!aRef->isInitialized())
+    if ((!aRef->isInitialized()) && myHasDefault)
       aRef->setValue(myDefaultVal);
   }
   return true;

diff --git a/src/ModuleBase/ModuleBase_WidgetDoubleValue.h b/src/ModuleBase/ModuleBase_WidgetDoubleValue.h
index 8e9055ab7d5f97931d1ee9558c18de1d1e887d7a..595cb8590bc8a7270f549122c8a67d0f0f02afd5 100644 (file)
--- a/src/ModuleBase/ModuleBase_WidgetDoubleValue.h
+++ b/src/ModuleBase/ModuleBase_WidgetDoubleValue.h
@@ -90,6 +90,7 @@ protected:
   FeaturePtr myParameter;
 
   double myDefaultVal;
+  bool myHasDefault;
 };
 
 #endif