Skip to content

Pullrequest thunderbird zero duration workaround#147

Open
JoachimFalk wants to merge 2 commits intoskarim:masterfrom
JoachimFalk:pullrequest-thunderbird-zero-duration-workaround
Open

Pullrequest thunderbird zero duration workaround#147
JoachimFalk wants to merge 2 commits intoskarim:masterfrom
JoachimFalk:pullrequest-thunderbird-zero-duration-workaround

Conversation

@JoachimFalk
Copy link

Description: Work around for DTEND and DURATION presence generated by Thunderbird Lightning
Author: Joachim Falk joachim.falk@gmx.de

Thunderbird (Lightning calendar) sometimes generates a VEVENT with
a zero DURATION entry while DTEND is also present. This can happen if
you edit one event belonging to a series of recurring events.

Fixed exception
'vobject.base.ValidateError: 'VEVENT components cannot contain both
DTEND and DURATION components'
triggered by bad ics files generated by Thunderbird (Lightning calendar)
where sometimes a zero DURATION entry can be present even if DTEND was
given. The patch will delete such redundant zero DURATION entries.

Thunderbird (Lightning calendar) sometimes generates a VEVENT with
a zero DURATION entry while DTEND is also present. This can happen if
you edit one event belonging to a series of recurring events.
…Lightning

Fixed exception
  'vobject.base.ValidateError: 'VEVENT components cannot contain both
   DTEND and DURATION components'
triggered by bad ics files generated by Thunderbird (Lightning calendar)
where sometimes a zero DURATION entry can be present even if DTEND was
given. The patch will delete such redundant zero DURATION entries.
@da4089
Copy link
Collaborator

da4089 commented Feb 15, 2024

@JoachimFalk do you know if there's a Mozilla bug for this? I had a quick search in their Bugzilla, but didn't turn up anything.

@JoachimFalk
Copy link
Author

@da4089 I don't know and I also did not submit one to Mozilla. Maybe they fixed the bug in the meantime. At the moment, I can't reproduce it. The bug was there ca. around Aug. 2019.

@da4089
Copy link
Collaborator

da4089 commented Feb 15, 2024

Some maybe it's been fixed in the last ... err ... 5 years.
Thanks for checking!

@da4089
Copy link
Collaborator

da4089 commented Feb 23, 2024

@JoachimFalk, I'll skip merging this into the https://github.com/py-vobject/vobject repo on the basis that you cannot repro it. If you come across this problem again, please raise an issue there, and link back to this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants