Skip to content

[stable25] fix share roots always being marked as writable - #39671

Merged
nfebe merged 1 commit into
stable25from
manual/backport/39614
Aug 3, 2023
Merged

[stable25] fix share roots always being marked as writable #39671
nfebe merged 1 commit into
stable25from
manual/backport/39614

Conversation

@nfebe

@nfebenfebe commented Aug 2, 2023

Copy link
Copy Markdown
Contributor

Backport of #39614

@nfebenfebe changed the title Backport of #39614[stable26] [27] fix share roots always being marked as writable Aug 2, 2023
Comment threadlib/public/Files/DavUtil.php Fixed
@nfebenfebe changed the title [stable26] [27] fix share roots always being marked as writable [stable25] [27] fix share roots always being marked as writable Aug 2, 2023
@solracsfsolracsf changed the title [stable25] [27] fix share roots always being marked as writable [stable25] fix share roots always being marked as writable Aug 2, 2023
@solracsfsolracsf added this to the Nextcloud 25.0.10 milestone Aug 2, 2023

@nickvergessennickvergessen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't check the test file, but you can execute the test locally before pushing

Comment threadlib/public/Files/DavUtil.php Outdated
Comment threadlib/public/Files/DavUtil.php Outdated
@blizzzblizzz mentioned this pull request Aug 2, 2023
Signed-off-by: Robin Appelman <robin@icewind.nl>

@nickvergessennickvergessen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cherry-picked the original command from icewind and adjusted the : void in the test file which was the conflict.

@blizzzblizzz added 4. to release Ready to be released and/or waiting for tests to finish and removed backport-request labels Aug 2, 2023
@nfebe
nfebe enabled auto-merge August 2, 2023 23:37
@nfebe
nfebe disabled auto-merge August 3, 2023 07:16
@nfebe
nfebe merged commit 0f10006 into stable25Aug 3, 2023
@nfebe
nfebe deleted the manual/backport/39614 branch August 3, 2023 07:16
@nfebe

nfebe commented Aug 3, 2023

Copy link
Copy Markdown
ContributorAuthor

CI, fail is erratic and unrelated.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finish25-feedback

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@nfebe@nickvergessen@ArtificialOwl@AndyScherzinger@blizzz@github-advanced-security@solracsf@icewind1991