From 08ddd8d9aaf48f4c35eb6c31482ccae8e0da0ac8 Mon Sep 17 00:00:00 2001
From: Uwe Stieber
Date: Mon, 3 Feb 2014 09:59:44 +0100
Subject: Target Explorer: Fix remote application launch bindings and step
group
---
.../org.eclipse.tcf.te.tcf.launch.core/plugin.xml | 18 ++---
.../app/LaunchConfigurationMainTabSection.java | 8 +++
.../ui/controls/ContextSelectorSectionControl.java | 2 +-
.../tcf/te/tcf/ui/navigator/ContentProvider.java | 80 ----------------------
4 files changed, 14 insertions(+), 94 deletions(-)
(limited to 'target_explorer')
diff --git a/target_explorer/plugins/org.eclipse.tcf.te.tcf.launch.core/plugin.xml b/target_explorer/plugins/org.eclipse.tcf.te.tcf.launch.core/plugin.xml
index 3fbe5db4f..e136f2952 100644
--- a/target_explorer/plugins/org.eclipse.tcf.te.tcf.launch.core/plugin.xml
+++ b/target_explorer/plugins/org.eclipse.tcf.te.tcf.launch.core/plugin.xml
@@ -77,11 +77,9 @@
+
-
-
-
-
+
@@ -101,15 +99,9 @@
+
-
-
-
-
-
-
-
-
+
@@ -147,6 +139,7 @@
label="%LaunchStepGroup.Remote.App.name"
iterator="org.eclipse.tcf.te.launch.core.steps.iterators.LaunchContextIterator">
+
@@ -157,7 +150,6 @@
-
diff --git a/target_explorer/plugins/org.eclipse.tcf.te.tcf.launch.ui/src/org/eclipse/tcf/te/tcf/launch/ui/remote/app/LaunchConfigurationMainTabSection.java b/target_explorer/plugins/org.eclipse.tcf.te.tcf.launch.ui/src/org/eclipse/tcf/te/tcf/launch/ui/remote/app/LaunchConfigurationMainTabSection.java
index 6211968da..230a64ec6 100644
--- a/target_explorer/plugins/org.eclipse.tcf.te.tcf.launch.ui/src/org/eclipse/tcf/te/tcf/launch/ui/remote/app/LaunchConfigurationMainTabSection.java
+++ b/target_explorer/plugins/org.eclipse.tcf.te.tcf.launch.ui/src/org/eclipse/tcf/te/tcf/launch/ui/remote/app/LaunchConfigurationMainTabSection.java
@@ -113,6 +113,10 @@ public class LaunchConfigurationMainTabSection extends AbstractSection implement
super.modifyText(e);
getManagedForm().dirtyStateChanged();
}
+ @Override
+ protected boolean isAdjustEditFieldControlWidthHint() {
+ return true;
+ }
};
processImage.setEditFieldLabel(Messages.LaunchConfigurationMainTabSection_processImage_label);
processImage.setIsGroup(false);
@@ -129,6 +133,10 @@ public class LaunchConfigurationMainTabSection extends AbstractSection implement
super.modifyText(e);
getManagedForm().dirtyStateChanged();
}
+ @Override
+ protected boolean isAdjustEditFieldControlWidthHint() {
+ return true;
+ }
};
processArguments.setEditFieldLabel(Messages.LaunchConfigurationMainTabSection_processArguments_label);
processArguments.setIsGroup(false);
diff --git a/target_explorer/plugins/org.eclipse.tcf.te.tcf.ui/src/org/eclipse/tcf/te/tcf/ui/controls/ContextSelectorSectionControl.java b/target_explorer/plugins/org.eclipse.tcf.te.tcf.ui/src/org/eclipse/tcf/te/tcf/ui/controls/ContextSelectorSectionControl.java
index 0128965e0..471d853a4 100644
--- a/target_explorer/plugins/org.eclipse.tcf.te.tcf.ui/src/org/eclipse/tcf/te/tcf/ui/controls/ContextSelectorSectionControl.java
+++ b/target_explorer/plugins/org.eclipse.tcf.te.tcf.ui/src/org/eclipse/tcf/te/tcf/ui/controls/ContextSelectorSectionControl.java
@@ -61,7 +61,7 @@ public class ContextSelectorSectionControl extends AbstractContextSelectorContro
*/
@Override
protected void doConfigureTreeContentAndLabelProvider(TreeViewer viewer) {
- viewer.setContentProvider(new ContentProvider(true));
+ viewer.setContentProvider(new ContentProvider());
DelegatingLabelProvider labelProvider = new DelegatingLabelProvider();
viewer.setLabelProvider(new DecoratingLabelProvider(labelProvider, labelProvider));
}
diff --git a/target_explorer/plugins/org.eclipse.tcf.te.tcf.ui/src/org/eclipse/tcf/te/tcf/ui/navigator/ContentProvider.java b/target_explorer/plugins/org.eclipse.tcf.te.tcf.ui/src/org/eclipse/tcf/te/tcf/ui/navigator/ContentProvider.java
index b21e87492..3bb6d34db 100644
--- a/target_explorer/plugins/org.eclipse.tcf.te.tcf.ui/src/org/eclipse/tcf/te/tcf/ui/navigator/ContentProvider.java
+++ b/target_explorer/plugins/org.eclipse.tcf.te.tcf.ui/src/org/eclipse/tcf/te/tcf/ui/navigator/ContentProvider.java
@@ -81,79 +81,13 @@ public class ContentProvider implements ICommonContentProvider, ITreePathContent
// Internal map of PeerRedirectorGroupNodes per peer id
private final Map roots = new HashMap();
- // Flag to remember if invisible nodes are to be included in the list of
- // returned children.
- private final boolean showInvisible;
-
INavigatorFilterService navFilterService = null;
/**
* Constructor.
*/
public ContentProvider() {
- this(false);
- }
-
- /**
- * Constructor.
- *
- * @param showInvisible If true, {@link #getChildren(Object)} will include invisible nodes too.
- */
- public ContentProvider(boolean showInvisible) {
super();
- this.showInvisible = showInvisible;
- }
-
- /**
- * Determines if the given peer model node is a value-add.
- *
- * @param peerNode The peer model node. Must not be null.
- * @return True if the peer model node is a value-add, false otherwise.
- */
- /* default */ final boolean isValueAdd(IPeer peer) {
- Assert.isNotNull(peer);
-
- String value = peer.getAttributes().get("ValueAdd"); //$NON-NLS-1$
- boolean isValueAdd = value != null && ("1".equals(value.trim()) || Boolean.parseBoolean(value.trim())); //$NON-NLS-1$
-
- return isValueAdd;
- }
-
- /**
- * Determines if the given peer model node is filtered from the view completely.
- *
- * @param peerNode The peer model node. Must not be null.
- * @return True if filtered, false otherwise.
- */
- /* default */ final boolean isFiltered(IPeerNode peerNode) {
- Assert.isNotNull(peerNode);
-
- boolean filtered = false;
-
- if (!showInvisible) {
- filtered |= !peerNode.isVisible();
- }
-
- return filtered;
- }
-
- /**
- * Determines if the given peer node is filtered from the view completely.
- *
- * @param peerNode The peer node. Must not be null.
- * @return True if filtered, false otherwise.
- */
- /* default */ final boolean isFiltered(IPeer peer) {
- Assert.isNotNull(peer);
-
- boolean filtered = false;
-
- filtered |= isValueAdd(peer);
-
- filtered |= peer.getName() != null
- && (peer.getName().endsWith("Command Server") || peer.getName().endsWith("CLI Server")); //$NON-NLS-1$ //$NON-NLS-2$
-
- return filtered;
}
/* (non-Javadoc)
@@ -209,11 +143,6 @@ public class ContentProvider implements ICommonContentProvider, ITreePathContent
}
else if (IUIConstants.ID_CAT_MY_TARGETS.equals(catID)) {
for (IPeerNode peerNode : peerNodes) {
- // Check for filtered nodes (Value-add's and Proxies)
- if (isFiltered(peerNode)) {
- continue;
- }
-
ICategorizable categorizable = (ICategorizable)peerNode.getAdapter(ICategorizable.class);
if (categorizable == null) {
categorizable = (ICategorizable)Platform.getAdapterManager().getAdapter(peerNode, ICategorizable.class);
@@ -234,11 +163,6 @@ public class ContentProvider implements ICommonContentProvider, ITreePathContent
}
else if (IUIConstants.ID_CAT_NEIGHBORHOOD.equals(catID)) {
for (IPeer peer : ModelManager.getLocatorModel().getPeers()) {
- // Check for filtered nodes (Value-add's and Proxies)
- if (isFiltered(peer)) {
- continue;
- }
-
if (!candidates.contains(peer)) {
candidates.add(peer);
}
@@ -261,10 +185,6 @@ public class ContentProvider implements ICommonContentProvider, ITreePathContent
}
else {
for (IPeerNode peerNode : peerNodes) {
- // Check for filtered nodes (Value-add's and Proxies)
- if (isFiltered(peerNode)) {
- continue;
- }
if (!candidates.contains(peerNode)) {
candidates.add(peerNode);
}
--
cgit v1.2.1