bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#2169: marked as done ([Orgmode] TODO progress regression in org-mod


From: Emacs bug Tracking System
Subject: bug#2169: marked as done ([Orgmode] TODO progress regression in org-mode in Emacs pretest)
Date: Mon, 02 Feb 2009 21:35:04 +0000

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
--- Begin Message --- Subject: Re: [Orgmode] TODO progress regression in org-mode in Emacs pretest Date: Mon, 2 Feb 2009 22:04:37 +0100
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





--- End Message ---
--- Begin Message --- Subject: Re: [Orgmode] TODO progress regression in org-mode in Emacs pretest Date: Mon, 2 Feb 2009 22:28:42 +0100
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



--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]