Earl Chase <[email protected]> writes:

>> Could you elaborate what exactly does not work?
>> Repeaters do work for inlinetasks on my side.
>
> Same for me. I am saying that I can't test them. I attached a patch to
> this email that contains two simple ert tests for repeaters. These
> tests were not written on the branch that I created for this commit.
> One is for inlinetasks and the other uses the same repeaters with
> regular headings. For the inlinetasks, the TODO state changes and the
> repeaters are ignored.

I think I tracker this down to another, unrelated bug.
When inlinetask is before first heading, we get
`org-get-repeat' entering (org-before-first-heading-p)
branch and ignoring the repeaters inside the inlinetasks.

>> Well. The org-todo itself does not throw an error, because the error
>> from org-auto-repeat-maybe is suppressed. But it does show up in the
>> messages on main. After your patch, it does not.
>
> I am confused here. When I run my patch, I still see: "user-error:
> Cannot repeat in 2 hour(s) because no hour has been set" in Messages
> when I use org-todo on a heading with a repeater like <2012-03-29 Thu
> +2h>. I just did a git pull --rebase on the branch that I created for
> this patch. Can you try the below version of my patch and see if the
> error still shows up.

I suspect that I tested with inlinetask before first heading :)

-- 
Ihor Radchenko // yantar92,
Org mode maintainer,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>

Reply via email to