unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* TODO progress regression in org-mode in Emacs pretest
@ 2009-02-02 18:31 Michael Ekstrand
  2009-02-02 21:04 ` Carsten Dominik
  0 siblings, 1 reply; 3+ messages in thread
From: Michael Ekstrand @ 2009-02-02 18:31 UTC (permalink / raw)
  To: emacs-orgmode; +Cc: bug-gnu-emacs


[-- Attachment #1.1: Type: text/plain, Size: 2399 bytes --]

I upgraded this morning from an Emacs CVS build from Jan 28 or 19 to the
pretest release, and noticed a regression or undocumented change with
respect to org-after-todo-statistics-hook.  I am using the org-mode
which is included in the pretest distribution.

I have a function based on the code in section 5.5 of the Org manual to
close projects when their subtasks are complete.  Now, if I have a
headline which does not have a subtask counter in it and toggle one of
its children to DONE, my org-after-todo-statistics-hook function gets
called with n-done and n-not-done values of 0.  The parent headline thus
has its TODO status changed.

I don't think it matters too much, as my code worked before, but my hook
function is `org-summary-todo' from the following.  The first case of
the cond is being activated.

(defun mde-org-entry-is-project-p ()
  "Query whether the current headline is a project, returning
non-`nil' if it is and `nil' otherwise."
  (member "PROJECT" (org-get-tags-at)))

