Earl Chase <[email protected]> writes: >> > + (let* ((headline (org-element-lineage >> > + (org-element-at-point) >> > + (list 'headline 'inlinetasks) t)) >> >> *"inlinetask", not "inlinetasks" >> >> I guess we need a test here as well. > > Fixed. Not sure what is going on here but `org-auto-repeat-maybe' does not > work properly when tested with inlinetasks. What I see is the todo being > toggled and the timestamp with the repeater being ignored. That is what > happened when I switched back to main and tried to > test `org-auto-repeat-maybe'. This does not happen outside of tests, pre or > post my refactor. It's of course possible that there is some issue on my > end. But this problem seems to predate my refactor.
Could you elaborate what exactly does not work? Repeaters do work for inlinetasks on my side. >> Also, >> * TODO test >> <2012-03-29 Thu +2h> >> >> no longer throws a user-error. > > This test still passes on my end. I also added two more test cases for > timestamps with hour repeaters and no start times. I even confirmed > manually that a user-error is thrown for those types of timestamps. When > are you seeing this? 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. -- 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>
