From f39f9838e43879ca3a1582de7150ee651a7942cb Mon Sep 17 00:00:00 2001
From: anv <anv@opencascade.com>
Date: Thu, 27 Jun 2024 16:09:48 +0100
Subject: [PATCH] 0033751: Shape Healing - Use static values in case of an
 absent Resource file

If a Resource file with parameters for ShapeFix is absent, use static parameters instead if possible.
---
 src/Resource/Resource_Manager.cxx   |  6 ++++--
 src/Resource/Resource_Manager.hxx   |  4 ++++
 src/XSAlgo/XSAlgo_AlgoContainer.cxx | 11 +++--------
 3 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/src/Resource/Resource_Manager.cxx b/src/Resource/Resource_Manager.cxx
index e773d624d6..7526211ce6 100644
--- a/src/Resource/Resource_Manager.cxx
+++ b/src/Resource/Resource_Manager.cxx
@@ -62,7 +62,8 @@ Resource_Manager::Resource_Manager (const TCollection_AsciiString& theName,
                                     const TCollection_AsciiString& theUserDefaultsDirectory,
                                     const Standard_Boolean theIsVerbose)
 : myName (theName),
-  myVerbose (theIsVerbose)
+  myVerbose (theIsVerbose),
+  myInitialized(Standard_False)
 {
   if (!theDefaultsDirectory.IsEmpty())
   {
@@ -102,7 +103,7 @@ Resource_Manager::Resource_Manager (const TCollection_AsciiString& theName,
 }
 
 Resource_Manager::Resource_Manager(const Standard_CString aName,
-				   const Standard_Boolean Verbose) : myName(aName), myVerbose(Verbose)
+				   const Standard_Boolean Verbose) : myName(aName), myVerbose(Verbose), myInitialized(Standard_False)
 {
   OSD_Environment envDebug("ResourceDebug");
   Debug = (!envDebug.Value().IsEmpty()) ;
@@ -157,6 +158,7 @@ void Resource_Manager::Load(const TCollection_AsciiString& thePath,
 	   << "\". File not found or permission denied." << std::endl;
     return;
   }
+  myInitialized = Standard_True;
   Standard_Integer LineNumber = 1;
   while ((aKind = WhatKindOfLine(File, Token1, Token2)) != Resource_KOL_End) {
     switch (aKind) {
diff --git a/src/Resource/Resource_Manager.hxx b/src/Resource/Resource_Manager.hxx
index 0072a4c71f..6e31ae1e98 100644
--- a/src/Resource/Resource_Manager.hxx
+++ b/src/Resource/Resource_Manager.hxx
@@ -116,6 +116,9 @@ public:
 
   //! Returns internal Ref or User map with parameters 
   Standard_EXPORT Resource_DataMapOfAsciiStringAsciiString& GetMap(Standard_Boolean theRefMap = Standard_True);
+
+  //! Returns true if Resource have been found
+  Standard_Boolean IsInitialized() const { return myInitialized; }
   
 private:
 
@@ -129,6 +132,7 @@ private:
   Resource_DataMapOfAsciiStringAsciiString myUserMap;
   Resource_DataMapOfAsciiStringExtendedString myExtStrMap;
   Standard_Boolean myVerbose;
+  Standard_Boolean myInitialized;
 
 };
 
diff --git a/src/XSAlgo/XSAlgo_AlgoContainer.cxx b/src/XSAlgo/XSAlgo_AlgoContainer.cxx
index 9f639cbd68..d329d85a01 100644
--- a/src/XSAlgo/XSAlgo_AlgoContainer.cxx
+++ b/src/XSAlgo/XSAlgo_AlgoContainer.cxx
@@ -104,17 +104,12 @@ TopoDS_Shape XSAlgo_AlgoContainer::ProcessShape(const TopoDS_Shape& theShape,
   if (aContext.IsNull())
   {
     Standard_CString aRscfile = Interface_Static::CVal(thePrscfile);
-    if (aRscfile != nullptr && strlen(aRscfile) == 0)
+    aContext = new ShapeProcess_ShapeContext(theShape, aRscfile);
+    if (!aContext->ResourceManager()->IsInitialized())
     {
-      aContext = new ShapeProcess_ShapeContext(theShape, nullptr);
+      // If resource file wasn't found, use static values instead
       Interface_Static::FillMap(aContext->ResourceManager()->GetMap());
     }
-    else
-    {
-      if (!aRscfile)
-        aRscfile = thePrscfile;
-      aContext = new ShapeProcess_ShapeContext(theShape, aRscfile);
-    }
     aContext->SetDetalisation(theDetalisationLevel);
   }
   aContext->SetNonManifold(theNonManifold);