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

Changing OperationUtils to use log4j

Browse source
This commit is contained in:
bdc34 2010-10-06 21:52:03 +00:00
parent b46412f71b
commit e2eb84ce73
2 changed files with 24 additions and 21 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
webapp/src/edu/cornell/mannlib/vedit/controller/OperationController.java
View file

@ -174,14 +174,17 @@ public class OperationController extends BaseEditController {
}
private void retry(HttpServletRequest request, HttpServletResponse response) throws IOException {
String referer = request.getHeader("Referer");
int epoKeyIndex = referer.indexOf("_epoKey");
if (epoKeyIndex<0)
response.sendRedirect(referer+"&_epoKey="+request.getParameter("_epoKey"));
else{
String url = referer.substring(0,epoKeyIndex) + "_epoKey="+request.getParameter("_epoKey");
response.sendRedirect(url);
String referer = request.getHeader("Referer");
if( referer != null ){
int epoKeyIndex = referer.indexOf("_epoKey");
if (epoKeyIndex >= 0){
String url = referer.substring(0,epoKeyIndex) + "_epoKey="+request.getParameter("_epoKey");
response.sendRedirect(url);
return;
}
}
response.sendRedirect(referer+"&_epoKey="+request.getParameter("_epoKey"));
return;
}

28
webapp/src/edu/cornell/mannlib/vedit/util/OperationUtils.java
View file

@ -54,9 +54,9 @@ public class OperationUtils{
try {
setterMethod.invoke(newObj,arglist);
} catch (Exception e) {
System.out.println("Couldn't invoke method");
System.out.println(e.getMessage());
System.out.println(field+" "+arglist[0]);
log.error("Couldn't invoke method");
log.error(e.getMessage());
log.error(field+" "+arglist[0]);
}
}
@ -97,18 +97,18 @@ public class OperationUtils{
setArgs[0] = fieldVal;
setterMethod.invoke(newBean,setArgs);
} catch (IllegalAccessException iae) {
System.out.println("edu.cornell.mannlib.vitro.edit.utils.OperationUtils encountered IllegalAccessException invoking "+setterMethod.getName());
log.error("edu.cornell.mannlib.vitro.edit.utils.OperationUtils encountered IllegalAccessException invoking "+setterMethod.getName());
} catch (InvocationTargetException ite) {
System.out.println("edu.cornell.mannlib.vitro.edit.utils.OperationUtils encountered InvocationTargetException invoking "+setterMethod.getName());
System.out.println(ite.getTargetException().getClass().toString());
log.error("edu.cornell.mannlib.vitro.edit.utils.OperationUtils encountered InvocationTargetException invoking "+setterMethod.getName());
log.error(ite.getTargetException().getClass().toString());
}
} catch (IllegalAccessException iae) {
System.out.println(OperationUtils.class.getName()+" encountered IllegalAccessException invoking "+beanMeths[i].getName());
log.error(OperationUtils.class.getName()+" encountered IllegalAccessException invoking "+beanMeths[i].getName());
} catch (InvocationTargetException ite) {
System.out.println(OperationUtils.class.getName()+" encountered InvocationTargetException invoking "+beanMeths[i].getName());
System.out.println(ite.getTargetException().getClass().toString());
log.error(OperationUtils.class.getName()+" encountered InvocationTargetException invoking "+beanMeths[i].getName());
log.error(ite.getTargetException().getClass().toString());
} catch (IllegalArgumentException iae) {
// System.out.println(OperationUtils.class.getName()+" found that "+beanMeths[i].getName()+" requires one or more arguments. Skipping.");
// log.error(OperationUtils.class.getName()+" found that "+beanMeths[i].getName()+" requires one or more arguments. Skipping.");
}
} catch (NoSuchMethodException nsme){
// ignore this field because there is no setter method
@ -116,11 +116,11 @@ public class OperationUtils{
}
}
} catch (InstantiationException ie){
System.out.println("edu.cornell.mannlib.vitro.edit.utils.OperationUtils.cloneBean("+bean.getClass().toString()+") could not instantiate new instance of bean.");
System.out.println(ie.getStackTrace());
log.error("edu.cornell.mannlib.vitro.edit.utils.OperationUtils.cloneBean("+bean.getClass().toString()+") could not instantiate new instance of bean.");
log.error(ie.getStackTrace());
} catch (IllegalAccessException iae){
System.out.println("edu.cornell.mannlib.vitro.edit.utils.OperationUtils.cloneBean("+bean.getClass().toString()+") encountered illegal access exception instantiating new bean.");
System.out.println(iae.getStackTrace());
log.error("edu.cornell.mannlib.vitro.edit.utils.OperationUtils.cloneBean("+bean.getClass().toString()+") encountered illegal access exception instantiating new bean.");
log.error(iae.getStackTrace());
}
return newBean;
}