]> code.delx.au - gnu-emacs/blobdiff - lisp/org/org-clock.el
Small doc fixes.
[gnu-emacs] / lisp / org / org-clock.el
index 457a4dcb2f0e2ce304ce6f36f3581a01849ef897..09b646d40ba84e44c0fd47959bb56d996d4c49e8 100644 (file)
@@ -1,12 +1,11 @@
 ;;; org-clock.el --- The time clocking code for Org-mode
 
-;; Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010
-;;   Free Software Foundation, Inc.
+;; Copyright (C) 2004-2011  Free Software Foundation, Inc.
 
 ;; Author: Carsten Dominik <carsten at orgmode dot org>
 ;; Keywords: outlines, hypermedia, calendar, wp
 ;; Homepage: http://orgmode.org
-;; Version: 7.3
+;; Version: 7.7
 ;;
 ;; This file is part of GNU Emacs.
 ;;
 ;; This file contains the time clocking code for Org-mode
 
 (require 'org)
+(require 'org-exp)
 ;;; Code:
 
 (eval-when-compile
   (require 'cl))
 
-(declare-function calendar-absolute-from-iso    "cal-iso"    (&optional date))
+(declare-function calendar-absolute-from-iso "cal-iso" (&optional date))
 (declare-function notifications-notify "notifications" (&rest params))
 (defvar org-time-stamp-formats)
+(defvar org-ts-what)
 
 (defgroup org-clock nil
   "Options concerning clocking working time in Org-mode."
@@ -63,6 +64,22 @@ which see."
          (const :tag "Into LOGBOOK drawer" "LOGBOOK")
          (string :tag "Into Drawer named...")))
 
+(defun org-clock-into-drawer ()
+  "Return the value of `org-clock-into-drawer', but let properties overrule.
+If the current entry has or inherits a CLOCK_INTO_DRAWER
+property, it will be used instead of the default value; otherwise
+if the current entry has or inherits a LOG_INTO_DRAWER property,
+it will be used instead of the default value.
+The default is the value of the customizable variable `org-clock-into-drawer',
+which see."
+  (let ((p (org-entry-get nil "CLOCK_INTO_DRAWER" 'inherit))
+       (q (org-entry-get nil "LOG_INTO_DRAWER" 'inherit)))
+    (cond
+     ((or (not (or p q)) (equal p "nil") (equal q "nil")) org-clock-into-drawer)
+     ((or (equal p "t") (equal q "t")) "LOGBOOK")
+     ((not p) q)
+     (t p))))
+
 (defcustom org-clock-out-when-done t
   "When non-nil, clock will be stopped when the clocked entry is marked DONE.
 DONE here means any DONE-like state.
@@ -222,11 +239,59 @@ string as argument."
          (string :tag "Program")
          (function :tag "Function")))
 
-(defcustom org-clock-clocktable-default-properties '(:maxlevel 2 :scope file)
-  "Default properties for new clocktables."
+(defgroup org-clocktable nil
+  "Options concerning the clock table in Org-mode."
+  :tag "Org Clock Table"
+  :group 'org-clock)
+
+(defcustom org-clocktable-defaults
+  `(list
+    :maxlevel 2
+    :lang ,org-export-default-language
+    :scope 'file
+    :block nil
+    :tstart nil
+    :tend nil
+    :step nil
+    :stepskip0 nil
+    :fileskip0 nil
+    :tags nil
+    :emphasize nil
+    :link nil
+    :narrow '40!
+    :indent t
+    :formula nil
+    :timestamp nil
+    :level nil
+    :tcolumns nil
+    :formatter nil)
+  "Default properties for clock tables."
   :group 'org-clock
   :type 'plist)
 
+(defcustom org-clock-clocktable-formatter 'org-clocktable-write-default
+  "Function to turn clocking data into a table.
+For more information, see `org-clocktable-write-default'."
+  :group 'org-clocktable
+  :type 'function)
+
+;; FIXME: translate es and nl last string "Clock summary at"
+(defcustom org-clock-clocktable-language-setup
+  '(("en" "File"     "L"  "Timestamp"  "Headline" "Time"  "ALL"   "Total time"   "File time" "Clock summary at")
+    ("es" "Archivo"  "N"  "Fecha y hora" "Tarea" "Tiempo" "TODO" "Tiempo total" "Tiempo archivo" "Clock summary at")
+    ("fr" "Fichier"  "N"  "Horodatage" "En-tête"  "Durée" "TOUT"  "Durée totale" "Durée fichier" "Horodatage sommaire à")
+    ("nl" "Bestand"  "N"  "Tijdstip"   "Hoofding" "Duur"  "ALLES" "Totale duur"  "Bestandstijd" "Clock summary at"))
+  "Terms used in clocktable, translated to different languages."
+  :group 'org-clocktable
+  :type 'alist)
+
+(defcustom org-clock-clocktable-default-properties '(:maxlevel 2 :scope file)
+  "Default properties for new clocktables.
+These will be inserted into the BEGIN line, to make it easy for users to
+play with them."
+  :group 'org-clocktable
+  :type 'plist)
+
 (defcustom org-clock-idle-time nil
   "When non-nil, resolve open clocks if the user is idle more than X minutes."
   :group 'org-clock
@@ -263,7 +328,7 @@ to add an effort property.")
   "Hook run when stopping the current clock.")
 
 (defvar org-clock-cancel-hook nil
-  "Hook run when cancelling the current clock.")
+  "Hook run when canceling the current clock.")
 (defvar org-clock-goto-hook nil
   "Hook run when selecting the currently clocked-in entry.")
 (defvar org-clock-has-been-used nil
@@ -281,7 +346,7 @@ to add an effort property.")
 (defvar org-clock-start-time "")
 
 (defvar org-clock-leftover-time nil
-  "If non-nil, user cancelled a clock; this is when leftover time started.")
+  "If non-nil, user canceled a clock; this is when leftover time started.")
 
 (defvar org-clock-effort ""
   "Effort estimate of the currently clocking task.")
@@ -351,6 +416,9 @@ of a different task.")
   "Return t when clocking a task."
   (not (equal (org-clocking-buffer) nil)))
 
+(defvar org-clock-before-select-task-hook nil
+  "Hook called in task selection just before prompting the user.")
+
 (defun org-clock-select-task (&optional prompt)
   "Select a task that recently was associated with clocking."
   (interactive)
@@ -383,6 +451,7 @@ of a different task.")
           (if (fboundp 'int-to-char) (setf (car s) (int-to-char (car s))))
           (push s sel-list)))
        org-clock-history)
+      (run-hooks 'org-clock-before-select-task-hook)
       (org-fit-window-to-buffer)
       (message (or prompt "Select task for clocking:"))
       (setq rpl (read-char-exclusive))
@@ -405,13 +474,11 @@ pointing to it."
            (ignore-errors
              (goto-char marker)
              (setq file (buffer-file-name (marker-buffer marker))
-                   cat (or (org-get-category)
-                           (progn (org-refresh-category-properties)
-                                  (org-get-category)))
+                   cat (org-get-category)
                    heading (org-get-heading 'notags)
                    prefix (save-excursion
                             (org-back-to-heading t)
-                            (looking-at "\\*+ ")
+                            (looking-at org-outline-regexp)
                             (match-string 0))
                    task (substring
                          (org-fontify-like-in-org-mode
@@ -437,7 +504,7 @@ If not, show simply the clocked time like 01:50."
         (m (- clocked-time (* 60 h))))
     (if org-clock-effort
        (let* ((effort-in-minutes
-               (org-hh:mm-string-to-minutes org-clock-effort))
+               (org-duration-string-to-minutes org-clock-effort))
               (effort-h (floor effort-in-minutes 60))
               (effort-m (- effort-in-minutes (* effort-h 60)))
               (work-done-str
@@ -511,10 +578,10 @@ the mode line."
        ;; A string.  See if it is a delta
        (setq sign (string-to-char value))
        (if (member sign '(?- ?+))
-          (setq current (org-hh:mm-string-to-minutes current)
+          (setq current (org-duration-string-to-minutes current)
                 value (substring value 1))
         (setq current 0))
-       (setq value (org-hh:mm-string-to-minutes value))
+       (setq value (org-duration-string-to-minutes value))
        (if (equal ?- sign)
           (setq value (- current value))
         (if (equal ?+ sign) (setq value (+ current value)))))
@@ -531,7 +598,7 @@ the mode line."
   "Show notification if we spent more time than we estimated before.
 Notification is shown only once."
   (when (org-clocking-p)
-    (let ((effort-in-minutes (org-hh:mm-string-to-minutes org-clock-effort))
+    (let ((effort-in-minutes (org-duration-string-to-minutes org-clock-effort))
          (clocked-time (org-clock-get-clocked-time)))
       (if (setq org-task-overrun
                (if (or (null effort-in-minutes) (zerop effort-in-minutes))
@@ -710,7 +777,8 @@ If necessary, clock-out of the currently active clock."
 
 (defun org-clock-jump-to-current-clock (&optional effective-clock)
   (interactive)
-  (let ((clock (or effective-clock (cons org-clock-marker
+  (let ((org-clock-into-drawer (org-clock-into-drawer))
+       (clock (or effective-clock (cons org-clock-marker
                                         org-clock-start-time))))
     (unless (marker-buffer (car clock))
       (error "No clock is currently running"))
@@ -925,6 +993,16 @@ so long."
                         60.0))))
           org-clock-user-idle-start)))))
 
+(defvar org-clock-current-task nil
+  "Task currently clocked in.")
+(defun org-clock-set-current ()
+  "Set `org-clock-current-task' to the task currently clocked in."
+  (setq org-clock-current-task (nth 4 (org-heading-components))))
+
+(defun org-clock-delete-current ()
+  "Reset `org-clock-current-task' to nil."
+  (setq org-clock-current-task nil))
+
 (defun org-clock-in (&optional select start-time)
   "Start the clock on the current item.
 If necessary, clock-out of the currently active clock.
@@ -942,6 +1020,7 @@ the clocking selection, associated with the letter `d'."
          ts selected-task target-pos (msg-extra "")
          (leftover (and (not org-clock-resolving-clocks)
                          org-clock-leftover-time)))
+
       (when (and org-clock-auto-clock-resolution
                 (or (not interrupting)
                     (eq t org-clock-auto-clock-resolution))
@@ -950,11 +1029,17 @@ the clocking selection, associated with the letter `d'."
        (setq org-clock-leftover-time nil)
        (let ((org-clock-clocking-in t))
          (org-resolve-clocks)))        ; check if any clocks are dangling
+
       (when (equal select '(4))
        (setq selected-task (org-clock-select-task "Clock-in on task: "))
        (if selected-task
            (setq selected-task (copy-marker selected-task))
          (error "Abort")))
+
+      (when (equal select '(16))
+       ;; Mark as default clocking task
+       (org-clock-mark-default-task))
+
       (when interrupting
        ;; We are interrupting the clocking of a different task.
        ;; Save a marker to this task, so that we can go back.
@@ -969,7 +1054,8 @@ the clocking selection, associated with the letter `d'."
                     (= (marker-position org-clock-hd-marker)
                        (if selected-task
                            (marker-position selected-task)
-                         (point)))))
+                         (point)))
+                    (equal org-clock-current-task (nth 4 (org-heading-components)))))
          (message "Clock continues in \"%s\"" org-clock-heading)
          (throw 'abort nil))
        (move-marker org-clock-interrupted-task
@@ -978,10 +1064,6 @@ the clocking selection, associated with the letter `d'."
        (let ((org-clock-clocking-in t))
          (org-clock-out t)))
 
-      (when (equal select '(16))
-       ;; Mark as default clocking task
-       (org-clock-mark-default-task))
-
       ;; Clock in at which position?
       (setq target-pos
            (if (and (eobp) (not (org-on-heading-p)))
@@ -1009,7 +1091,7 @@ the clocking selection, associated with the letter `d'."
                                            (match-string 2))))
                     (if newstate (org-todo newstate))))
                  ((and org-clock-in-switch-to-state
-                       (not (looking-at (concat outline-regexp "[ \t]*"
+                       (not (looking-at (concat org-outline-regexp "[ \t]*"
                                                 org-clock-in-switch-to-state
                                                 "\\>"))))
                   (org-todo org-clock-in-switch-to-state)))
@@ -1102,16 +1184,6 @@ the clocking selection, associated with the letter `d'."
            (message "Clock starts at %s - %s" ts msg-extra)
            (run-hooks 'org-clock-in-hook)))))))
 
-(defvar org-clock-current-task nil
-  "Task currently clocked in.")
-(defun org-clock-set-current ()
-  "Set `org-clock-current-task' to the task currently clocked in."
-  (setq org-clock-current-task (nth 4 (org-heading-components))))
-
-(defun org-clock-delete-current ()
-  "Reset `org-clock-current-task' to nil."
-  (setq org-clock-current-task nil))
-
 (defun org-clock-mark-default-task ()
   "Mark current task as default task."
   (interactive)
@@ -1161,16 +1233,17 @@ When FIND-UNCLOSED is non-nil, first check if there is an unclosed clock
 line and position cursor in that line."
   (org-back-to-heading t)
   (catch 'exit
-    (let ((beg (save-excursion
-                (beginning-of-line 2)
-                (or (bolp) (newline))
-                (point)))
-         (end (progn (outline-next-heading) (point)))
-         (re (concat "^[ \t]*" org-clock-string))
-         (cnt 0)
-         (drawer (if (stringp org-clock-into-drawer)
-                     org-clock-into-drawer "LOGBOOK"))
-         first last ind-last)
+    (let* ((org-clock-into-drawer (org-clock-into-drawer))
+          (beg (save-excursion
+                 (beginning-of-line 2)
+                 (or (bolp) (newline))
+                 (point)))
+          (end (progn (outline-next-heading) (point)))
+          (re (concat "^[ \t]*" org-clock-string))
+          (cnt 0)
+          (drawer (if (stringp org-clock-into-drawer)
+                      org-clock-into-drawer "LOGBOOK"))
+          first last ind-last)
       (goto-char beg)
       (when (and find-unclosed
                 (re-search-forward
@@ -1188,7 +1261,7 @@ line and position cursor in that line."
            (and (re-search-forward org-property-end-re nil t)
                 (goto-char (match-beginning 0))))
        (throw 'exit t))
-      ;; Lets count the CLOCK lines
+      ;; Let's count the CLOCK lines
       (goto-char beg)
       (while (re-search-forward re end t)
        (setq first (or first (match-beginning 0))
@@ -1203,7 +1276,10 @@ line and position cursor in that line."
        (beginning-of-line 2)
        (if (and (>= (org-get-indentation) ind-last)
                 (org-at-item-p))
-           (org-end-of-item))
+           (when (and (>= (org-get-indentation) ind-last)
+                  (org-at-item-p))
+             (let ((struct (org-list-struct)))
+               (goto-char (org-list-get-bottom-point struct)))))
        (insert ":END:\n")
        (beginning-of-line 0)
        (org-indent-line-to ind-last)
@@ -1303,7 +1379,7 @@ If there is no running clock, throw an error, unless FAIL-QUIETLY is set."
                                           (match-string 2))))
                    (if newstate (org-todo newstate))))
                 ((and org-clock-out-switch-to-state
-                      (not (looking-at (concat outline-regexp "[ \t]*"
+                      (not (looking-at (concat org-outline-regexp "[ \t]*"
                                                org-clock-out-switch-to-state
                                                "\\>"))))
                  (org-todo org-clock-out-switch-to-state))))))
@@ -1313,6 +1389,76 @@ If there is no running clock, throw an error, unless FAIL-QUIETLY is set."
           (run-hooks 'org-clock-out-hook)
          (org-clock-delete-current))))))
 
+(add-hook 'org-clock-out-hook 'org-clock-remove-empty-clock-drawer)
+
+(defun org-clock-remove-empty-clock-drawer nil
+  "Remove empty clock drawer in the current subtree."
+  (let* ((olid (or (org-entry-get (point) "LOG_INTO_DRAWER")
+                  org-log-into-drawer))
+        (clock-drawer (if (eq t olid) "LOGBOOK" olid))
+        (end (save-excursion (org-end-of-subtree t t))))
+    (when clock-drawer
+      (save-excursion
+       (org-back-to-heading t)
+       (while (search-forward clock-drawer end t)
+         (goto-char (match-beginning 0))
+         (org-remove-empty-drawer-at clock-drawer (point))
+         (forward-line 1))))))
+
+(defun org-at-clock-log-p nil
+  "Is the cursor on the clock log line?"
+  (save-excursion
+    (move-beginning-of-line 1)
+    (looking-at "^[ \t]*CLOCK:")))
+
+(defun org-clock-timestamps-up nil
+  "Increase CLOCK timestamps at cursor."
+  (interactive)
+  (org-clock-timestamps-change 'up))
+
+(defun org-clock-timestamps-down nil
+  "Increase CLOCK timestamps at cursor."
+  (interactive)
+  (org-clock-timestamps-change 'down))
+
+(defun org-clock-timestamps-change (updown)
+  "Change CLOCK timestamps synchronously at cursor.
+UPDOWN tells whether to change 'up or 'down."
+  (setq org-ts-what nil)
+  (when (org-at-timestamp-p t)
+    (let ((tschange (if (eq updown 'up) 'org-timestamp-up
+                     'org-timestamp-down))
+         ts1 begts1 ts2 begts2 updatets1 tdiff)
+      (save-excursion
+       (move-beginning-of-line 1)
+       (re-search-forward org-ts-regexp3 nil t)
+       (setq ts1 (match-string 0) begts1 (match-beginning 0))
+       (when (re-search-forward org-ts-regexp3 nil t)
+         (setq ts2 (match-string 0) begts2 (match-beginning 0))))
+      ;; Are we on the second timestamp?
+      (if (<= begts2 (point)) (setq updatets1 t))
+      (if (not ts2)
+         ;; fall back on org-timestamp-up if there is only one
+         (funcall tschange)
+       ;; setq this so that (boundp 'org-ts-what is non-nil)
+       (funcall tschange)
+       (let ((ts (if updatets1 ts2 ts1))
+             (begts (if updatets1 begts1 begts2)))
+         (setq tdiff
+               (subtract-time
+                (org-time-string-to-time org-last-changed-timestamp)
+                (org-time-string-to-time ts)))
+         (save-excursion
+           (goto-char begts)
+           (org-timestamp-change
+            (round (/ (org-float-time tdiff)
+                      (cond ((eq org-ts-what 'minute) 60)
+                            ((eq org-ts-what 'hour) 3600)
+                            ((eq org-ts-what 'day) (* 24 3600))
+                            ((eq org-ts-what 'month) (* 24 3600 31))
+                            ((eq org-ts-what 'year) (* 24 3600 365.2)))))
+            org-ts-what 'updown)))))))
+
 (defun org-clock-cancel ()
   "Cancel the running clock by removing the start timestamp."
   (interactive)
@@ -1466,7 +1612,9 @@ nil are excluded from the clock summation."
 (defun org-clock-display (&optional total-only)
   "Show subtree times in the entire buffer.
 If TOTAL-ONLY is non-nil, only show the total time for the entire file
-in the echo area."
+in the echo area.
+
+Use \\[org-clock-remove-overlays] to remove the subtree times."
   (interactive)
   (org-clock-remove-overlays)
   (let (time h m p)
@@ -1586,13 +1734,16 @@ fontified, and then returned."
     (font-lock-fontify-buffer)
     (forward-line 2)
     (buffer-substring (point) (progn
-                               (re-search-forward "^#\\+END" nil t)
+                               (re-search-forward "^[ \t]*#\\+END" nil t)
                                (point-at-bol)))))
 
 (defun org-clock-report (&optional arg)
   "Create a table containing a report about clocked time.
 If the cursor is inside an existing clocktable block, then the table
-will be updated.  If not, a new clocktable will be inserted.
+will be updated.  If not, a new clocktable will be inserted.  The scope
+of the new clock will be subtree when called from within a subtree, and
+file elsewhere.
+
 When called with a prefix argument, move to the first clock table in the
 buffer and update it."
   (interactive "P")
@@ -1602,8 +1753,12 @@ buffer and update it."
     (org-show-entry))
   (if (org-in-clocktable-p)
       (goto-char (org-in-clocktable-p))
-    (org-create-dblock (append (list :name "clocktable")
-                              org-clock-clocktable-default-properties)))
+    (let ((props (if (ignore-errors
+                      (save-excursion (org-back-to-heading)))
+                    (list :name "clocktable" :scope 'subtree)
+                  (list :name "clocktable"))))
+      (org-create-dblock
+       (org-combine-plists org-clock-clocktable-default-properties props))))
   (org-update-dblock))
 
 (defun org-in-clocktable-p ()
@@ -1611,12 +1766,68 @@ buffer and update it."
   (let ((pos (point)) start)
     (save-excursion
       (end-of-line 1)
-      (and (re-search-backward "^#\\+BEGIN:[ \t]+clocktable" nil t)
+      (and (re-search-backward "^[ \t]*#\\+BEGIN:[ \t]+clocktable" nil t)
           (setq start (match-beginning 0))
-          (re-search-forward "^#\\+END:.*" nil t)
+          (re-search-forward "^[ \t]*#\\+END:.*" nil t)
           (>= (match-end 0) pos)
           start))))
 
+(defun org-day-of-week (day month year)
+  "Returns the day of the week as an integer."
+  (nth 6
+       (decode-time
+       (date-to-time
+        (format "%d-%02d-%02dT00:00:00" year month day)))))
+
+(defun org-quarter-to-date (quarter year)
+  "Get the date (week day year) of the first day of a given quarter."
+  (let (startday)
+    (cond
+     ((= quarter 1)
+      (setq startday (org-day-of-week 1 1 year))
+      (cond
+       ((= startday 0)
+       (list 52 7 (- year 1)))
+       ((= startday 6)
+       (list 52 6 (- year 1)))
+       ((<= startday 4)
+       (list 1 startday year))
+       ((> startday 4)
+       (list 53 startday (- year 1)))
+       )
+      )
+     ((= quarter 2)
+      (setq startday (org-day-of-week 1 4 year))
+      (cond
+       ((= startday 0)
+       (list 13 startday year))
+       ((< startday 4)
+       (list 14 startday year))
+       ((>= startday 4)
+       (list 13 startday year))
+       )
+      )
+     ((= quarter 3)
+      (setq startday (org-day-of-week 1 7 year))
+      (cond
+       ((= startday 0)
+       (list 26 startday year))
+       ((< startday 4)
+       (list 27 startday year))
+       ((>= startday 4)
+       (list 26 startday year))
+       )
+      )
+     ((= quarter 4)
+      (setq startday (org-day-of-week 1 10 year))
+      (cond
+       ((= startday 0)
+       (list 39 startday year))
+       ((<= startday 4)
+       (list 40 startday year))
+       ((> startday 4)
+       (list 39 startday year)))))))
+
 (defun org-clock-special-range (key &optional time as-strings)
   "Return two times bordering a special time range.
 Key is a symbol specifying the range and can be one of `today', `yesterday',
@@ -1633,7 +1844,12 @@ the returned times will be formatted strings."
         (dow (nth 6 tm))
         (skey (symbol-name key))
         (shift 0)
-        s1 m1 h1 d1 month1 y1 diff ts te fm txt w date)
+         (q (cond ((>= (nth 4 tm) 10) 4)
+                  ((>= (nth 4 tm) 7) 3)
+                  ((>= (nth 4 tm) 4) 2)
+                  ((>= (nth 4 tm) 1) 1)))
+        s1 m1 h1 d1 month1 y1 diff ts te fm txt w date
+        interval tmp shiftedy shiftedm shiftedq)
     (cond
      ((string-match "^[0-9]+$" skey)
       (setq y (string-to-number skey) m 1 d 1 key 'year))
@@ -1650,6 +1866,15 @@ the returned times will be formatted strings."
       (setq d (nth 1 date) month (car date) y (nth 2 date)
            dow 1
            key 'week))
+      ((string-match "^\\([0-9]+\\)-[qQ]\\([1-4]\\)$" skey)
+       (require 'cal-iso)
+       (setq y (string-to-number (match-string 1 skey)))
+       (setq q (string-to-number (match-string 2 skey)))
+       (setq date (calendar-gregorian-from-absolute
+                   (calendar-absolute-from-iso (org-quarter-to-date q y))))
+       (setq d (nth 1 date) month (car date) y (nth 2 date)
+            dow 1
+            key 'quarter))
      ((string-match "^\\([0-9]+\\)-\\([0-9]\\{1,2\\}\\)-\\([0-9]\\{1,2\\}\\)$" skey)
       (setq y (string-to-number (match-string 1 skey))
            month (string-to-number (match-string 2 skey))
@@ -1657,12 +1882,17 @@ the returned times will be formatted strings."
            key 'day))
      ((string-match "\\([-+][0-9]+\\)$" skey)
       (setq shift (string-to-number (match-string 1 skey))
-           key (intern (substring skey 0 (match-beginning 1))))))
+            key (intern (substring skey 0 (match-beginning 1))))
+       (if(and (memq key '(quarter thisq)) (> shift 0))
+         (error "Looking forward with quarters isn't implemented.")
+        ())))
+
     (when (= shift 0)
-      (cond ((eq key 'yesterday) (setq key 'today shift -1))
-           ((eq key 'lastweek)  (setq key 'week  shift -1))
-           ((eq key 'lastmonth) (setq key 'month shift -1))
-           ((eq key 'lastyear)  (setq key 'year  shift -1))))
+       (cond ((eq key 'yesterday) (setq key 'today   shift -1))
+            ((eq key 'lastweek)  (setq key 'week    shift -1))
+            ((eq key 'lastmonth) (setq key 'month   shift -1))
+            ((eq key 'lastyear)  (setq key 'year    shift -1))
+            ((eq key 'lastq)     (setq key 'quarter shift -1))))
     (cond
      ((memq key '(day today))
       (setq d (+ d shift) h 0 m 0 h1 24 m1 0))
@@ -1671,6 +1901,28 @@ the returned times will be formatted strings."
            m 0 h 0 d (- d diff) d1 (+ 7 d)))
      ((memq key '(month thismonth))
       (setq d 1 h 0 m 0 d1 1 month (+ month shift) month1 (1+ month) h1 0 m1 0))
+     ((memq key '(quarter thisq))
+      ; compute if this shift remains in this year
+      ; if not, compute how many years and quarters we have to shift (via floor*)
+      ; and compute the shifted years, months and quarters
+      (cond
+       ((< (+ (- q 1) shift) 0) ; shift not in this year
+       (setq interval (* -1 (+ (- q 1) shift)))
+       ; set tmp to ((years to shift) (quarters to shift))
+       (setq tmp (org-floor* interval 4))
+       ; due to the use of floor, 0 quarters actually means 4
+       (if (= 0 (nth 1 tmp))
+           (setq shiftedy (- y (nth 0 tmp))
+                 shiftedm 1
+                 shiftedq 1)
+         (setq shiftedy (- y (+ 1 (nth 0 tmp)))
+               shiftedm (- 13 (* 3 (nth 1 tmp)))
+               shiftedq (- 5 (nth 1 tmp))))
+       (setq d 1 h 0 m 0 d1 1 month shiftedm month1 (+ 3 shiftedm) h1 0 m1 0 y shiftedy))
+       ((> (+ q shift) 0) ; shift is within this year
+       (setq shiftedq (+ q shift))
+       (setq shiftedy y)
+       (setq d 1 h 0 m 0 d1 1 month (+ 1 (* 3 (- (+ q shift) 1))) month1 (+ 4 (* 3 (- (+ q shift) 1))) h1 0 m1 0))))
      ((memq key '(year thisyear))
       (setq m 0 h 0 d 1 month 1 y (+ y shift) y1 (1+ y)))
      (t (error "No such time block %s" key)))
@@ -1686,11 +1938,21 @@ the returned times will be formatted strings."
      ((memq key '(month thismonth))
       (setq txt (format-time-string "%B %Y" ts)))
      ((memq key '(year thisyear))
-      (setq txt (format-time-string "the year %Y" ts))))
+      (setq txt (format-time-string "the year %Y" ts)))
+     ((memq key '(quarter thisq))
+      (setq txt (concatenate 'string (org-count-quarter shiftedq) " quarter of " (number-to-string shiftedy))))
+     )
     (if as-strings
        (list (format-time-string fm ts) (format-time-string fm te) txt)
       (list ts te txt))))
 
+(defun org-count-quarter (n)
+  (cond
+   ((= n 1) "1st")
+   ((= n 2) "2nd")
+   ((= n 3) "3rd")
+   ((= n 4) "4th")))
+
 (defun org-clocktable-shift (dir n)
   "Try to shift the :block date of the clocktable at point.
 Point must be in the #+BEGIN: line of a clocktable, or this function
@@ -1704,7 +1966,7 @@ the currently selected interval size."
   (and (memq dir '(left down)) (setq n (- n)))
   (save-excursion
     (goto-char (point-at-bol))
-    (if (not (looking-at "#\\+BEGIN: clocktable\\>.*?:block[ \t]+\\(\\S-+\\)"))
+    (if (not (looking-at "^[ \t]*#\\+BEGIN:[ \t]+clocktable\\>.*?:block[ \t]+\\(\\S-+\\)"))
        (error "Line needs a :block definition before this command works")
       (let* ((b (match-beginning 1)) (e (match-end 1))
             (s (match-string 1))
@@ -1713,90 +1975,95 @@ the currently selected interval size."
         ((equal s "yesterday") (setq s "today-1"))
         ((equal s "lastweek") (setq s "thisweek-1"))
         ((equal s "lastmonth") (setq s "thismonth-1"))
-        ((equal s "lastyear") (setq s "thisyear-1")))
-       (cond
-        ((string-match "^\\(today\\|thisweek\\|thismonth\\|thisyear\\)\\([-+][0-9]+\\)?$" s)
-         (setq block (match-string 1 s)
-               shift (if (match-end 2)
-                         (string-to-number (match-string 2 s))
-                       0))
-         (setq shift (+ shift n))
-         (setq ins (if (= shift 0) block (format "%s%+d" block shift))))
-        ((string-match "\\([0-9]+\\)\\(-\\([wW]?\\)\\([0-9]\\{1,2\\}\\)\\(-\\([0-9]\\{1,2\\}\\)\\)?\\)?" s)
-         ;;               1        1  2   3       3  4                4  5   6                6  5   2
-         (setq y (string-to-number (match-string 1 s))
-               wp (and (match-end 3) (match-string 3 s))
-               mw (and (match-end 4) (string-to-number (match-string 4 s)))
-               d (and (match-end 6) (string-to-number (match-string 6 s))))
-         (cond
-          (d (setq ins (format-time-string
-                        "%Y-%m-%d"
-                        (encode-time 0 0 0 (+ d n) m y))))
-          ((and wp mw (> (length wp) 0))
-           (require 'cal-iso)
-           (setq date (calendar-gregorian-from-absolute (calendar-absolute-from-iso (list (+ mw n) 1 y))))
-           (setq ins (format-time-string
-                      "%G-W%V"
-                      (encode-time 0 0 0 (nth 1 date) (car date) (nth 2 date)))))
-          (mw
-           (setq ins (format-time-string
-                      "%Y-%m"
-                      (encode-time 0 0 0 1 (+ mw n) y))))
-          (y
-           (setq ins (number-to-string (+ y n))))))
-        (t (error "Cannot shift clocktable block")))
-       (when ins
-         (goto-char b)
-         (insert ins)
-         (delete-region (point) (+ (point) (- e b)))
-         (beginning-of-line 1)
-         (org-update-dblock)
-         t)))))
+        ((equal s "lastyear") (setq s "thisyear-1"))
+        ((equal s "lastq") (setq s "thisq-1")))
+
+       (cond
+        ((string-match "^\\(today\\|thisweek\\|thismonth\\|thisyear\\|thisq\\)\\([-+][0-9]+\\)?$" s)
+         (setq block (match-string 1 s)
+               shift (if (match-end 2)
+                         (string-to-number (match-string 2 s))
+                       0))
+         (setq shift (+ shift n))
+         (setq ins (if (= shift 0) block (format "%s%+d" block shift))))
+       ((string-match "\\([0-9]+\\)\\(-\\([wWqQ]?\\)\\([0-9]\\{1,2\\}\\)\\(-\\([0-9]\\{1,2\\}\\)\\)?\\)?" s)
+        ;;               1        1  2   3       3  4                  4  5   6                6  5   2
+         (setq y (string-to-number (match-string 1 s))
+               wp (and (match-end 3) (match-string 3 s))
+               mw (and (match-end 4) (string-to-number (match-string 4 s)))
+              d (and (match-end 6) (string-to-number (match-string 6 s))))
+        (cond
+         (d (setq ins (format-time-string
+                        "%Y-%m-%d"
+                        (encode-time 0 0 0 (+ d n) m y))))
+          ((and wp (string-match "w\\|W" wp) mw (> (length wp) 0))
+           (require 'cal-iso)
+           (setq date (calendar-gregorian-from-absolute (calendar-absolute-from-iso (list (+ mw n) 1 y))))
+           (setq ins (format-time-string
+                      "%G-W%V"
+                      (encode-time 0 0 0 (nth 1 date) (car date) (nth 2 date)))))
+         ((and wp (string-match "q\\|Q" wp) mw (> (length wp) 0))
+           (require 'cal-iso)
+          ; if the 4th + 1 quarter is requested we flip to the 1st quarter of the next year
+           (if (> (+ mw n) 4)
+               (setq mw 0
+                     y (+ 1 y))
+            ())
+          ; if the 1st - 1 quarter is requested we flip to the 4th quarter of the previous year
+           (if (= (+ mw n) 0)
+               (setq mw 5
+                     y (- y 1))
+             ())
+           (setq date (calendar-gregorian-from-absolute (calendar-absolute-from-iso (org-quarter-to-date (+ mw n) y))))
+           (setq ins (format-time-string
+                      (concatenate 'string (number-to-string y) "-Q" (number-to-string (+ mw n)))
+                      (encode-time 0 0 0 (nth 1 date) (car date) (nth 2 date)))))
+          (mw
+           (setq ins (format-time-string
+                      "%Y-%m"
+                     (encode-time 0 0 0 1 (+ mw n) y))))
+         (y
+          (setq ins (number-to-string (+ y n))))))
+       (t (error "Cannot shift clocktable block")))
+       (when ins
+        (goto-char b)
+        (insert ins)
+        (delete-region (point) (+ (point) (- e b)))
+        (beginning-of-line 1)
+        (org-update-dblock)
+        t)))))
 
 (defun org-dblock-write:clocktable (params)
   "Write the standard clocktable."
+  (setq params (org-combine-plists org-clocktable-defaults params))
   (catch 'exit
-    (let* ((hlchars '((1 . "*") (2 . "/")))
-          (ins (make-marker))
-          (total-time nil)
-          (scope (plist-get params :scope))
-          (tostring (plist-get  params :tostring))
-          (multifile (plist-get  params :multifile))
-          (header (plist-get  params :header))
-          (maxlevel (or (plist-get params :maxlevel) 3))
-          (step (plist-get params :step))
-          (emph (plist-get params :emphasize))
-          (timestamp (plist-get params :timestamp))
+    (let* ((scope (plist-get params :scope))
+          (block (plist-get params :block))
           (ts (plist-get params :tstart))
           (te (plist-get params :tend))
-          (block (plist-get params :block))
           (link (plist-get params :link))
-          (tags (plist-get params :tags))
-          (matcher (if tags (cdr (org-make-tags-matcher tags))))
-          ipos time p level hlc hdl tsp props content recalc formula pcol
-          cc beg end pos tbl tbl1 range-text rm-file-column scope-is-list st)
-      (setq org-clock-file-total-minutes nil)
+          (maxlevel (or (plist-get params :maxlevel) 3))
+          (step (plist-get params :step))
+          (timestamp (plist-get params :timestamp))
+          (formatter (or (plist-get params :formatter)
+                         org-clock-clocktable-formatter
+                         'org-clocktable-write-default))
+          cc range-text ipos pos one-file-with-archives
+          scope-is-list tbls level)
+
+      ;; Check if we need to do steps
+      (when block
+       ;; Get the range text for the header
+       (setq cc (org-clock-special-range block nil t)
+             ts (car cc) te (nth 1 cc) range-text (nth 2 cc)))
       (when step
+       ;; Write many tables, in steps
        (unless (or block (and ts te))
          (error "Clocktable `:step' can only be used with `:block' or `:tstart,:end'"))
        (org-clocktable-steps params)
        (throw 'exit nil))
-      (when block
-       (setq cc (org-clock-special-range block nil t)
-             ts (car cc) te (nth 1 cc) range-text (nth 2 cc)))
-      (when (integerp ts) (setq ts (calendar-gregorian-from-absolute ts)))
-      (when (integerp te) (setq te (calendar-gregorian-from-absolute te)))
-      (when (and ts (listp ts))
-       (setq ts (format "%4d-%02d-%02d" (nth 2 ts) (car ts) (nth 1 ts))))
-      (when (and te (listp te))
-       (setq te (format "%4d-%02d-%02d" (nth 2 te) (car te) (nth 1 te))))
-      ;; Now the times are strings we can parse.
-      (if ts (setq ts (org-float-time
-                      (apply 'encode-time (org-parse-time-string ts)))))
-      (if te (setq te (org-float-time
-                      (apply 'encode-time (org-parse-time-string te)))))
-      (move-marker ins (point))
-      (setq ipos (point))
+
+      (setq ipos (point)) ; remember the insertion position
 
       ;; Get the right scope
       (setq pos (point))
@@ -1810,166 +2077,287 @@ the currently selected interval size."
        (setq scope (org-add-archive-files scope)))
        ((eq scope 'file-with-archives)
        (setq scope (org-add-archive-files (list (buffer-file-name)))
-             rm-file-column t)))
+             one-file-with-archives t)))
       (setq scope-is-list (and scope (listp scope)))
-      (save-restriction
-       (cond
-        ((not scope))
-        ((eq scope 'file) (widen))
-        ((eq scope 'subtree) (org-narrow-to-subtree))
-        ((eq scope 'tree)
-         (while (org-up-heading-safe))
-         (org-narrow-to-subtree))
-        ((and (symbolp scope) (string-match "^tree\\([0-9]+\\)$"
-                                            (symbol-name scope)))
-         (setq level (string-to-number (match-string 1 (symbol-name scope))))
-         (catch 'exit
-           (while (org-up-heading-safe)
-             (looking-at outline-regexp)
-             (if (<= (org-reduced-level (funcall outline-level)) level)
-                 (throw 'exit nil))))
-         (org-narrow-to-subtree))
-        (scope-is-list
+      (if scope-is-list
+         ;; we collect from several files
          (let* ((files scope)
-                (scope 'agenda)
-                (p1 (copy-sequence params))
                 file)
-           (setq p1 (plist-put p1 :tostring t))
-           (setq p1 (plist-put p1 :multifile t))
-           (setq p1 (plist-put p1 :scope 'file))
            (org-prepare-agenda-buffers files)
            (while (setq file (pop files))
              (with-current-buffer (find-buffer-visiting file)
-               (setq org-clock-file-total-minutes 0)
-               (setq tbl1 (org-dblock-write:clocktable p1))
-               (when tbl1
-                 (push (org-clocktable-add-file
-                        file
-                        (concat "| |*File time*|*"
-                                (org-minutes-to-hh:mm-string
-                                 org-clock-file-total-minutes)
-                                "*|\n"
-                                tbl1)) tbl)
-                 (setq total-time (+ (or total-time 0)
-                                     org-clock-file-total-minutes))))))))
-       (goto-char pos)
-
-       (unless scope-is-list
-         (org-clock-sum ts te
-                        (unless (null matcher)
-                          (lambda ()
-                            (let ((tags-list
-                                   (org-split-string
-                                    (or (org-entry-get (point) "ALLTAGS") "")
-                                    ":")))
-                              (eval matcher)))))
-         (goto-char (point-min))
-         (setq st t)
-         (while (or (and (bobp) (prog1 st (setq st nil))
-                         (get-text-property (point) :org-clock-minutes)
-                         (setq p (point-min)))
-                    (setq p (next-single-property-change (point) :org-clock-minutes)))
-           (goto-char p)
-           (when (setq time (get-text-property p :org-clock-minutes))
-             (save-excursion
-               (beginning-of-line 1)
-               (when (and (looking-at (org-re "\\(\\*+\\)[ \t]+\\(.*?\\)\\([ \t]+:[[:alnum:]_@#%:]+:\\)?[ \t]*$"))
-                          (setq level (org-reduced-level
-                                       (- (match-end 1) (match-beginning 1))))
-                          (<= level maxlevel))
-                 (setq hlc (if emph (or (cdr (assoc level hlchars)) "") "")
-                       hdl (if (not link)
-                               (match-string 2)
-                             (org-make-link-string
-                              (format "file:%s::%s"
-                                      (buffer-file-name)
-                                      (save-match-data
-                                        (org-make-org-heading-search-string
-                                         (match-string 2))))
-                              (match-string 2)))
-                       tsp (when timestamp
-                             (setq props (org-entry-properties (point)))
-                             (or (cdr (assoc "SCHEDULED" props))
-                                 (cdr (assoc "TIMESTAMP" props))
-                                 (cdr (assoc "DEADLINE" props))
-                                 (cdr (assoc "TIMESTAMP_IA" props)))))
-                 (if (and (not multifile) (= level 1)) (push "|-" tbl))
-                 (push (concat
-                        "| " (int-to-string level) "|"
-                        (if timestamp (concat tsp "|") "")
-                        hlc hdl hlc " |"
-                        (make-string (1- level) ?|)
-                        hlc (org-minutes-to-hh:mm-string time) hlc
-                        " |") tbl))))))
-       (setq tbl (nreverse tbl))
-       (if tostring
-           (if tbl (mapconcat 'identity tbl "\n") nil)
-         (goto-char ins)
-         (insert-before-markers
-          (or header
-              (concat
-               "Clock summary at ["
-               (substring
-                (format-time-string (cdr org-time-stamp-formats))
-                1 -1)
-               "]"
-               (if block (concat ", for " range-text ".") "")
-               "\n\n"))
-          (if scope-is-list "|File" "")
-          "|L|" (if timestamp "Timestamp|" "") "Headline|Time|\n")
-         (setq total-time (or total-time org-clock-file-total-minutes))
-         (insert-before-markers
-          "|-\n|"
-          (if scope-is-list "|" "")
-          (if timestamp "|Timestamp|" "|")
-          "*Total time*| *"
-          (org-minutes-to-hh:mm-string (or total-time 0))
-          "*|\n|-\n")
-         (setq tbl (delq nil tbl))
-         (if (and (stringp (car tbl)) (> (length (car tbl)) 1)
-                  (equal (substring (car tbl) 0 2) "|-"))
-             (pop tbl))
-         (insert-before-markers (mapconcat
-                                 'identity (delq nil tbl)
-                                 (if scope-is-list "\n|-\n" "\n")))
-         (backward-delete-char 1)
-         (if (setq formula (plist-get params :formula))
-             (cond
-              ((eq formula '%)
-               (setq pcol (+ (if scope-is-list 1 0) maxlevel 3))
-               (insert
-                (format
-                 "\n#+TBLFM: $%d='(org-clock-time%% @%d$%d $%d..$%d);%%.1f"
-                 pcol
-                 2
-                 (+ 3 (if scope-is-list 1 0))
-                 (+ (if scope-is-list 1 0) 3)
-                 (1- pcol)))
-               (setq recalc t))
-              ((stringp formula)
-               (insert "\n#+TBLFM: " formula)
-               (setq recalc t))
-              (t (error "invalid formula in clocktable")))
-           ;; Should we rescue an old formula?
-           (when (stringp (setq content (plist-get params :content)))
-             (when (string-match "^\\([ \t]*#\\+TBLFM:.*\\)" content)
-               (setq recalc t)
-               (insert "\n" (match-string 1 (plist-get params :content)))
-               (beginning-of-line 0))))
-         (goto-char ipos)
-         (skip-chars-forward "^|")
-         (org-table-align)
-         (when recalc
-           (if (eq formula '%)
-               (save-excursion (org-table-goto-column pcol nil 'force)
-                               (insert "%")))
-           (org-table-recalculate 'all))
-         (when rm-file-column
-           (forward-char 1)
-           (org-table-delete-column))
-         total-time)))))
+               (save-excursion
+                 (save-restriction
+                   (push (org-clock-get-table-data file params) tbls))))))
+       ;; Just from the current file
+       (save-restriction
+         ;; get the right range into the restriction
+         (org-prepare-agenda-buffers (list (buffer-file-name)))
+         (cond
+          ((not scope))  ; use the restriction as it is now
+          ((eq scope 'file) (widen))
+          ((eq scope 'subtree) (org-narrow-to-subtree))
+          ((eq scope 'tree)
+           (while (org-up-heading-safe))
+           (org-narrow-to-subtree))
+          ((and (symbolp scope) (string-match "^tree\\([0-9]+\\)$"
+                                              (symbol-name scope)))
+           (setq level (string-to-number (match-string 1 (symbol-name scope))))
+           (catch 'exit
+             (while (org-up-heading-safe)
+               (looking-at org-outline-regexp)
+               (if (<= (org-reduced-level (funcall outline-level)) level)
+                   (throw 'exit nil))))
+           (org-narrow-to-subtree)))
+         ;; do the table, with no file name.
+         (push (org-clock-get-table-data nil params) tbls)))
+
+      ;; OK, at this point we tbls as a list of tables, one per file
+      (setq tbls (nreverse tbls))
+
+      (setq params (plist-put params :multifile scope-is-list))
+      (setq params (plist-put params :one-file-with-archives
+                             one-file-with-archives))
+
+      (funcall formatter ipos tbls params))))
+
+(defun org-clocktable-write-default (ipos tables params)
+  "Write out a clock table at position IPOS in the current buffer.
+TABLES is a list of tables with clocking data as produced by
+`org-clock-get-table-data'.  PARAMS is the parameter property list obtained
+from the dynamic block definition."
+  ;; This function looks quite complicated, mainly because there are a
+  ;; lot of options which can add or remove columns.  I have massively
+  ;; commented this function, the I hope it is understandable.  If
+  ;; someone wants to write their own special formatter, this maybe
+  ;; much easier because there can be a fixed format with a
+  ;; well-defined number of columns...
+  (let* ((hlchars '((1 . "*") (2 . "/")))
+        (lwords (assoc (or (plist-get params :lang)
+                           org-export-default-language)
+                       org-clock-clocktable-language-setup))
+        (multifile (plist-get params :multifile))
+        (block (plist-get params :block))
+        (ts (plist-get params :tstart))
+        (te (plist-get params :tend))
+        (header (plist-get  params :header))
+        (narrow (plist-get params :narrow))
+        (link (plist-get params :link))
+        (maxlevel (or (plist-get params :maxlevel) 3))
+        (emph (plist-get params :emphasize))
+        (level-p (plist-get params :level))
+        (timestamp (plist-get params :timestamp))
+        (properties (plist-get params :properties))
+        (ntcol (max 1 (or (plist-get params :tcolumns) 100)))
+        (rm-file-column (plist-get params :one-file-with-archives))
+        (indent (plist-get params :indent))
+        range-text total-time tbl level hlc formula pcol
+        file-time entries entry headline
+        recalc content narrow-cut-p tcol)
+
+    ;; Implement abbreviations
+    (when (plist-get params :compact)
+      (setq level nil indent t narrow (or narrow '40!) ntcol 1))
+
+    ;; Some consistency test for parameters
+      (unless (integerp ntcol)
+       (setq params (plist-put params :tcolumns (setq ntcol 100))))
+
+      (when (and narrow (integerp narrow) link)
+       ;; We cannot have both integer narrow and link
+       (message
+        "Using hard narrowing in clocktable to allow for links")
+       (setq narrow (intern (format "%d!" narrow))))
+
+      (when narrow
+       (cond
+        ((integerp narrow))
+        ((and (symbolp narrow)
+              (string-match "\\`[0-9]+!\\'" (symbol-name narrow)))
+         (setq narrow-cut-p t
+               narrow (string-to-number (substring (symbol-name narrow)
+                                                   0 -1))))
+        (t
+         (error "Invalid value %s of :narrow property in clock table"
+                narrow))))
+
+      (when block
+       ;; Get the range text for the header
+       (setq range-text (nth 2 (org-clock-special-range block nil t))))
+
+      ;; Compute the total time
+      (setq total-time (apply '+ (mapcar 'cadr tables)))
+
+      ;; Now we need to output this tsuff
+      (goto-char ipos)
+
+      ;; Insert the text *before* the actual table
+      (insert-before-markers
+       (or header
+          ;; Format the standard header
+          (concat
+           (nth 9 lwords) " ["
+           (substring
+            (format-time-string (cdr org-time-stamp-formats))
+            1 -1)
+           "]"
+           (if block (concat ", for " range-text ".") "")
+           "\n\n")))
+
+      ;; Insert the narrowing line
+      (when (and narrow (integerp narrow) (not narrow-cut-p))
+       (insert-before-markers
+        "|"                            ; table line starter
+        (if multifile "|" "")          ; file column, maybe
+        (if level-p   "|" "")          ; level column, maybe
+        (if timestamp "|" "")          ; timestamp column, maybe
+        (if properties (make-string (length properties) ?|) "")  ;properties columns, maybe
+        (format "<%d>| |\n" narrow)))  ; headline and time columns
+
+      ;; Insert the table header line
+      (insert-before-markers
+       "|"                              ; table line starter
+       (if multifile (concat (nth 1 lwords) "|") "")  ; file column, maybe
+       (if level-p   (concat (nth 2 lwords) "|") "")  ; level column, maybe
+       (if timestamp (concat (nth 3 lwords) "|") "")  ; timestamp column, maybe
+       (if properties (concat (mapconcat 'identity properties "|") "|") "") ;properties columns, maybe
+       (concat (nth 4 lwords) "|"
+              (nth 5 lwords) "|\n"))                 ; headline and time columns
+
+      ;; Insert the total time in the table
+      (insert-before-markers
+       "|-\n"                            ; a hline
+       "|"                               ; table line starter
+       (if multifile (concat "| " (nth 6 lwords) " ") "")
+                                        ; file column, maybe
+       (if level-p   "|"      "")        ; level column, maybe
+       (if timestamp "|"      "")        ; timestamp column, maybe
+       (if properties (make-string (length properties) ?|) "")  ;properties columns, maybe
+       (concat "*" (nth 7 lwords) "*| ") ; instead of a headline
+       "*"
+       (org-minutes-to-hh:mm-string (or total-time 0)) ; the time
+       "*|\n")                          ; close line
+
+      ;; Now iterate over the tables and insert the data
+      ;; but only if any time has been collected
+      (when (and total-time (> total-time 0))
+
+       (while (setq tbl (pop tables))
+         ;; now tbl is the table resulting from one file.
+         (setq file-time (nth 1 tbl))
+         (when (or (and file-time (> file-time 0))
+                   (not (plist-get params :fileskip0)))
+           (insert-before-markers "|-\n")  ; a hline because a new file starts
+           ;; First the file time, if we have multiple files
+           (when multifile
+             ;; Summarize the time collected from this file
+             (insert-before-markers
+              (format (concat "| %s %s | %s%s*" (nth 8 lwords) "* | *%s*|\n")
+                      (file-name-nondirectory (car tbl))
+                      (if level-p   "| " "") ; level column, maybe
+                      (if timestamp "| " "") ; timestamp column, maybe
+                      (if properties (make-string (length properties) ?|) "")  ;properties columns, maybe
+                      (org-minutes-to-hh:mm-string (nth 1 tbl))))) ; the time
+
+           ;; Get the list of node entries and iterate over it
+           (setq entries (nth 2 tbl))
+           (while (setq entry (pop entries))
+             (setq level (car entry)
+                   headline (nth 1 entry)
+                   hlc (if emph (or (cdr (assoc level hlchars)) "") ""))
+             (when narrow-cut-p
+               (if (and (string-match (concat "\\`" org-bracket-link-regexp
+                                              "\\'")
+                                      headline)
+                        (match-end 3))
+                   (setq headline
+                         (format "[[%s][%s]]"
+                                 (match-string 1 headline)
+                                 (org-shorten-string (match-string 3 headline)
+                                                     narrow)))
+                 (setq headline (org-shorten-string headline narrow))))
+             (insert-before-markers
+              "|"                      ; start the table line
+              (if multifile "|" "")    ; free space for file name column?
+              (if level-p (format "%d|" (car entry)) "")   ; level, maybe
+              (if timestamp (concat (nth 2 entry) "|") "") ; timestamp, maybe
+              (if properties
+                  (concat
+                   (mapconcat
+                    (lambda (p) (or (cdr (assoc p (nth 4 entry))) ""))
+                    properties "|") "|") "")  ;properties columns, maybe
+              (if indent (org-clocktable-indent-string level) "") ; indentation
+              hlc headline hlc "|"                                ; headline
+              (make-string (min (1- ntcol) (or (- level 1))) ?|)
+                                       ; empty fields for higher levels
+              hlc (org-minutes-to-hh:mm-string (nth 3 entry)) hlc ; time
+              "|\n"                                               ; close line
+              )))))
+      (backward-delete-char 1)
+      (if (setq formula (plist-get params :formula))
+         (cond
+          ((eq formula '%)
+           ;; compute the column where the % numbers need to go
+           (setq pcol (+ 2
+                         (if multifile 1 0)
+                         (if level-p 1 0)
+                         (if timestamp 1 0)
+                         (min maxlevel (or ntcol 100))))
+           ;; compute the column where the total time is
+           (setq tcol (+ 2
+                         (if multifile 1 0)
+                         (if level-p 1 0)
+                         (if timestamp 1 0)))
+           (insert
+            (format
+             "\n#+TBLFM: $%d='(org-clock-time%% @%d$%d $%d..$%d);%%.1f"
+             pcol            ; the column where the % numbers should go
+             (if (and narrow (not narrow-cut-p)) 3 2) ; row of the total time
+             tcol            ; column of the total time
+             tcol (1- pcol)  ; range of columns where times can be found
+             ))
+           (setq recalc t))
+          ((stringp formula)
+           (insert "\n#+TBLFM: " formula)
+           (setq recalc t))
+          (t (error "invalid formula in clocktable")))
+       ;; Should we rescue an old formula?
+       (when (stringp (setq content (plist-get params :content)))
+         (when (string-match "^\\([ \t]*#\\+TBLFM:.*\\)" content)
+           (setq recalc t)
+           (insert "\n" (match-string 1 (plist-get params :content)))
+           (beginning-of-line 0))))
+      ;; Back to beginning, align the table, recalculate if necessary
+      (goto-char ipos)
+      (skip-chars-forward "^|")
+      (org-table-align)
+      (when org-hide-emphasis-markers
+       ;; we need to align a second time
+       (org-table-align))
+      (when recalc
+       (if (eq formula '%)
+           (save-excursion
+             (if (and narrow (not narrow-cut-p)) (beginning-of-line 2))
+             (org-table-goto-column pcol nil 'force)
+             (insert "%")))
+       (org-table-recalculate 'all))
+      (when rm-file-column
+       ;; The file column is actually not wanted
+       (forward-char 1)
+       (org-table-delete-column))
+      total-time))
+
+(defun org-clocktable-indent-string (level)
+  (if (= level 1)
+      ""
+    (let ((str "\\__"))
+      (while (> level 2)
+       (setq level (1- level)
+             str (concat str "___")))
+      (concat str " "))))
 
 (defun org-clocktable-steps (params)
+  "Step through the range to make a number of clock tables."
   (let* ((p1 (copy-sequence params))
         (ts (plist-get p1 :tstart))
         (te (plist-get p1 :tend))
@@ -2008,29 +2396,116 @@ the currently selected interval size."
       (setq p1 (plist-put p1 :tend (format-time-string
                                    (org-time-stamp-format nil t)
                                    (seconds-to-time (setq ts (+ ts step))))))
-      (insert "\n" (if (eq step0 'day) "Daily report: " "Weekly report starting on: ")
+      (insert "\n" (if (eq step0 'day) "Daily report: "
+                    "Weekly report starting on: ")
              (plist-get p1 :tstart) "\n")
       (setq step-time (org-dblock-write:clocktable p1))
-      (re-search-forward "#\\+END:")
+      (re-search-forward "^[ \t]*#\\+END:")
       (when (and (equal step-time 0) stepskip0)
        ;; Remove the empty table
        (delete-region (point-at-bol)
                       (save-excursion
-                        (re-search-backward "^\\(Daily\\|Weekly\\) report" nil t)
+                        (re-search-backward "^\\(Daily\\|Weekly\\) report"
+                                            nil t)
                         (point))))
       (end-of-line 0))))
 
-(defun org-clocktable-add-file (file table)
-  (if table
-      (let ((lines (org-split-string table "\n"))
-           (ff (file-name-nondirectory file)))
-       (mapconcat 'identity
-                  (mapcar (lambda (x)
-                            (if (string-match org-table-dataline-regexp x)
-                                (concat "|" ff x)
-                              x))
-                          lines)
-                  "\n"))))
+(defun org-clock-get-table-data (file params)
+  "Get the clocktable data for file FILE, with parameters PARAMS.
+FILE is only for identification - this function assumes that
+the correct buffer is current, and that the wanted restriction is
+in place.
+The return value will be a list with the file name and the total
+file time (in minutes) as 1st and 2nd elements.  The third element
+of this list will be a list of headline entries.  Each entry has the
+following structure:
+
+  (LEVEL HEADLINE TIMESTAMP TIME)
+
+LEVEL:     The level of the headline, as an integer.  This will be
+           the reduced leve, so 1,2,3,... even if only odd levels
+           are being used.
+HEADLINE:  The text of the headline.  Depending on PARAMS, this may
+           already be formatted like a link.
+TIMESTAMP: If PARAMS require it, this will be a time stamp found in the
+           entry, any of SCHEDULED, DEADLINE, NORMAL, or first inactive,
+           in this sequence.
+TIME:      The sum of all time spend in this tree, in minutes.  This time
+           will of cause be restricted to the time block and tags match
+           specified in PARAMS."
+  (let* ((maxlevel (or (plist-get params :maxlevel) 3))
+        (timestamp (plist-get params :timestamp))
+        (ts (plist-get params :tstart))
+        (te (plist-get params :tend))
+        (block (plist-get params :block))
+        (link (plist-get params :link))
+        (tags (plist-get params :tags))
+        (properties (plist-get params :properties))
+        (inherit-property-p (plist-get params :inherit-props))
+        (matcher (if tags (cdr (org-make-tags-matcher tags))))
+        cc range-text st p time level hdl props tsp tbl)
+
+    (setq org-clock-file-total-minutes nil)
+    (when block
+      (setq cc (org-clock-special-range block nil t)
+           ts (car cc) te (nth 1 cc) range-text (nth 2 cc)))
+    (when (integerp ts) (setq ts (calendar-gregorian-from-absolute ts)))
+    (when (integerp te) (setq te (calendar-gregorian-from-absolute te)))
+    (when (and ts (listp ts))
+      (setq ts (format "%4d-%02d-%02d" (nth 2 ts) (car ts) (nth 1 ts))))
+    (when (and te (listp te))
+      (setq te (format "%4d-%02d-%02d" (nth 2 te) (car te) (nth 1 te))))
+    ;; Now the times are strings we can parse.
+    (if ts (setq ts (org-float-time
+                    (apply 'encode-time (org-parse-time-string ts)))))
+    (if te (setq te (org-float-time
+                    (apply 'encode-time (org-parse-time-string te)))))
+    (save-excursion
+      (org-clock-sum ts te
+                    (unless (null matcher)
+                      (lambda ()
+                        (let ((tags-list (org-get-tags-at)))
+                          (eval matcher)))))
+      (goto-char (point-min))
+      (setq st t)
+      (while (or (and (bobp) (prog1 st (setq st nil))
+                     (get-text-property (point) :org-clock-minutes)
+                     (setq p (point-min)))
+                (setq p (next-single-property-change
+                         (point) :org-clock-minutes)))
+       (goto-char p)
+       (when (setq time (get-text-property p :org-clock-minutes))
+         (save-excursion
+           (beginning-of-line 1)
+           (when (and (looking-at (org-re "\\(\\*+\\)[ \t]+\\(.*?\\)\\([ \t]+:[[:alnum:]_@#%:]+:\\)?[ \t]*$"))
+                      (setq level (org-reduced-level
+                                   (- (match-end 1) (match-beginning 1))))
+                      (<= level maxlevel))
+             (setq hdl (if (not link)
+                           (match-string 2)
+                         (org-make-link-string
+                          (format "file:%s::%s"
+                                  (buffer-file-name)
+                                  (save-match-data
+                                    (org-make-org-heading-search-string
+                                     (match-string 2))))
+                          (match-string 2)))
+                   tsp (when timestamp
+                         (setq props (org-entry-properties (point)))
+                         (or (cdr (assoc "SCHEDULED" props))
+                             (cdr (assoc "DEADLINE" props))
+                             (cdr (assoc "TIMESTAMP" props))
+                             (cdr (assoc "TIMESTAMP_IA" props))))
+           props (when properties
+                   (remove nil
+                           (mapcar
+                            (lambda (p)
+                              (when (org-entry-get (point) p inherit-property-p)
+                                (cons p (org-entry-get (point) p inherit-property-p))))
+                            properties))))
+             (when (> time 0) (push (list level hdl tsp time props) tbl))))))
+      (setq tbl (nreverse tbl))
+      (list file org-clock-file-total-minutes tbl))))
 
 (defun org-clock-time% (total &rest strings)
   "Compute a time fraction in percent.
@@ -2051,10 +2526,13 @@ This function is made for clock tables."
          (if (string-match "\\([0-9]+\\):\\([0-9]+\\)" s)
              (throw 'exit
                     (/ (* 100.0 (+ (string-to-number (match-string 2 s))
-                                   (* 60 (string-to-number (match-string 1 s)))))
+                                   (* 60 (string-to-number
+                                          (match-string 1 s)))))
                        tot))))
        0))))
 
+;; Saving and loading the clock
+
 (defvar org-clock-loaded nil
   "Was the clock file loaded?")
 
@@ -2081,7 +2559,8 @@ The details of what will be saved are regulated by the variable
                   (buffer-file-name b)
                   (or (not org-clock-persist-query-save)
                       (y-or-n-p (concat "Save current clock ("
-                                        (substring-no-properties org-clock-heading)
+                                        (substring-no-properties
+                                         org-clock-heading)
                                         ") "))))
              (insert "(setq resume-clock '(\""
                      (buffer-file-name (org-clocking-buffer))
@@ -2145,7 +2624,7 @@ The details of what will be saved are regulated by the variable
              (goto-char (cdr resume-clock))
              (let ((org-clock-auto-clock-resolution nil))
                (org-clock-in)
-               (if (org-invisible-p)
+               (if (outline-invisible-p)
                    (org-show-context))))))))))
 
 ;;;###autoload
@@ -2159,6 +2638,6 @@ The details of what will be saved are regulated by the variable
 
 (provide 'org-clock)
 
-;; arch-tag: 7b42c5d4-9b36-48be-97c0-66a869daed4c
+
 
 ;;; org-clock.el ends here