Skip to content

fix(caldav): handle non-vevents in CalDavEventListener - #17542

Merged
miaulalala merged 1 commit into
mainfrom
handle-vevent-null
Mar 30, 2026
Merged

fix(caldav): handle non-vevents in CalDavEventListener#17542
miaulalala merged 1 commit into
mainfrom
handle-vevent-null

Conversation

@tcitworld

Copy link
Copy Markdown
Member

For instance if vobject contains vtodos or vjournals.

Fixes :

[PHP] Attempt to read property "LOCATION" on null at spreed/lib/Listener/CalDavEventListener.php#86

🛠️ API Checklist

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not possible
  • 📘 API documentation in docs/ has been updated or is not required
  • 🔖 Capability is added or not needed

@tcitworld tcitworld added this to the 🏖️ Next Major (34) milestone Mar 29, 2026
@tcitworld tcitworld added bug feature: api 🛠️ OCS API for conversations, chats and participants labels Mar 29, 2026
For instance if vobject contains vtodos or vjournals

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
@tcitworld
tcitworld force-pushed the handle-vevent-null branch from 23291d8 to f8c65c4 Compare March 29, 2026 09:35
@miaulalala

Copy link
Copy Markdown
Contributor

/backport to stable33

@miaulalala

Copy link
Copy Markdown
Contributor

/backport to stable32

@miaulalala

Copy link
Copy Markdown
Contributor

/backport to stable31

@miaulalala
miaulalala merged commit a383c87 into main Mar 30, 2026
100 of 104 checks passed
@miaulalala
miaulalala deleted the handle-vevent-null branch March 30, 2026 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug feature: api 🛠️ OCS API for conversations, chats and participants

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants