diff --git a/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/iproduct/IProductPlugin.java b/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/iproduct/IProductPlugin.java index 4cadafa959..ad00c222cb 100644 --- a/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/iproduct/IProductPlugin.java +++ b/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/iproduct/IProductPlugin.java @@ -23,17 +23,4 @@ public interface IProductPlugin extends IProductObject { String getVersion(); void setVersion(String version); - - /** - * @return whether this product plug-in is a fragment. false by default. - * @see #setFragment(boolean) - */ - boolean isFragment(); - - /** - * Sets whether this product plug-in is a fragment. false by default. - * @param isFragment whether this product is a fragment - * @see #isFragment() - */ - void setFragment(boolean isFragment); } diff --git a/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/product/ProductPlugin.java b/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/product/ProductPlugin.java index ff97d5f704..04793c637c 100644 --- a/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/product/ProductPlugin.java +++ b/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/product/ProductPlugin.java @@ -18,8 +18,6 @@ import java.io.PrintWriter; -import org.eclipse.pde.core.plugin.IFragmentModel; -import org.eclipse.pde.core.plugin.PluginRegistry; import org.eclipse.pde.internal.core.ICoreConstants; import org.eclipse.pde.internal.core.iproduct.IProductModel; import org.eclipse.pde.internal.core.iproduct.IProductPlugin; @@ -32,12 +30,6 @@ public class ProductPlugin extends ProductObject implements IProductPlugin { private String fId; private String fVersion; - /** - * Used to cache the fragment attribute value internally in order to not lose it in case the current - * plugin/fragment is not in the target platform anymore (see bug 264462) - */ - private boolean fFragment; - public ProductPlugin(IProductModel model) { super(model); } @@ -48,8 +40,6 @@ public void parse(Node node) { Element element = (Element) node; fId = element.getAttribute("id"); //$NON-NLS-1$ fVersion = element.getAttribute("version"); //$NON-NLS-1$ - String fragment = element.getAttribute("fragment"); //$NON-NLS-1$ - fFragment = Boolean.parseBoolean(fragment); } } @@ -59,16 +49,6 @@ public void write(String indent, PrintWriter writer) { if (fVersion != null && fVersion.length() > 0 && !fVersion.equals(ICoreConstants.DEFAULT_VERSION)) { writer.print(" version=\"" + fVersion + "\""); //$NON-NLS-1$ //$NON-NLS-2$ } - - // If the plugin is a known fragment or has a cached fragment setting, mark it as a fragment - if (PluginRegistry.findModel(fId) != null) { - if (PluginRegistry.findModel(fId) instanceof IFragmentModel) { - writer.print(" fragment=\"" + Boolean.TRUE + "\""); //$NON-NLS-1$ //$NON-NLS-2$ - } - } else if (fFragment) { - writer.print(" fragment=\"" + Boolean.TRUE + "\""); //$NON-NLS-1$ //$NON-NLS-2$ - } - writer.println("/>"); //$NON-NLS-1$ } @@ -96,14 +76,4 @@ public void setVersion(String version) { } } - @Override - public boolean isFragment() { - return fFragment; - } - - @Override - public void setFragment(boolean isFragment) { - fFragment = isFragment; - } - }