Unify nature checks and reuse existing constants from core.natures.PDE

lathapatil · Jul 14, 2024 · a37e0a1 · a37e0a1
1 parent ff55594
commit a37e0a1
Show file tree

Hide file tree

Showing 86 changed files with 496 additions and 730 deletions.
diff --git a/...lipse.pde.api.tools.tests/src/org/eclipse/pde/api/tools/builder/tests/ApiBuilderTest.java b/...lipse.pde.api.tools.tests/src/org/eclipse/pde/api/tools/builder/tests/ApiBuilderTest.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2008, 2019 IBM Corporation and others.
+ * Copyright (c) 2008, 2024 IBM Corporation and others.
  *
  * This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License 2.0
@@ -69,6 +69,7 @@
 import org.eclipse.pde.api.tools.tests.util.ProjectUtils;
 import org.eclipse.pde.internal.core.ICoreConstants;
 import org.eclipse.pde.internal.core.PDECore;
+import org.eclipse.pde.internal.core.natures.PluginProject;
 import org.eclipse.pde.ui.tests.util.FreezeMonitor;
 import org.eclipse.ui.dialogs.IOverwriteQuery;
 import org.eclipse.ui.wizards.datatransfer.FileSystemStructureProvider;
@@ -1154,7 +1155,7 @@ private static String toString(List<IProject> projects) throws Exception {
 			contents.append(System.lineSeparator());
 			contents.append("    is open: " + project.isOpen()); //$NON-NLS-1$
 			contents.append(System.lineSeparator());
-			if (project.hasNature(JavaCore.NATURE_ID) && project.isAccessible()) {
+			if (PluginProject.isJavaProject(project) && project.isAccessible()) {
 				IJavaProject javaProject = JavaCore.create(project);
 				boolean ignoreUnresolvedEntry = true;
 				IClasspathEntry[] projectClassPath = javaProject.getResolvedClasspath(ignoreUnresolvedEntry);

diff --git a/...de.api.tools.tests/src/org/eclipse/pde/api/tools/builder/tests/ApiTestingEnvironment.java b/...de.api.tools.tests/src/org/eclipse/pde/api/tools/builder/tests/ApiTestingEnvironment.java
@@ -47,7 +47,7 @@
 import org.eclipse.pde.core.target.ITargetDefinition;
 import org.eclipse.pde.core.target.ITargetPlatformService;
 import org.eclipse.pde.internal.core.PDECore;
-import org.eclipse.pde.internal.core.natures.PDE;
+import org.eclipse.pde.internal.core.natures.PluginProject;
 import org.eclipse.pde.ui.tests.util.TargetPlatformUtil;
 
 /**
@@ -133,8 +133,8 @@ protected IJavaProject createProject(String projectName) {
 			if (project.exists()) {
 				project.delete(true, new NullProgressMonitor());
 			}
-			jproject = ProjectUtils.createPluginProject(projectName, new String[] {
-					PDE.PLUGIN_NATURE, ApiPlugin.NATURE_ID });
+			jproject = ProjectUtils.createPluginProject(projectName,
+					new String[] { PluginProject.NATURE, ApiPlugin.NATURE_ID });
 			addProject(jproject.getProject());
 		} catch (CoreException ce) {
 			ApiPlugin.log(ce);

diff --git a/.../org.eclipse.pde.api.tools.tests/src/org/eclipse/pde/api/tools/tests/AbstractApiTest.java b/.../org.eclipse.pde.api.tools.tests/src/org/eclipse/pde/api/tools/tests/AbstractApiTest.java
@@ -44,7 +44,7 @@
 import org.eclipse.pde.core.plugin.PluginRegistry;
 import org.eclipse.pde.internal.core.ICoreConstants;
 import org.eclipse.pde.internal.core.PDECore;
-import org.eclipse.pde.internal.core.natures.PDE;
+import org.eclipse.pde.internal.core.natures.PluginProject;
 import org.eclipse.pde.ui.tests.util.FreezeMonitor;
 import org.junit.After;
 import org.junit.Before;
@@ -155,8 +155,8 @@ protected void createProject(final String name, String[] packages) throws Except
 		}
 
 		// create project and import source
-		IJavaProject jproject = ProjectUtils.createPluginProject(name, new String[] {
-				PDE.PLUGIN_NATURE, ApiPlugin.NATURE_ID });
+		IJavaProject jproject = ProjectUtils.createPluginProject(name,
+				new String[] { PluginProject.NATURE, ApiPlugin.NATURE_ID });
 		assertNotNull("The java project must have been created", jproject); //$NON-NLS-1$
 
 		IPackageFragmentRoot root = jproject.getPackageFragmentRoot(jproject.getProject().getFolder(ProjectUtils.SRC_FOLDER));

diff --git a/...rg.eclipse.pde.api.tools.tests/src/org/eclipse/pde/api/tools/tests/util/ProjectUtils.java b/...rg.eclipse.pde.api.tools.tests/src/org/eclipse/pde/api/tools/tests/util/ProjectUtils.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2008, 2017 IBM Corporation and others.
+ * Copyright (c) 2008, 2024 IBM Corporation and others.
  *
  * This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License 2.0
@@ -40,6 +40,7 @@
 import org.eclipse.pde.core.project.IBundleProjectService;
 import org.eclipse.pde.core.project.IPackageExportDescription;
 import org.eclipse.pde.internal.core.PDECore;
+import org.eclipse.pde.internal.core.natures.PluginProject;
 import org.osgi.framework.Version;
 
 /**
@@ -201,7 +202,7 @@ public static IJavaProject createPluginProject(String projectName, String[] addi
 	public static IJavaProject createJavaProject(String projectName, String[] additionalNatures) throws CoreException {
 		IProgressMonitor monitor = new NullProgressMonitor();
 		IProject project = createProject(projectName, monitor);
-		if (!project.hasNature(JavaCore.NATURE_ID)) {
+		if (!PluginProject.isJavaProject(project)) {
 			addNatureToProject(project, JavaCore.NATURE_ID, monitor);
 		}
 		if (additionalNatures != null) {

diff --git a/...se.pde.api.tools.tests/src/org/eclipse/pde/api/tools/util/tests/ProjectCreationTests.java b/...se.pde.api.tools.tests/src/org/eclipse/pde/api/tools/util/tests/ProjectCreationTests.java
@@ -37,7 +37,7 @@
 import org.eclipse.pde.api.tools.tests.util.ProjectUtils;
 import org.eclipse.pde.core.project.IPackageExportDescription;
 import org.eclipse.pde.internal.core.ICoreConstants;
-import org.eclipse.pde.internal.core.natures.PDE;
+import org.eclipse.pde.internal.core.natures.PluginProject;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
@@ -118,7 +118,7 @@ public void testImportClassesTestSource() {
 	public void testCreatePluginProject() throws CoreException {
 		IJavaProject jproject = getTestingJavaProject(TESTING_PROJECT_NAME);
 		IProject project = jproject.getProject();
-		assertTrue("project must have the PDE nature", project.hasNature(PDE.PLUGIN_NATURE)); //$NON-NLS-1$
+		assertTrue("project must have the PDE nature", project.hasNature(PluginProject.NATURE)); //$NON-NLS-1$
 		assertTrue("project must have the java nature", project.hasNature(JavaCore.NATURE_ID)); //$NON-NLS-1$
 		assertTrue("project must have additional nature for API Tools", project.hasNature(ApiPlugin.NATURE_ID)); //$NON-NLS-1$
 		IFile file = project.getFile("build.properties"); //$NON-NLS-1$

diff --git a/....api.tools.ui/src/org/eclipse/pde/api/tools/ui/internal/markers/ApiQuickFixProcessor.java b/....api.tools.ui/src/org/eclipse/pde/api/tools/ui/internal/markers/ApiQuickFixProcessor.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) Sep 26, 2018 IBM Corporation and others.
+ * Copyright (c) Sep 26, 2024 IBM Corporation and others.
  *
  * This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License 2.0
@@ -52,6 +52,7 @@
 import org.eclipse.osgi.util.TextProcessor;
 import org.eclipse.pde.api.tools.internal.provisional.ApiPlugin;
 import org.eclipse.pde.api.tools.internal.util.Signatures;
+import org.eclipse.pde.api.tools.internal.util.Util;
 import org.eclipse.pde.api.tools.ui.internal.ApiUIPlugin;
 import org.eclipse.pde.api.tools.ui.internal.IApiToolsConstants;
 import org.eclipse.pde.api.tools.ui.internal.refactoring.CreateFileChange;
@@ -134,7 +135,7 @@ public IJavaCompletionProposal[] getCorrections(IInvocationContext context, IPro
 		List<IJavaCompletionProposal> proposals = new ArrayList<>();
 		ICompilationUnit unit = context.getCompilationUnit();
 		IProject project = unit.getJavaProject().getProject();
-		if (!project.hasNature(ApiPlugin.NATURE_ID)) {
+		if (!Util.isApiProject(project)) {
 			return new IJavaCompletionProposal[0];
 		}
 		IFile build = project.getFile("build.properties"); //$NON-NLS-1$
@@ -166,8 +167,7 @@ public IJavaCompletionProposal[] getCorrections(IInvocationContext context, IPro
 	public static Change createChange(ICompilationUnit unit, String qualifiedname) throws CoreException {
 		IProject project = unit.getJavaProject().getProject();
 		IFile buildProperties = project.getFile("build.properties"); //$NON-NLS-1$
-		boolean isBundle = project.hasNature(ApiPlugin.NATURE_ID);
-		if (!isBundle) {
+		if (!Util.isApiProject(project)) {
 			return new NullChange();
 		}
 		return new CompositeChange(MarkerMessages.UnknownAnnotationResolution_3, new Change[] {

diff --git a/...rc/org/eclipse/pde/api/tools/ui/internal/preferences/ApiErrorsWarningsPreferencePage.java b/...rc/org/eclipse/pde/api/tools/ui/internal/preferences/ApiErrorsWarningsPreferencePage.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2007, 2018 IBM Corporation and others.
+ * Copyright (c) 2007, 2024 IBM Corporation and others.
  *
  * This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License 2.0
@@ -20,13 +20,12 @@
 
 import org.eclipse.core.resources.IProject;
 import org.eclipse.core.resources.ResourcesPlugin;
-import org.eclipse.core.runtime.CoreException;
 import org.eclipse.jdt.core.IJavaProject;
 import org.eclipse.jdt.core.JavaCore;
 import org.eclipse.jdt.core.JavaModelException;
 import org.eclipse.jface.dialogs.IDialogConstants;
 import org.eclipse.jface.preference.PreferencePage;
-import org.eclipse.pde.api.tools.internal.provisional.ApiPlugin;
+import org.eclipse.pde.api.tools.internal.util.Util;
 import org.eclipse.pde.api.tools.ui.internal.ApiUIPlugin;
 import org.eclipse.pde.api.tools.ui.internal.IApiToolsConstants;
 import org.eclipse.pde.api.tools.ui.internal.IApiToolsHelpContextIds;
@@ -110,15 +109,10 @@ protected Control createContents(Composite parent) {
 			Set<IJavaProject> set = new HashSet<>();
 			try {
 				IJavaProject[] projects = JavaCore.create(ResourcesPlugin.getWorkspace().getRoot()).getJavaProjects();
-				IProject project = null;
 				for (IJavaProject p : projects) {
-					project = p.getProject();
-					try {
-						if (project.hasNature(ApiPlugin.NATURE_ID) && block.hasProjectSpecificSettings(project)) {
-							set.add(p);
-						}
-					} catch (CoreException ce) {
-						// do nothing ignore the project
+					IProject project = p.getProject();
+					if (Util.isApiProject(project) && block.hasProjectSpecificSettings(project)) {
+						set.add(p);
 					}
 				}
 			} catch (JavaModelException jme) {

diff --git a/...tools.ui/src/org/eclipse/pde/api/tools/ui/internal/wizards/ApiToolingSetupWizardPage.java b/...tools.ui/src/org/eclipse/pde/api/tools/ui/internal/wizards/ApiToolingSetupWizardPage.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2007, 2018 IBM Corporation and others.
+ * Copyright (c) 2007, 2024 IBM Corporation and others.
  *
  * This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License 2.0
@@ -16,7 +16,6 @@
 import java.io.File;
 import java.lang.reflect.InvocationTargetException;
 import java.text.MessageFormat;
-import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.HashMap;
 import java.util.HashSet;
@@ -40,7 +39,6 @@
 import org.eclipse.jface.operation.IRunnableWithProgress;
 import org.eclipse.jface.viewers.ArrayContentProvider;
 import org.eclipse.jface.viewers.CheckboxTableViewer;
-import org.eclipse.jface.viewers.ISelection;
 import org.eclipse.jface.viewers.ISelectionProvider;
 import org.eclipse.jface.viewers.IStructuredSelection;
 import org.eclipse.jface.viewers.ViewerComparator;
@@ -57,6 +55,7 @@
 import org.eclipse.pde.api.tools.ui.internal.IApiToolsConstants;
 import org.eclipse.pde.api.tools.ui.internal.IApiToolsHelpContextIds;
 import org.eclipse.pde.api.tools.ui.internal.SWTFactory;
+import org.eclipse.pde.internal.core.natures.PluginProject;
 import org.eclipse.swt.SWT;
 import org.eclipse.swt.events.KeyListener;
 import org.eclipse.swt.events.SelectionListener;
@@ -248,24 +247,15 @@ private void updateCheckStatus(int count) {
 	 */
 	private IProject[] getInputProjects() {
 		IProject[] projects = ResourcesPlugin.getWorkspace().getRoot().getProjects();
-		ArrayList<IProject> pjs = new ArrayList<>();
-		for (IProject project : projects) {
-			try {
-				if (acceptProject(project)) {
-					pjs.add(project);
-				}
-			} catch (CoreException ce) {
-			}
-		}
-		return pjs.toArray(new IProject[pjs.size()]);
+		return Arrays.stream(projects).filter(this::acceptProject).toArray(IProject[]::new);
 	}
 
-	private boolean acceptProject(IProject project) throws CoreException {
+	private boolean acceptProject(IProject project) {
 		if (project == null) {
 			return false;
 		}
-		return (project.hasNature(JavaCore.NATURE_ID) && project.hasNature("org.eclipse.pde.PluginNature")) //$NON-NLS-1$
-				&& !project.hasNature(ApiPlugin.NATURE_ID) && !Util.isBinaryProject(project);
+		return PluginProject.isJavaProject(project) && PluginProject.isPluginProject(project)
+				&& !Util.isApiProject(project) && !Util.isBinaryProject(project);
 	}
 
 	/**
@@ -281,25 +271,10 @@ protected Object[] getWorkbenchSelection() {
 					IWorkbenchSite site = part.getSite();
 					if (site != null) {
 						ISelectionProvider provider = site.getSelectionProvider();
-						if (provider != null) {
-							ISelection selection = provider.getSelection();
-							if (selection instanceof IStructuredSelection) {
-								Object[] jps = ((IStructuredSelection) provider.getSelection()).toArray();
-								ArrayList<IProject> pjs = new ArrayList<>();
-								for (Object jp : jps) {
-									if (jp instanceof IAdaptable) {
-										IAdaptable adapt = (IAdaptable) jp;
-										IProject pj = adapt.getAdapter(IProject.class);
-										try {
-											if (acceptProject(pj)) {
-												pjs.add(pj);
-											}
-										} catch (CoreException ce) {
-										}
-									}
-								}
-								return pjs.toArray();
-							}
+						if (provider != null && provider.getSelection() instanceof IStructuredSelection selection) {
+							return Arrays.stream(selection.toArray()).filter(IAdaptable.class::isInstance)
+									.map(IAdaptable.class::cast).map(a -> a.getAdapter(IProject.class))
+									.filter(this::acceptProject).toArray();
 						}
 					}
 				}

diff --git a/....eclipse.pde.api.tools/src/org/eclipse/pde/api/tools/internal/ApiAnalysisApplication.java b/....eclipse.pde.api.tools/src/org/eclipse/pde/api/tools/internal/ApiAnalysisApplication.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2019, 2021 Red Hat Inc. and others.
+ * Copyright (c) 2019, 2024 Red Hat Inc. and others.
  *
  * This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License 2.0
@@ -20,6 +20,7 @@
 import java.nio.file.Files;
 import java.util.Arrays;
 import java.util.Objects;
+import java.util.Set;
 import java.util.function.Predicate;
 
 import org.eclipse.core.resources.ICommand;
@@ -376,11 +377,14 @@ private IProject importProject(File projectPath) throws CoreException, IOExcepti
 		return project;
 	}
 
+	private static final Set<String> REMOVED_BUILDERS = Set.of(
+			org.eclipse.pde.internal.core.natures.PluginProject.MANIFEST_BUILDER_ID,
+			org.eclipse.pde.internal.core.natures.PluginProject.SCHEMA_BUILDER_ID);
+
 	private static ICommand[] removeManifestAndSchemaBuilders(ICommand[] buildSpec) {
 		// remove manifest and schema builders
-		return Arrays.stream(buildSpec).filter(x -> !("org.eclipse.pde.ManifestBuilder".equals(x.getBuilderName()) //$NON-NLS-1$
-				|| "org.eclipse.pde.SchemaBuilder".equals(x.getBuilderName())) //$NON-NLS-1$
-		).toArray(ICommand[]::new);
+		return Arrays.stream(buildSpec).filter(x -> !REMOVED_BUILDERS.contains(x.getBuilderName()))
+				.toArray(ICommand[]::new);
 	}
 
 	@Override

diff --git a/....api.tools/src/org/eclipse/pde/api/tools/internal/ApiAnnotationsClasspathContributor.java b/....api.tools/src/org/eclipse/pde/api/tools/internal/ApiAnnotationsClasspathContributor.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2023 Christoph Läubrich and others.
+ * Copyright (c) 2023, 2024 Christoph Läubrich and others.
  *
  * This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License 2.0
@@ -22,13 +22,12 @@
 import java.util.stream.Stream;
 
 import org.eclipse.core.resources.IResource;
-import org.eclipse.core.runtime.CoreException;
 import org.eclipse.jdt.core.IClasspathEntry;
 import org.eclipse.osgi.service.resolver.BundleDelta;
 import org.eclipse.osgi.service.resolver.BundleDescription;
 import org.eclipse.osgi.service.resolver.State;
 import org.eclipse.osgi.service.resolver.StateDelta;
-import org.eclipse.pde.api.tools.internal.provisional.ApiPlugin;
+import org.eclipse.pde.api.tools.internal.util.Util;
 import org.eclipse.pde.core.IClasspathContributor;
 import org.eclipse.pde.core.plugin.IPluginModelBase;
 import org.eclipse.pde.core.plugin.PluginRegistry;
@@ -77,11 +76,7 @@ private boolean hasApiNature(IPluginModelBase model) {
 		if (model != null) {
 			IResource resource = model.getUnderlyingResource();
 			if (resource != null) {
-				try {
-					return resource.getProject().hasNature(ApiPlugin.NATURE_ID);
-				} catch (CoreException e) {
-					// assume not compatible project then...
-				}
+				return Util.isApiProject(resource.getProject());
 			}
 		}
 		return false;

diff --git a/...g.eclipse.pde.api.tools/src/org/eclipse/pde/api/tools/internal/ApiDescriptionManager.java b/...g.eclipse.pde.api.tools/src/org/eclipse/pde/api/tools/internal/ApiDescriptionManager.java
@@ -50,6 +50,7 @@
 import org.eclipse.pde.api.tools.internal.provisional.model.IApiElement;
 import org.eclipse.pde.api.tools.internal.provisional.scanner.ScannerMessages;
 import org.eclipse.pde.api.tools.internal.util.Util;
+import org.eclipse.pde.internal.core.natures.PluginProject;
 import org.w3c.dom.Document;
 import org.w3c.dom.Element;
 import org.w3c.dom.Node;
@@ -221,7 +222,7 @@ public void rollback(ISaveContext context) {
 
 	@Override
 	public synchronized void saving(ISaveContext context) throws CoreException {
-		if (context.getKind() == ISaveContext.PROJECT_SAVE && !Util.isJavaProject(context.getProject())) {
+		if (context.getKind() == ISaveContext.PROJECT_SAVE && !PluginProject.isJavaProject(context.getProject())) {
 			return;
 		}
 

diff --git a/...eclipse.pde.api.tools/src/org/eclipse/pde/api/tools/internal/WorkspaceDeltaProcessor.java b/...eclipse.pde.api.tools/src/org/eclipse/pde/api/tools/internal/WorkspaceDeltaProcessor.java
@@ -34,6 +34,7 @@
 import org.eclipse.pde.api.tools.internal.model.ApiBaseline;
 import org.eclipse.pde.api.tools.internal.provisional.ApiPlugin;
 import org.eclipse.pde.api.tools.internal.util.Util;
+import org.eclipse.pde.internal.core.natures.PluginProject;
 
 /**
  * Standard delta processor for us to track element state changes in the workspace
@@ -239,7 +240,7 @@ public void resourceChanged(IResourceChangeEvent event) {
 						resource = element.getResource();
 						if (element.getResource().getType() == IResource.PROJECT) {
 							IProject project = (IProject) resource;
-							if (Util.isApiProject(project) || Util.isJavaProject(project)) {
+							if (Util.isApiProject(project) || PluginProject.isJavaProject(project)) {
 								if ((element.getFlags() & IResourceDelta.DESCRIPTION) != 0) {
 									cleanAndDisposeWorkspaceBaseline(resource);
 									break;
@@ -252,7 +253,7 @@ public void resourceChanged(IResourceChangeEvent event) {
 						resource = element.getResource();
 						if (element.getResource().getType() == IResource.PROJECT) {
 							IProject project = (IProject) resource;
-							if (Util.isApiProject(project) || Util.isJavaProject(project)) {
+							if (Util.isApiProject(project) || PluginProject.isJavaProject(project)) {
 									cleanAndDisposeWorkspaceBaseline(resource);
 									break;
 							}
@@ -265,7 +266,7 @@ public void resourceChanged(IResourceChangeEvent event) {
 			case IResourceChangeEvent.PRE_DELETE: {
 				if (resource.getType() == IResource.PROJECT) {
 					IProject project = (IProject) resource;
-					if (Util.isApiProject(project) || Util.isJavaProject(project)) {
+					if (Util.isApiProject(project) || PluginProject.isJavaProject(project)) {
 						if (ApiPlugin.DEBUG_WORKSPACE_DELTA_PROCESSOR) {
 							if (event.getType() == IResourceChangeEvent.PRE_CLOSE) {
 								System.out.println("processed PRE_CLOSE delta for project: [" + resource.getName() + "]"); //$NON-NLS-1$ //$NON-NLS-2$