From: nds <nds@opencascade.com>
Date: Fri, 26 Feb 2016 15:48:46 +0000 (+0300)
Subject: Issue #1330 correction for attribute ref attr list. Crash of rectangle creation
X-Git-Tag: V_2.2.0~54
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=8c68ad2a530b1656e2ffa26ebdbc22f20482f361;p=modules%2Fshaper.git

Issue #1330 correction for attribute ref attr list. Crash of rectangle creation

Scenario: create rectangle, start fillet. Select a point, select another point, result: crash in the fillet feature
---

diff --git a/src/ModuleBase/ModuleBase_WidgetMultiSelector.cpp b/src/ModuleBase/ModuleBase_WidgetMultiSelector.cpp
index 1259441ca..6eff484be 100755
--- a/src/ModuleBase/ModuleBase_WidgetMultiSelector.cpp
+++ b/src/ModuleBase/ModuleBase_WidgetMultiSelector.cpp
@@ -310,6 +310,10 @@ bool ModuleBase_WidgetMultiSelector::setSelection(QList<ModuleBase_ViewerPrs>& t
 {
   QList<ModuleBase_ViewerPrs> aSkippedValues;
 
+  /// remove unused objects from the model attribute.
+  /// It should be performed before new attributes append.
+  removeUnusedAttributeObjects(theValues);
+
   QList<ModuleBase_ViewerPrs>::const_iterator anIt = theValues.begin(), aLast = theValues.end();
   bool isDone = false;
   for (; anIt != aLast; anIt++) {
@@ -332,7 +336,7 @@ bool ModuleBase_WidgetMultiSelector::setSelection(QList<ModuleBase_ViewerPrs>& t
   //}
 
   /// remove unused objects from the model attribute
-  removeUnusedAttributeObjects(theValues);
+  //removeUnusedAttributeObjects(theValues);
 
   theValues.clear();
   if (!aSkippedValues.empty())