Re: org-mobile-push and MobileOrg compatibility issue

[Vamsi Vytla <vamsi.vytla@gmail.com>]

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

Hi Nicolas,

I can guarantee you that your suggestion did NOT work for me. I haven't
heard anything from Sergey here. Looks like people at MobileOrg-Android
maybe running into the same issue:

https://github.com/matburt/mobileorg-android/issues/472#issuecomment-147403622

Any thoughts?

Vamsi

On Sat, Sep 5, 2015 at 1:44 AM Nicolas Goaziou <mail@nicolasgoaziou.fr>
wrote:

> Hello,
>
> Vamsi Vytla <vamsi.vytla@gmail.com> writes:
>
> > M-x 'org-mobile-push', with the latest org-mode leaves the files
> > incompatible with MobileOrg Android application. There have been no
> changes
> > in MobileOrg for over a year.
> >
> > I bisected the latest org-mobile related changes and noticed that
> reverting
> > this one line below "fixes" the issue (at least for me). Since there
> aren't
> > any tests, it's hard to validate changes and understand things easily.
> >
> > If anybody can point me in the right direction, I would love to look into
> > it further.
> >
> > Cheers!
> >
> > diff --git a/lisp/org-mobile.el b/lisp/org-mobile.el
> > index 6c7c8d0..7e1127c 100644
> > --- a/lisp/org-mobile.el
> > +++ b/lisp/org-mobile.el
> > @@ -446,7 +446,7 @@ agenda view showing the flagged items."
> >                                          x))
> >                            (cdr entry)))
> >         (insert "#+TODO: " (mapconcat 'identity kwds " ") "\n")
> > -       (setq dwds (or (member "|" kwds) (last kwds))
> > +       (setq dwds (member "|" kwds)
> >               twds (org-delete-all dwds kwds)
> >               todo-kwds (org-delete-all twds todo-kwds)
> >               done-kwds (org-delete-all dwds done-kwds)))
>
> IIRC this change was introduced to fix another bug. It might be useful
> to discuss with the author of this change. Another option is to use
>
>   (or (member "|" kwds) (cons "|" (last kwds)))
>
>
> Regards,
>
> --
> Nicolas Goaziou
>

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