xan/awips2
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Issue #1365 RangeRingDialog now perform a close properly when not on the UI thread.

Browse source 
Change-Id: Idc32bb73f6a3cba2c8f2be5119606a17356e3ec7

Former-commit-id: 9a1620b31f1ef86c19c4d3c3dd53ee29161e5f60
This commit is contained in:
Roger Ferrel 2012-11-29 11:09:44 -06:00
parent 94fb2d77eb
commit 4dcb533215
1 changed files with 30 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

32
cave/com.raytheon.viz.awipstools/src/com/raytheon/viz/awipstools/ui/dialog/RangeRingDialog.java
View file

@ -73,6 +73,7 @@ import com.vividsolutions.jts.geom.Coordinate;
* unapplied points.
* 07-31-12 #875 rferrel Use MenuButton to show points in groups.
* 11-05-12 #1304 rferrel Added Point Change Listener.
* 11-29-12 #1365 rferrel Properly close dialog when not on UI thread.
*
* </pre>
*
@ -614,13 +615,21 @@ public class RangeRingDialog extends CaveJFACEDialog implements
});
}
/*
* (non-Javadoc)
*
* @see
* com.raytheon.uf.viz.core.rsc.IResourceDataChanged#resourceChanged(com
* .raytheon.uf.viz.core.rsc.IResourceDataChanged.ChangeType,
* java.lang.Object)
*/
@Override
public void resourceChanged(ChangeType type, Object object) {
if (object instanceof Boolean) {
boolean b = (Boolean) object;
if (b == false) {
this.resourceData.removeChangeListener(this);
close();
performClose();
}
return;
} else if (object instanceof RangeRing[]) {
@ -641,11 +650,27 @@ public class RangeRingDialog extends CaveJFACEDialog implements
} else if (object instanceof EditableCapability) {
if (!((EditableCapability) object).isEditable()) {
this.resourceData.removeChangeListener(this);
close();
performClose();
}
}
}
/**
* Close dialog when not on the UI thread.
*/
private void performClose() {
VizApp.runAsync(new Runnable() {
@Override
public void run() {
close();
}
});
}
/* (non-Javadoc)
* @see org.eclipse.jface.dialogs.Dialog#close()
*/
@Override
public boolean close() {
// passing true tells the resource that editable should be turned off
@ -654,6 +679,9 @@ public class RangeRingDialog extends CaveJFACEDialog implements
return super.close();
}
/* (non-Javadoc)
* @see com.raytheon.uf.viz.points.IPointChangedListener#pointChanged()
*/
@Override
public void pointChanged() {
VizApp.runAsync(new Runnable() {