Renato Ferreira <renatof...@gmail.com> writes:

> (org-todo): Fast selection should only be shown if a state argument was not 
> used
> * list/org.el (org-todo): Respect argument when called from elisp
> when calling from elisp (such as in `org-clock-in').

Thanks for the patch!  I hit this earlier today but hadn't yet looked
into it.  Testing this out, it fixes the issue introduced by f1c030bed
(Prefix argument to `org-todo' forces stage change logging, 2019-08-14),
while keeping the intended C-u behavior change from that patch.

> @@ -10121,7 +10118,9 @@ When called through ELisp, arg is also interpreted in 
> the following way:
>                               (user-error "State `%s' not valid in this file" 
> arg))
>                              ((nth (1- (prefix-numeric-value arg))
>                                    org-todo-keywords-1))))
> +                          ((and org-todo-key-trigger 
> org-use-fast-todo-selection)
> +                           ;; Use fast selection.
> +                           (org-fast-todo-selection this))
>                            ((null member) (or head (car org-todo-keywords-1)))
>                            ((equal this final-done-word) nil) ;-> make empty
>                            ((null tail) nil) ;-> first entry

This patch didn't apply for me, and the hunk counts look off.  It should
be -10121,6, no?  Anyway, I've adjusted it so that it applies and pushed
it to master.  (I made a typo fix and a few small tweaks to your commit
message.)

-- 
Kyle

Reply via email to