From 569dddb77cc6997d2b95487ee39529d80ffa0386 Mon Sep 17 00:00:00 2001
From: apo <apo@opencascade.com>
Date: Fri, 24 Nov 2006 13:48:15 +0000
Subject: [PATCH] Tune holder presentations creation

---
 src/VISU_I/VISU_ColoredPrs3dCache_i.cc  | 108 +++++++++++++++---------
 src/VISU_I/VISU_ColoredPrs3dCache_i.hh  |  11 +++
 src/VISU_I/VISU_ColoredPrs3dHolder_i.cc |  33 +++-----
 src/VISU_I/VISU_ColoredPrs3dHolder_i.hh |   6 ++
 src/VISU_I/VISU_ColoredPrs3d_i.cc       |  14 +--
 src/VISU_I/VISU_ColoredPrs3d_i.hh       |   6 --
 src/VISU_I/VISU_Prs3d_i.cc              |   8 ++
 src/VISU_I/VISU_Prs3d_i.hh              |   3 +
 8 files changed, 110 insertions(+), 79 deletions(-)

diff --git a/src/VISU_I/VISU_ColoredPrs3dCache_i.cc b/src/VISU_I/VISU_ColoredPrs3dCache_i.cc
index dd16a331..a6ef3c79 100644
--- a/src/VISU_I/VISU_ColoredPrs3dCache_i.cc
+++ b/src/VISU_I/VISU_ColoredPrs3dCache_i.cc
@@ -85,45 +85,51 @@ VISU::ColoredPrs3dCache_i
 
 
 //----------------------------------------------------------------------------
+template<unsigned int colored_prs3d_type_enum> 
 int
