Bug 69244 - "role A properties" should give class name
Summary: "role A properties" should give class name
Status: RESOLVED FIXED
Alias: None
Product: umbrello
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR wishlist
Target Milestone: ---
Assignee: Umbrello Development Group
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-11-28 22:56 UTC by Jonathan Riddell
Modified: 2006-09-01 17:12 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 Jonathan Riddell 2003-11-28 22:56:33 UTC
Version:           1.2-beta (using KDE 3.1.93 (CVS >= 20031028),  (testing/unstable))
Compiler:          gcc version 3.3.2 (Debian)
OS:          Linux (i686) release 2.4.22-1-686

From Alexander Fieroch <fieroch-AT-web.de>

Adding properies/roles for an association should not only show "Role A
Properties" and "Role B Properties" but also the names of the classes
because I do not know what class was A or B.
Comment 1 Filip Miletic 2005-06-23 15:23:14 UTC
I second that (i.e. was about to file the same report).

f
Comment 2 Daniel Calviño Sánchez 2006-09-01 17:12:24 UTC
SVN commit 579759 by danxuliu:

Role properties now also show the name of the widget.
BUG: 69244

 M  +9 -10     assocrolepage.cpp  


--- branches/KDE/3.5/kdesdk/umbrello/umbrello/dialogs/assocrolepage.cpp #579758:579759
@@ -1,8 +1,3 @@
-/*
- *  copyright (C) 2003-2004
- *  Umbrello UML Modeller Authors <uml-devel@ uml.sf.net>
- */
-
 /***************************************************************************
  *                                                                         *
  *   This program is free software; you can redistribute it and/or modify  *
@@ -10,6 +5,8 @@
  *   the Free Software Foundation; either version 2 of the License, or     *
  *   (at your option) any later version.                                   *
  *                                                                         *
+ *   copyright (C) 2003-2006                                               *
+ *   Umbrello UML Modeller Authors <uml-devel@ uml.sf.net>                 *
  ***************************************************************************/
 
 // own header
@@ -52,10 +49,12 @@
     QString nameB = m_pAssociationWidget->getRoleName(Uml::B);
     QString titleA = i18n("Role A Properties");
     QString titleB = i18n("Role B Properties");
-    if(!nameA.isEmpty())
-        titleA.append( "("+nameA+")");
-    if(!nameB.isEmpty())
-        titleB.append( "("+nameB+")");
+    QString widgetNameA = m_pAssociationWidget->getWidget(Uml::A)->getName();
+    QString widgetNameB = m_pAssociationWidget->getWidget(Uml::B)->getName();
+    if(!widgetNameA.isEmpty())
+        titleA.append(" (" + widgetNameA + ")");
+    if(!widgetNameB.isEmpty())
+        titleB.append(" (" + widgetNameB + ")");
 
     // general configuration of the GUI
     int margin = fontMetrics().height();
@@ -183,7 +182,7 @@
 
     m_ImplementationBRB = new QRadioButton(i18n("Implementation"), scopeBBG);
     scopeBLayout -> addWidget(m_ImplementationBRB);
-    
+
     scope = m_pAssociationWidget->getVisibility(Uml::B);
     if( scope == Uml::Visibility::Public )
         m_PublicBRB -> setChecked( true );