From 711560136859ac1fdc980aaacb38d039c0ccee3d Mon Sep 17 00:00:00 2001
From: spo <sergey.pokhodenko@opencascade.com>
Date: Fri, 18 Sep 2015 11:30:49 +0300
Subject: [PATCH] Fix TestParameterRename

---
 .../ParametersPlugin_EvalListener.cpp          | 18 +++++-------------
 .../ParametersPlugin_EvalListener.h            |  1 +
 2 files changed, 6 insertions(+), 13 deletions(-)

diff --git a/src/ParametersPlugin/ParametersPlugin_EvalListener.cpp b/src/ParametersPlugin/ParametersPlugin_EvalListener.cpp
index 8b5e915ee..183a027f4 100644
--- a/src/ParametersPlugin/ParametersPlugin_EvalListener.cpp
+++ b/src/ParametersPlugin/ParametersPlugin_EvalListener.cpp
@@ -280,17 +280,9 @@ void ParametersPlugin_EvalListener::renameInAttribute(
 }
 
 void ParametersPlugin_EvalListener::renameInDependants(std::shared_ptr<ModelAPI_ResultParameter> theResultParameter,
+                                                       const std::string& theOldName,
                                                        const std::string& theNewName)
 {
-  // get parameter feature for the result
-  std::shared_ptr<ParametersPlugin_Parameter> aParameter =
-      std::dynamic_pointer_cast<ParametersPlugin_Parameter>(
-          ModelAPI_Feature::feature(theResultParameter));
-  if (!aParameter.get())
-    return;
-
-  std::string anOldName = aParameter->string(ParametersPlugin_Parameter::VARIABLE_ID())->value();
-
   std::set<std::shared_ptr<ModelAPI_Attribute> > anAttributes =
       theResultParameter->data()->refsToMe();
   std::set<std::shared_ptr<ModelAPI_Attribute> >::const_iterator anAttributeIt =
@@ -303,10 +295,10 @@ void ParametersPlugin_EvalListener::renameInDependants(std::shared_ptr<ModelAPI_
               anAttribute->owner());
       if (aParameter.get())
         // Rename
-        renameInParameter(aParameter, anOldName, theNewName);
+        renameInParameter(aParameter, theOldName, theNewName);
     } else
         // Rename
-        renameInAttribute(anAttribute, anOldName, theNewName);
+        renameInAttribute(anAttribute, theOldName, theNewName);
   }
 }
 
@@ -366,7 +358,7 @@ void ParametersPlugin_EvalListener::processObjectRenamedEvent(
     return;
   }
 
-  renameInDependants(aResultParameter, aMessage->newName());
+  renameInDependants(aResultParameter, aMessage->oldName(), aMessage->newName());
 }
 
 void ParametersPlugin_EvalListener::processReplaceParameterEvent(
@@ -391,5 +383,5 @@ void ParametersPlugin_EvalListener::processReplaceParameterEvent(
   double aRealValue = aResultParameter->data()->real(ModelAPI_ResultParameter::VALUE())->value();
   std::string aValue = toStdString(aRealValue);
 
-  renameInDependants(aResultParameter, aValue);
+  renameInDependants(aResultParameter, aResultParameter->data()->name(), aValue);
 }
diff --git a/src/ParametersPlugin/ParametersPlugin_EvalListener.h b/src/ParametersPlugin/ParametersPlugin_EvalListener.h
index 56bb88dd4..c24831780 100644
--- a/src/ParametersPlugin/ParametersPlugin_EvalListener.h
+++ b/src/ParametersPlugin/ParametersPlugin_EvalListener.h
@@ -43,6 +43,7 @@ class ParametersPlugin_EvalListener : public Events_Listener
                          const std::string& theOldName,
                          const std::string& theNewName);
   void renameInDependants(std::shared_ptr<ModelAPI_ResultParameter> theResultParameter,
+                          const std::string& theOldName,
                           const std::string& theNewName);
 
  private:
-- 
2.39.2