Skip to content

fix(dav): properly handle files metadata#61057

Merged
sorbaugh merged 1 commit into
masterfrom
fix/dav-property
Jun 8, 2026
Merged

fix(dav): properly handle files metadata#61057
sorbaugh merged 1 commit into
masterfrom
fix/dav-property

Conversation

@susnux

@susnux susnux commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Summary

PropFind::handle expects a callable and fallsback to value, but if you pass a string like Date thats also a callable for PHP. So always put the value as the return value of a callable.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@susnux susnux requested a review from a team as a code owner June 7, 2026 15:15
@susnux susnux added bug 3. to review Waiting for reviews labels Jun 7, 2026
@susnux susnux requested review from Altahrim, artonge and salmart-dev and removed request for a team June 7, 2026 15:15
@susnux susnux requested a review from leftybournes June 7, 2026 15:15
@susnux susnux added this to the Nextcloud 35 milestone Jun 7, 2026
@susnux susnux requested a review from miaulalala June 7, 2026 15:15
@susnux

susnux commented Jun 7, 2026

Copy link
Copy Markdown
Contributor Author

/backport to stable32

@susnux

susnux commented Jun 7, 2026

Copy link
Copy Markdown
Contributor Author

/backport to stable33

@susnux

susnux commented Jun 7, 2026

Copy link
Copy Markdown
Contributor Author

/backport to stable34

@susnux susnux force-pushed the fix/dav-property branch from 618ca63 to eeb1196 Compare June 8, 2026 12:38
`PropFind::handle` expects a callable and fallsback to value,
but if you pass a string like `Date` thats also a callable for PHP.
So always put the value as the return value of a callable.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the fix/dav-property branch from eeb1196 to 9177e4e Compare June 8, 2026 12:59
@sorbaugh sorbaugh merged commit b4a2b67 into master Jun 8, 2026
192 of 210 checks passed
@sorbaugh sorbaugh deleted the fix/dav-property branch June 8, 2026 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Photos/WebDAV 500 error when metadata value photos-place.value is "Date"

4 participants