Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TI-67] Summary with parameters shows wrong ids #72

Closed
taskwarrior opened this issue Feb 11, 2018 · 2 comments
Closed

[TI-67] Summary with parameters shows wrong ids #72

taskwarrior opened this issue Feb 11, 2018 · 2 comments
Milestone

Comments

@taskwarrior
Copy link

@taskwarrior taskwarrior commented Feb 11, 2018

Bodo Graumann on 2017-03-09T12:15:07Z says:

The id is import for modifying and deleting tracked times. So it should be consistent across all views. This is not the case, when using the summary and giving an interval or filtering by tags.
In that case all the displayed times are simply numbered starting from 1 in reverse order.

For example if timew summary :ids give:

Wk  Date       Day ID Tags       Start    End      Time    Total
W10 2017-03-09 Thu @4 Tag1        8:43:08  9:38:15 0:55:07
                   @3 Tag2       11:38:39 11:45:35 0:06:56
                   @2 Tag2, Tag3 11:46:21 12:00:17 0:13:56
                   @1 Tag2, Tag4 12:01:49 12:28:46 0:26:57 1:42:56
                                                                                                      
                                                                                               1:42:56

then, issuing timew summary Tag1 :ids will give

Wk  Date       Day ID Tags       Start     End    Time   Total
W10 2017-03-09 Thu @1 Task1 8:43:08 9:38:15 0:55:07 0:55:07
                                                              
                                                       0:55:07

By using the delete command one can see that @4 should be the correct id instead of @1. So the latter view should list it as such.

@taskwarrior taskwarrior added this to the 1.1.0 milestone Feb 11, 2018
@taskwarrior
Copy link
Author

@taskwarrior taskwarrior commented Feb 11, 2018

Migrated metadata:

Created: 2017-03-09T12:15:07Z
Modified: 2017-03-25T14:36:53Z

@taskwarrior
Copy link
Author

@taskwarrior taskwarrior commented Feb 11, 2018

Paul Beckingham on 2017-03-10T13:37:56Z says:

Thanks for the nice write-up. The IDs clearly need to be applied before the filter, but after exclusions.

@lauft lauft mentioned this issue Mar 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant