litvinovg/vitro
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fixing BaseDumpDirective so it does not attempt do dump methods defined in the java.lang and sun packages

Browse source
This commit is contained in:
briancaruso 2011-11-17 00:08:37 +00:00
parent 02023bb2e1
commit 012c5dc43b
2 changed files with 27 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

3
webapp/src/edu/cornell/mannlib/vitro/webapp/controller/freemarker/UrlBuilder.java
View file

@ -44,7 +44,8 @@ public class UrlBuilder {
VISUALIZATION("/visualization"),
VISUALIZATION_SHORT("/vis"),
VISUALIZATION_AJAX("/visualizationAjax"),
VISUALIZATION_DATA("/visualizationData");
VISUALIZATION_DATA("/visualizationData"),
EDIT_REQUEST_DISPATCH("/editRequestDispatch");
private final String path;

26
webapp/src/freemarker/ext/dump/BaseDumpDirective.java
View file

@ -344,10 +344,12 @@ public abstract class BaseDumpDirective implements TemplateDirectiveModel {
}
private Map<String, Object> getObjectDump(TemplateHashModelEx model, Object object) throws TemplateModelException {
Map<String, Object> map = new HashMap<String, Object>();
map.put(Key.TYPE.toString(), object.getClass().getName());
if( object instanceof java.lang.reflect.Method)
return map;
// Compile the collections of properties and methods available to the template
SortedMap<String, Object> properties = new TreeMap<String, Object>();
SortedMap<String, Object> methods = new TreeMap<String, Object>();
@ -376,11 +378,25 @@ public abstract class BaseDumpDirective implements TemplateDirectiveModel {
// like name/getName - we'll keep only the first form.
for ( Method method : classMethods ) {
if( "declaringClass".equals( method.getName() ))
continue;
// Eliminate methods declared on Object
Class<?> c = method.getDeclaringClass();
if (c.equals(java.lang.Object.class)) {
if (c.equals(java.lang.Object.class) ||
c.equals(java.lang.reflect.Constructor.class) ||
c.equals(java.lang.reflect.Field.class ) )
continue;
}
if(
c.getPackage().getName().startsWith("sun.") ||
c.getPackage().getName().startsWith("java.lang") ||
c.getPackage().getName().startsWith("java.security") )
continue;
log.error("decalring class of method " + method.getName() + " is " + c.getName());
// Eliminate deprecated methods
if (method.isAnnotationPresent(Deprecated.class)) {
@ -399,8 +415,12 @@ public abstract class BaseDumpDirective implements TemplateDirectiveModel {
// The method is available as a property
if (keySet.contains(propertyName)) {
try{
TemplateModel value = model.get(propertyName);
properties.put(propertyName, getDump(value));
}catch(Throwable th){
log.error("problem dumping " + propertyName + " on " + object.getClass().getName(), th);
}
continue;
}
}