Le sam. 25 juil. 2026 à 09:06, Ihor Radchenko <[email protected]> a écrit :
>
> 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.
>

I see. I had originally planned to include a revision of
org-get-repeat in this patch but once I got going, I thought that
would be too much. But since there is a bug, I could go ahead and make
`org-get-repeat' use the org-element API. Or I could just fix the bug.
What do you think?

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