Using the play icon to continue an activity is broken

[kralan]

Using the play icon on a line in timer view to continue an activity is broken since 0.0.76.
Before 0.0.76 it used to:
Stop the currently running activity, start a new activity with the entries copied from the entry the play icon was clicked on.
Now it does:
Stop the currently running activity, start a new activity with the entries copied from the most recent entry.

[printplaatreparatie]

A small addition: The hover-over text always displays "Start", even when the button shows the rectangular stop icon.

[kralan]

@printplaatreparatie the glitch you mentioned is annoying, but justa bit. It has been present ever since I started using TimeTracker.

The other breakages appeared with the release of version 0.0.76. I supect the merge of the features for #110. I would rather have the new feature #110 reverted than the old feature of continuing/repeating an activity broken.

[mtierltd]

@Hawiak i think it was your baae5a7 commit that broke this.

[Hawiak]

@mtierltd ah, it's been a while since I made that. I'll try to look into it
@kralan could you please elaborate further on the issue you're experiencing? I'm not quite sure what is going wrong at this moment.

[kralan]

@Hawiak the problem is as follows:

The goal is to restart or continue a previously recorded activity without having to enter all data manually. There is a play icon to the right of every timer entry o accomplish this.

1. You are in timer view and have a few activities recorded:

topDescription topProject topTag topCost topStartTime-
otherDescription otherProject otherTag otherCost otherTimeInterval
clickedDescription clickedProject clickedTag clickedCost clickedTimeInterval |> <--- click here

2. you click on the play icon to the right of the clickedXXX line

3. results
   expected result (this is what you got up to version 0.0.75:

clickedDescription clickedProject clickedTag 0.00 now-
topDescription topProject topTag topCost topStartTime-now
otherDescription otherProject otherTag otherCost otherTimeInterval
clickedDescription clickedProject clickedTag clickedCost clickedTimeInterval

observed result (this is what you get with 0.0.76):

noDescrption topProject topTag 0.00 now-
topDescription topProject topTag topCost topStartTime-now
otherDescription otherProject otherTag otherCost otherTimeInterval
clickedDescription clickedProject clickedTag clickedCost clickedTimeInterval

In addition to that, after editing the new runnig activity to desired values, after clicking stop the line gets messed up with wrong values, but haven't identified any pattern yet.

[Hawiak]

I believe I have found and fixed the issue. I've opened a PR.

[printplaatreparatie]

PR #199

[mtierltd]

fixed by 14d6072