[O] Bug: numeric org-clock-into-drawer is ignored [8.3beta (release_8.3beta-846-g38ab8b @ /Users/vvv/lib/emacs/org-mode/lisp/)]

[Valery V. Vorotyntsev]

Remember to cover the basics, that is, what you expected to happen and
what in fact did happen.  You don't know how to make a good report?  See

     http://orgmode.org/manual/Feedback.html#Feedback

Your bug report will be posted to the Org-mode mailing list.
------------------------------------------------------------------------

I have set `org-clock-into-drawer' to 2.
`(org-clock-into-drawer)' should return the same value (see the
documentation of this function), yet it returns "LOGBOOK".
So a CLOCK line is created in a drawer. This contradicts to the
documentation of `org-clock-into-drawer' variable:

| When an integer, the drawer is created only when the number of
| clocking entries in an item reaches or exceeds this value.

I believe this bug to be introduced by commit e990bed.

--
Cheers,
vvv


Emacs  : GNU Emacs 24.4.90.1 (x86_64-apple-darwin13.4.0, NS
apple-appkit-1265.21)
 of 2015-02-17 on builder10-9.porkrind.org
Package: Org-mode version 8.3beta (release_8.3beta-846-g38ab8b @
/Users/vvv/lib/emacs/org-mode/lisp/)

current state:
==============
(setq
 org-tab-first-hook '(org-hide-block-toggle-maybe
     org-babel-hide-result-toggle-maybe
     org-babel-header-arg-expand)
 org-speed-command-hook '(org-speed-command-default-hook
 org-babel-speed-command-hook)
 org-occur-hook '(org-first-headline-recenter)
 org-metaup-hook '(org-babel-load-in-session-maybe)
 org-clock-into-drawer 2
 org-confirm-shell-link-function 'yes-or-no-p
 org-clock-idle-time 10
 org-todo-keyword-faces '(("TODO" :foreground "red" :weight bold)
 ("NEXT" :foreground "blue" :weight bold)
 ("DONE" :foreground "forest green" :weight bold)
 ("WAITING" :foreground "orange" :weight bold)
 ("HOLD" :foreground "magenta" :weight bold)
 ("CANCELLED" :foreground "forest green" :weight bold)
 )
 org-startup-indented t
 org-after-todo-state-change-hook '(org-clock-out-if-current
   bh/mark-next-parent-tasks-todo)
 org-src-mode-hook '(org-src-babel-configure-edit-buffer
    org-src-mode-configure-edit-buffer)
 org-agenda-before-write-hook '(org-agenda-add-entry-text)
 org-babel-pre-tangle-hook '(save-buffer)
 org-mode-hook '((lambda nil
 (org-add-hook (quote change-major-mode-hook)
  (quote org-show-block-all) (quote append) (quote local))
 )
#[nil "\300\301\302\303\304$\207"
  [org-add-hook change-major-mode-hook
   org-babel-show-result-all append local]
  5]
org-babel-result-hide-spec org-babel-hide-all-hashes
(lambda nil (define-key org-mode-map (kbd "C-c SPC") nil)))
 org-todo-state-tags-triggers '(("CANCELLED" ("CANCELLED" . t))
("WAITING" ("WAITING" . t))
("HOLD" ("HOLD" . t) ("WAITING"))
("TODO" ("WAITING") ("HOLD") ("CANCELLED"))
("NEXT" ("WAITING") ("HOLD") ("CANCELLED"))
("DONE" ("WAITING") ("HOLD") ("CANCELLED"))
(done ("WAITING") ("HOLD")))
 org-archive-hook '(org-attach-archive-delete-maybe)
 org-ctrl-c-ctrl-c-hook '(org-babel-hash-at-point
 org-babel-execute-safely-maybe)
 org-directory "~/.org"
 org-enforce-todo-dependencies t
 org-cycle-hook '(org-cycle-hide-archived-subtrees org-cycle-hide-drawers
 org-cycle-hide-inline-tasks org-cycle-show-empty-lines
 org-optimize-window-after-visibility-change)
 org-todo-keywords '((sequence "TODO(t)" "NEXT(n)" "|" "DONE(d)")
    (sequence "WAITING(w@/!)" "HOLD(h@/!)" "|"
     "CANCELLED(c@/!)")
    )
 org-modules nil
 org-confirm-elisp-link-function 'yes-or-no-p
 org-metadown-hook '(org-babel-pop-to-session-maybe)
 org-treat-S-cursor-todo-selection-as-state-change nil
 org-blocker-hook '(org-block-todo-from-children-or-siblings-or-parent)
 org-babel-load-languages '((emacs-lisp . t) (shell . t))
 org-clock-in-hook '(bh/mark-next-parent-tasks-todo)
 org-clock-out-hook '(org-clock-remove-empty-clock-drawer)
 org-clock-out-remove-zero-time-clocks t
 )