bug with org-clone-subtree-with-time-shift?

bug with org-clone-subtree-with-time-shift?

[Ingmar Meissner]

[-- Attachment #1: Type: text/plain, Size: 419 bytes --]

Hi,

it seems to me that "org-clone-subtree-with-time-shift" is not working correct with org-mode 8.0. Usually i could specify a time shift, but not anymore. Did i miss something?

Ingmar Meissner

P.S. By the way, thanks for all the good work on org-mode!

********************************************
Ingmar Meissner
info@ingmarmeissner.de
********************************************











[-- Attachment #2: Type: text/html, Size: 6114 bytes --]

Re: bug with org-clone-subtree-with-time-shift?

[David Rogers]

Ingmar Meissner <info@ingmarmeissner.de> writes:

> Hi,
>
> it seems to me that "org-clone-subtree-with-time-shift" is not working
> correct with org-mode 8.0. Usually i could specify a time shift, but
> not anymore. Did i miss something?

Hi Ingmar

I had the same problem; it's not a bug, it still works, just the default
behaviour has been changed for 8.0.

The change is that the time-shift is no longer automatic. Now, before
you type

C-c C-x c

you need to type

C-u

first, so that the whole command is "C-u C-c C-x c", if you want the
time shift. Otherwise, you get a plain clone with no time
shift. (Before, you had to press extra keys to get a plain clone, now
you have to press extra keys to get a time-shifted clone.)

-- 
David

[Bug] Re: bug with org-clone-subtree-with-time-shift?

[Karl Voit]

* David Rogers <davidandrewrogers@gmail.com> wrote:
> Ingmar Meissner <info@ingmarmeissner.de> writes:

Hi!

>> it seems to me that "org-clone-subtree-with-time-shift" is not working
>> correct with org-mode 8.0. Usually i could specify a time shift, but
>> not anymore. Did i miss something?
>
> I had the same problem; it's not a bug, it still works, just the default
> behaviour has been changed for 8.0.
>
> The change is that the time-shift is no longer automatic. Now, before
> you type
>
> C-c C-x c
>
> you need to type
>
> C-u
>
> first, so that the whole command is "C-u C-c C-x c", if you want the
> time shift. Otherwise, you get a plain clone with no time
> shift.

Thanks for clearing this issue, David.

I totally agree with Ingmar that this is definitely a bug:

The function is called "org-clone-subtree-with-time-shift" and in
its default behavior (without the universal argument), it does *not*
shift time.

When a user does "M-x org-clone-subtree-with-time-shift" and there
is no time shift, she/he gets irritated - just like Ingmar, just like
I was.


To me, the desired behavior should be like this:

org-clone-subtree ... the same behavior as David described above for
          the current situation of org-clone-subtree-with-time-shift
          with an optional universal argument to ask for time shift
          if you want.

org-clone-subtree-with-time-shift ... the old behavior where the
          user gets asked about the time shift

What function of those two is mapped to "C-c C-x c" is an
independent defined issue.

I would love to see this bug fixed.

-- 
mail|git|SVN|photos|postings|SMS|phonecalls|RSS|CSV|XML to Org-mode:
       > get Memacs from https://github.com/novoid/Memacs <

https://github.com/novoid/extract_pdf_annotations_to_orgmode + more on github

Re: [Bug] Re: bug with org-clone-subtree-with-time-shift?

[Bastien]

Hi Karl,

Karl Voit <devnull@Karl-Voit.at> writes:

> I would love to see this bug fixed.

I've applied a fix and replied here:
http://article.gmane.org/gmane.emacs.orgmode/70904

Can you double-check it works as you like?

Thanks!

-- 
 Bastien

Re: [Bug] Re: bug with org-clone-subtree-with-time-shift?

[Karl Voit]

* Bastien <bzg@gnu.org> wrote:
> Hi Karl,

Hi!

> Karl Voit <devnull@Karl-Voit.at> writes:
>
>> I would love to see this bug fixed.
>
> I've applied a fix and replied here:
> http://article.gmane.org/gmane.emacs.orgmode/70904
>
> Can you double-check it works as you like?

I've seen commit 2a192ff6c (Apr 20) from you and I checked out again
(commit 4873682e, Apr 23 11:51) but following heading with
org-clone-subtree-with-time-shift does not ask for a time shift
here:

* <2013-05-07 Tue +1w> Test
:PROPERTIES:
:CREATED:  [2013-05-07 Tue 10:29]
:END:

Is a misunderstanding on my side what "when there is a time-stamp"
means?

-- 
mail|git|SVN|photos|postings|SMS|phonecalls|RSS|CSV|XML to Org-mode:
       > get Memacs from https://github.com/novoid/Memacs <

https://github.com/novoid/extract_pdf_annotations_to_orgmode + more on github

Re: [Bug] Re: bug with org-clone-subtree-with-time-shift?

[Bastien]

Hi Karl,

Karl Voit <devnull@Karl-Voit.at> writes:

> I've seen commit 2a192ff6c (Apr 20) from you and I checked out again
> (commit 4873682e, Apr 23 11:51) but following heading with
> org-clone-subtree-with-time-shift does not ask for a time shift
> here:
>
> * <2013-05-07 Tue +1w> Test
> :PROPERTIES:
> :CREATED:  [2013-05-07 Tue 10:29]
> :END:

I just tried this and it does ask for the time-shift here.
There is tiny bug about the "+1w" not being repeated for new
headlines, but that's different.

-- 
 Bastien

Re: [Bug] Re: bug with org-clone-subtree-with-time-shift?

[Karl Voit]

* Bastien <bzg@gnu.org> wrote:
> Hi Karl,

Hi!

> Karl Voit <devnull@Karl-Voit.at> writes:
>
>> I've seen commit 2a192ff6c (Apr 20) from you and I checked out again
>> (commit 4873682e, Apr 23 11:51) but following heading with
>> org-clone-subtree-with-time-shift does not ask for a time shift
>> here:
>>
>> * <2013-05-07 Tue +1w> Test
>> :PROPERTIES:
>> :CREATED:  [2013-05-07 Tue 10:29]
>> :END:
>
> I just tried this and it does ask for the time-shift here.
> There is tiny bug about the "+1w" not being repeated for new
> headlines, but that's different.

Sorry, my fault. After compiling my org-mode checkout the issue
is fixed at my side as well.

-- 
mail|git|SVN|photos|postings|SMS|phonecalls|RSS|CSV|XML to Org-mode:
       > get Memacs from https://github.com/novoid/Memacs <

https://github.com/novoid/extract_pdf_annotations_to_orgmode + more on github

Re: bug with org-clone-subtree-with-time-shift?

[Bastien]

Hi Ingmar,

Ingmar Meissner <info@ingmarmeissner.de> writes:

> it seems to me that "org-clone-subtree-with-time-shift" is not
> working correct with org-mode 8.0. Usually i could specify a time
> shift, but not anymore. Did i miss something?

David explanations are correct, but your email made me think I've
been too heavy-handed with this change.

It makes more sense to skip the time-shift prompt when there is no
time-stamp and to have it by default when there is a time-stamp.  So
this is now how the command behaves.  And when called with C-u, it
will skip the time-shift prompt unconditionally.

Thanks for bringing this up!

-- 
 Bastien