Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPatrick Tasse2014-12-01 18:17:13 +0000
committerPatrick Tasse2014-12-01 22:41:49 +0000
commit67a0ee8c965da2ec4bee77707481a01cf3b00380 (patch)
treef9041cb792ecf3176ce9eecbfd9e447e8809f0e0
parentf55292fa09996731abf7dffb6baf9b72ebda125a (diff)
downloadorg.eclipse.linuxtools-67a0ee8c965da2ec4bee77707481a01cf3b00380.tar.gz
org.eclipse.linuxtools-67a0ee8c965da2ec4bee77707481a01cf3b00380.tar.xz
org.eclipse.linuxtools-67a0ee8c965da2ec4bee77707481a01cf3b00380.zip
tmf: Fix AbstractTracePackageWizardPage tree viewer check integrity
Grayed state did not propagate correctly to parent elements. Change-Id: Ibeaf498285622dd8a3cc7c63c92a97f7fded7593 Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com> Reviewed-on: https://git.eclipse.org/r/37425 Tested-by: Hudson CI Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com> Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
-rw-r--r--lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/project/wizards/tracepkg/AbstractTracePackageWizardPage.java7
1 files changed, 6 insertions, 1 deletions
diff --git a/lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/project/wizards/tracepkg/AbstractTracePackageWizardPage.java b/lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/project/wizards/tracepkg/AbstractTracePackageWizardPage.java
index c30d201e85..9c05d7ec30 100644
--- a/lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/project/wizards/tracepkg/AbstractTracePackageWizardPage.java
+++ b/lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/project/wizards/tracepkg/AbstractTracePackageWizardPage.java
@@ -139,7 +139,12 @@ abstract public class AbstractTracePackageWizardPage extends WizardPage {
if (parentElement != null) {
if (parentElement.getChildren() != null) {
for (TracePackageElement child : parentElement.getChildren()) {
- boolean checked = fElementViewer.getChecked(child) && !fElementViewer.getGrayed(child);
+ if (fElementViewer.getGrayed(child)) {
+ oneChecked = true;
+ allChecked = false;
+ break;
+ }
+ boolean checked = fElementViewer.getChecked(child);
oneChecked |= checked;
allChecked &= checked;
}

Back to the top