Re: [O] [PATCH] ox-icalendar: fix handling of timestamps

[Nicolas Goaziou]

Hello,

Viktor Rosenfeld <address@hidden> writes:

>   * ox-icalendar.el (org-icalendar-entry): Honor setting of
>   `org-icalendar-with-timestamps' for timestamps on headlines
>   and checkboxes.
>
>   The setting `org-icalendar-with-timestamps' was only applied
>   to timestamps which do not appear on a heading or on a
>   checkbox. E.g., with `org-icalendar-with-timestamps' set to
>   'active, an heading containing an inactive timestamp on would
>   be exported. This patch fixes this.

This would make icalendar back-end inconsistent with other back-ends,
see `org-export-with-timestamps' docstring.

If inconsistency is desirable in this case, `org-icalendar-with-timestamps'
docstring should clearly state it.


Regards,

-- 
Nicolas Goaziou