Re: [O] org-capture does not fully clean up after C-c C-k

2011-07-22 Thread Jason Dunsmore
Carsten Dominik carsten.domi...@gmail.com writes:

 I am unable to reproduce this.  I do get an empty line the first time,
 but when the empty line is already there, then there is no problem.
 
 Do you find the insertion of this one empty line a problem?
 
 It's a significant problem for me.  I still use org-remember because 1)
 org-capture inserts an extra blank line

 Do you also mean at the beginning of the file, or elsewhere?

 and 2) canceling a capture
 doesn't work for some reason - I get the following message:
 
 org-capture-finalize: Capture process aborted, but target buffer could not 
 be cleaned up correctly

 can you make a minimal example to demonstrate this issue?
 I would like to fix it.

While attempting to make a minimal example of this, I narrowed down the
problem to using :unnarrowed t in my org-capture-templates.  Since I
removed it, there is no extra line and the buffer gets cleaned up when I
type C-c C-k.  I've finally made the switch to org-capture.  Thanks!



Re: [O] org-capture does not fully clean up after C-c C-k

2011-07-12 Thread Bastien
Hi Leo,

Leo sdl@gmail.com writes:

 Yes, it fixes this issue.

I applied the patch, then.

Thanks for confirming,

-- 
 Bastien



Re: [O] org-capture does not fully clean up after C-c C-k

2011-07-11 Thread Leo
On 2011-06-16 15:51 +0800, Carsten Dominik wrote:
 Hi Leo,

 I am unable to reproduce this.  I do get an empty line the first time,
 but when the empty line is already there, then there is no problem.

 Do you find the insertion of this one empty line a problem?

 - Carsten

