Bug 138965 - Changing attributename or methodname results in truncating list in edit window (parameters not shown)
Summary: Changing attributename or methodname results in truncating list in edit windo...
Status: RESOLVED DUPLICATE of bug 114477
Alias: None
Product: umbrello
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Umbrello Development Group
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-12-18 15:29 UTC by gmud
Modified: 2006-12-21 14:23 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description gmud 2006-12-18 15:29:37 UTC
Version:           1.5.52-svn-171106 (using KDE 3.5.5, Gentoo)
Compiler:          gcc version 3.4.6 (Gentoo 3.4.6-r1, ssp-3.4.5-1.0, pie-8.7.9)
OS:                Linux (i686) release 2.6.18-suspend2

Changing attributename or methodname of an already exiting object in class diagram view results in truncating list in edit window (parameters not shown).

Seems not to be related to #114477.
Comment 1 Michael Palomas 2006-12-20 22:21:51 UTC
Hello,  i was able to reproduce this bug with 1.5.6 svn version.

According to me, it seems that problem comes in dialogs/classifierlistpage.cpp, in slotDoubleClick, the list item display string is updated only with the classifier name (getName()) , maybe it should be the entire classifier string representation ( toString(Uml::st_SigNoVis) ).

Sorry i'm quite a noob in diff/patch, but the following seems to help :


Index: umbrello/dialogs/classifierlistpage.cpp
===================================================================
--- umbrello/dialogs/classifierlistpage.cpp	(révision 615256)
+++ umbrello/dialogs/classifierlistpage.cpp	(copie de travail)
@@ -500,7 +500,7 @@
     }
 
     if( listItem->showPropertiesDialog(this) ) {
-        m_pItemListLB->changeItem( listItem->getName(), m_pItemListLB->index(item) );
+      m_pItemListLB->changeItem( listItem->toString(Uml::st_SigNoVis), m_pItemListLB->index(item) );
     }
 }


I'm not an Umbrello developer, i was just looking at the source code, so maybe this is not a correct fix and has bad side effects, so please test and post feedback.
Comment 2 Oliver Kellogg 2006-12-21 14:23:09 UTC
Thanks for the fix - I applied it on branches/KDE/3.5/kdesdk (r615411)

*** This bug has been marked as a duplicate of 114477 ***