Org file clock are added by accident.

[tru2dagame]

I found couple times, that organice would mess up the state and clock by accident.

[munen]

Yes, that's a good catch! From memory, I can see why the parser did it. Any line with an inactive timestamp in the :LOGBOOK: drawer without a second timestamp is seen as a beginning timestamp. The parsing happens here.

Out of curiosity, how did you manage to get those - State "DONE" from "TODO" [2019-11-21 Thu 21:46] statements within the :LOGBOOK: drawer? I only know them from schedules/deadlines in which they would get a separate drawer like this:

* TODO Header with schedule and clocked time
  SCHEDULED: <2019-11-22 Fri +1d>
  :PROPERTIES:
  :LAST_REPEAT: [2019-11-21 Thu 21:46]
  :END:
  
  - State "DONE"       from "TODO"       [2019-11-21 Thu 21:46]
  :LOGBOOK:
  CLOCK: [2019-11-12 Tue 14:15]--[2019-11-12 Tue 13:20] => -0:55
  CLOCK: [2019-11-12 Tue 14:15]--[2019-11-12 Tue 14:25] =>  0:10
  :END:

I'm assuming there's a configuration flag that'll make org track the toggled time and add it to the same drawer as the clocked time.

In any case, the parser will have to be hardened, here. Thank you for the detailed bug report, @tru2dagame! 🙏

[tru2dagame]

Hi @munen,
I use C-c C-t d to make it done.
The emacs configuration I use is https://github.com/purcell/emacs.d

[munen]

What function does C-c C-t d call? You can find out with C-h c C-c C-t (C-h c will always give you the description to key presses).

By default C-c C-t already calls org-todo which toggles the state, so the d wouldn't be part of the command.

[tru2dagame]

It triggers org-todo then mark it done.

[tru2dagame]

@munen
I think it's this one: https://github.com/purcell/emacs.d/blob/master/lisp/init-org.el#L44

https://www.gnu.org/software/emacs/manual/html_node/org/Closing-items.html
https://emacs.stackexchange.com/questions/47951/marking-a-todo-item-done-does-not-create-a-closing-timestamp-in-org-mode

[munen]

@tru2dagame This issue has been addressed in a PR #112 by @jamesnvc.

Could you please give it a go one more time and confirm that the issue is gone for you, too? Please make sure to empty the cache and reload organice beforehand.

[munen]

@munen I think it's this one:
https://github.com/purcell/emacs.d/blob/master/lisp/init-org.el#L44

https://www.gnu.org/software/emacs/manual/html_node/org/Closing-items.html
https://emacs.stackexchange.com/questions/47951/marking-a-todo-item-done-does-not-create-a-closing-timestamp-in-org-mode

Yes, that's the flag! Thanks for looking it up and getting back to me!🙏

It's a nice flag to have set. And with PR #112 organice should treat the data set correctly, too.

[tru2dagame]

@munen yes, I tried playing with the latest version. It works without any issue. Thanks!

[munen]

Perfect, thank you for testing! 🙏