0028988: Visualization - AIS_InteractiveContext::SetLocation() does not reset locatio...

SelectMgr_SelectionManager - removed broken HasTransformation() checks.

diff --git a/src/SelectMgr/SelectMgr_SelectableObject.cxx b/src/SelectMgr/SelectMgr_SelectableObject.cxx
index dc3126d6019f5036f4de80e5f7b947d22ebef5d3..14f4d61ad6dc6eb5bab6798bafe6f6ed07d7e34b 100644 (file)
--- a/src/SelectMgr/SelectMgr_SelectableObject.cxx
+++ b/src/SelectMgr/SelectMgr_SelectableObject.cxx
@@ -265,11 +265,6 @@ void SelectMgr_SelectableObject::UpdateTransformation()
 void SelectMgr_SelectableObject::UpdateTransformations (const Handle(SelectMgr_Selection)& theSel)
 {
   const TopLoc_Location aSelfLocation (Transformation());
-  if (aSelfLocation.IsIdentity())
-  {
-    return;
-  }
-
   for (NCollection_Vector<Handle(SelectMgr_SensitiveEntity)>::Iterator aSelEntIter (theSel->Entities()); aSelEntIter.More(); aSelEntIter.Next())
   {
     if (Handle(Select3D_SensitiveEntity) aSensEntity = Handle(Select3D_SensitiveEntity)::DownCast (aSelEntIter.Value()->BaseSensitive()))

diff --git a/src/SelectMgr/SelectMgr_SelectionManager.cxx b/src/SelectMgr/SelectMgr_SelectionManager.cxx
index a730aa4b3935f752b323cc0f2c564aea11868678..86e34a126bfa79e3c9698da7f9f7d4159eb342b0 100644 (file)
--- a/src/SelectMgr/SelectMgr_SelectionManager.cxx
+++ b/src/SelectMgr/SelectMgr_SelectionManager.cxx
@@ -345,10 +345,7 @@ void SelectMgr_SelectionManager::Activate (const Handle(SelectMgr_SelectableObje
     Standard_FALLTHROUGH
     case SelectMgr_TOU_Partial:
     {
-      if(theObject->HasTransformation())
-      {
-        theObject->UpdateTransformations (aSelection);
-      }
+      theObject->UpdateTransformations (aSelection);
       theSelector->RebuildObjectsTree();
       break;
     }
@@ -846,11 +843,8 @@ void SelectMgr_SelectionManager::Update (const Handle(SelectMgr_SelectableObject
         Standard_FALLTHROUGH
         case SelectMgr_TOU_Partial:
         {
-          if (theObject->HasTransformation())
-          {
-            theObject->UpdateTransformations (aSelection);
-            theSelector->RebuildObjectsTree();
-          }
+          theObject->UpdateTransformations (aSelection);
+          theSelector->RebuildObjectsTree();
           break;
         }
         default:

diff --git a/tests/bugs/vis/bug28988 b/tests/bugs/vis/bug28988
new file mode 100644 (file)
index 0000000..3712dfc
--- /dev/null
+++ b/tests/bugs/vis/bug28988
@@ -0,0 +1,22 @@
+puts "=================="
+puts "0028988: Visualization - AIS_InteractiveContext::SetLocation() does not reset location properly"
+puts "=================="
+puts ""
+
+pload MODELING VISUALIZATION
+
+box b 1 2 3
+vclear
+vinit View1
+vaxo
+vdisplay -dispMode 1 -highMode 1 b
+vfit
+vmoveto 110 110
+if { "[vreadpixel 110 110 rgb name]" != "DARKTURQUOISE" } { puts "Error: shape is not selected!" }
+
+vlocation b -translate 1 0 0
+vmoveto 110 110
+vlocation b -reset
+vmoveto 110 110
+vreadpixel 110 110 rgb name
+if { "[vreadpixel 110 110 rgb name]" != "DARKTURQUOISE" } { puts "Error: resetted shape is not selected!" }