Clean Code for update/org.eclipse.update.configurator

update/org.eclipse.update.configurator/src/org/eclipse/update/configurator/ConfiguratorUtils.java

@@ -44,6 +44,7 @@ public class ConfiguratorUtils {
 	 * @return platform configuration used in current instance of platform
 	 * @since 3.0
 	 */
+	@Deprecated
 	public static IPlatformConfiguration getCurrentPlatformConfiguration() {
 		// acquire factory service first
 		BundleContext context = ConfigurationActivator.getBundleContext();
@@ -69,6 +70,7 @@ public static IPlatformConfiguration getCurrentPlatformConfiguration() {
 	 * is specified, an empty configuration object is returned
 	 * @return platform configuration used in current instance of platform
 	 */
+	@Deprecated
 	public static IPlatformConfiguration getPlatformConfiguration(URL url) throws IOException {
 		// acquire factory service first
 		BundleContext context = ConfigurationActivator.getBundleContext();
@@ -95,6 +97,7 @@ public static IPlatformConfiguration getPlatformConfiguration(URL url) throws IO
 	 * @param loc location of the platform installation.  Used to resolve entries in the save location
 	 * @return platform configuration used in current instance of platform
 	 */
+	@Deprecated
 	public static IPlatformConfiguration getPlatformConfiguration(URL url, URL loc) throws IOException {
 		// acquire factory service first
 		BundleContext context = ConfigurationActivator.getBundleContext();
@@ -115,6 +118,7 @@ public static IPlatformConfiguration getPlatformConfiguration(URL url, URL loc)
 	/**
 	 * @return the URL of this eclipse installation
 	 */
+	@Deprecated
 	public static URL getInstallURL() {
 		return Utils.getInstallURL();
 	}

update/org.eclipse.update.configurator/src/org/eclipse/update/configurator/IPlatformConfiguration.java

@@ -44,6 +44,7 @@
 	 *
 	 * @since 2.0
 	 */
+	@Deprecated
 	public interface ISiteEntry {
 
 		/**
@@ -52,7 +53,7 @@
 		 * @return site url
 		 * @since 2.0
 		 */
 		public URL getURL();
 
 		/**
 		 * Returns the policy for this site
@@ -60,7 +61,7 @@
 		 * @return site policy
 		 * @since 2.0
 		 */
 		public ISitePolicy getSitePolicy();
 
 		/**
 		 * Sets the site policy
@@ -68,7 +69,7 @@
 		 * @param policy site policy
 		 * @since 2.0
 		 */
 		public void setSitePolicy(ISitePolicy policy);
 
 		/**
 		 * Returns a list of features visible on the site. Note, that this is simply a
@@ -78,7 +79,7 @@
 		 * A feature entry is returned as a path relative to the site URL
 		 * @since 2.0
 		 */
 		public String[] getFeatures();
 
 		/**
 		 * Returns a list of plug-ins visible on the site. Note, that this is simply a
@@ -90,7 +91,7 @@
 		 * A plug-in entry is returned as a path relative to the site URL		 *
 		 * @since 2.0
 		 */
 		public String[] getPlugins();
 
 		/**
 		 * Returns a stamp reflecting the current state of the site. If called repeatedly,
@@ -100,7 +101,7 @@
 		 * @return site change stamp
 		 * @since 2.0
 		 */
 		public long getChangeStamp();
 
 		/**
 		 * Returns a stamp reflecting the current state of the features on the site.
@@ -132,7 +133,7 @@
 		 * @return <code>true</code> if site can be updated, <code>false</code> otherwise
 		 * @since 2.0
 		 */
 		public boolean isUpdateable();
 
 		/**
 		 * Returns an indication whether the site represents an install site
@@ -142,7 +143,7 @@
 		 * @return <code>true</code> if the site is linked, <code>false</code> otherwise
 		 * @since 2.0
 		 */
 		public boolean isNativelyLinked();
 	}
 
 	/**
@@ -167,6 +168,7 @@
 	 *
 	 * @since 2.0
 	 */
+	@Deprecated
 	public interface ISitePolicy {
 
 		/**
@@ -178,21 +180,21 @@
 		 * policy type is interpreted as path entries to included plugin.xml
 		 * or fragment.xml <b>relative</b> to the site URL
 		 */
 		public static final int USER_INCLUDE = 0;
 
 		/**
 		 * User-defined exclusion list. The list associated with this
 		 * policy type is interpreted as path entries to excluded plugin.xml
 		 * or fragment.xml <b>relative</b> to the site URL
 		 */
 		public static final int USER_EXCLUDE = 1;
 
 		/**
 		 * When this site policy is used, only plug-ins specified by the configured features
 		 * are contributed to the runtime.
 		 * @since 3.1
 		 */
 		public static final int MANAGED_ONLY = 2;
 
 		/**
 		 * Return policy type
@@ -200,7 +202,7 @@
 		 * @return policy type
 		 * @since 2.0
 		 */
 		public int getType();
 
 		/**
 		 * Return policy inclusion/ exclusion list
@@ -208,7 +210,7 @@
 		 * @return the list as an array
 		 * @since 2.0
 		 */
 		public String[] getList();
 
 		/**
 		 * Set new policy list. The list entries are interpreted based on the policy
@@ -220,7 +222,7 @@
 		 * @see #USER_EXCLUDE
 		 * @since 2.0
 		 */
 		public void setList(String[] list);
 	}
 
 	/**
@@ -235,6 +237,7 @@
 	 *
 	 * @since 2.0
 	 */
+	@Deprecated
 	public interface IFeatureEntry {
 
 		/**
@@ -242,14 +245,14 @@
 		 * @return feature identifier
 		 * @since 2.0
 		 */
 		public String getFeatureIdentifier();
 
 		/**
 		 * Returns the currently configured version for the feature.
 		 * @return feature version (as string), or <code>null</code>
 		 * @since 2.0
 		 */
 		public String getFeatureVersion();
 
 		/**
 		 * Returns the identifier of the feature plug-in for this feature entry.
@@ -263,7 +266,7 @@
 		 * @return feature identifier (as string), or <code>null</code>
 		 * @since 2.1
 		 */
 		public String getFeaturePluginIdentifier();
 
 		/**
 		 * Returns the version of the feature plug-in for this feature
@@ -277,7 +280,7 @@
 		 * @return feature version (as string), or <code>null</code>
 		 * @since 2.0
 		 */
 		public String getFeaturePluginVersion();
 
 		/**
 		 * Returns the application to run when this feature is the
@@ -285,7 +288,7 @@
 		 * @return application identifier, or <code>null</code>
 		 * @since 2.0
 		 */
 		public String getFeatureApplication();
 
 		/**
 		 * Returns URLs to the feature "root" locations. The root
@@ -295,7 +298,7 @@
 		 * @return array of URLs, or an empty array
 		 * @since 2.0
 		 */
 		public URL[] getFeatureRootURLs();
 
 		/**
 		 * Returns an indication whether this feature has been defined
@@ -304,7 +307,7 @@
 		 * <code>false</code> otherwise.
 		 * @since 2.0
 		 */
 		public boolean canBePrimary();
 	}
 
 	/**
@@ -315,6 +318,7 @@
 	 * @return created site entry
 	 * @since 2.0
 	 */
+	@Deprecated
 	public ISiteEntry createSiteEntry(URL url, ISitePolicy policy);
 
 	/**
@@ -327,6 +331,7 @@
 	 * @return created site policy entry
 	 * @since 2.0
 	 */
+	@Deprecated
 	public ISitePolicy createSitePolicy(int type, String[] list);
 
 	/**
@@ -348,6 +353,7 @@
 	 * @return create feature entry
 	 * @since 2.0
 	 */
+	@Deprecated
 	public IFeatureEntry createFeatureEntry(String id, String version, String pluginVersion, boolean primary, String application, URL[] root);
 
 	/**
@@ -371,6 +377,7 @@
 	 * @return create feature entry
 	 * @since 2.1
 	 */
+	@Deprecated
 	public IFeatureEntry createFeatureEntry(String id, String version, String pluginIdentifier, String pluginVersion, boolean primary, String application, URL[] root);
 
 	/**
@@ -380,6 +387,7 @@
 	 * @param entry site entry
 	 * @since 2.0
 	 */
+	@Deprecated
 	public void configureSite(ISiteEntry entry);
 
 	/**
@@ -392,6 +400,7 @@
 	 * the same URL should be replaced (<code>true</code>) or not (<code>false</code>).
 	 * @since 2.0
 	 */
+	@Deprecated
 	public void configureSite(ISiteEntry entry, boolean replace);
 
 	/**
@@ -401,6 +410,7 @@
 	 * @param entry site entry
 	 * @since 2.0
 	 */
+	@Deprecated
 	public void unconfigureSite(ISiteEntry entry);
 
 	/**
@@ -410,6 +420,7 @@
 	 * configured
 	 * @since 2.0
 	 */
+	@Deprecated
 	public ISiteEntry[] getConfiguredSites();
 
 	/**
@@ -419,6 +430,7 @@
 	 * @return matching site entry, or <code>null</code> if no match found
 	 * @since 2.0
 	 */
+	@Deprecated
 	public ISiteEntry findConfiguredSite(URL url);
 
 	/**
@@ -428,13 +440,15 @@
 	 * @param entry feature entry
 	 * @since 2.0
 	 */
+	@Deprecated
 	public void configureFeatureEntry(IFeatureEntry entry);
 
 	/**
 	 * Unconfigures the specified feature entry if it exists.
 	 * @param entry feature entry
 	 * @since 2.0
 	 */
+	@Deprecated
 	public void unconfigureFeatureEntry(IFeatureEntry entry);
 
 	/**
@@ -443,6 +457,7 @@
 	 * are configured
 	 * @since 2.0
 	 */
+	@Deprecated
 	public IFeatureEntry[] getConfiguredFeatureEntries();
 
 	/**
@@ -451,6 +466,7 @@
 	 * @return ferature entry, or <code>null</code>.
 	 * @since 2.0
 	 */
+	@Deprecated
 	public IFeatureEntry findConfiguredFeatureEntry(String id);
 
 	/**
@@ -460,6 +476,7 @@
 	 * configuration location could not be determined.
 	 * @since 2.0
 	 */
+	@Deprecated
 	public URL getConfigurationLocation();
 
 	/**
@@ -470,6 +487,7 @@
 	 * @return configuration change stamp
 	 * @since 2.0
 	 */
+	@Deprecated
 	public long getChangeStamp();
 
 	/**
@@ -504,6 +522,7 @@
 	 * @return primary feature identifier, or <code>null</code> if none configured
 	 * @since 2.0
 	 */
+	@Deprecated
 	public String getPrimaryFeatureIdentifier();
 
 	/**
@@ -513,6 +532,7 @@
 	 * @return an array of plug-in path elements (full URL entries), or an empty array.
 	 * @since 2.0
 	 */
+	@Deprecated
 	public URL[] getPluginPath();
 
 	/**
@@ -549,6 +569,7 @@
 	 * otherwise
 	 * @since 2.0
 	 */
+	@Deprecated
 	public boolean isUpdateable();
 
 	/**
@@ -561,6 +582,7 @@
 	 * otherwise
 	 * @since 2.0
 	 */
+	@Deprecated
 	public boolean isTransient();
 
 	/**
@@ -574,6 +596,7 @@
 	 * otherwise
 	 * @since 2.0
 	 */
+	@Deprecated
 	public void isTransient(boolean value);
 
 	/**
@@ -582,12 +605,14 @@
 	 * configuration state, and updates the lists of available plug-ins.
 	 * @since 2.0
 	 */
+	@Deprecated
 	public void refresh();
 
 	/**
 	 * Called to save the configuration information
 	 * @since 2.0
 	 */
+	@Deprecated
 	public void save() throws IOException;
 
 	/**
@@ -597,6 +622,7 @@
 	 * @param url save location.
 	 * @since 2.0
 	 */
+	@Deprecated
 	public void save(URL url) throws IOException;
 
 }

update/org.eclipse.update.configurator/src/org/eclipse/update/configurator/IPlatformConfigurationFactory.java

@@ -34,6 +34,7 @@ public interface IPlatformConfigurationFactory {
 	 *
 	 * @return platform configuration used in current instance of platform
 	 */
+	@Deprecated
 	public IPlatformConfiguration getCurrentPlatformConfiguration();
 	/**
 	 * Returns a platform configuration object, optionally initialized with previously saved
@@ -43,6 +44,7 @@ public interface IPlatformConfigurationFactory {
 	 * is specified, an empty configuration object is returned
 	 * @return platform configuration used in current instance of platform
 	 */
+	@Deprecated
 	public IPlatformConfiguration getPlatformConfiguration(URL url) throws IOException;
 
 	/**
@@ -55,5 +57,6 @@ public interface IPlatformConfigurationFactory {
 	 * location
 	 * @return platform configuration used in current instance of platform
 	 */
+	@Deprecated
 	public IPlatformConfiguration getPlatformConfiguration(URL url, URL loc) throws IOException;
 }