emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed

From: Bernt Hansen <bernt@norang.ca>
To: emacs-orgmode@gnu.org
Cc: Bernt Hansen <bernt@norang.ca>,
	Carsten Dominik <carsten.dominik@gmail.com>
Subject: [PATCH] Fix reported file time for clock reports
Date: Wed,  8 Jun 2011 15:41:45 -0400	[thread overview]
Message-ID: <1307562105-6281-1-git-send-email-bernt@norang.ca> (raw)
In-Reply-To: <20110513121920.EF6F444B988@u016822.science.uva.nl>

* lisp/org-clock.el (org-clocktable-write-default):
---
Hi,

My agenda clock reports were not displaying total file time correctly anymore.
Since the patch for adding properties used up a placeholder when no properties are
provided there was no place to put the total file time into the summary line.

I'm not completely sure the placeholder is in the right place for this but this works for me.
Please double check before applying.

This patch is available at git://git.norang.ca/org-mode.git fix-clock-report-file-time

Regards,
Bernt


 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..8d00c0a 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*" (nth 8 lwords) "* | %s *%s*|\n")
 		       (file-name-nondirectory (car tbl))
 		       (if level-p   "| " "") ; level column, maybe
 		       (if timestamp "| " "") ; timestamp column, maybe
-- 
1.7.6.rc0.12.g2c6b5

next prev parent reply	other threads:[~2011-06-08 19:42 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-26 20:29 lisp/org-clock.el: Add param :properties to list properties in clocktable Niels Giesen
2011-03-26 20:49 ` Bernt Hansen
2011-03-27 10:13   ` Niels Giesen
2011-03-27 12:13     ` Bernt Hansen
2011-03-28 18:24       ` Niels Giesen
2011-05-02  9:33 ` Carsten Dominik
2011-05-13  8:00   ` Niels Giesen
2011-05-13  8:03     ` Carsten Dominik
2011-05-13 12:19     ` [Accepted] " Carsten Dominik
2011-06-08 19:41       ` Bernt Hansen [this message]
2011-05-13 12:20     ` Carsten Dominik
2011-05-13 12:35       ` Giovanni Ridolfi
2011-05-18  7:08       ` Niels Giesen
2011-05-18  8:43         ` Giovanni Ridolfi

find likely ancestor, descendant, or conflicting patches for this message:
dfblob:a3caa48 dfblob:8d00c0a
	(help)

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.orgmode.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1307562105-6281-1-git-send-email-bernt@norang.ca \
    --to=bernt@norang.ca \
    --cc=carsten.dominik@gmail.com \
    --cc=emacs-orgmode@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).