Re: Sticky agenda branch merged

[Rainer Stengele <rainer.stengele@online.de>]

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. 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