RenJuan/TASKPM
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Add "throws InstanceNotFoundException" to delete method and catch the exception into confirmDelete method

Browse source
This commit is contained in:
Cristina Alvarino 2011-07-15 08:51:44 +02:00 • committed by Manuel Rego Casasnovas
parent 9e3e7bc916
commit f6e78f3bc0
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
navalplanner-webapp/src/main/java/org/navalplanner/web/common/BaseCRUDController.java
View file

@ -23,6 +23,7 @@ import static org.navalplanner.web.I18nHelper._;
import org.apache.commons.lang.StringUtils;
import org.navalplanner.business.common.IHumanIdentifiable;
import org.navalplanner.business.common.exceptions.InstanceNotFoundException;
import org.navalplanner.business.common.exceptions.ValidationException;
import org.zkoss.zk.ui.Component;
import org.zkoss.zk.ui.util.GenericForwardComposer;
@ -319,6 +320,11 @@ public abstract class BaseCRUDController<T extends IHumanIdentifiable> extends
}
} catch (InterruptedException e) {
throw new RuntimeException(e);
} catch (InstanceNotFoundException ie) {
messagesForUser.showMessage(
Level.ERROR,
_("{0} \"{1}\" could not be deleted, it was already removed", getEntityType(),
entity.getHumanId()));
}
}
@ -343,6 +349,6 @@ public abstract class BaseCRUDController<T extends IHumanIdentifiable> extends
* @param entity
* Entity to be removed
*/
protected abstract void delete(T entity);
protected abstract void delete(T entity) throws InstanceNotFoundException;
}