0027309: Result of UnifySameDomain has InvalidMultiConnexity error

The update of the set of boundary edges has been added for the case when faces containing milti-connected edges are excluded from merging.

Update test cases for unifysamedom to generate snapshots.

diff --git a/src/ShapeUpgrade/ShapeUpgrade_UnifySameDomain.cxx b/src/ShapeUpgrade/ShapeUpgrade_UnifySameDomain.cxx
index 718189492a0ebabde03ac3bc3dd812d8f5ef603f..6189e4c9db8eaab3b4b4fb28ff90693f746fc848 100644 (file)
--- a/src/ShapeUpgrade/ShapeUpgrade_UnifySameDomain.cxx
+++ b/src/ShapeUpgrade/ShapeUpgrade_UnifySameDomain.cxx
@@ -1208,8 +1208,25 @@ void ShapeUpgrade_UnifySameDomain::UnifyFaces()
               anAvoidFaces.Add(aLF.Last());
             }
             for (i = 1; i <= faces.Length(); ) {
-              if (anAvoidFaces.Contains(faces(i)))
+              if (anAvoidFaces.Contains(faces(i))) {
+                // update the boundaries of merged area, for that
+                // remove from 'edges' the edges of this face and add to 'edges' 
+                // the edges of this face that were not present in 'edges' before
+                TopExp_Explorer ex(faces(i), TopAbs_EDGE);
+                for (; ex.More(); ex.Next()) {
+                  TopoDS_Shape aE = ex.Current();
+                  Standard_Integer j;
+                  for (j = 1; j <= edges.Length(); j++) {
+                    if (edges(j).IsSame(aE))
+                      break;
+                  }
+                  if (j <= edges.Length())
+                    edges.Remove(j);
+                  else
+                    edges.Append(aE);
+                }
                 faces.Remove(i);
+              }
               else
                 i++;
             }

diff --git a/tests/bugs/modalg_6/bug27082_1 b/tests/bugs/modalg_6/bug27082_1
index a58a1a23acaad81dd9757c27b043abce910ffc62..6181fcffc4b41fc56f52114aefa304e84a9c750c 100644 (file)
--- a/tests/bugs/modalg_6/bug27082_1
+++ b/tests/bugs/modalg_6/bug27082_1
@@ -16,3 +16,5 @@ don result
 smallview; l; fit
 bopcheck result
 checknbshapes result -m UnifySameDomain -face 22 -edge 40
+
+checkview -screenshot -2d -path ${imagedir}/${test_image}.png

diff --git a/tests/bugs/modalg_6/bug27082_1i b/tests/bugs/modalg_6/bug27082_1i
index 0a0ff3b8883b0b367813dcd536d8ed13a872c864..1633beb636bc0e062054a77e154208dcd8bfce21 100644 (file)
--- a/tests/bugs/modalg_6/bug27082_1i
+++ b/tests/bugs/modalg_6/bug27082_1i
@@ -16,3 +16,5 @@ don result
 smallview; l; fit
 bopcheck result
 checknbshapes result -m UnifySameDomain -face 14 -edge 28
+
+checkview -screenshot -2d -path ${imagedir}/${test_image}.png

diff --git a/tests/bugs/modalg_6/bug27082_2 b/tests/bugs/modalg_6/bug27082_2
index cbe7bc5d5a33d94ea56e8ef741d2034c85ca96b4..359282c885c796bbbdd96ef773c7ad90544f6d61 100644 (file)
--- a/tests/bugs/modalg_6/bug27082_2
+++ b/tests/bugs/modalg_6/bug27082_2
@@ -15,3 +15,5 @@ don result
 smallview; l; fit
 bopcheck result
 checknbshapes result -m UnifySameDomain -face 37 -edge 94
+
+checkview -screenshot -2d -path ${imagedir}/${test_image}.png

diff --git a/tests/bugs/modalg_6/bug27082_2i b/tests/bugs/modalg_6/bug27082_2i
index 82ee825b220fbd893345ac743a7d85a1924d39ce..0ded3c135d24e3462124dfcbfa558a3720dc4ff1 100644 (file)
--- a/tests/bugs/modalg_6/bug27082_2i
+++ b/tests/bugs/modalg_6/bug27082_2i
@@ -15,3 +15,5 @@ don result
 smallview; l; fit
 bopcheck result
 checknbshapes result -m UnifySameDomain -face 33 -edge 86
+
+checkview -screenshot -2d -path ${imagedir}/${test_image}.png

diff --git a/tests/bugs/modalg_6/bug27082_3 b/tests/bugs/modalg_6/bug27082_3
index 4728c05a2e8a0ebf950b137b8a290e2c48d107a0..dcdf7653b040eebdaa36e5b95ebc63570551b250 100644 (file)
--- a/tests/bugs/modalg_6/bug27082_3
+++ b/tests/bugs/modalg_6/bug27082_3
@@ -13,3 +13,5 @@ don result
 smallview; l; fit
 bopcheck result
 checknbshapes result -m UnifySameDomain -face 249 -edge 600
+
+checkview -screenshot -2d -path ${imagedir}/${test_image}.png

diff --git a/tests/bugs/modalg_6/bug27082_3i b/tests/bugs/modalg_6/bug27082_3i
index 49d91b0e32030c641e2f764d892c96f96ad02bc5..a48b0bb422ec45ed6eae09e60a6e927079ccaa84 100644 (file)
--- a/tests/bugs/modalg_6/bug27082_3i
+++ b/tests/bugs/modalg_6/bug27082_3i
@@ -13,3 +13,5 @@ don result
 smallview; l; fit
 bopcheck result
 checknbshapes result -m UnifySameDomain -face 225 -edge 576
+
+checkview -screenshot -2d -path ${imagedir}/${test_image}.png

diff --git a/tests/bugs/modalg_6/bug27309 b/tests/bugs/modalg_6/bug27309
new file mode 100644 (file)
index 0000000..aa3a0ab
--- /dev/null
+++ b/tests/bugs/modalg_6/bug27309
@@ -0,0 +1,17 @@
+puts "============"
+puts "OCC27309"
+puts "============"
+puts ""
+###############################
+## Result of UnifySameDomain has InvalidMultiConnexity error
+###############################
+
+restore [locate_data_file bug27309_shape.brep] a
+unifysamedom result a
+
+don result
+smallview; l; fit
+checkshape result
+checknbshapes result -m UnifySameDomain -face 28 -edge 66
+
+checkview -screenshot -2d -path ${imagedir}/${test_image}.png