Apache
/
openjpa
mirror of https://github.com/apache/openjpa.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

OPENJPA-758: Added code to check the ClassResolver to get a class loader when attempting to load a ValueHandler or FieldStrategy. 

git-svn-id: https://svn.apache.org/repos/asf/openjpa/branches/2.0.x@1155108 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Heath Thomann 2011-08-08 21:20:47 +00:00
parent d12334e6f9
commit 1345e75631
2 changed files with 41 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/meta/MappingRepository.java
View File

@ -1204,7 +1204,7 @@ public class MappingRepository extends MetaDataRepository {
try {
Class<?> c = JavaTypes.classForName(name, val,
AccessController.doPrivileged(
J2DoPrivHelper.getClassLoaderAction(FieldStrategy.class)));
J2DoPrivHelper.getClassLoaderAction(FieldStrategy.class)),false);
Object o = AccessController.doPrivileged(
J2DoPrivHelper.newInstanceAction(c));
Configurations.configureInstance(o, getConfiguration(), props);
@ -1231,7 +1231,7 @@ public class MappingRepository extends MetaDataRepository {
try {
Class<?> c = JavaTypes.classForName(name, val,
AccessController.doPrivileged(
J2DoPrivHelper.getClassLoaderAction(ValueHandler.class)));
J2DoPrivHelper.getClassLoaderAction(ValueHandler.class)),false);
if (ValueHandler.class.isAssignableFrom(c)) {
ValueHandler vh = (ValueHandler) AccessController.doPrivileged(
J2DoPrivHelper.newInstanceAction(c));

40
openjpa-kernel/src/main/java/org/apache/openjpa/meta/JavaTypes.java
View File

@ -205,12 +205,38 @@ public class JavaTypes {
context.getFieldMetaData().getDeclaringType(), context, loader);
}
/**
* Try to load a class using the provided loader. Optionally tries the
* configuration's ClassResolver if the supplied loader cannot find the class.
*
* @param name Name of the class to load.
* @param context
* @param loader ClassLoader to use. If null, the configuration's ClassResolver will be used.
* @param mustExist Whether the supplied loader <b>must</b> be able to load the class. If true no attempt to use a
* different classloader will be made. If false the ClassResolver from the configuration will be used.
*/
public static Class<?> classForName(String name, ValueMetaData context,
ClassLoader loader, boolean mustExist) {
return classForName(name,
context.getFieldMetaData().getDefiningMetaData(),
context.getFieldMetaData().getDeclaringType(), context, loader, mustExist);
}
/**
* OJ-758: Delegates to the final classForName. This is needed
* to maintain the existing code path prior to OJ-758.
*/
private static Class<?> classForName(String name, ClassMetaData meta,
Class<?> dec, ValueMetaData vmd, ClassLoader loader) {
return classForName(name, meta, dec, vmd, loader, true);
}
/**
* Check the given name against the same set of standard packages used
* when parsing metadata.
*/
private static Class<?> classForName(String name, ClassMetaData meta, Class<?> dec, ValueMetaData vmd,
ClassLoader loader) {
ClassLoader loader, boolean mustExist) {
// special case for PersistenceCapable and Object
if ("PersistenceCapable".equals(name)
|| "javax.jdo.PersistenceCapable".equals(name)) // backwards compatibility
@ -232,9 +258,21 @@ public class JavaTypes {
pkg = Strings.getPackageName(vmd.getDeclaredType());
cls = CFMetaDataParser.classForName(name, pkg, runtime, loader);
}
//OJ-758 start: If the class is still null, as a last/final attempt to
//load the class, check with the ClassResolver to get a loader
//and use it to attempt to load the class.
if (cls == null && !mustExist){
loader = rep.getConfiguration().getClassResolverInstance().
getClassLoader(dec, meta.getEnvClassLoader());
cls = CFMetaDataParser.classForName(name, pkg, runtime, loader);
}
//OJ-758 end
if (cls == null)
throw new MetaDataException(_loc.get("bad-class", name,
(vmd == null) ? (Object) meta : (Object) vmd));
return cls;
}