MobileOrg Export Broken

MobileOrg Export Broken

[Ian Dunn]

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


    I run Org from the git repo, updating daily.  I unsuccessfully tried syncing with MobileOrg (Android) about two hours ago.  I reverted org-mobile.el back to the version from yesterday and found the problem: TODO keywords aren't being listed correctly.  Here's an example:

Old:
#+TODO: TODO | DEFERRED DONE PENDING CANCELLED

New:
#+TODO: TODO

    This messes with MobileOrg, preventing it from syncing.

    I worked through the problem, which appears to be caused by org-delete-all being called before the TODO keywords are inserted into the file.  I attached a patch that fixes this problem.

-- 
Ian Dunn


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: org-mobile.el.patch --]
[-- Type: text/x-diff, Size: 585 bytes --]

diff --git a/lisp/org-mobile.el b/lisp/org-mobile.el
index 9c69128..9192561 100644
--- a/lisp/org-mobile.el
+++ b/lisp/org-mobile.el
@@ -446,8 +446,9 @@ agenda view showing the flagged items."
 				      x))
 			     (cdr entry)))
 	       (dwds (or (member "|" kwds) (last kwds)))
-	       (twds (org-delete-all dwds kwds)))
+	       (twds kwds))
 	  (insert "#+TODO: " (mapconcat 'identity kwds " ") "\n")
+	  (org-delete-all dwds twds)
 	  (setq todo-kwds (org-delete-all twds todo-kwds))
 	  (setq done-kwds (org-delete-all dwds done-kwds))))
       (when (or todo-kwds done-kwds)

Re: MobileOrg Export Broken

[Nicolas Goaziou]

Hello,

Ian Dunn <dunni@gnu.org> writes:

>     I run Org from the git repo, updating daily. I unsuccessfully
> tried syncing with MobileOrg (Android) about two hours ago. I reverted
> org-mobile.el back to the version from yesterday and found the
> problem: TODO keywords aren't being listed correctly. Here's an
> example:
>
> Old:
> #+TODO: TODO | DEFERRED DONE PENDING CANCELLED
>
> New:
> #+TODO: TODO
>
>     This messes with MobileOrg, preventing it from syncing.
>
>     I worked through the problem, which appears to be caused by
>     org-delete-all being called before the TODO keywords are inserted
>     into the file.  I attached a patch that fixes this problem.

Correct. I pushed a slightly different patch. Thank you.

Regards,

-- 
Nicolas Goaziou