fixed bug where "Always create result" was required

[modules/shaper.git] / src / FeaturesPlugin / FeaturesPlugin_Sewing.cpp

diff --git a/src/FeaturesPlugin/FeaturesPlugin_Sewing.cpp b/src/FeaturesPlugin/FeaturesPlugin_Sewing.cpp
index c8319e6fc09382bef1d12fda2e3882804fee8e3f..10694b9954d69601630538cd9c63624ff37cedc2 100644 (file)
--- a/src/FeaturesPlugin/FeaturesPlugin_Sewing.cpp
+++ b/src/FeaturesPlugin/FeaturesPlugin_Sewing.cpp
@@ -115,9 +115,9 @@ bool FeaturesPlugin_Sewing::isSewn(const ListOfShape& theInputs,
        ++anIt)
   {
     GeomShapePtr aShape = *anIt;
-    if (aShape.get() && aShape->isShell())
+    if (aShape.get())
     {
-      nbInputShells++;
+      if (aShape->isShell()) nbInputShells++;
       nbInputFaces += aShape->subShapes(GeomAPI_Shape::FACE, true).size();
     }
   }
@@ -144,5 +144,5 @@ bool FeaturesPlugin_Sewing::isSewn(const ListOfShape& theInputs,
     nbResultFaces = theResult->subShapes(GeomAPI_Shape::FACE, true).size();
   }
 
-  return (nbResultShells > nbInputShells || (nbResultShells == nbInputShells && nbResultFaces > nbInputFaces));
+  return ((nbInputShells == 0 && nbResultShells > 0) || (nbResultShells < nbInputShells) || (nbResultShells == nbInputShells && nbResultFaces > nbInputFaces));
 }