Fixed resume to create a new item instead of persisting the old item …

…again
SimpleTimeTracking · Apr 10, 2015 · 04fc9bc · 04fc9bc
1 parent 74eb6fa
commit 04fc9bc
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/src/main/java/org/stt/ToItemWriterCommandHandler.java b/src/main/java/org/stt/ToItemWriterCommandHandler.java
@@ -63,9 +63,7 @@ public Optional<TimeTrackingItem> endCurrentItem(DateTime endTime) {
 
     @Override
     public void resumeGivenItem(TimeTrackingItem item) {
-        TimeTrackingItem newItem = new TimeTrackingItem(
-                item.getComment().get(), DateTime.now());
-        parser.resumeItemCommand(newItem).execute();
+        parser.resumeItemCommand(item).execute();
     }
 
     @Override

diff --git a/src/main/java/org/stt/command/CommandParser.java b/src/main/java/org/stt/command/CommandParser.java
@@ -69,7 +69,9 @@ public Optional<Command> endCurrentItemCommand(DateTime endTime) {
     }
 
     public Command resumeItemCommand(final TimeTrackingItem item) {
-        return new ResumeCommand(persister, item);
+        TimeTrackingItem newItem = new TimeTrackingItem(
+                item.getComment().get(), DateTime.now());
+        return new ResumeCommand(persister, newItem);
     }
 
     public Command deleteCommandFor(TimeTrackingItem item) {

diff --git a/src/main/java/org/stt/gui/jfx/STTApplication.java b/src/main/java/org/stt/gui/jfx/STTApplication.java
@@ -271,6 +271,7 @@ private void updateItems() {
 
     @Override
     public void continueItem(TimeTrackingItem item) {
+        LOG.severe("Continuing item: " + item);
         commandParser.resumeItemCommand(item).execute();
         viewAdapter.shutdown();
     }