Re: [O] Sticky agenda branch merged

[Nick Dokos]

Rainer Stengele <address@hidden> wrote:

> Am 17.04.2012 12:09, schrieb Rainer Stengele:
> > Am 16.04.2012 11:47, schrieb Max Mikhanosha:
> >> I had just pushed a merge of max-sticky-agenda branch to master, let
> >> me know if there are any problems, also feel free to hack/iterate on
> >> it, if you have question as to why something was done, I'll do my best
> >> to answer.
> >>
> >> In particular I just seen a new commit introducing filter on
> >> categories, and org-agenda-filtered-by-top-category probably needs to
> >> be included into local variable list to have it work per agenda
> >> buffer.
> >>
> >> Also the individual calls to org-todo-list and such, do not work right
> >> with sticky agenda as there is nothing to catch the 'exit tag, that is
> >> thrown by `org-prepare-agenda' when it finds cached buffer. I'm not
> >> sure there is more elegant way of fixing it, other then wrapping all
> >> the individual scanner pieces with (catch) blocks, that re-throw if
> >> they had detected that they been called from (org-agenda) command
> >>
> >> Regards,
> >>    Max
> >>
> >>
> > Hi,
> >
> > I see a strange behaviour when clocking in with C-c C-x C-i.
> > Sometimes (!) there is no new CLOCK: entry created.
> > The TODO state changes as expected, but no new CLOCK: line is created, even 
> > when clocking out.
> > Looks like this happens when the initial TODO state is set.
> >
> > Maybe I missed a change in the behaviour?
> >
> > Regards,
> > Rainer
> >
> >
> Hi Max,
> 
> I found the culprit:
> 
> 
> (defun bh/mark-parent-tasks-started ()
>    "Visit each parent task and change TODO states to STARTED"
>    (unless bh/mark-parent-tasks-started
>      (when (equal state "INARBEIT")
>        (let ((bh/mark-parent-tasks-started t))
>          (save-excursion
>            (while (org-up-heading-safe)
>              (when (member (nth 2 (org-heading-components)) (list "TODO" 
> "INARBEIT" "WARTEN"))
>                (org-todo "INARBEIT"))))))))
> 
> (add-hook 'org-after-todo-state-change-hook 'bh/mark-parent-tasks-started 
> 'append)
> 
>  something seems to have changed conecerning the "state" variable.

Presumably another victim of the variable renaming patches. Try
renaming "state" to "org-state" and see if that works (untested,
no promises).

Nick


> The function fails and stops the clockin process.  Deactivating the
> hook brings back the right behaviour.  Sorry for that.

> 
> Bernt, that function of yours does seem to have stopped working?
> 
> Thanks and regards,
> Rainer
>