Login | Register
My pages Projects Community openCollabNet

Discussions > issues > [gef-issues] [Issue 353] New - SelectionManaged.deselectAll should do nothing if none selected

gef
Discussion topic

Back to topic list

[gef-issues] [Issue 353] New - SelectionManaged.deselectAll should do nothing if none selected

Reply

Author bobtarling
Full name Bob Tarling
Date 2010-11-28 01:52:21 PST
Message http://gef.tigris.or​g/issues/show_bug.cg​i?id=353
                 Issue #|353
                 Summary|SelectionMan​aged.deselectAll should do nothing if none
                        | selected
               Component|gef
                 Version|current
                Platform|All
                     URL|
              OS/Version|All
                  Status|NEW
       Status whiteboard|
                Keywords|
              Resolution|
              Issue type|DEFECT
                Priority|P3
            Subcomponent|core
             Assigned to|issues@gef
             Reported by|bobtarling






------- Additional comments from bobtarling at tigris dot org Sun Nov 28 01:52:21 -0800 2010 -------
ArgoUML is producing the following stacktrace

org.tigris.gef.base.​SelectionManager.des​electAll(SelectionMa​nager.java:227)
  [argouml] at
org.tigris.gef.graph​.presentation.JGraph​.setDiagram(JGraph.j​ava:304)
  [argouml] at
org.argouml.uml.diag​ram.ui.TabDiagram.se​tTarget(TabDiagram.j​ava:208)
  [argouml] at
org.argouml.uml.diag​ram.ui.TabDiagram$1​.run(TabDiagram.java​:406)
  [argouml] at
java.awt.event.Invoc​ationEvent.dispatch(​InvocationEvent.java​:209)
  [argouml] at java.awt.EventQueue.​dispatchEvent(EventQ​ueue.java:597)

This is happening because the work done in SelectionManager to remove all
selections is resulting in an event being fired to indicate a change to the
model. However if there are no selections then no model change has really
happened.

The quick solution here is for deslectAll to return immediately if it has
nothing to do.

« Previous message in topic | 1 of 2 | Next message in topic »

Messages

Show all messages in topic

[gef-issues] [Issue 353] New - SelectionManaged.deselectAll should do nothing if none selected bobtarling Bob Tarling 2010-11-28 01:52:21 PST
     [gef-issues] [Issue 353] SelectionManaged.deselectAll should do nothing if none selected bobtarling Bob Tarling 2010-11-28 01:56:40 PST
Messages per page: