--- dao/DaoAdresseVedligehold/src/main/java/dk/daoas/adressevedligehold/tasks/Task.java 2016/01/25 13:35:25 2843 +++ dao/DaoAdresseVedligehold/src/main/java/dk/daoas/adressevedligehold/tasks/Task.java 2016/01/25 21:43:59 2844 @@ -11,6 +11,7 @@ public static class TaskBean { public String description; public String detail; + public String errorMessage; public double percentCompleted; public String state; } @@ -18,6 +19,7 @@ protected boolean abort = false; protected TaskState state = TaskState.STATE_QUEUED; protected TaskManager manager; + private String errorMsg; public void setManager(TaskManager manager) { this.manager = manager; @@ -35,6 +37,8 @@ taskRun(); this.state = TaskState.STATE_DONE; } catch (Exception e) { + this.errorMsg = e.getMessage(); + e.printStackTrace(); this.state = TaskState.STATE_ABORTED; } @@ -59,11 +63,22 @@ bean.detail = this.getDetail(); bean.percentCompleted = this.getPercentCompleted(); bean.state = this.state.toString(); + bean.errorMessage = this.getErrorMessage(); return bean; } - protected abstract void taskRun(); + public String getErrorMessage() { + return this.errorMsg; + } + + /** + * @throws Exception + * + * Implementing classes should not catch terminating exceptions but let it propagate to Task the Task::run() + * method. + */ + protected abstract void taskRun() throws Exception; public abstract String getDescription(); public abstract String getDetail();