Skip to content

Commit 04fc9bc

Browse files
author
Bytekeeper
committed
Fixed resume to create a new item instead of persisting the old item again
1 parent 74eb6fa commit 04fc9bc

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

src/main/java/org/stt/ToItemWriterCommandHandler.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,9 +63,7 @@ public Optional<TimeTrackingItem> endCurrentItem(DateTime endTime) {
6363

6464
@Override
6565
public void resumeGivenItem(TimeTrackingItem item) {
66-
TimeTrackingItem newItem = new TimeTrackingItem(
67-
item.getComment().get(), DateTime.now());
68-
parser.resumeItemCommand(newItem).execute();
66+
parser.resumeItemCommand(item).execute();
6967
}
7068

7169
@Override

src/main/java/org/stt/command/CommandParser.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,9 @@ public Optional<Command> endCurrentItemCommand(DateTime endTime) {
6969
}
7070

7171
public Command resumeItemCommand(final TimeTrackingItem item) {
72-
return new ResumeCommand(persister, item);
72+
TimeTrackingItem newItem = new TimeTrackingItem(
73+
item.getComment().get(), DateTime.now());
74+
return new ResumeCommand(persister, newItem);
7375
}
7476

7577
public Command deleteCommandFor(TimeTrackingItem item) {

src/main/java/org/stt/gui/jfx/STTApplication.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -271,6 +271,7 @@ private void updateItems() {
271271

272272
@Override
273273
public void continueItem(TimeTrackingItem item) {
274+
LOG.severe("Continuing item: " + item);
274275
commandParser.resumeItemCommand(item).execute();
275276
viewAdapter.shutdown();
276277
}

0 commit comments

Comments
 (0)