Bug 112531

Summary:	Cannot change type of existing attribute
Product:	[Applications] umbrello	Reporter:	Emil Karlén <emilkarlen>
Component:	general	Assignee:	Umbrello Development Group <umbrello-devel>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Can't change properties of attributes of class 'A'.

Description Emil Karlén 2005-09-13 14:51:38 UTC

Version:           1.4.89 (using KDE KDE 3.4.2)
Compiler:          gcc version 4.0.1 20050727 (Red Hat 4.0.1-5) 
OS:                Linux

using KDE v. 3.4.2-0.fc4.1.

1. Double-click on a class to open the Properties dialog.
2. Select "attributes".
3. Double-click an attribute.
4. click OK.
   I first tested it while changeing the type of the attribute, but
   I get the same response even if I don't.

Response:
 * in Swedish: "Attributnamnet du har valt används redan i den här operationen".
 * in free translation to english: "The attribute name you have chosen is already in use in this operation.".

: I can't change the attribute, and I get a complaint about a an operation.

5. If I change the name of the attribute the result is OK, but whenever I don't change anything or change anything else than the name, I get the error.

Comment 1 Emil Karlén 2005-09-13 14:53:41 UTC

Created attachment 12548 [details]
Can't change properties of attributes of class 'A'.

Comment 2 Oliver Kellogg 2005-09-13 21:28:33 UTC

SVN commit 460424 by okellogg:

apply(): Make sure the item returned by UMLClassifier::findChildObject()
 is not the dialog's own dedicated one.
BUG:112531


 M  +1 -1      umlattributedialog.cpp  
 M  +1 -1      umlentityattributedialog.cpp  
 M  +1 -1      umltemplatedialog.cpp  


--- branches/KDE/3.5/kdesdk/umbrello/umbrello/dialogs/umlattributedialog.cpp #460423:460424
@@ -153,7 +153,7 @@
     }
     UMLClassifier * pConcept = dynamic_cast<UMLClassifier *>( m_pAttribute->parent() );
     UMLObject *o = pConcept->findChildObject(name);
-    if (o) {
+    if (o && o != m_pAttribute) {
         KMessageBox::error(this, i18n("The attribute name you have chosen is already being used in this operation."),
                            i18n("Attribute Name Not Unique"), false);
         m_pNameLE->setText( m_pAttribute->getName() );
--- branches/KDE/3.5/kdesdk/umbrello/umbrello/dialogs/umlentityattributedialog.cpp #460423:460424
@@ -197,7 +197,7 @@
     }
     UMLClassifier * pConcept = dynamic_cast<UMLClassifier *>( m_pEntityAttribute->parent() );
     UMLObject *o = pConcept->findChildObject(name);
-    if (o) {
+    if (o && o != m_pEntityAttribute) {
         KMessageBox::error(this, i18n("The entity attribute name you have chosen is already being used in this operation."),
                            i18n("Entity Attribute Name Not Unique"), false);
         m_pNameLE->setText( m_pEntityAttribute->getName() );
--- branches/KDE/3.5/kdesdk/umbrello/umbrello/dialogs/umltemplatedialog.cpp #460423:460424
@@ -129,7 +129,7 @@
     UMLClassifier * pClass = dynamic_cast<UMLClassifier *>( m_pTemplate->parent() );
     if (pClass) {
         UMLObject *o = pClass->findChildObject(name);
-        if (o) {
+        if (o && o != m_pTemplate) {
             KMessageBox::error(this, i18n("The template parameter name you have chosen is already being used in this operation."),
                                i18n("Template Name Not Unique"), false);
             m_pNameLE->setText( m_pTemplate->getName() );