From 7b91dd13af3a2e8b6bdcfe05f701e58f57cbdc1c Mon Sep 17 00:00:00 2001 From: Jaxsun McCarthy Huggan Date: Mon, 16 May 2016 16:59:44 -0700 Subject: 493037: deprecate AbstractTaskListMigrator Task List migrators are no longer used when reading the task list. Change-Id: I88e0f60af3cf512e4176c5abefd117c229311ac0 Task-Url: https://bugs.eclipse.org/bugs/show_bug.cgi?id=493037 Signed-off-by: Jaxsun McCarthy Huggan --- .../core/externalization/TaskListExternalizer.java | 6 -- .../util/RepositoryConnectorExtensionReader.java | 84 +++++----------------- .../mylyn/tasks/core/AbstractTaskListMigrator.java | 1 + .../core/spi/RepositoryConnectorDescriptor.java | 13 ++-- 4 files changed, 25 insertions(+), 79 deletions(-) (limited to 'org.eclipse.mylyn.tasks.core/src/org/eclipse') diff --git a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/internal/tasks/core/externalization/TaskListExternalizer.java b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/internal/tasks/core/externalization/TaskListExternalizer.java index 86d57ebe5..2c655cff1 100644 --- a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/internal/tasks/core/externalization/TaskListExternalizer.java +++ b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/internal/tasks/core/externalization/TaskListExternalizer.java @@ -18,7 +18,6 @@ import java.io.FileInputStream; import java.io.FileOutputStream; import java.io.IOException; import java.io.InputStream; -import java.util.List; import java.util.zip.ZipEntry; import java.util.zip.ZipInputStream; import java.util.zip.ZipOutputStream; @@ -30,7 +29,6 @@ import org.eclipse.mylyn.internal.tasks.core.ITasksCoreConstants; import org.eclipse.mylyn.internal.tasks.core.ITransferList; import org.eclipse.mylyn.internal.tasks.core.RepositoryModel; import org.eclipse.mylyn.internal.tasks.core.XmlReaderUtil; -import org.eclipse.mylyn.tasks.core.AbstractTaskListMigrator; import org.eclipse.mylyn.tasks.core.IRepositoryManager; import org.w3c.dom.Document; import org.xml.sax.InputSource; @@ -56,10 +54,6 @@ public class TaskListExternalizer { this.repositoryManager = repositoryManager; } - public void initialize(List migrators) { - // TODO: Deprecate migration - } - public void writeTaskList(ITransferList taskList, File outFile) throws CoreException { try (FileOutputStream outStream = new FileOutputStream(outFile)) { try (ZipOutputStream zipOutStream = new ZipOutputStream(outStream)) { diff --git a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/internal/tasks/core/util/RepositoryConnectorExtensionReader.java b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/internal/tasks/core/util/RepositoryConnectorExtensionReader.java index 8f6c78c79..7b62ece16 100644 --- a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/internal/tasks/core/util/RepositoryConnectorExtensionReader.java +++ b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/internal/tasks/core/util/RepositoryConnectorExtensionReader.java @@ -34,15 +34,12 @@ import org.eclipse.mylyn.internal.tasks.core.TaskRepositoryManager; import org.eclipse.mylyn.internal.tasks.core.externalization.TaskListExternalizer; import org.eclipse.mylyn.tasks.core.AbstractRepositoryConnector; import org.eclipse.mylyn.tasks.core.AbstractRepositoryMigrator; -import org.eclipse.mylyn.tasks.core.AbstractTaskListMigrator; import org.eclipse.mylyn.tasks.core.spi.RepositoryConnectorContributor; import org.eclipse.mylyn.tasks.core.spi.RepositoryConnectorDescriptor; import org.eclipse.osgi.util.NLS; public class RepositoryConnectorExtensionReader { - public static final String ELMNT_MIGRATOR = "taskListMigrator"; //$NON-NLS-1$ - public static final String ELMNT_REPOSITORY_MIGRATOR = "repositoryMigrator"; //$NON-NLS-1$ public static final String ELMNT_REPOSITORY_CONNECTOR = "connectorCore"; //$NON-NLS-1$ @@ -60,8 +57,6 @@ public class RepositoryConnectorExtensionReader { private AbstractRepositoryConnector connector; - private AbstractTaskListMigrator taskListMigrator; - private AbstractRepositoryMigrator repositoryMigrator; private final RepositoryConnectorDescriptor descriptor; @@ -87,10 +82,6 @@ public class RepositoryConnectorExtensionReader { return pluginId; } - public AbstractTaskListMigrator getTaskListMigrator() { - return taskListMigrator; - } - public AbstractRepositoryMigrator getRepositoryMigrator() { return repositoryMigrator; } @@ -102,26 +93,12 @@ public class RepositoryConnectorExtensionReader { if (connector != null) { return Status.OK_STATUS; } else { - return new Status(IStatus.ERROR, ITasksCoreConstants.ID_PLUGIN, NLS.bind( - "Could not load connector core contributed by ''{0}''", getPluginId())); //$NON-NLS-1$ + return new Status(IStatus.ERROR, ITasksCoreConstants.ID_PLUGIN, + NLS.bind("Could not load connector core contributed by ''{0}''", getPluginId())); //$NON-NLS-1$ } } catch (Throwable e) { - return new Status(IStatus.ERROR, ITasksCoreConstants.ID_PLUGIN, NLS.bind( - "Could not load connector core contributed by ''{0}''", getPluginId()), e); //$NON-NLS-1$ - } - } - - public IStatus createTaskListMigrator() { - try { - taskListMigrator = descriptor.createTaskListMigrator(); - if (taskListMigrator != null) { - return Status.OK_STATUS; - } else { - return Status.CANCEL_STATUS; - } - } catch (Throwable e) { - return new Status(IStatus.ERROR, ITasksCoreConstants.ID_PLUGIN, NLS.bind( - "Could not load task list migrator extension contributed by ''{0}''", getPluginId()), e); //$NON-NLS-1$ + return new Status(IStatus.ERROR, ITasksCoreConstants.ID_PLUGIN, + NLS.bind("Could not load connector core contributed by ''{0}''", getPluginId()), e); //$NON-NLS-1$ } } @@ -135,8 +112,9 @@ public class RepositoryConnectorExtensionReader { return Status.CANCEL_STATUS; } } catch (Throwable e) { - return new Status(IStatus.ERROR, ITasksCoreConstants.ID_PLUGIN, NLS.bind( - "Could not load repository migrator extension contributed by ''{0}''", getPluginId()), e); //$NON-NLS-1$ + return new Status(IStatus.ERROR, ITasksCoreConstants.ID_PLUGIN, + NLS.bind("Could not load repository migrator extension contributed by ''{0}''", getPluginId()), //$NON-NLS-1$ + e); } } @@ -146,8 +124,6 @@ public class RepositoryConnectorExtensionReader { IConfigurationElement element; - IConfigurationElement taskListMigratorElement; - IConfigurationElement repositoryMigratorElement; public ExtensionPointBasedConnectorDescriptor(IConfigurationElement element) { @@ -163,18 +139,6 @@ public class RepositoryConnectorExtensionReader { } } - @Override - public AbstractTaskListMigrator createTaskListMigrator() { - if (taskListMigratorElement == null) { - return null; - } - try { - return (AbstractTaskListMigrator) taskListMigratorElement.createExecutableExtension(ATTR_CLASS); - } catch (CoreException e) { - throw new RuntimeException(e); - } - } - @Override public AbstractRepositoryMigrator createRepositoryMigrator() { if (repositoryMigratorElement == null) { @@ -199,8 +163,6 @@ public class RepositoryConnectorExtensionReader { private final ContributorBlackList blackList = new ContributorBlackList(); - private final TaskListExternalizer taskListExternalizer; - private final TaskRepositoryManager repositoryManager; private final List factories = new ArrayList(); @@ -211,13 +173,13 @@ public class RepositoryConnectorExtensionReader { public RepositoryConnectorExtensionReader(TaskListExternalizer taskListExternalizer, TaskRepositoryManager repositoryManager) { - this.taskListExternalizer = taskListExternalizer; this.repositoryManager = repositoryManager; this.result = new MultiStatus(ITasksCoreConstants.ID_PLUGIN, 0, "Repository connectors failed to load.", null); //$NON-NLS-1$ } public void loadConnectors(IExtensionPoint repositoriesExtensionPoint) { - Map> factoryById = readFromRepositoriesExtensionPoint(repositoriesExtensionPoint); + Map> factoryById = readFromRepositoriesExtensionPoint( + repositoriesExtensionPoint); checkForConflicts(factoryById); } @@ -241,8 +203,8 @@ public class RepositoryConnectorExtensionReader { } private void readFromContributorsExtensionPoint() { - IExtensionPoint repositoriesExtensionPoint = Platform.getExtensionRegistry().getExtensionPoint( - EXTENSION_CONTRIBUTORS); + IExtensionPoint repositoriesExtensionPoint = Platform.getExtensionRegistry() + .getExtensionPoint(EXTENSION_CONTRIBUTORS); IExtension[] extensions = repositoriesExtensionPoint.getExtensions(); for (IExtension extension : extensions) { IConfigurationElement[] elements = extension.getConfigurationElements(); @@ -257,7 +219,8 @@ public class RepositoryConnectorExtensionReader { @Override public void run() throws Exception { - RepositoryConnectorContributor contributor = (RepositoryConnectorContributor) element.createExecutableExtension(ATTR_CLASS); + RepositoryConnectorContributor contributor = (RepositoryConnectorContributor) element + .createExecutableExtension(ATTR_CLASS); Collection descriptors = contributor.getDescriptors(); if (descriptors == null) { result.add(new Status(IStatus.ERROR, ITasksCoreConstants.ID_PLUGIN, NLS.bind( @@ -294,23 +257,13 @@ public class RepositoryConnectorExtensionReader { } private void registerConnectorInstances() { - List taskListmigrators = new ArrayList(); List repositoryMigrators = new ArrayList(); for (ConnectorFactory descriptor : factories) { if (descriptor.getConnector() != null) { repositoryManager.addRepositoryConnector(descriptor.getConnector()); - IStatus status = descriptor.createTaskListMigrator(); - if (status.isOK() && descriptor.getTaskListMigrator() != null) { - taskListmigrators.add(descriptor.getTaskListMigrator()); - } else if (status.getSeverity() == IStatus.CANCEL) { - // ignore - } else { - result.add(status); - } - - status = descriptor.createRepositoryMigrator(); + IStatus status = descriptor.createRepositoryMigrator(); if (status.isOK() && descriptor.getRepositoryMigrator() != null) { repositoryMigrators.add(descriptor.getRepositoryMigrator()); } else if (status.getSeverity() == IStatus.CANCEL) { @@ -322,7 +275,6 @@ public class RepositoryConnectorExtensionReader { } repositoryManager.initialize(repositoryMigrators); - taskListExternalizer.initialize(taskListmigrators); } private Map> readFromRepositoriesExtensionPoint( @@ -334,19 +286,15 @@ public class RepositoryConnectorExtensionReader { for (IExtension repositoryExtension : repositoryExtensions) { IConfigurationElement[] elements = repositoryExtension.getConfigurationElements(); ExtensionPointBasedConnectorDescriptor descriptor = null; - IConfigurationElement tasklistMigratorElement = null; IConfigurationElement repositoryMigratorElement = null; for (IConfigurationElement element : elements) { if (element.getName().equals(ELMNT_REPOSITORY_CONNECTOR)) { descriptor = new ExtensionPointBasedConnectorDescriptor(element); - } else if (element.getName().equals(ELMNT_MIGRATOR)) { - tasklistMigratorElement = element; } else if (element.getName().equals(ELMNT_REPOSITORY_MIGRATOR)) { repositoryMigratorElement = element; } } if (descriptor != null) { - descriptor.taskListMigratorElement = tasklistMigratorElement; descriptor.repositoryMigratorElement = repositoryMigratorElement; ConnectorFactory factory = new ConnectorFactory(descriptor, descriptor.getPluginId()); factories.add(factory); @@ -361,8 +309,8 @@ public class RepositoryConnectorExtensionReader { private void checkForConflicts(Map> descriptorById) { for (Map.Entry> entry : descriptorById.entrySet()) { if (entry.getValue().size() > 1) { - MultiStatus status = new MultiStatus(ITasksCoreConstants.ID_PLUGIN, 0, NLS.bind( - "Connector ''{0}'' registered by multiple extensions.", entry.getKey()), null); //$NON-NLS-1$ + MultiStatus status = new MultiStatus(ITasksCoreConstants.ID_PLUGIN, 0, + NLS.bind("Connector ''{0}'' registered by multiple extensions.", entry.getKey()), null); //$NON-NLS-1$ for (ConnectorFactory factory : entry.getValue()) { status.add(new Status(IStatus.ERROR, ITasksCoreConstants.ID_PLUGIN, NLS.bind( "All extensions contributed by ''{0}'' have been disabled.", factory.getPluginId()), null)); //$NON-NLS-1$ diff --git a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/AbstractTaskListMigrator.java b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/AbstractTaskListMigrator.java index ab18983ac..00d534cb9 100644 --- a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/AbstractTaskListMigrator.java +++ b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/AbstractTaskListMigrator.java @@ -19,6 +19,7 @@ import org.w3c.dom.Element; * @author Steffen Pingel * @since 3.0 */ +@Deprecated public abstract class AbstractTaskListMigrator { public static final String KEY_QUERY = "Query"; //$NON-NLS-1$ diff --git a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/spi/RepositoryConnectorDescriptor.java b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/spi/RepositoryConnectorDescriptor.java index c1c532662..90e00e361 100644 --- a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/spi/RepositoryConnectorDescriptor.java +++ b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/spi/RepositoryConnectorDescriptor.java @@ -19,7 +19,7 @@ import org.eclipse.mylyn.tasks.core.AbstractTaskListMigrator; /** * A descriptor for an {@link AbstractRepositoryConnector} instance. - * + * * @since 3.10 * @see RepositoryConnectorBranding */ @@ -27,7 +27,7 @@ public abstract class RepositoryConnectorDescriptor { /** * Creates and returns a connector instance. Only invoked once. - * + * * @return a connector instance */ @NonNull @@ -35,15 +35,18 @@ public abstract class RepositoryConnectorDescriptor { /** * Creates and returns a task list migrator instance. Only invoked once. - * + * * @return a migrator or null if no migrator is provided */ @Nullable - public abstract AbstractTaskListMigrator createTaskListMigrator(); + @Deprecated + public AbstractTaskListMigrator createTaskListMigrator() { + return null; + } /** * Creates and returns a repository migrator instance. Only invoked once. - * + * * @return a migrator or null if no migrator is provided */ @Nullable -- cgit v1.2.3