From 064bf2298570ece8c1f98a240ed73c1989aa689c Mon Sep 17 00:00:00 2001
From: jfa <jfa@opencascade.com>
Date: Wed, 17 Feb 2021 13:18:13 +0300
Subject: [PATCH] bos #23963 - EDF 22935 : Extrusion3D, submesh problem. Fix
 groups handling with ticks

---
 src/GEOMClient/GEOM_Client.cxx | 19 +++++++++++++++++--
 1 file changed, 17 insertions(+), 2 deletions(-)

diff --git a/src/GEOMClient/GEOM_Client.cxx b/src/GEOMClient/GEOM_Client.cxx
index 897a47541..d97962314 100644
--- a/src/GEOMClient/GEOM_Client.cxx
+++ b/src/GEOMClient/GEOM_Client.cxx
@@ -201,13 +201,27 @@ TopoDS_Shape GEOM_Client::GetShape( GEOM::GEOM_Gen_ptr geom, GEOM::GEOM_Object_p
 {
   CORBA::String_var     anIOR = geom->GetStringFromIOR(aShape);
   TCollection_AsciiString IOR = anIOR.in();
+
+  // Special treatment for groups: group value (shape) is computed on
+  // first demand, and its tick is set to main shape's tick after that;
+  // tick of modified group is decremented.
+  // There is a not fixed problem: if anyone modifies a group and updates
+  // it before call to this method, the group will not be changed in the buffer
+  bool isModifiedSubShape = false;
   int aShapeTick = aShape->GetTick();
+  if ( !aShape->IsMainShape() ) {
+    int anOldTick = aShapeTick;
+    aShape->GetShapeType(); // compute subshape and update its tick, if not yet
+    aShapeTick = aShape->GetTick();
+    isModifiedSubShape = (anOldTick != aShapeTick);
+  }
 
   std::map< TCollection_AsciiString , int >::iterator i2t = myTicksMap.find( IOR );
   if ( i2t != myTicksMap.end() ) {
-    // The shape was modified, clean the stored one
-    if (i2t->second != aShapeTick)
+    if (i2t->second != aShapeTick || isModifiedSubShape) {
+      // The shape was modified, clean the stored one
       RemoveShapeFromBuffer(IOR);
+    }
   }
 
   TopoDS_Shape S;
@@ -255,6 +269,7 @@ TopoDS_Shape GEOM_Client::GetShape( GEOM::GEOM_Gen_ptr geom, GEOM::GEOM_Object_p
 
     S = aCompound;
   }
+
   Bind(IOR, S, aShapeTick);
   return S;
 }