From f87daaedd5edce42d9927441f749397fdf888b80 Mon Sep 17 00:00:00 2001
From: apo <apo@opencascade.com>
Date: Thu, 19 Jul 2007 11:16:02 +0000
Subject: [PATCH] To improve naming of stored files

---
 src/VISU_I/VISU_MultiResult_i.cc |  2 +-
 src/VISU_I/VISU_ResultUtils.cc   | 23 +++++++++++++++++++----
 src/VISU_I/VISU_ResultUtils.hh   |  6 ++++++
 src/VISU_I/VISU_Result_i.cc      | 13 ++++++-------
 src/VISU_I/VISU_Result_i.hh      | 16 ++++++++++++++--
 5 files changed, 46 insertions(+), 14 deletions(-)

diff --git a/src/VISU_I/VISU_MultiResult_i.cc b/src/VISU_I/VISU_MultiResult_i.cc
index 3a2ee405..8f211473 100644
--- a/src/VISU_I/VISU_MultiResult_i.cc
+++ b/src/VISU_I/VISU_MultiResult_i.cc
@@ -683,7 +683,7 @@ VISU::MultiResult_i
 
   {
     std::string aBase, aSuffix;
-    SplitName(GetFileName(), aBase, aSuffix);
+    VISU::SplitName(GetFileName(), aBase, aSuffix);
     INITMSG(MYDEBUG, "aBase = '"<<aBase<<"'; aSuffix = '"<<aSuffix<<"'\n");
 
     QFileInfo aFileInfo(myMultiprObj.getSequentialMEDFilename());
diff --git a/src/VISU_I/VISU_ResultUtils.cc b/src/VISU_I/VISU_ResultUtils.cc
index fd2be078..9f8d37d0 100644
--- a/src/VISU_I/VISU_ResultUtils.cc
+++ b/src/VISU_I/VISU_ResultUtils.cc
@@ -155,7 +155,7 @@ namespace VISU
 	    std::string& theSuffix,
 	    char theDelimeter)
   {
-    size_t aPosition = theName.find(theDelimeter);
+    size_t aPosition = theName.rfind(theDelimeter);
     if(aPosition == std::string::npos){
       theBase = theName;
       theSuffix = "";
@@ -168,9 +168,21 @@ namespace VISU
   }
   
 
+  //----------------------------------------------------------------------------
+  std::string
+  MakeFileName(const std::string& theName,
+	       const void* thePointer)
+  {
+    std::ostringstream aStream;    
+    aStream<<theName<<":"<<thePointer;
+    return aStream.str();
+  }
+  
+
   //----------------------------------------------------------------------------
   QString 
-  GenerateFieldName (const string& theName, const string& theUnits)
+  GenerateFieldName(const std::string& theName, 
+		    const std::string& theUnits)
   {
     QString aName;
     const string tmp (theUnits.size(),' ');
@@ -186,8 +198,8 @@ namespace VISU
   //----------------------------------------------------------------------------
   void
   CreateReference(_PTR(Study) theStudyDocument,
-		  const string& theFatherEntry, 
-		  const string& theRefEntry)
+		  const std::string& theFatherEntry, 
+		  const std::string& theRefEntry)
   {
     _PTR(StudyBuilder) aStudyBuilder = theStudyDocument->NewBuilder();
     _PTR(SObject) aFather = theStudyDocument->FindObjectID(theFatherEntry);
@@ -644,6 +656,9 @@ namespace VISU
   {
     QFileInfo aSourceFileInfo(theSourceFileName);
     QFileInfo aTargetFileInfo(theTargetFileName);
+    if(aSourceFileInfo.absFilePath() == aTargetFileInfo.absFilePath())
+      return true;
+
     QString aCommand;
     aCommand.sprintf("%s %s %s",
 		     COPY_COMMAND,
diff --git a/src/VISU_I/VISU_ResultUtils.hh b/src/VISU_I/VISU_ResultUtils.hh
index f6807a7b..357dfd49 100644
--- a/src/VISU_I/VISU_ResultUtils.hh
+++ b/src/VISU_I/VISU_ResultUtils.hh
@@ -82,6 +82,12 @@ namespace VISU
 	    char theDelimeter = ':');
 
 
+  //----------------------------------------------------------------------------
+  std::string
+  MakeFileName(const std::string& theName,
+	       const void* thePointer);
+    
+    
   //----------------------------------------------------------------------------
   void
   BuildEntities(Result_i* theResult,
diff --git a/src/VISU_I/VISU_Result_i.cc b/src/VISU_I/VISU_Result_i.cc
index 08b9fab4..12fad523 100644
--- a/src/VISU_I/VISU_Result_i.cc
+++ b/src/VISU_I/VISU_Result_i.cc
@@ -508,8 +508,8 @@ VISU::Result_i
 
     if(mySourceId == eRestoredFile){
       QString aTmpDir(SALOMEDS_Tool::GetTmpDir().c_str());
-      QString aSuffix = QString::number(size_t(this));
-      QString aPathToCopy(aTmpDir + myFileInfo.fileName() + ":" + aSuffix);
+      std::string aFileName = VISU::MakeFileName(myFileInfo.fileName().latin1(), this);
+      QString aPathToCopy(aTmpDir + aFileName.c_str());
       if(!VISU::CopyFile(myFileInfo.absFilePath().latin1(), aPathToCopy.latin1()))
 	return NULL;
 
@@ -809,7 +809,7 @@ VISU::Result_i
       } else if (!theIsMultiFile) {
 	mySourceId = eRestoredFile;
       } else {
-	mySourceId = eFile;
+	mySourceId = eSavedFile;
       }
       if(MYDEBUG) MESSAGE("VISU::Result_i::Restore - mySourceId = " << mySourceId);
       myInput.reset(CreateConvertor(myFileInfo.filePath().latin1()));
@@ -1324,8 +1324,8 @@ VISU::Result_i
 ::GetFileName() const
 {
   if(GetSourceId() == eFile)
-    return QString("%1:%2").arg(myFileName.c_str()).arg(size_t(this)).latin1();
-  
+    return VISU::MakeFileName(myFileName, this);
+
   return myFileName;
 }
 
@@ -1405,9 +1405,8 @@ VISU::Result_i
 ::~Result_i()
 {
   MESSAGE("Result_i::~Result_i() - this = "<<this);
-  if (GetSourceId() == eRestoredFile) {
+  if (GetSourceId() == eRestoredFile) 
     VISU::RemoveFile(myFileInfo.filePath().latin1());
-  }
 }
 
 //=======================================================================
diff --git a/src/VISU_I/VISU_Result_i.hh b/src/VISU_I/VISU_Result_i.hh
index 17d35e2b..40eff416 100644
--- a/src/VISU_I/VISU_Result_i.hh
+++ b/src/VISU_I/VISU_Result_i.hh
@@ -55,8 +55,20 @@ namespace VISU
     Result_i(const Result_i &);
 
   public:
-    enum ESourceId {eRestoredComponent = -2, eRestoredFile = -1, eFile = 1, eComponent = 2};
-    enum ECreationId {eImportFile, eCopyAndImportFile, eImportMed, eImportMedField};
+    enum ESourceId {
+      eRestoredComponent = -2, 
+      eRestoredFile = -1,  
+      eSavedFile = 0, 
+      eFile = 1, 
+      eComponent = 2
+    };
+
+    enum ECreationId {
+      eImportFile, 
+      eCopyAndImportFile, 
+      eImportMed, 
+      eImportMedField
+    };
 
     static
     Result_i*
-- 
2.39.2