Added Filter Textfield support

Co-authored-by: Brandon Algarra <[email protected]>
Co-authored-by: Khang Nguyen <[email protected]>

Author: 3 people

apitools/org.eclipse.pde.api.tools/.classpath

@@ -6,4 +6,4 @@
 	<classpathentry kind="src" output="bin_ant" path="src_ant"/>
 	<classpathentry kind="src" path="xslt"/>
 	<classpathentry kind="output" path="bin"/>
-</classpath>
+</classpath>

ui/org.eclipse.pde.core/src/org/eclipse/pde/core/plugin/IPluginImport.java

@@ -34,6 +34,11 @@ public interface IPluginImport extends IPluginObject, IPluginReference {
 	 */
 	String P_OPTIONAL = "optional"; //$NON-NLS-1$
 
+	/**
+	 * @since 3.21
+	 */
+	String P_FILTER = "filter"; //$NON-NLS-1$
+
 	/**
 	 * Tests whether the imported plug-in is reexported for
 	 * plug-ins that will use this plug-in.
@@ -70,4 +75,17 @@ public interface IPluginImport extends IPluginObject, IPluginReference {
 	 */
 	void setOptional(boolean value) throws CoreException;
 
+	/**
+	 * @since 3.21
+	 */
+	void setFilter(String filter) throws CoreException;
+
+	/**
+	 * Returns the filter string for this plugin reference.
+	 *
+	 * @return the filter string, or null if not set
+	 * @since 3.21
+	 */
+	String getFilter();
+
 }

ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/plugin/PluginImport.java

@@ -46,6 +46,7 @@ public class PluginImport extends IdentifiablePluginObject implements IPluginImp
 	private boolean reexported = false;
 	private boolean optional = false;
 	private String version;
+	private String filter;
 
 	public PluginImport() {
 	}
@@ -74,6 +75,12 @@ public String getVersion() {
 		return version;
 	}
 
