Skip to content

fix: include registered extra props in PROPFIND request body#2726

Open
flash7777 wants to merge 1 commit into
opencloud-eu:mainfrom
flash7777:fix/propfind-extra-props
Open

fix: include registered extra props in PROPFIND request body#2726
flash7777 wants to merge 1 commit into
opencloud-eu:mainfrom
flash7777:fix/propfind-extra-props

Conversation

@flash7777

Copy link
Copy Markdown

Summary

registerExtraProp() registered properties for response parsing but buildPropFindBody() never included them in the PROPFIND request XML. Extensions using registerExtraProp() would never receive their custom properties because the server was never asked for them.

Replaces #2721 (which had unrelated files).

Changes

File Change
builders.ts Include registered extra props in PROPFIND XML body

Test plan

  • Extension registers extra prop via registerExtraProp('oc:my.prop')
  • PROPFIND request XML now includes <oc:my.prop/>
  • Server returns the property value in response
  • Existing props (etag, fileid etc.) still work

registerExtraProp() registered properties for response parsing but
buildPropFindBody() never included them in the PROPFIND request XML.
Extensions using registerExtraProp() would never receive their
custom properties because the server was never asked for them.
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.

1 participant