diff options
| author | Niels Giesen <niels.giesen@gmail.com> | 2011-06-10 11:32:42 +0200 |
|---|---|---|
| committer | Carsten Dominik <carsten.dominik@gmail.com> | 2011-06-10 13:56:02 +0200 |
| commit | 1de29a0059e300da5e0245a209f66275c4eb3ec8 (patch) | |
| tree | 419c25681a82bcec19c52d8959c3c126338f18d4 | |
| parent | 374b70a3832366ccc4fadafd916394ac9e9affd4 (diff) | |
| download | org-mode-1de29a0059e300da5e0245a209f66275c4eb3ec8.tar.gz | |
Fix reported file time for clock reports
Fix the format specification for clock tables.
Niels writes:
Bernt was right in that there were more arguments to format than the
format string specified. I guess I must not have noticed that there was
a call to format (all the other times properties columns are 'maybe
inserted' go directly to `insert-before-markers'), and I must have
forgotten to test the multifile case (I never used multifile clock
reports myself).
However, to keep the layout of the table consistent the *File time*
cells should be in the Headline column. Attached patch fixes this by
putting the (empty) properties cells *before* the *File time* cell.
| -rw-r--r-- | lisp/org-clock.el | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lisp/org-clock.el b/lisp/org-clock.el index a3caa48..abab70c 100644 --- a/lisp/org-clock.el +++ b/lisp/org-clock.el @@ -2162,9 +2162,9 @@ from the dynamic block defintion." (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 colleted from this file + ;; Summarize the time collected from this file (insert-before-markers - (format (concat "| %s %s | %s*" (nth 8 lwords) "* | *%s*|\n") + (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 |