+	@Override
+	public String getFilter() {
+		// TODO Auto-generated method stub
+		return version;
+	}
+
 	@Override
 	public boolean isReexported() {
 		return reexported;
@@ -203,6 +210,15 @@ public void setVersion(String version) throws CoreException {
 		firePropertyChanged(P_VERSION, oldValue, version);
 	}
 
+	@Override
+	public void setFilter(String filter) throws CoreException {
+		ensureModelEditable();
+		String oldValue = this.filter;
+		this.filter = filter;
+		firePropertyChanged(P_FILTER, oldValue, filter);
+
+	}
+
 	@Override
 	public void restoreProperty(String name, Object oldValue, Object newValue) throws CoreException {
 		if (name.equals(P_MATCH)) {
@@ -312,4 +328,6 @@ public void writeDelimeter(PrintWriter writer) {
 		writer.print(' ');
 	}
 
+
+
 }

ui/org.eclipse.pde.core/text/org/eclipse/pde/internal/core/text/plugin/PluginImportNode.java

@@ -21,6 +21,17 @@ public class PluginImportNode extends PluginObjectNode implements IPluginImport
 
 	private static final long serialVersionUID = 1L;
 
+	@Override
+	public void setFilter(String filter) {
+		setXMLAttribute(P_FILTER, filter);
+
+	}
+
+	@Override
+	public String getFilter() {
+		return getXMLAttributeValue(P_FILTER);
+	}
+
 	public PluginImportNode(String id) {
 		super();
 		String name = "plugin"; //$NON-NLS-1$

ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/PDEUIMessages.java

@@ -977,6 +977,7 @@ public class PDEUIMessages extends NLS {
 	public static String ManifestEditor_MatchSection_equivalent;
 	public static String ManifestEditor_MatchSection_compatible;
 	public static String ManifestEditor_MatchSection_greater;
+	public static String ManifestEditor_MatchSection_filter;
 
 	public static String ManifestEditor_PluginSpecSection_title;
 	public static String ManifestEditor_PluginSpecSection_desc;

ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/editor/plugin/MatchSection.java

@@ -25,6 +25,7 @@
 import org.eclipse.pde.core.plugin.IMatchRules;
 import org.eclipse.pde.core.plugin.IPluginImport;
 import org.eclipse.pde.core.plugin.IPluginReference;
+import org.eclipse.pde.internal.core.ifeature.IFeatureImport;
 import org.eclipse.pde.internal.core.plugin.ImportObject;
 import org.eclipse.pde.internal.ui.PDEPlugin;
 import org.eclipse.pde.internal.ui.PDEUIMessages;
@@ -44,13 +45,17 @@
 import org.eclipse.ui.forms.IPartSelectionListener;
 import org.eclipse.ui.forms.widgets.FormToolkit;
 import org.eclipse.ui.forms.widgets.Section;
+import org.osgi.framework.FrameworkUtil;
+import org.osgi.framework.InvalidSyntaxException;
 
 public class MatchSection extends PDESection implements IPartSelectionListener {
 
+
 	private Button fReexportButton;
 	private Button fOptionalButton;
 
 	private FormEntry fVersionText;
+	private FormEntry fFilterText;
 
 	private ComboPart fMatchCombo;
 	protected IPluginReference fCurrentImport;
@@ -134,6 +139,26 @@ public void textDirty(FormEntry text) {
 		}));
 		toolkit.paintBordersFor(container);
 		initialize();
+
+		fFilterText = new FormEntry(container, toolkit, PDEUIMessages.ManifestEditor_MatchSection_filter, null, false);
+		fFilterText.setFormEntryListener(
+				new FormEntryAdapter(this, getPage().getEditor().getEditorSite().getActionBars()) {
+					@Override
+					public void textValueChanged(FormEntry text) {
+						applyFilter(text.getValue());
+					}
+
+					@Override
+					public void textDirty(FormEntry text) {
+						if (fBlockChanges) {
+							return;
+						}
+						markDirty();
+						fBlockChanges = true;
+						resetMatchCombo(fCurrentImport);
+						fBlockChanges = false;
+					}
+				});
 		update((IPluginReference) null);
 
 		section.setClient(container);
@@ -200,6 +225,23 @@ private void applyMatch(int match) {
 		}
 	}
 
+	private void applyFilter(String filter) {
+		try {
+			if (fCurrentImport == null)
+				return;
+			if (filter == null || filter.isEmpty())
+				((IFeatureImport) fCurrentImport).setFilter(null);
+			else {
+				FrameworkUtil.createFilter(filter);
+				((IFeatureImport) fCurrentImport).setFilter(filter);
+			}
+		} catch (InvalidSyntaxException e) {
+			PDEPlugin.logException(e);
+		} catch (CoreException e) {
+			PDEPlugin.logException(e);
+		}
+	}
+
 	private int getMatch() {
 		return fMatchCombo.getSelectionIndex();
 	}
@@ -272,8 +314,11 @@ protected void update(IPluginReference iimport) {
 			}
 			fVersionText.setValue(null, true);
 			fVersionText.setEditable(false);
+
 			fMatchCombo.getControl().setEnabled(false);
 			fMatchCombo.setText(""); //$NON-NLS-1$
+			fFilterText.setValue(null, false);
+			fFilterText.setEditable(false);
 			fCurrentImport = null;
 			fBlockChanges = false;
 			return;
@@ -292,7 +337,10 @@ protected void update(IPluginReference iimport) {
 		}
 		fVersionText.setEditable(isEditable());
 		fVersionText.setValue(fCurrentImport.getVersion());
+
 		resetMatchCombo(fCurrentImport);
+		fFilterText.setEditable(isEditable());
+		fFilterText.setValue(((IFeatureImport) fCurrentImport).getFilter());
 		fBlockChanges = false;
 	}
 }

ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/pderesources.properties

@@ -388,6 +388,7 @@ ManifestEditor_MatchSection_perfect = Perfect
 ManifestEditor_MatchSection_equivalent = Equivalent
 ManifestEditor_MatchSection_compatible = Compatible
 ManifestEditor_MatchSection_greater = Greater or Equal
+ManifestEditor_MatchSection_filter = Filter:
 
 ManifestEditor_PluginSpecSection_title = General Information
 ManifestEditor_PluginSpecSection_desc = This section describes general information about this plug-in.

ui/org.eclipse.pde.ui/src/org/eclipse/pde/ui/templates/PluginReference.java

@@ -113,5 +113,4 @@ public String getId() {
 	public void setId(String id) throws CoreException {
 		this.id = id;
 	}
-
-}
+}