emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* [Bug] marking repeating task done destroys property/logbook drawers
@ 2014-03-04  4:23 Matt Lundin
  2014-03-08  9:27 ` Karl Voit
  2014-03-13 16:16 ` Bastien
  0 siblings, 2 replies; 5+ messages in thread
From: Matt Lundin @ 2014-03-04  4:23 UTC (permalink / raw)
  To: org-mode Mailing List

[-- Attachment #1: Type: text/plain, Size: 289 bytes --]

When I mark a repeating task done (with the settings in the minimal.el
file below), org mangles the :LOGBOOK: and :PROPERTY: drawers when
trying to insert a :LAST_REPEAT: property.

Here are the steps to reproduce consistently:

1. emacs -Q -l ~/minimal.el 

where minimal.el contains...


[-- Attachment #2: minimal.el --]
[-- Type: application/emacs-lisp, Size: 174 bytes --]

[-- Attachment #3: Type: text/plain, Size: 1586 bytes --]


2. Open an org-file containing the following entry.

--8<---------------cut here---------------start------------->8---
* TODO Repeating task
  SCHEDULED: <2014-03-02 Sun .+2w>
  :LOGBOOK:
  - State "DONE"       from "NOW"        [2014-02-23 Sun 08:39]
  CLOCK: [2014-02-23 Sun 08:26]--[2014-02-23 Sun 08:39] =>  0:13
  - State "DONE"       from "STARTED"    [2014-01-28 Tue 20:17]
  :END:
  :PROPERTIES:
  :LAST_REPEAT: [2014-02-23 Sun 08:39]
  :END:
--8<---------------cut here---------------end--------------->8---

3. Mark the item DONE. 

Voilà... note the empty property drawer, the duplicate logbook drawers,
and the misplaced :LAST_REPEAT:

--8<---------------cut here---------------start------------->8---
* TODO Repeating task
  SCHEDULED: <2014-03-17 Mon .+2w>
  :LOGBOOK:
  - State "DONE"       from "TODO"       [2014-03-03 Mon 22:11]
  :END:
  :LAST_REPEAT: [2014-03-03 Mon 22:11]  :LOGBOOK:
  - State "DONE"       from "NOW"        [2014-02-23 Sun 08:39]
  CLOCK: [2014-02-23 Sun 08:26]--[2014-02-23 Sun 08:39] =>  0:13
  - State "DONE"       from "STARTED"    [2014-01-28 Tue 20:17]
  :END:
  :PROPERTIES:

  :END:
--8<---------------cut here---------------end--------------->8---

I am using...

 - Emacs version: 24.3.1
 - Org-mode version: 8.2.5h

...on Arch Linux.

Running edebug suggested that org-entry-put (which is invoked by
org-auto-repeat-maybe) places the :LAST_REPEAT: in the wrong place.
However, I couldn't discern why, since invocations of org-entry-put
otherwise seem to work fine.

Thanks,
Matt



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Bug] marking repeating task done destroys property/logbook drawers
  2014-03-04  4:23 [Bug] marking repeating task done destroys property/logbook drawers Matt Lundin
@ 2014-03-08  9:27 ` Karl Voit
  2014-03-13 16:16 ` Bastien
  1 sibling, 0 replies; 5+ messages in thread
From: Karl Voit @ 2014-03-08  9:27 UTC (permalink / raw)
  To: emacs-orgmode

* Matt Lundin <mdl@imapmail.org> wrote:
>
> When I mark a repeating task done (with the settings in the minimal.el
> file below), org mangles the :LOGBOOK: and :PROPERTY: drawers when
> trying to insert a :LAST_REPEAT: property.

I can confirm this bug at my side.[1]

  1. https://github.com/novoid/dot-emacs
-- 
mail|git|SVN|photos|postings|SMS|phonecalls|RSS|CSV|XML to Org-mode:
       > get Memacs from https://github.com/novoid/Memacs <

https://github.com/novoid/extract_pdf_annotations_to_orgmode + more on github

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Bug] marking repeating task done destroys property/logbook drawers
  2014-03-04  4:23 [Bug] marking repeating task done destroys property/logbook drawers Matt Lundin
  2014-03-08  9:27 ` Karl Voit
@ 2014-03-13 16:16 ` Bastien
  2014-03-13 17:47   ` Matt Lundin
  1 sibling, 1 reply; 5+ messages in thread
From: Bastien @ 2014-03-13 16:16 UTC (permalink / raw)
  To: Matt Lundin; +Cc: org-mode Mailing List

Hi Matt,

Matt Lundin <mdl@imapmail.org> writes:

> When I mark a repeating task done (with the settings in the minimal.el
> file below), org mangles the :LOGBOOK: and :PROPERTY: drawers when
> trying to insert a :LAST_REPEAT: property.

I can't reproduce this.  Karl can, so there is something weird here.

> I am using...
>
>  - Emacs version: 24.3.1
>  - Org-mode version: 8.2.5h

Can you use M-x org-version RET (or C-u M-x org-version RET to insert
the output in your mail) ?

> Running edebug suggested that org-entry-put (which is invoked by
> org-auto-repeat-maybe) places the :LAST_REPEAT: in the wrong place.
> However, I couldn't discern why, since invocations of org-entry-put
> otherwise seem to work fine.

Thanks for any further investigation, it's hard to know what can be
wrong without being to reproduce this.

-- 
 Bastien

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Bug] marking repeating task done destroys property/logbook drawers
  2014-03-13 16:16 ` Bastien
@ 2014-03-13 17:47   ` Matt Lundin
  2014-03-13 18:05     ` Bastien
  0 siblings, 1 reply; 5+ messages in thread
From: Matt Lundin @ 2014-03-13 17:47 UTC (permalink / raw)
  To: Bastien; +Cc: org-mode Mailing List

Hi Bastien,

Bastien <bzg@gnu.org> writes:

> Hi Matt,
>
> Matt Lundin <mdl@imapmail.org> writes:
>
>> When I mark a repeating task done (with the settings in the minimal.el
>> file below), org mangles the :LOGBOOK: and :PROPERTY: drawers when
>> trying to insert a :LAST_REPEAT: property.
>
> I can't reproduce this.  Karl can, so there is something weird here.
>
>> I am using...
>>
>>  - Emacs version: 24.3.1
>>  - Org-mode version: 8.2.5h
>
> Can you use M-x org-version RET (or C-u M-x org-version RET to insert
> the output in your mail) ?
>
>> Running edebug suggested that org-entry-put (which is invoked by
>> org-auto-repeat-maybe) places the :LAST_REPEAT: in the wrong place.
>> However, I couldn't discern why, since invocations of org-entry-put
>> otherwise seem to work fine.
>
> Thanks for any further investigation, it's hard to know what can be
> wrong without being to reproduce this.

This was a problem with org-element's cache. It was solved with the
following commit:

a8568372ad8bba841af7f7d1fdf9b64d38426c3a

Here's the original post of the thread containing the solution:

http://permalink.gmane.org/gmane.emacs.orgmode/83015

Matt

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Bug] marking repeating task done destroys property/logbook drawers
  2014-03-13 17:47   ` Matt Lundin
@ 2014-03-13 18:05     ` Bastien
  0 siblings, 0 replies; 5+ messages in thread
From: Bastien @ 2014-03-13 18:05 UTC (permalink / raw)
  To: Matt Lundin; +Cc: org-mode Mailing List

Matt Lundin <mdl@imapmail.org> writes:

> This was a problem with org-element's cache.

Okay, thanks for following up!

-- 
 Bastien

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2014-03-13 18:06 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-04  4:23 [Bug] marking repeating task done destroys property/logbook drawers Matt Lundin
2014-03-08  9:27 ` Karl Voit
2014-03-13 16:16 ` Bastien
2014-03-13 17:47   ` Matt Lundin
2014-03-13 18:05     ` Bastien

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).