Fixed #183

Author: thomaswoehlke

src/main/java/org/woehlke/simpleworklist/task/Task.java

@@ -351,28 +351,18 @@ public boolean hasContext(Context context) {
         return (this.getContext().getId().longValue() == context.getId().longValue());
     }
 
-    @Transient
-    public String getView(boolean project){
-        if(project){
-            if(this.project == null){
-                return "redirect:/project/root";
-            } else {
-                return this.project.getUrl();
-            }
-        } else {
-            return "redirect:/"+this.taskState.getUrl();
-        }
-    }
-
     @Transient
     public String getUrl(){
-        return getView(false);
+        return this.taskState.getUrl();
     }
 
     @Transient
     public String getProjectUrl() {
-
-        return getView(true);
+        if(this.project == null){
+            return "redirect:/project/root";
+        } else {
+            return this.project.getUrl();
+        }
     }
 
     public void merge(Task task) {

src/main/java/org/woehlke/simpleworklist/taskstate/TaskState.java

@@ -60,7 +60,7 @@ public String getCode() {
     }
 
     public String getUrl() {
-        return "/taskstate/"+this.name().toLowerCase();
+        return "redirect:/taskstate/"+this.name().toLowerCase();
     }
 
     public static List<TaskState> list() {