From f384f1447547bb51c8268b11e51893492a740553 Mon Sep 17 00:00:00 2001 From: Jim Blake Date: Wed, 8 Oct 2014 10:36:38 -0400 Subject: [PATCH 01/30] Remove obsolete code lines, reduce compiler warnings, add generics. --- .../edit/PropertyRetryController.java | 38 ++++++------------- 1 file changed, 12 insertions(+), 26 deletions(-) diff --git a/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/edit/PropertyRetryController.java b/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/edit/PropertyRetryController.java index 935dfcf4c..739f50be6 100644 --- a/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/edit/PropertyRetryController.java +++ b/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/edit/PropertyRetryController.java @@ -22,6 +22,7 @@ import edu.cornell.mannlib.vedit.beans.Option; import edu.cornell.mannlib.vedit.controller.BaseEditController; import edu.cornell.mannlib.vedit.forwarder.PageForwarder; import edu.cornell.mannlib.vedit.forwarder.impl.UrlForwarder; +import edu.cornell.mannlib.vedit.listener.ChangeListener; import edu.cornell.mannlib.vedit.util.FormUtils; import edu.cornell.mannlib.vedit.validator.Validator; import edu.cornell.mannlib.vedit.validator.impl.IntValidator; @@ -29,14 +30,12 @@ import edu.cornell.mannlib.vedit.validator.impl.XMLNameValidator; import edu.cornell.mannlib.vitro.webapp.auth.permissions.SimplePermission; import edu.cornell.mannlib.vitro.webapp.auth.policy.bean.PropertyRestrictionListener; import edu.cornell.mannlib.vitro.webapp.beans.ObjectProperty; -import edu.cornell.mannlib.vitro.webapp.beans.VClass; import edu.cornell.mannlib.vitro.webapp.controller.Controllers; import edu.cornell.mannlib.vitro.webapp.controller.VitroRequest; import edu.cornell.mannlib.vitro.webapp.controller.edit.utils.RoleLevelOptionsSetup; import edu.cornell.mannlib.vitro.webapp.dao.ModelAccess; import edu.cornell.mannlib.vitro.webapp.dao.ObjectPropertyDao; import edu.cornell.mannlib.vitro.webapp.dao.OntologyDao; -import edu.cornell.mannlib.vitro.webapp.dao.VClassDao; public class PropertyRetryController extends BaseEditController { @@ -67,10 +66,8 @@ public class PropertyRetryController extends BaseEditController { } ObjectPropertyDao propDao = ModelAccess.on(getServletContext()).getWebappDaoFactory().getObjectPropertyDao(); - //getUnfilteredWebappDaoFactory().getObjectPropertyDao(); epo.setDataAccessObject(propDao); OntologyDao ontDao = request.getUnfilteredWebappDaoFactory().getOntologyDao(); - VClassDao vclassDao = request.getUnfilteredWebappDaoFactory().getVClassDao(); ObjectProperty propertyForEditing = null; if (!epo.getUseRecycledBean()){ @@ -82,6 +79,7 @@ public class PropertyRetryController extends BaseEditController { epo.setAction("update"); } catch (NullPointerException e) { log.error("Need to implement 'record not found' error message."); + throw(e); } } else { propertyForEditing = new ObjectProperty(); @@ -105,9 +103,9 @@ public class PropertyRetryController extends BaseEditController { //set any validators - List localNameValidatorList = new ArrayList(); + List localNameValidatorList = new ArrayList<>(); localNameValidatorList.add(new XMLNameValidator()); - List localNameInverseValidatorList = new ArrayList(); + List localNameInverseValidatorList = new ArrayList<>(); localNameInverseValidatorList.add(new XMLNameValidator(true)); epo.getValidatorMap().put("LocalName", localNameValidatorList); epo.getValidatorMap().put("LocalNameInverse", localNameInverseValidatorList); @@ -116,7 +114,7 @@ public class PropertyRetryController extends BaseEditController { epo.getValidatorMap().put("DisplayRank", displayRankValidatorList); //set up any listeners - List changeListenerList = new ArrayList(); + List changeListenerList = new ArrayList<>(); changeListenerList.add(new PropertyRestrictionListener(getServletContext())); epo.setChangeListenerList(changeListenerList); @@ -127,7 +125,7 @@ public class PropertyRetryController extends BaseEditController { //set the getMethod so we can retrieve a new bean after we've inserted it try { - Class[] args = new Class[1]; + Class[] args = new Class[1]; args[0] = String.class; epo.setGetMethod(propDao.getClass().getDeclaredMethod("getObjectPropertyByURI",args)); } catch (NoSuchMethodException e) { @@ -149,7 +147,7 @@ public class PropertyRetryController extends BaseEditController { optionMap.put("ProhibitedFromUpdateBelowRoleLevelUsingRoleUri",RoleLevelOptionsSetup.getUpdateOptionsList(propertyForEditing)); optionMap.put("HiddenFromPublishBelowRoleLevelUsingRoleUri",RoleLevelOptionsSetup.getPublishOptionsList(propertyForEditing)); - List groupOptList = FormUtils.makeOptionListFromBeans(request.getUnfilteredWebappDaoFactory().getPropertyGroupDao().getPublicGroups(true),"URI","Name", ((propertyForEditing.getGroupURI()==null) ? "" : propertyForEditing.getGroupURI()), null, (propertyForEditing.getGroupURI()!=null)); + List