-VISU::ColoredPrs3dCache_i
-::IsPossible(VISU::VISUType theType,
-	     const VISU::ColoredPrs3dHolder::BasicInput& theInput)
+CheckIsPossible(const VISU::ColoredPrs3dHolder::BasicInput& theInput)
 {
-  //cout << "VISU::ColoredPrs3dCache_i::IsPossible " << endl;
   VISU::Result_i* aResult = dynamic_cast<VISU::Result_i*>( VISU::GetServant(theInput.myResult).in() );
   std::string aMeshName = theInput.myMeshName.in();
   VISU::Entity anEntity = theInput.myEntity;
   std::string aFieldName = theInput.myFieldName.in();
-  long aTimeStampNumber = theInput.myTimeStampNumber;
-
-  bool ok = 0;
-  switch( theType ){
-    case TSCALARMAP: ok = VISU::ScalarMap_i::IsPossible(aResult,aMeshName,anEntity,aFieldName,aTimeStampNumber);
-      break;
-    case TISOSURFACE: ok = VISU::IsoSurfaces_i::IsPossible(aResult,aMeshName,anEntity,aFieldName,aTimeStampNumber); 
-      break;
-    case TDEFORMEDSHAPE: ok = VISU::DeformedShape_i::IsPossible(aResult,aMeshName,anEntity,aFieldName,aTimeStampNumber);
-      break;
-    case TSCALARMAPONDEFORMEDSHAPE: ok = VISU::ScalarMapOnDeformedShape_i::IsPossible(aResult,aMeshName,anEntity,aFieldName,aTimeStampNumber);
-      break;
-    case TGAUSSPOINTS: ok = VISU::GaussPoints_i::IsPossible(aResult,aMeshName,anEntity,aFieldName,aTimeStampNumber);
-      break;
-    case TPLOT3D: ok = VISU::Plot3D_i::IsPossible(aResult,aMeshName,anEntity,aFieldName,aTimeStampNumber);
-      break;
-    case TCUTPLANES: ok = VISU::CutPlanes_i::IsPossible(aResult,aMeshName,anEntity,aFieldName,aTimeStampNumber);
-      break;
-    case TCUTLINES: ok = VISU::CutLines_i::IsPossible(aResult,aMeshName,anEntity,aFieldName,aTimeStampNumber);
-      break;
-    case TVECTORS: ok = VISU::Vectors_i::IsPossible(aResult,aMeshName,anEntity,aFieldName,aTimeStampNumber);
-      break;
-    case TSTREAMLINES: ok = VISU::StreamLines_i::IsPossible(aResult,aMeshName,anEntity,aFieldName,aTimeStampNumber);
-      break;
-  default: break;
+  CORBA::Long aTimeStampNumber = theInput.myTimeStampNumber;
+
+  typedef typename VISU::TL::TColoredEnum2Type<colored_prs3d_type_enum>::TResult TColoredPrs3d;
+  return TColoredPrs3d::IsPossible(aResult,aMeshName,anEntity,aFieldName,aTimeStampNumber);
+}
+
+//----------------------------------------------------------------------------
+int
+VISU::ColoredPrs3dCache_i
+::IsPossible(VISU::VISUType theType,
+	     const VISU::ColoredPrs3dHolder::BasicInput& theInput)
+{
+  //cout << "VISU::ColoredPrs3dCache_i::IsPossible " << endl;
+  switch(theType){
+  case TSCALARMAP:
+    return CheckIsPossible<TSCALARMAP>(theInput);
+  case TGAUSSPOINTS:
+    return CheckIsPossible<TGAUSSPOINTS>(theInput);
+  case TDEFORMEDSHAPE:
+    return CheckIsPossible<TDEFORMEDSHAPE>(theInput);
+  case TSCALARMAPONDEFORMEDSHAPE:
+    return CheckIsPossible<TSCALARMAPONDEFORMEDSHAPE>(theInput);
+  case TISOSURFACE:
+    return CheckIsPossible<TISOSURFACE>(theInput);
+  case TSTREAMLINES:
+    return CheckIsPossible<TSTREAMLINES>(theInput);
+  case TPLOT3D:
+    return CheckIsPossible<TPLOT3D>(theInput);
+  case TCUTPLANES:
+    return CheckIsPossible<TCUTPLANES>(theInput);
+  case TCUTLINES:
+    return CheckIsPossible<TCUTLINES>(theInput);
+  case TVECTORS:
+    return CheckIsPossible<TVECTORS>(theInput);
   }
 
-  //cout << "returned " << ok << endl;
-  return ok;
+  return 0;
 }
 
 
@@ -164,7 +170,11 @@ VISU::ColoredPrs3dCache_i
   if(IsPossible(theType, theInput))
   {
     VISU::ColoredPrs3dHolder_i* aHolder = new VISU::ColoredPrs3dHolder_i(theType, theInput, *this);
-    CreatePrs(theType, theInput, aHolder);
+    VISU::ColoredPrs3d_i* aColoredPrs3d = CreateColoredPrs3d(theType, theInput);
+    std::string aComment = std::string("myComment=") + aColoredPrs3d->GetComment();
+    std::string aName = aColoredPrs3d->GenerateName().latin1();
+    aHolder->PublishInStudy(aName, aComment);
+    RegisterInHolder(aColoredPrs3d, aHolder);
     return aHolder->_this();
   }
   return VISU::ColoredPrs3dHolder::_nil();
@@ -212,7 +222,7 @@ std::string
 VISU::ColoredPrs3dCache_i
 ::GetFolderName() 
 { 
-  return "Cache"; 
+  return "3D Cache System"; 
 }
 
 void
@@ -260,25 +270,43 @@ CreateHolder2GetDeviceByEnum(VISU::VISUType theType,
 //----------------------------------------------------------------------------
 VISU::ColoredPrs3d_i*
 VISU::ColoredPrs3dCache_i
-::CreatePrs(VISU::VISUType theType,
-	    VISU::ColoredPrs3dHolder::BasicInput theInput,
-	    VISU::ColoredPrs3dHolder_i* theHolder)
+::CreateColoredPrs3d(VISU::VISUType theType,
+		     VISU::ColoredPrs3dHolder::BasicInput theInput)
 {
   VISU::ColoredPrs3d_i* aPrs3d = VISU::CreatePrs3d_i(theType, GetStudyDocument(), ColoredPrs3d_i::ERegisterInCache);
-  aPrs3d->SetHolderEntry( theHolder->GetEntry() );
   aPrs3d->SetResultObject( theInput.myResult );  
   aPrs3d->SetMeshName( theInput.myMeshName );  
   aPrs3d->SetEntity( theInput.myEntity );  
   aPrs3d->SetFieldName( theInput.myFieldName );  
   aPrs3d->SetTimeStampNumber( theInput.myTimeStampNumber );
   aPrs3d->Apply();
-
-  GetLastVisitedPrsList(theHolder).push_back(aPrs3d);
-  
   return aPrs3d;
 }
 
 
+//----------------------------------------------------------------------------
+VISU::ColoredPrs3d_i*
+VISU::ColoredPrs3dCache_i
+::RegisterInHolder(VISU::ColoredPrs3d_i* thePrs3d,
+		   VISU::ColoredPrs3dHolder_i* theHolder)
+{
+  thePrs3d->SetHolderEntry( theHolder->GetEntry() );
+  GetLastVisitedPrsList(theHolder).push_back(thePrs3d);  
+  return thePrs3d;
+}
+
+
+//----------------------------------------------------------------------------
+VISU::ColoredPrs3d_i*
+VISU::ColoredPrs3dCache_i
+::CreatePrs(VISU::VISUType theType,
+	    VISU::ColoredPrs3dHolder::BasicInput theInput,
+	    VISU::ColoredPrs3dHolder_i* theHolder)
+{
+  return RegisterInHolder(CreateColoredPrs3d(theType, theInput), theHolder);
+}
+
+
 //----------------------------------------------------------------------------
 VISU::TLastVisitedPrsList&
 VISU::ColoredPrs3dCache_i
diff --git a/src/VISU_I/VISU_ColoredPrs3dCache_i.hh b/src/VISU_I/VISU_ColoredPrs3dCache_i.hh
index b4b970c9..4eb1f3cb 100644
--- a/src/VISU_I/VISU_ColoredPrs3dCache_i.hh
+++ b/src/VISU_I/VISU_ColoredPrs3dCache_i.hh
@@ -127,6 +127,17 @@ namespace VISU
     VISU::ColoredPrs3dCache_ptr
     GetInstance(SALOMEDS::Study_ptr theStudy);
 
+    //! Creates corresponding ColoredPrs3d instance and apply the BasicInput
+    VISU::ColoredPrs3d_i*
+    CreateColoredPrs3d(VISU::VISUType theType,
+		       VISU::ColoredPrs3dHolder::BasicInput theInput);
+
+    //! Registers the given ColoredPrs3d instance for the ColoredPrs3dHolder
+    VISU::ColoredPrs3d_i*
+    RegisterInHolder(VISU::ColoredPrs3d_i* thePrs3d,
+		     VISU::ColoredPrs3dHolder_i* theHolder);
+
+    //! Creates and registers corresponding ColoredPrs3d instance for the ColoredPrs3dHolder
     VISU::ColoredPrs3d_i*
     CreatePrs(VISU::VISUType theType,
 	      VISU::ColoredPrs3dHolder::BasicInput theInput,
diff --git a/src/VISU_I/VISU_ColoredPrs3dHolder_i.cc b/src/VISU_I/VISU_ColoredPrs3dHolder_i.cc
index e67888aa..5825e08b 100644
--- a/src/VISU_I/VISU_ColoredPrs3dHolder_i.cc
+++ b/src/VISU_I/VISU_ColoredPrs3dHolder_i.cc
@@ -69,27 +69,6 @@ VISU::ColoredPrs3dHolder_i
   myCache(theCache)
 {
   if(MYDEBUG) MESSAGE("ColoredPrs3dHolder_i::ColoredPrs3dHolder_i - this = "<<this);
-
-  QString aType;
-  switch( theType ){
-    case TSCALARMAP:                aType = "SCALARMAP"; break;
-    case TISOSURFACE:               aType = "ISOSURFACES"; break;
-    case TDEFORMEDSHAPE:            aType = "DEFORMEDSHAPE"; break;
-    case TSCALARMAPONDEFORMEDSHAPE: aType = "SCALARMAPONDEFORMEDSHAPE"; break;
-    case TGAUSSPOINTS:              aType = "GAUSSPOINTS"; break;
-    case TPLOT3D:                   aType = "PLOT3D"; break;
-    case TCUTPLANES:                aType = "CUTPLANES"; break;
-    case TCUTLINES:                 aType = "CUTLINES"; break;
-    case TVECTORS:                  aType = "VECTORS"; break;
-    case TSTREAMLINES:              aType = "STREAMLINES"; break;
-    default: break;
-  }
-
-  SetName(GenerateName().latin1(), false);
-  CORBA::String_var anIOR = GetID();
-  std::string aFatherEntry = theCache.GetEntry();
-  QString aComment = QString("myComment=") + aType;
-  CreateAttributes(GetStudyDocument(), aFatherEntry, "", anIOR.in(), GetName(), "", aComment, true);
 }
 
 
@@ -101,6 +80,18 @@ VISU::ColoredPrs3dHolder_i
 }
 
 
+//----------------------------------------------------------------------------
+void
+VISU::ColoredPrs3dHolder_i
+::PublishInStudy(const std::string& theName, 
+		 const std::string& theComment)
+{
+  SetName(theName, false);
+  CORBA::String_var anIOR = GetID();
+  std::string aFatherEntry = myCache.GetEntry();
+  CreateAttributes(GetStudyDocument(), aFatherEntry, "", anIOR.in(), GetName(), "", theComment, true);
+}
+
 //----------------------------------------------------------------------------
 CORBA::Boolean 
 VISU::ColoredPrs3dHolder_i
diff --git a/src/VISU_I/VISU_ColoredPrs3dHolder_i.hh b/src/VISU_I/VISU_ColoredPrs3dHolder_i.hh
index 204924c9..e78aac0d 100644
--- a/src/VISU_I/VISU_ColoredPrs3dHolder_i.hh
+++ b/src/VISU_I/VISU_ColoredPrs3dHolder_i.hh
@@ -47,6 +47,8 @@ namespace VISU
   {
     ColoredPrs3dHolder_i();
     ColoredPrs3dHolder_i(const ColoredPrs3dHolder_i&);
+
+    friend class ColoredPrs3dCache_i;
   public:
     //----------------------------------------------------------------------------
     //! A constructor to create an instance of the class
@@ -120,6 +122,10 @@ namespace VISU
 
     //----------------------------------------------------------------------------
   private:
+    void
+    PublishInStudy(const std::string& theName, 
+		   const std::string& theComment);
+
     VISU::ColoredPrs3dCache_i& myCache;
     static const std::string myComment;
     static int myNbHolders;
diff --git a/src/VISU_I/VISU_ColoredPrs3d_i.cc b/src/VISU_I/VISU_ColoredPrs3d_i.cc
index 3098f582..9fa199d3 100644
--- a/src/VISU_I/VISU_ColoredPrs3d_i.cc
+++ b/src/VISU_I/VISU_ColoredPrs3d_i.cc
@@ -362,21 +362,11 @@ void
 VISU::ColoredPrs3d_i
 ::SetHolderEntry(const std::string& theEntry)
 {
-  myHolderEntry = theEntry; 
+  myHolderEntry = theEntry;
+  SetIO(new SALOME_InteractiveObject(theEntry.c_str(), "VISU", GetName().c_str()));
 }
 
 
-//----------------------------------------------------------------------------
-Handle(SALOME_InteractiveObject)
-VISU::ColoredPrs3d_i
-::GetIO()
-{
-  Handle(SALOME_InteractiveObject) anIO = TSuperClass::GetIO();
-  if(myHolderEntry != "")
-    anIO->setEntry(myHolderEntry.c_str());
-  return anIO;
-}
-
 //----------------------------------------------------------------------------
 CORBA::Boolean 
 VISU::ColoredPrs3d_i
diff --git a/src/VISU_I/VISU_ColoredPrs3d_i.hh b/src/VISU_I/VISU_ColoredPrs3d_i.hh
index eb46188e..19ad5515 100644
--- a/src/VISU_I/VISU_ColoredPrs3d_i.hh
+++ b/src/VISU_I/VISU_ColoredPrs3d_i.hh
@@ -227,12 +227,6 @@ namespace VISU
     void
     SetHolderEntry(const std::string& theEntry);
 
-    //----------------------------------------------------------------------------
-    //! Create and return the interactive object
-    virtual
-    Handle(SALOME_InteractiveObject)
-    GetIO();
-
     //----------------------------------------------------------------------------
   public:
     /*!
diff --git a/src/VISU_I/VISU_Prs3d_i.cc b/src/VISU_I/VISU_Prs3d_i.cc
index 126ad995..1ac89d41 100644
--- a/src/VISU_I/VISU_Prs3d_i.cc
+++ b/src/VISU_I/VISU_Prs3d_i.cc
@@ -395,6 +395,14 @@ VISU::Prs3d_i
   return myIO;
 }
 
+//----------------------------------------------------------------------------
+void
+VISU::Prs3d_i
+::SetIO(const Handle(SALOME_InteractiveObject)& theIO)
+{
+  myIO = theIO;
+}
+
 //----------------------------------------------------------------------------
 void
 VISU::Prs3d_i
diff --git a/src/VISU_I/VISU_Prs3d_i.hh b/src/VISU_I/VISU_Prs3d_i.hh
index ce9415d7..3afc8c0f 100644
--- a/src/VISU_I/VISU_Prs3d_i.hh
+++ b/src/VISU_I/VISU_Prs3d_i.hh
@@ -273,6 +273,9 @@ namespace VISU
     void 
     SetSObject(SALOMEDS::SObject_var theSObject);
 
+    void
+    SetIO(const Handle(SALOME_InteractiveObject)& theIO);
+
   protected:
     vtkTimeStamp myUpdateTime;
     vtkTimeStamp myParamsTime;
-- 
2.39.2