When org-clock-report-include-clocking-task is set we always add the
current clocking task to the clock report. This is incorrect if you
are looking at an agenda clock report for a time range that does not
include the current clocking task (e.g. yesterday or last week).
Now we only include the current clocking task if the clock report
date range includes the current clocking task start time.
---
This patch is available at git://git.norang.ca for-carsten
lisp/org-clock.el | 14 +++++++++-----
1 files changed, 9 insertions(+), 5 deletions(-)
diff --git a/lisp/org-clock.el b/lisp/org-clock.el
index 504f0c9..e999690 100644
--- a/lisp/org-clock.el
+++ b/lisp/org-clock.el
@@ -1396,12 +1396,16 @@ nil are excluded from the clock summation."
(* 60 (string-to-number (match-string 4))))))
(t ;; A headline
;; Add the currently clocking item time to the total
- (when (and org-clock-report-include-clocking-task
- (equal (org-clocking-buffer) (current-buffer))
- (equal (marker-position org-clock-hd-marker) (point)))
+ (let ((org-clock-start-time-as-float (org-float-time
org-clock-start-time)))
+ (message "clock test %s %s %s" tstart org-clock-start-time-as-float
tend)
+ (when (and org-clock-report-include-clocking-task
+ (equal (org-clocking-buffer) (current-buffer))
+ (equal (marker-position org-clock-hd-marker) (point))
+ (>= org-clock-start-time-as-float tstart)
+ (<= org-clock-start-time-as-float tend))
(let ((time (floor (- (org-float-time)
- (org-float-time org-clock-start-time)) 60)))
- (setq t1 (+ t1 time))))
+ org-clock-start-time-as-float) 60)))
+ (setq t1 (+ t1 time)))))
(let* ((headline-forced
(get-text-property (point)
:org-clock-force-headline-inclusion))
--
1.7.1.86.g0e460
_______________________________________________
Emacs-orgmode mailing list
Please use `Reply All' to send replies to the list.
Emacs-orgmode@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-orgmode
