emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
From: Carsten Dominik <dominik@science.uva.nl>
To: Michael Ekstrand <michael@elehack.net>
Cc: bug-gnu-emacs@gnu.org, emacs-orgmode@gnu.org
Subject: Re: TODO progress regression in org-mode in Emacs pretest
Date: Mon, 2 Feb 2009 22:04:37 +0100	[thread overview]
Message-ID: <CBD1DE23-6D4D-4036-ACEB-27EC34F02932@uva.nl> (raw)
In-Reply-To: <87fxiwlmgp.fsf@elehack.net>

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

      reply	other threads:[~2009-02-02 21:04 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-02 18:31 Michael Ekstrand
2009-02-02 21:04 ` Carsten Dominik [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.orgmode.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CBD1DE23-6D4D-4036-ACEB-27EC34F02932@uva.nl \
    --to=dominik@science.uva.nl \
    --cc=bug-gnu-emacs@gnu.org \
    --cc=emacs-orgmode@gnu.org \
    --cc=michael@elehack.net \
    --subject='Re: TODO progress regression in org-mode in Emacs pretest' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Code repositories for project(s) associated with this 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).