Bug 121478 - State transition documentation does not appear in the Documentation window.
Summary: State transition documentation does not appear in the Documentation window.
Status: RESOLVED FIXED
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-02-06 16:22 UTC by Steve Hartmann
Modified: 2006-02-19 10:54 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 Steve Hartmann 2006-02-06 16:22:39 UTC
Version:           1.5.1 (using KDE 3.5.0 Level "a" , SUSE 10.0 UNSUPPORTED)
Compiler:          Target: i586-suse-linux
OS:                Linux (i686) release 2.6.13-15.7-smp

In a state machine diagram, the documentation window does not work for the state transitions.  One can go into the state transition properties window and enter the documentation, but that text does not appear in the "documentation window".  Also, if one enters text into the documention window while a state transition is selected, that text does not get associated with the selected state transition.

note:  The documentation window acts properly with states selected, I am suggesting that it should work the same way with transitions.
Comment 1 Oliver Kellogg 2006-02-19 10:54:48 UTC
SVN commit 511242 by okellogg:

setSelected(): Sprinkling in some kapp->processEvents() disspells the mystery
 of the missing selection adornments.
BUG:121478
BUG:122063


 M  +2 -0      ChangeLog  
 M  +3 -10     umbrello/associationwidget.cpp  


--- branches/KDE/3.5/kdesdk/umbrello/ChangeLog #511241:511242
@@ -6,6 +6,7 @@
   http://sourceforge.net/mailarchive/forum.php?thread_id=9558795&forum_id=472
 * Bugs/wishes from http://bugs.kde.org:
 * Import Rose model files (81364)
+* Documentation for association roles not saved (105661)
 * Default data types not added for old Java generator (115991)
 * Custom operations in sequence diagrams become class operations (120337)
 * Multiplicity labels positioned incorrectly when moving entities (120598)
@@ -13,6 +14,7 @@
 * Unable to delete entity from entity list in "UML Model" frame (120750)
 * Interface names not italicized in diagram view (120761)
 * Cannot Resize Sequence Diagram Synchronous Messages (120910)
+* Documentation for associations is not retained (121478, 122063)
 * Crash when importing Python files (121952)
 * "void" is imported as class and not datatypes (122184)
 * Crash when creating a datatype with the same name as a class (122188)
--- branches/KDE/3.5/kdesdk/umbrello/umbrello/associationwidget.cpp #511241:511242
@@ -20,6 +20,7 @@
 #include <kdebug.h>
 #include <klocale.h>
 #include <kcolordialog.h>
+#include <kapplication.h>
 // app includes
 #include "activitywidget.h"
 #include "uml.h"
@@ -2956,11 +2957,7 @@
         m_role[A].m_pChangeWidget-> setSelected( _select );
     if( m_role[B].m_pChangeWidget)
         m_role[B].m_pChangeWidget-> setSelected( _select );
-    /**
-     * @fixme MYSTERY:
-     *        If the following code is activated and the m_LinePath.setSelected()
-     *        precedes it then the blue selection markers sometimes disappear
-     *        (notably on messages in collaboration diagrams.)
+    kapp->processEvents();
     //Update the docwindow for this association.
     // This is done last because each of the above setSelected calls
     // overwrites the docwindow, but we want the main association doc
@@ -2970,11 +2967,7 @@
                 m_pView -> showDocumentation( this, false );
     } else
         m_pView -> updateDocumentation( true );
-     */
-    /** @fixme Had to pull the m_LinePath.setSelected() call down here
-     *         because otherwise the blue selection markers would not show
-     *         when selecting collaboration messages.
-     */
+    kapp->processEvents();
     m_LinePath.setSelected( _select );
     if (! _select) {
         // For now, if _select is true we don't make the assoc class line