-
Notifications
You must be signed in to change notification settings - Fork 350
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make sure a {DAV:}prop is not empty #1355
Conversation
ee03532
to
30450f0
Compare
30450f0
to
159cfc7
Compare
Codecov Report
@@ Coverage Diff @@
## master #1355 +/- ##
=========================================
Coverage 97.31% 97.31%
- Complexity 2806 2809 +3
=========================================
Files 174 174
Lines 7975 7975
=========================================
Hits 7761 7761
Misses 214 214
Continue to review full report at Codecov.
|
77eb320
to
43ea1d2
Compare
@DeepDiver1975 @staabm I rebased this. I looks good to me. Please review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me. but I cannot really judge.
Assigned myself so that I find this easily and followup. It would be nice to sort out the pending PRs in this repo and make a release in the next few days. |
Port sabre-io/xml#158 to the prop parser to sabre/dav Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
43ea1d2
to
c1da060
Compare
Port sabre-io/xml#158 to sabre/dav prop parser.
Is a multistatus propfind response by Alfresco. XMLReader or Sabre/XML is having troubles to parse the empty element.
Report: nextcloud/server#16471