Sorry for the long delay. I can reproduce this in Emacs -q with

  (setq org-blank-before-new-entry '((heading)))

Leo



Re: [O] org-capture does not fully clean up after C-c C-k

2011-07-11 Thread Bastien
Hi Leo,

Leo sdl@gmail.com writes:

 On 2011-06-16 15:51 +0800, Carsten Dominik wrote:
 Hi Leo,

 I am unable to reproduce this.  I do get an empty line the first time,
 but when the empty line is already there, then there is no problem.

 Do you find the insertion of this one empty line a problem?

 - Carsten

 Sorry for the long delay. I can reproduce this in Emacs -q with

   (setq org-blank-before-new-entry '((heading)))

Please let me know if the attached patch fixes this issue.

Thanks,

From 38334f29f4c949cad76c199433f60250a824878d Mon Sep 17 00:00:00 2001
From: Bastien Guerry b...@altern.org
Date: Tue, 12 Jul 2011 00:57:57 +0200
Subject: [PATCH] org-capture.el: Fix bug about insert blank line when
 aborting.

* org-capture.el (org-capture-finalize): bugfix about
inserted blank line when killing the capture buffer and
`org-blank-before-new-entry' tells to not insert anything
before a heading.

Thanks to Leo for reporting this.
---
 lisp/org-capture.el |   12 +++-
 1 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/lisp/org-capture.el b/lisp/org-capture.el
index d547f3d..dd74ecc 100644
--- a/lisp/org-capture.el
+++ b/lisp/org-capture.el
@@ -551,6 +551,9 @@ captured item after finalizing.
 	  (m2 (org-capture-get :end-marker 'local)))
 	  (if (and m1 m2 (= m1 beg) (= m2 end))
 	  (progn
+		(setq m2 (if (cdr (assoc 'heading org-blank-before-new-entry))
+			 m2 (1+ m2))
+		  m2 (if ( (point-max) m2) (point-max) m2))
 		(setq abort-note 'clean)
 		(kill-region m1 m2))
 	(setq abort-note 'dirty)))
@@ -576,16 +579,14 @@ captured item after finalizing.
 		   (org-at-table-p))
 	  (if (org-table-get-stored-formulas)
 	  (org-table-recalculate 'all) ;; FIXME: Should we iterate???
-	(org-table-align)))
-	)
+	(org-table-align
   ;; Store this place as the last one where we stored something
   ;; Do the marking in the base buffer, so that it makes sense after
   ;; the indirect buffer has been killed.
   (org-capture-bookmark-last-stored-position)
 
   ;; Run the hook
-  (run-hooks 'org-capture-before-finalize-hook)
-  )
+  (run-hooks 'org-capture-before-finalize-hook))
 
 ;; Kill the indirect buffer
 (save-buffer)
@@ -665,7 +666,8 @@ already gone.  Any prefix argument will be passed to the refile command.
   (interactive)
   ;; FIXME: This does not do the right thing, we need to remove the new stuff
   ;; By hand it is easy: undo, then kill the buffer
-  (let ((org-note-abort t) (org-capture-before-finalize-hook nil))
+  (let ((org-note-abort t) 
+	(org-capture-before-finalize-hook nil))
 (org-capture-finalize)))
 
 (defun org-capture-goto-last-stored ()
-- 
1.7.5.2


-- 
 Bastien


Re: [O] org-capture does not fully clean up after C-c C-k

2011-07-11 Thread Leo
On 2011-07-12 07:00 +0800, Bastien wrote:
[snipped 14 lines]
 Sorry for the long delay. I can reproduce this in Emacs -q with

   (setq org-blank-before-new-entry '((heading)))

 Please let me know if the attached patch fixes this issue.

 Thanks,

Yes, it fixes this issue.

Leo



Re: [O] org-capture does not fully clean up after C-c C-k

2011-06-21 Thread Carsten Dominik
Hi Jason,


On Jun 16, 2011, at 8:44 PM, Jason Dunsmore wrote:

 Carsten Dominik carsten.domi...@gmail.com writes:
 
 On 4.5.2011, at 14:30, Leo wrote:
 
 Hello,
 
 I have one template as follows
 
 (n Notes entry (file Notes.org) * %?\n  %i :prepend t)
 
 Every time I `C-c C-k' to abort the capture, a blank line is inserted at
 the front of file Notes.org.
 
 Hi Leo,
 
 I am unable to reproduce this.  I do get an empty line the first time,
 but when the empty line is already there, then there is no problem.
 
 Do you find the insertion of this one empty line a problem?
 
 It's a significant problem for me.  I still use org-remember because 1)
 org-capture inserts an extra blank line

Do you also mean at the beginning of the file, or elsewhere?

 and 2) canceling a capture
 doesn't work for some reason - I get the following message:
 
 org-capture-finalize: Capture process aborted, but target buffer could not 
 be cleaned up correctly


can you make a minimal example to demonstrate this issue?
I would like to fix it.

 
 Both of these issues cause me to do manual clean up following every
 capture.
 
 Please let me know if there are fixes/workarounds for these issues.  I'd
 like to finally make the switch to org-capture.



 
 Regards,
 Jason

- Carsten






Re: [O] org-capture does not fully clean up after C-c C-k

2011-06-17 Thread Leo
On 2011-06-16 15:51 +0800, Carsten Dominik wrote:
 Hi Leo,

 I am unable to reproduce this.  I do get an empty line the first time,
 but when the empty line is already there, then there is no problem.

 Do you find the insertion of this one empty line a problem?

 - Carsten

I am running org-mode from git 2011-05-20 and I can still reproduce the
problem. I suspect this might be related to other things in my setup. I
will track it down later on.

Leo



Re: [O] org-capture does not fully clean up after C-c C-k

2011-06-16 Thread Carsten Dominik

On 4.5.2011, at 14:30, Leo wrote:

 Hello,
 
 I have one template as follows
 
 (n Notes entry (file Notes.org) * %?\n  %i :prepend t)
 
 Every time I `C-c C-k' to abort the capture, a blank line is inserted at
 the front of file Notes.org.

Hi Leo,

I am unable to reproduce this.  I do get an empty line the first time,
but when the empty line is already there, then there is no problem.

Do you find the insertion of this one empty line a problem?

- Carsten

 
 orgmode 2011-04-29 from git on Emacs 23.3.50.
 
 With best wishes,
 Leo
 
 




Re: [O] org-capture does not fully clean up after C-c C-k

2011-06-16 Thread Jason Dunsmore
Carsten Dominik carsten.domi...@gmail.com writes:

 On 4.5.2011, at 14:30, Leo wrote:

 Hello,
 
 I have one template as follows
 
 (n Notes entry (file Notes.org) * %?\n  %i :prepend t)
 
 Every time I `C-c C-k' to abort the capture, a blank line is inserted at
 the front of file Notes.org.

 Hi Leo,

 I am unable to reproduce this.  I do get an empty line the first time,
 but when the empty line is already there, then there is no problem.

 Do you find the insertion of this one empty line a problem?

It's a significant problem for me.  I still use org-remember because 1)
org-capture inserts an extra blank line and 2) canceling a capture
doesn't work for some reason - I get the following message:

org-capture-finalize: Capture process aborted, but target buffer could not be 
cleaned up correctly

Both of these issues cause me to do manual clean up following every
capture.

Please let me know if there are fixes/workarounds for these issues.  I'd
like to finally make the switch to org-capture.

Regards,
Jason



[O] org-capture does not fully clean up after C-c C-k

2011-05-04 Thread Leo
Hello,

I have one template as follows

 (n Notes entry (file Notes.org) * %?\n  %i :prepend t)

Every time I `C-c C-k' to abort the capture, a blank line is inserted at
the front of file Notes.org.

orgmode 2011-04-29 from git on Emacs 23.3.50.

With best wishes,
Leo