Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Rennie2008-11-24 17:13:17 +0000
committerMichael Rennie2008-11-24 17:13:17 +0000
commit466714cec198cb0ac65be98d9db5b0273a42b643 (patch)
tree30611f61d3a6ec23270554c8dccbf4e4bccaed8d /org.eclipse.ui.externaltools
parent36bdae3933c8113aa01b0fa9df50f9b79ff76790 (diff)
downloadeclipse.platform.debug-466714cec198cb0ac65be98d9db5b0273a42b643.tar.gz
eclipse.platform.debug-466714cec198cb0ac65be98d9db5b0273a42b643.tar.xz
eclipse.platform.debug-466714cec198cb0ac65be98d9db5b0273a42b643.zip
Bug 253100 'Builders' property page should remember it has warned the user
Diffstat (limited to 'org.eclipse.ui.externaltools')
-rw-r--r--org.eclipse.ui.externaltools/External Tools Base/org/eclipse/ui/externaltools/internal/ui/BuilderPropertyPage.java22
1 files changed, 16 insertions, 6 deletions
diff --git a/org.eclipse.ui.externaltools/External Tools Base/org/eclipse/ui/externaltools/internal/ui/BuilderPropertyPage.java b/org.eclipse.ui.externaltools/External Tools Base/org/eclipse/ui/externaltools/internal/ui/BuilderPropertyPage.java
index 43e5b21d2..f090fa642 100644
--- a/org.eclipse.ui.externaltools/External Tools Base/org/eclipse/ui/externaltools/internal/ui/BuilderPropertyPage.java
+++ b/org.eclipse.ui.externaltools/External Tools Base/org/eclipse/ui/externaltools/internal/ui/BuilderPropertyPage.java
@@ -102,6 +102,8 @@ public final class BuilderPropertyPage extends PropertyPage implements ICheckSta
private CheckboxTableViewer viewer= null;
+ private boolean fWarned = false;
+
private ILabelProvider labelProvider= new BuilderLabelProvider();
/**
@@ -423,13 +425,21 @@ public final class BuilderPropertyPage extends PropertyPage implements ICheckSta
}
if (checked) {
enableCommand((ICommand)element, checked);
- } else if (MessageDialog.openConfirm(shell, ExternalToolsUIMessages.BuilderPropertyPage_6, ExternalToolsUIMessages.BuilderPropertyPage_7)) {
- enableCommand((ICommand)element, checked);
- } else {
- viewer.removeCheckStateListener(this);
- viewer.setChecked(element, true);
- viewer.addCheckStateListener(this);
+ return;
+ } else if (!fWarned) {
+ if(MessageDialog.openConfirm(shell, ExternalToolsUIMessages.BuilderPropertyPage_6, ExternalToolsUIMessages.BuilderPropertyPage_7)) {
+ fWarned = true;
+ }
+ }
+ if(fWarned) {
+ enableCommand((ICommand)element, checked);
+ }
+ else {
+ viewer.removeCheckStateListener(this);
+ viewer.setChecked(element, true);
+ viewer.addCheckStateListener(this);
}
+
}
}

Back to the top