Re: [Orgmode] org-toggle-checkbox bug

2011-02-24 Thread Nick Dokos
Matt Lundin  wrote:

> According to the docstring of org-toggle-checkbox, the function can be
> called on headlines:
> 
> ,
> | If the cursor is in a headline, apply this to all checkbox items
> | in the text below the heading, taking as reference the first item
> | in subtree, ignoring drawers.
> `
> 
> When I call org-toggle-checkbox on the following entry...
> 
> --8<---cut here---start->8---
> * Test
>  - [ ] Now
>  - [ ] Not now
>  - [ ] Later
>  - Not yet
>  - Never
> --8<---cut here---end--->8---
> 
> ...I get the following results...
> 
> --8<---cut here---start->8---
> * Test
>  - [ ] Now
>  - [X] Not now
>  - [X] Later
>  - Not yet
>  - Never
> --8<---cut here---end--->8---
> 
> Then if I call org-toggle-checkbox on the headline again, nothing
> happens. The two boxes remain checked.
> 

Try this patch:

diff --git a/lisp/org-list.el b/lisp/org-list.el
index f66c665..1449062 100644
--- a/lisp/org-list.el
+++ b/lisp/org-list.el
@@ -2134,7 +2134,7 @@ in subtree, ignoring drawers."
   (bottom (copy-marker (org-list-get-bottom-point struct)))
   (items-to-toggle (org-remove-if
 (lambda (e) (or (< e lim-up) (> e lim-down)))
-(mapcar 'car (cdr struct)
+(mapcar 'car struct
  (mapc (lambda (e) (org-list-set-checkbox
e struct
;; if there is no box at item, leave as-is

Nick

___
Emacs-orgmode mailing list
Please use `Reply All' to send replies to the list.
Emacs-orgmode@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-orgmode


Re: [Orgmode] org-toggle-checkbox bug

2011-02-24 Thread Nick Dokos
John Hendy  wrote:

> On Thu, Feb 24, 2011 at 5:52 PM, Matt Lundin  wrote:
> 
> According to the docstring of org-toggle-checkbox, the function can be
> called on headlines:
>
> ,
> | If the cursor is in a headline, apply this to all checkbox items
> | in the text below the heading, taking as reference the first item
> | in subtree, ignoring drawers.
> `
>
> When I call org-toggle-checkbox on the following entry, [I get (shortened 
> from your example)]:
>
> --8<---cut here---start->8---
> * Test
>  - [ ] Now
>  - [ ] Not now ---> [x]
>  - [ ] Later ---> [x]
>  - Not yet
>  - Never
> --8<---cut here---end--->8---
>
>  Then if I call org-toggle-checkbox on the headline again, nothing
> happens. The two boxes remain checked.
> 
> Odd; this is not the behavior I get. I just tried on a structure and got this 
> cycling:
> ,-
> | * top
> | ** next
> | *** next
> | - [x] item 1  --> [ ]  --> [x]  
> | - [ ] item 2  --> [ ]  --> [x]
> | - [ ] item 3  --> [ ]  --> [x]
> | - [ ] item 4  --> [ ]  --> [x]
> `-
> 
> So... it seems to be working for me, though I was not sure based on the 
> docstring whether "taking as reference the first item in subtree" meant it 
> would toggle the rest to be like it or everything to be the opposite of it. 
> It appears it's the latter definition.
> 
> Maybe provide your version? I'm using 7.4.
> 

Not working for me: I can reproduce the behavior that Matt reported.

Org-mode version 7.4 (release_7.4.529.gb23d.dirty)

Nick

___
Emacs-orgmode mailing list
Please use `Reply All' to send replies to the list.
Emacs-orgmode@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-orgmode


Re: [Orgmode] org-toggle-checkbox bug

2011-02-24 Thread John Hendy
On Thu, Feb 24, 2011 at 5:52 PM, Matt Lundin  wrote:

> According to the docstring of org-toggle-checkbox, the function can be
> called on headlines:
>
> ,
> | If the cursor is in a headline, apply this to all checkbox items
> | in the text below the heading, taking as reference the first item
> | in subtree, ignoring drawers.
> `
>
> When I call org-toggle-checkbox on the following entry, [I get (shortened
> from your example)]:
>
> --8<---cut here---start->8---
> * Test
>  - [ ] Now
>  - [ ] Not now ---> [x]
>  - [ ] Later ---> [x]
>  - Not yet
>  - Never
> --8<---cut here---end--->8---
>
>  Then if I call org-toggle-checkbox on the headline again, nothing
> happens. The two boxes remain checked.
>
>
Odd; this is not the behavior I get. I just tried on a structure and got
this cycling:
,-
| * top
| ** next
| *** next
| - [x] item 1  --> [ ]  --> [x]
| - [ ] item 2  --> [ ]  --> [x]
| - [ ] item 3  --> [ ]  --> [x]
| - [ ] item 4  --> [ ]  --> [x]
`-

So... it seems to be working for me, though I was not sure based on the
docstring whether "taking as reference the first item in subtree" meant it
would toggle the rest to be like it or everything to be the opposite of it.
It appears it's the latter definition.

Maybe provide your version? I'm using 7.4.

Best,
> Matt
>
> ___
> Emacs-orgmode mailing list
> Please use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode
>
___
Emacs-orgmode mailing list
Please use `Reply All' to send replies to the list.
Emacs-orgmode@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-orgmode


[Orgmode] org-toggle-checkbox bug

2011-02-24 Thread Matt Lundin
According to the docstring of org-toggle-checkbox, the function can be
called on headlines:

,
| If the cursor is in a headline, apply this to all checkbox items
| in the text below the heading, taking as reference the first item
| in subtree, ignoring drawers.
`

When I call org-toggle-checkbox on the following entry...

--8<---cut here---start->8---
* Test
 - [ ] Now
 - [ ] Not now
 - [ ] Later
 - Not yet
 - Never
--8<---cut here---end--->8---

...I get the following results...

--8<---cut here---start->8---
* Test
 - [ ] Now
 - [X] Not now
 - [X] Later
 - Not yet
 - Never
--8<---cut here---end--->8---

Then if I call org-toggle-checkbox on the headline again, nothing
happens. The two boxes remain checked.

Best,
Matt

___
Emacs-orgmode mailing list
Please use `Reply All' to send replies to the list.
Emacs-orgmode@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-orgmode