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

On 19 Feb 2006 09:54:50 -0000, Oliver Kellogg <
okellogg@users.sourceforge.net> wrote:
[bugs.kde.org quoted mail]

Wow this was fast!
<br><br><div><span class="gmail_quote">On 19 Feb 2006 09:54:50 -0000, <b class="gmail_sendername">Oliver Kellogg</b> &lt;<a href="mailto:okellogg@users.sourceforge.net">okellogg@users.sourceforge.net</a>&gt; wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
------- You are receiving this mail because: -------<br>You reported the bug, or are watching the reporter.<br>You are a voter for the bug, or are watching someone who is.<br><br><a href="http://bugs.kde.org/show_bug.cgi?id=122063">
http://bugs.kde.org/show_bug.cgi?id=122063</a><br>okellogg users sourceforge net changed:<br><br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; What&nbsp;&nbsp;&nbsp;&nbsp;|Removed&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; |Added<br>----------------------------------------------------------------------------
<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Status|NEW&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; |RESOLVED<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Resolution|&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;|FIXED<br></blockquote></div><br>Wow this was fast!<br>