[bug] org-caldav-sync broken if entries have tables?

[bug] org-caldav-sync broken if entries have tables?

[Eric S Fraga]

Hello,

I have just upgraded both org and org-caldav and now I can no longer
sync to my Google calendar.  I get the following error:

Debugger entered--Lisp error: (args-out-of-range [left left right left] 4)
  org-export-table-cell-alignment((table-cell (:begin 49232 :end 49241 
  org-ascii-table-cell((table-cell (:begin 49232 :end 49241
  org-export-data((table-cell (:begin 49232 :end 49241 
  ...
  org-export-data((org-data nil (section (:begin 1 :end 103 
  org-export-as(icalendar nil nil t (:ascii-charset utf-8 :icalendar-agenda-view nil))
  ...
  org-icalendar--combine-files(nil "~/s/notes/diary.org" "~/s/notes/googlecalendar.org")
  apply(org-icalendar--combine-files nil ("~/s/notes/diary.org" "~/s/notes/googlecalendar.org"))
  (if (featurep (quote ox-icalendar)) (apply (quote org-icalendar--combine-files) nil orgfiles) (apply (quote org-export-icalendar) t orgfiles))
  (let ((icalendar-file (if (featurep (quote ox-icalendar)) (quote org-icalendar-combined-agenda-file) (quote org-combined-agenda-icalendar-file))) (orgfiles (append org-caldav-files (list org-caldav-inbox))) (org-icalendar-store-UID t) (org-icalendar-include-bbdb-anniversaries nil) (icalendar-uid-format "orgsexp-%h") (org-icalendar-date-time-format (cond ((and org-icalendar-timezone (string= org-icalendar-timezone "UTC")) ":%Y%m%dT%H%M%SZ") (org-icalendar-timezone ";TZID=%Z:%Y%m%dT%H%M%S") (t ":%Y%m%dT%H%M%S")))) (set icalendar-file (make-temp-file "org-caldav-")) (org-caldav-debug-print 1 (format "Generating ICS file %s." (symbol-value icalendar-file))) (if (featurep (quote ox-icalendar)) (apply (quote org-icalendar--combine-files) nil orgfiles) (apply (quote org-export-icalendar) t orgf
 iles)) (find-file-noselect (symbol-value icalendar-file)))
  org-caldav-generate-ics()
  (setq org-caldav-ics-buffer (org-caldav-generate-ics))
  (if (and org-caldav-event-list (y-or-n-p "Last sync seems to have been aborted. Should I try to resume? ")) nil (setq org-caldav-ics-buffer (org-caldav-generate-ics)) (setq org-caldav-event-list nil) (setq org-caldav-sync-result nil) (org-caldav-load-sync-state) (progn (let ((--dolist-tail-- org-caldav-event-list) cur) (while --dolist-tail-- (setq cur (car --dolist-tail--)) (org-caldav-event-set-status cur nil) (setq --dolist-tail-- (cdr --dolist-tail--))))) (org-caldav-update-eventdb-from-org org-caldav-ics-buffer) (org-caldav-update-eventdb-from-cal))
  org-caldav-sync()
  call-interactively(org-caldav-sync nil nil)
  command-execute(org-caldav-sync)

  
There is nothing particularly strange about the table found at location
49232 in my diary.

Deleting the table and one other, possibly leaving no tables in my
diary, allows syncing to work again.

thanks,
eric

-- 
: Eric S Fraga (0xFFFCF67D), Emacs 24.3.50.1, Org release_8.1.1-7-gaecdf5

Re: [bug] org-caldav-sync broken if entries have tables?

[Nicolas Goaziou]

Hello,

Eric S Fraga <e.fraga@ucl.ac.uk> writes:

> I have just upgraded both org and org-caldav and now I can no longer
> sync to my Google calendar.  I get the following error:

[...]

> There is nothing particularly strange about the table found at location
> 49232 in my diary.

Are you able to export a buffer (with any export back-end) containing
only that table?


Regards,

-- 
Nicolas Goaziou

Re: [bug] org-caldav-sync broken if entries have tables?

[Eric S Fraga]

Nicolas Goaziou <n.goaziou@gmail.com> writes:

> Hello,
>
> Eric S Fraga <e.fraga@ucl.ac.uk> writes:
>
>> I have just upgraded both org and org-caldav and now I can no longer
>> sync to my Google calendar.  I get the following error:
>
> [...]
>
>> There is nothing particularly strange about the table found at location
>> 49232 in my diary.
>
> Are you able to export a buffer (with any export back-end) containing
> only that table?

No, and I have figured out why. 

<blush>

The table was ill-formed.  It was originally part of an email I had cut
and pasted into a diary item and I had never asked org to "format" it
which, if I had, would have added an extra column throughout.  It's
interesting that the exporter used to work.  It *is*, however, a long
time since I updated my org as I was away all of August (without a
computer!) and have been doing the headless chicken impression ever
since trying to catch up ...

</blush>

If it helps, here is a minimal example:

--8<---------------cut here---------------start------------->8---
* table test for export
        |Milestone / Deliverable                              |WP  |Responsibl|Due:    |
        |                                                     |    |e         |        |
        |M7.1  |UCL to have developed and exercised test bed  |7   |UCL       |30th    |
--8<---------------cut here---------------end--------------->8---

Note the extra | before the first UCL in the last line of the table,
something which I hadn't spotted.

Thanks for making me look at this closer and sorry for the noise.

-- 
: Eric S Fraga (0xFFFCF67D), Emacs 24.3.50.1, Org release_8.1.1-7-gaecdf5