Salome HOME
Bos #24055. Minor changes.
authorjfa <jfa@opencascade.com>
Fri, 19 Mar 2021 11:03:52 +0000 (14:03 +0300)
committerjfa <jfa@opencascade.com>
Fri, 19 Mar 2021 11:03:52 +0000 (14:03 +0300)
src/FeaturesPlugin/FeaturesPlugin_Extrusion.cpp
src/FeaturesPlugin/FeaturesPlugin_Extrusion.h
src/FeaturesPlugin/FeaturesPlugin_Revolution.h
src/ModuleBase/ModuleBase_ModelWidget.cpp
src/ModuleBase/ModuleBase_ModelWidget.h

index 0ef1875dcd97ddef2a63c684f6beb410a8de5851..3461326aa0a48d3a9ff3a4bdf6431482594ce0c7 100644 (file)
@@ -220,7 +220,7 @@ void FeaturesPlugin_Extrusion::getSizes(double& theToSize, double& theFromSize)
   if (string(CREATION_METHOD())->value() == CREATION_METHOD_BY_SIZES()) {
     theToSize = real(TO_SIZE_ID())->value();
     theFromSize = real(FROM_SIZE_ID())->value();
-  } if (string(CREATION_METHOD())->value() == CREATION_METHOD_BY_PLANES()) {
+  } else if (string(CREATION_METHOD())->value() == CREATION_METHOD_BY_PLANES()) {
     theToSize = real(TO_OFFSET_ID())->value();
     theFromSize = real(FROM_OFFSET_ID())->value();
   } else {
index 088e8b19cbbd04ecfac2220b9b297af9b3679d5a..0928ff61a4e06e6f8c0909de7c0fe85793f78957 100644 (file)
@@ -56,22 +56,22 @@ public:
   /// Attribute name for creation method.
   inline static const std::string& CREATION_METHOD_BY_SIZES()
   {
-    static const std::string MY_CREATION_METHOD_ID("BySizes");
-    return MY_CREATION_METHOD_ID;
+    static const std::string MY_CREATION_METHOD_BY_SIZES("BySizes");
+    return MY_CREATION_METHOD_BY_SIZES;
   }
 
   /// Attribute name for creation method.
   inline static const std::string& CREATION_METHOD_BY_PLANES()
   {
-    static const std::string MY_CREATION_METHOD_ID("ByPlanesAndOffsets");
-    return MY_CREATION_METHOD_ID;
+    static const std::string MY_CREATION_METHOD_BY_PLANES("ByPlanesAndOffsets");
+    return MY_CREATION_METHOD_BY_PLANES;
   }
 
   /// Attribute name for creation method.
   inline static const std::string& CREATION_METHOD_THROUGH_ALL()
   {
-    static const std::string MY_CREATION_METHOD_ID("ThroughAll");
-    return MY_CREATION_METHOD_ID;
+    static const std::string MY_CREATION_METHOD_THROUGH_ALL("ThroughAll");
+    return MY_CREATION_METHOD_THROUGH_ALL;
   }
 
   /// Attribute name of an object to which the extrusion grows.
index d9c2b99101c7237fe72888edb0c0d155da52de9e..6ab504e34029f4928cf06ccca6c5d9d4ef21b53c 100644 (file)
@@ -51,22 +51,22 @@ public:
   /// Attribute name for creation method.
   inline static const std::string& CREATION_METHOD_THROUGH_ALL()
   {
-    static const std::string MY_CREATION_METHOD_ID("ThroughAll");
-    return MY_CREATION_METHOD_ID;
+    static const std::string MY_CREATION_METHOD_THROUGH_ALL("ThroughAll");
+    return MY_CREATION_METHOD_THROUGH_ALL;
   }
 
   /// Attribute name for creation method.
   inline static const std::string& CREATION_METHOD_BY_ANGLES()
   {
-    static const std::string MY_CREATION_METHOD_ID("ByAngles");
-    return MY_CREATION_METHOD_ID;
+    static const std::string MY_CREATION_METHOD_BY_ANGLES("ByAngles");
+    return MY_CREATION_METHOD_BY_ANGLES;
   }
 
   /// Attribute name for creation method.
   inline static const std::string& CREATION_METHOD_BY_PLANES()
   {
-    static const std::string MY_CREATION_METHOD_ID("ByPlanesAndOffsets");
-    return MY_CREATION_METHOD_ID;
+    static const std::string MY_CREATION_METHOD_BY_PLANES("ByPlanesAndOffsets");
+    return MY_CREATION_METHOD_BY_PLANES;
   }
 
   /// Attribute name of an revolution axis.
index e232d1d89614d7d7288dc413ced7e09cd379af70..e54f8c6dd0df244689c33ed2ec36f2d9ed7ce478 100644 (file)
@@ -53,8 +53,8 @@ ModuleBase_ModelWidget::ModuleBase_ModelWidget(QWidget* theParent,
   const Config_WidgetAPI* theData)
   : QWidget(theParent),
   myWidgetValidator(0),
-  myIsEditing(false),
   myState(Stored),
+  myIsEditing(false),
   myIsValueStateBlocked(false),
   myFlushUpdateBlocked(false)
 {
index 73de512dd1b9feaaf86068878c3a655f5a36b7d8..de0aae6fa14e40577fd29cd1321af550a73883a3 100644 (file)
@@ -441,9 +441,6 @@ protected:
   /// A feature ID
   std::string myFeatureId;
 
-  /// Flag which shows that current operation is in editing mode
-  bool myIsEditing;
-
   /// Flag which shows whether current widget is obligatory
   /// The non-obligatory widgets should not accept the focus in the property panel
   bool myIsObligatory;
@@ -455,6 +452,9 @@ protected:
   ValueState myState;
 
 private:
+  /// Flag which shows that current operation is in editing mode
+  bool myIsEditing;
+
   /// Value should be computed on execute, like radius for circle's constraint (can not be zero)
   bool myIsComputedDefault;