Skip to content

fix(caldav): Enhance UID check for base component#57758

Merged
solracsf merged 1 commit intomasterfrom
fixNoBaseComponentFound
Jan 23, 2026
Merged

fix(caldav): Enhance UID check for base component#57758
solracsf merged 1 commit intomasterfrom
fixNoBaseComponentFound

Conversation

@solracsf
Copy link
Copy Markdown
Member

@solracsf solracsf commented Jan 23, 2026

Summary

Added a check to skip processing if the base component is null or UID is not set.

Checklist

@solracsf solracsf added this to the Nextcloud 34 milestone Jan 23, 2026
@solracsf solracsf added the bug label Jan 23, 2026
@solracsf solracsf requested a review from tcitworld as a code owner January 23, 2026 14:59
@solracsf solracsf added 3. to review Waiting for reviews feature: caldav Related to CalDAV internals labels Jan 23, 2026
@solracsf
Copy link
Copy Markdown
Member Author

/backport to stable33

@solracsf
Copy link
Copy Markdown
Member Author

/backport to stable32

@solracsf
Copy link
Copy Markdown
Member Author

/backport to stable31

@solracsf solracsf enabled auto-merge January 23, 2026 14:59
@solracsf solracsf self-assigned this Jan 23, 2026
Comment thread apps/dav/lib/CalDAV/WebcalCaching/RefreshWebcalService.php Outdated
@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 23, 2026
@solracsf solracsf disabled auto-merge January 23, 2026 16:21
Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
@solracsf solracsf force-pushed the fixNoBaseComponentFound branch from 242c053 to 59bb049 Compare January 23, 2026 16:24
@solracsf solracsf enabled auto-merge January 23, 2026 16:25
@solracsf solracsf merged commit 0c83a9d into master Jan 23, 2026
184 of 190 checks passed
@solracsf solracsf deleted the fixNoBaseComponentFound branch January 23, 2026 18:24
@nextcloud-bot nextcloud-bot mentioned this pull request Apr 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug feature: caldav Related to CalDAV internals

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RefreshWebcalService "Attempt to read property UID on null"

3 participants