;;; Taken from Org Mode manual chapter 5.5
(defun org-summary-todo (n-done n-not-done)
  "Switch entry to DONE when all subentries are done, to TODO otherwise."
  (let ((org-log-done nil)
        (org-log-states nil))
    (cond
     ((and (or (org-entry-is-todo-p)
               (mde-org-entry-is-project-p))
           (= n-not-done 0))
      (org-todo
       (if (mde-org-entry-is-project-p)
           "FINISHED"
         "DONE")))
     ;; Entry is a project that is marked done
     ((and (mde-org-entry-is-project-p)
           (not (org-entry-is-todo-p))
           (org-get-todo-state)
           (> n-not-done 0))
      (org-todo 'none))
     ;; Entry is a task
     ((and (not (mde-org-entry-is-project-p))
           (not (org-entry-is-todo-p))
           (org-get-todo-state)
           (> n-not-done 0))
      (org-todo (org-get-todo-sequence-head))))))

There is a small chance that the problem was actually introduced a bit
earlier, perhaps since my Jan 20 build, but I think I would have noticed
it after at least the Jan 28 build.

I have CC'd bug-gnu-emacs to get this in as a bug report there as well.

- Michael

-- 
mouse, n: A device for pointing at the xterm in which you want to type.
Confused by the strange files?  I cryptographically sign my messages.
For more information see <http://www.elehack.net/resources/gpg>.

[-- Attachment #1.2: Type: application/pgp-signature, Size: 196 bytes --]

[-- Attachment #2: Type: text/plain, Size: 204 bytes --]

_______________________________________________
Emacs-orgmode mailing list
Remember: use `Reply All' to send replies to the list.
Emacs-orgmode@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-orgmode

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

* Re: TODO progress regression in org-mode in Emacs pretest
  2009-02-02 18:31 TODO progress regression in org-mode in Emacs pretest Michael Ekstrand
@ 2009-02-02 21:04 ` Carsten Dominik
  2009-02-02 21:35   ` bug#2169: marked as done ([Orgmode] TODO progress regression in org-mode in Emacs pretest) Emacs bug Tracking System
  0 siblings, 1 reply; 3+ messages in thread
From: Carsten Dominik @ 2009-02-02 21:04 UTC (permalink / raw)
  To: Michael Ekstrand; +Cc: bug-gnu-emacs, emacs-orgmode

Hi Michael,

I have fixed this bug and checked the changes into Emacs CVS, so they
should be part of the next pretest.

Thank you for the report!

- Carsten

On Feb 2, 2009, at 7:31 PM, Michael Ekstrand wrote:

> I upgraded this morning from an Emacs CVS build from Jan 28 or 19 to  
> the
> pretest release, and noticed a regression or undocumented change with
> respect to org-after-todo-statistics-hook.  I am using the org-mode
> which is included in the pretest distribution.
>
> I have a function based on the code in section 5.5 of the Org manual  
> to
> close projects when their subtasks are complete.  Now, if I have a
> headline which does not have a subtask counter in it and toggle one of
> its children to DONE, my org-after-todo-statistics-hook function gets
> called with n-done and n-not-done values of 0.  The parent headline  
> thus
> has its TODO status changed.
>
> I don't think it matters too much, as my code worked before, but my  
> hook
> function is `org-summary-todo' from the following.  The first case of
> the cond is being activated.
>
> (defun mde-org-entry-is-project-p ()
>  "Query whether the current headline is a project, returning
> non-`nil' if it is and `nil' otherwise."
>  (member "PROJECT" (org-get-tags-at)))
>
> ;;; Taken from Org Mode manual chapter 5.5
> (defun org-summary-todo (n-done n-not-done)
>  "Switch entry to DONE when all subentries are done, to TODO  
> otherwise."
>  (let ((org-log-done nil)
>        (org-log-states nil))
>    (cond
>     ((and (or (org-entry-is-todo-p)
>               (mde-org-entry-is-project-p))
>           (= n-not-done 0))
>      (org-todo
>       (if (mde-org-entry-is-project-p)
>           "FINISHED"
>         "DONE")))
>     ;; Entry is a project that is marked done
>     ((and (mde-org-entry-is-project-p)
>           (not (org-entry-is-todo-p))
>           (org-get-todo-state)
>           (> n-not-done 0))
>      (org-todo 'none))
>     ;; Entry is a task
>     ((and (not (mde-org-entry-is-project-p))
>           (not (org-entry-is-todo-p))
>           (org-get-todo-state)
>           (> n-not-done 0))
>      (org-todo (org-get-todo-sequence-head))))))
>
> There is a small chance that the problem was actually introduced a bit
> earlier, perhaps since my Jan 20 build, but I think I would have  
> noticed
> it after at least the Jan 28 build.
>
> I have CC'd bug-gnu-emacs to get this in as a bug report there as  
> well.
>
> - Michael
>
> -- 
> mouse, n: A device for pointing at the xterm in which you want to  
> type.
> Confused by the strange files?  I cryptographically sign my messages.
> For more information see <http://www.elehack.net/resources/gpg>.
> _______________________________________________
> Emacs-orgmode mailing list
> Remember: use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode



_______________________________________________
Emacs-orgmode mailing list
Remember: use `Reply All' to send replies to the list.
Emacs-orgmode@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-orgmode


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

* bug#2169: marked as done ([Orgmode] TODO progress regression in  org-mode in Emacs pretest)
  2009-02-02 21:04 ` Carsten Dominik
@ 2009-02-02 21:35   ` Emacs bug Tracking System
  0 siblings, 0 replies; 3+ messages in thread
From: Emacs bug Tracking System @ 2009-02-02 21:35 UTC (permalink / raw)
  To: Carsten Dominik

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


Your message dated Mon, 2 Feb 2009 22:28:42 +0100
with message-id <12B14995-9201-4A56-8289-31B4498484D3@uva.nl>
and subject line Re: [Orgmode] TODO progress regression in org-mode in Emacs pretest
has caused the Emacs bug report #2169,
regarding [Orgmode] TODO progress regression in org-mode in Emacs pretest
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@emacsbugs.donarmstrong.com
immediately.)


-- 
2169: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=2169
Emacs Bug Tracking System
Contact owner@emacsbugs.donarmstrong.com with problems

[-- Attachment #2: Type: message/rfc822, Size: 6362 bytes --]

From: Carsten Dominik <dominik@science.uva.nl>
To: Michael Ekstrand <michael@elehack.net>
Cc: emacs-orgmode@gnu.org, bug-gnu-emacs@gnu.org
Subject: Re: [Orgmode] TODO progress regression in org-mode in Emacs pretest
Date: Mon, 2 Feb 2009 22:04:37 +0100
Message-ID: <CBD1DE23-6D4D-4036-ACEB-27EC34F02932@uva.nl>

Hi Michael,

I have fixed this bug and checked the changes into Emacs CVS, so they
should be part of the next pretest.

Thank you for the report!

- Carsten

On Feb 2, 2009, at 7:31 PM, Michael Ekstrand wrote:

> I upgraded this morning from an Emacs CVS build from Jan 28 or 19 to  
> the
> pretest release, and noticed a regression or undocumented change with
> respect to org-after-todo-statistics-hook.  I am using the org-mode
> which is included in the pretest distribution.
>
> I have a function based on the code in section 5.5 of the Org manual  
> to
> close projects when their subtasks are complete.  Now, if I have a
> headline which does not have a subtask counter in it and toggle one of
> its children to DONE, my org-after-todo-statistics-hook function gets
> called with n-done and n-not-done values of 0.  The parent headline  
> thus
> has its TODO status changed.
>
> I don't think it matters too much, as my code worked before, but my  
> hook
> function is `org-summary-todo' from the following.  The first case of
> the cond is being activated.
>
> (defun mde-org-entry-is-project-p ()
>  "Query whether the current headline is a project, returning
> non-`nil' if it is and `nil' otherwise."
>  (member "PROJECT" (org-get-tags-at)))
>
> ;;; Taken from Org Mode manual chapter 5.5
> (defun org-summary-todo (n-done n-not-done)
>  "Switch entry to DONE when all subentries are done, to TODO  
> otherwise."
>  (let ((org-log-done nil)
>        (org-log-states nil))
>    (cond
>     ((and (or (org-entry-is-todo-p)
>               (mde-org-entry-is-project-p))
>           (= n-not-done 0))
>      (org-todo
>       (if (mde-org-entry-is-project-p)
>           "FINISHED"
>         "DONE")))
>     ;; Entry is a project that is marked done
>     ((and (mde-org-entry-is-project-p)
>           (not (org-entry-is-todo-p))
>           (org-get-todo-state)
>           (> n-not-done 0))
>      (org-todo 'none))
>     ;; Entry is a task
>     ((and (not (mde-org-entry-is-project-p))
>           (not (org-entry-is-todo-p))
>           (org-get-todo-state)
>           (> n-not-done 0))
>      (org-todo (org-get-todo-sequence-head))))))
>
> There is a small chance that the problem was actually introduced a bit
> earlier, perhaps since my Jan 20 build, but I think I would have  
> noticed
> it after at least the Jan 28 build.
>
> I have CC'd bug-gnu-emacs to get this in as a bug report there as  
> well.
>
> - Michael
>
> -- 
> mouse, n: A device for pointing at the xterm in which you want to  
> type.
> Confused by the strange files?  I cryptographically sign my messages.
> For more information see <http://www.elehack.net/resources/gpg>.
> _______________________________________________
> Emacs-orgmode mailing list
> Remember: use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode





[-- Attachment #3: Type: message/rfc822, Size: 5549 bytes --]

From: Carsten Dominik <dominik@science.uva.nl>
To: 2169-done@emacsbugs.donarmstrong.com
Subject: Re: [Orgmode] TODO progress regression in org-mode in Emacs pretest
Date: Mon, 2 Feb 2009 22:28:42 +0100
Message-ID: <12B14995-9201-4A56-8289-31B4498484D3@uva.nl>

Hi Michael,

I have fixed this bug and checked the changes into Emacs CVS, so they
should be part of the next pretest.

Thank you for the report!

- Carsten

On Feb 2, 2009, at 7:31 PM, Michael Ekstrand wrote:

> I upgraded this morning from an Emacs CVS build from Jan 28 or 19 to  
> the
> pretest release, and noticed a regression or undocumented change with
> respect to org-after-todo-statistics-hook.  I am using the org-mode
> which is included in the pretest distribution.
>
> I have a function based on the code in section 5.5 of the Org manual  
> to
> close projects when their subtasks are complete.  Now, if I have a
> headline which does not have a subtask counter in it and toggle one of
> its children to DONE, my org-after-todo-statistics-hook function gets
> called with n-done and n-not-done values of 0.  The parent headline  
> thus
> has its TODO status changed.
>
> I don't think it matters too much, as my code worked before, but my  
> hook
> function is `org-summary-todo' from the following.  The first case of
> the cond is being activated.
>
> (defun mde-org-entry-is-project-p ()
> "Query whether the current headline is a project, returning
> non-`nil' if it is and `nil' otherwise."
> (member "PROJECT" (org-get-tags-at)))
>
> ;;; Taken from Org Mode manual chapter 5.5
> (defun org-summary-todo (n-done n-not-done)
> "Switch entry to DONE when all subentries are done, to TODO  
> otherwise."
> (let ((org-log-done nil)
>       (org-log-states nil))
>   (cond
>    ((and (or (org-entry-is-todo-p)
>              (mde-org-entry-is-project-p))
>          (= n-not-done 0))
>     (org-todo
>      (if (mde-org-entry-is-project-p)
>          "FINISHED"
>        "DONE")))
>    ;; Entry is a project that is marked done
>    ((and (mde-org-entry-is-project-p)
>          (not (org-entry-is-todo-p))
>          (org-get-todo-state)
>          (> n-not-done 0))
>     (org-todo 'none))
>    ;; Entry is a task
>    ((and (not (mde-org-entry-is-project-p))
>          (not (org-entry-is-todo-p))
>          (org-get-todo-state)
>          (> n-not-done 0))
>     (org-todo (org-get-todo-sequence-head))))))
>
> There is a small chance that the problem was actually introduced a bit
> earlier, perhaps since my Jan 20 build, but I think I would have  
> noticed
> it after at least the Jan 28 build.
>
> I have CC'd bug-gnu-emacs to get this in as a bug report there as  
> well.
>
> - Michael
>
> -- 
> mouse, n: A device for pointing at the xterm in which you want to  
> type.
> Confused by the strange files?  I cryptographically sign my messages.
> For more information see <http://www.elehack.net/resources/gpg>.
> _______________________________________________
> Emacs-orgmode mailing list
> Remember: use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode



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

end of thread, other threads:[~2009-02-02 21:35 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-02-02 18:31 TODO progress regression in org-mode in Emacs pretest Michael Ekstrand
2009-02-02 21:04 ` Carsten Dominik
2009-02-02 21:35   ` bug#2169: marked as done ([Orgmode] TODO progress regression in org-mode in Emacs pretest) Emacs bug Tracking System

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

	https://git.savannah.gnu.org/cgit/emacs.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).