ClassUtils: deprecate hasClass methods

An implication of these methods is that they might somehow test for the
existence of classes without loading them, which is not actually true.
Better to recommend using Types.load directly and comparing vs null.

Author: ctrueden

src/main/java/org/scijava/util/ClassUtils.java

@@ -89,18 +89,6 @@ private ClassUtils() {
 
 	// -- Class loading, querying and reflection --
 
-	/** Checks whether a class with the given name exists. */
-	public static boolean hasClass(final String className) {
-		return hasClass(className, null);
-	}
-
-	/** Checks whether a class with the given name exists. */
-	public static boolean hasClass(final String className,
-		final ClassLoader classLoader)
-	{
-		return Types.load(className, classLoader) != null;
-	}
-
 	/**
 	 * Gets the base location of the given class.
 	 * <p>
@@ -627,6 +615,20 @@ public static Class<?> loadClass(final String name,
 		return Types.load(name, classLoader, quietly);
 	}
 
+	/** @deprecated Use {@link Types#load(String)} instead. */
+	@Deprecated
+	public static boolean hasClass(final String className) {
+		return Types.load(className) != null;
+	}
+
+	/** @deprecated Use {@link Types#load(String, ClassLoader)} instead. */
+	@Deprecated
+	public static boolean hasClass(final String className,
+		final ClassLoader classLoader)
+	{
+		return Types.load(className, classLoader) != null;
+	}
+
 	/** @deprecated use {@link ConversionUtils#convert(Object, Class)} */
 	@Deprecated
 	public static <T> T convert(final Object value, final Class<T> type) {