From 549138c2302de10c21a72b8c3ac4da9694874da9 Mon Sep 17 00:00:00 2001
From: nds <nds@opencascade.com>
Date: Tue, 19 Jul 2016 16:28:08 +0300
Subject: [PATCH] Issue #1649: New creation of Construction Planes

Positive validator using breaks the test.
---
 src/ConstructionPlugin/ConstructionPlugin_Plane.cpp | 2 +-
 src/ConstructionPlugin/ConstructionPlugin_Plane.h   | 4 ++--
 src/ConstructionPlugin/plane_widget.xml             | 8 +++-----
 src/ModelAPI/Test/Test1064.py                       | 1 +
 4 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/src/ConstructionPlugin/ConstructionPlugin_Plane.cpp b/src/ConstructionPlugin/ConstructionPlugin_Plane.cpp
index f4b966ac1..68f3e5b0a 100644
--- a/src/ConstructionPlugin/ConstructionPlugin_Plane.cpp
+++ b/src/ConstructionPlugin/ConstructionPlugin_Plane.cpp
@@ -68,7 +68,7 @@ void ConstructionPlugin_Plane::initAttributes()
   data()->addAttribute(CREATION_METHOD_BY_OTHER_PLANE_OPTION(), ModelAPI_AttributeString::typeId());
   data()->addAttribute(PLANE(), ModelAPI_AttributeSelection::typeId());
   data()->addAttribute(DISTANCE(), ModelAPI_AttributeDouble::typeId());
-  //data()->addAttribute(DISTANCE_REVERSE(), ModelAPI_AttributeBoolean::typeId());
+  data()->addAttribute(DISTANCE_REVERSE(), ModelAPI_AttributeBoolean::typeId());
   data()->addAttribute(COINCIDENT_POINT(), ModelAPI_AttributeSelection::typeId());
   data()->addAttribute(AXIS(), ModelAPI_AttributeSelection::typeId());
   data()->addAttribute(ANGLE(), ModelAPI_AttributeDouble::typeId());
diff --git a/src/ConstructionPlugin/ConstructionPlugin_Plane.h b/src/ConstructionPlugin/ConstructionPlugin_Plane.h
index b9a7141f5..15894a2f6 100644
--- a/src/ConstructionPlugin/ConstructionPlugin_Plane.h
+++ b/src/ConstructionPlugin/ConstructionPlugin_Plane.h
@@ -160,11 +160,11 @@ public:
   }
 
   /// Attribute name for reverse in distance from other case
-  /*inline static const std::string& DISTANCE_REVERSE()
+  inline static const std::string& DISTANCE_REVERSE()
   {
     static const std::string ATTR_DISTANCE_REVERSE_ID("distance_reverse");
     return ATTR_DISTANCE_REVERSE_ID;
-  }*/
+  }
 
   /// Attribute name for coincident point.
   inline static const std::string& COINCIDENT_POINT()
diff --git a/src/ConstructionPlugin/plane_widget.xml b/src/ConstructionPlugin/plane_widget.xml
index 797dacb60..9085e2c76 100644
--- a/src/ConstructionPlugin/plane_widget.xml
+++ b/src/ConstructionPlugin/plane_widget.xml
@@ -68,13 +68,11 @@
                        label="Distance"
                        tooltip="Distance from selected face to plane."
                        icon="icons/Construction/distance_value.png"
-                       default="0">
-            <validator id="GeomValidators_Positive"/>
-          </doublevalue>
-          <!--<boolvalue id="distance_reverse"
+                       default="10"/>
+          <boolvalue id="distance_reverse"
                      label="Reverse"
                      tooltip="Distance from plane."
-                     default="false"/>-->
+                     default="false"/>
         </box>
         <box id="by_coincident_to_point"
              title="By coincident to point"
diff --git a/src/ModelAPI/Test/Test1064.py b/src/ModelAPI/Test/Test1064.py
index c2078e61d..d44e933a0 100644
--- a/src/ModelAPI/Test/Test1064.py
+++ b/src/ModelAPI/Test/Test1064.py
@@ -75,6 +75,7 @@ aPlane.string("creation_method").setValue("by_other_plane")
 aPlane.string("by_other_plane_option").setValue("by_distance_from_other")
 aPlane.selection("plane").selectSubShape("face", "Part_1/Extrusion_1_1/Generated_Face_3")
 aPlane.real("distance").setValue(0)
+aPlacementFt.boolean("distance_reverse").setValue(False)
 aSession.finishOperation()
 
 #=========================================================================
-- 
2.39.2