Another fix for converting custom ocs permissions to roles

When creating (public) shares with the permissions read,update,create,delete we wrongly converted that into the `SpacerEditor` sharing role. The correct role for that would be `legacy`. Fixes: https://github.com/owncloud/enterprise/issues/6209
cs3org · Nov 15, 2023 · 0ed0cec · 0ed0cec
1 parent 9fa275e
commit 0ed0cec
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/changelog/unreleased/fix-roleconversion-custompermissions.md b/changelog/unreleased/fix-roleconversion-custompermissions.md
@@ -4,4 +4,5 @@ When creating shares with just `view` permission we wrongly converted that
 into the `SpacerViewer` sharing role. The correct role for that would be `legacy`.
 
 https://github.com/cs3org/reva/pull/4342
+https://github.com/cs3org/reva/pull/4343
 https://github.com/owncloud/enterprise/issues/6209
diff --git a/pkg/conversions/role.go b/pkg/conversions/role.go
@@ -389,7 +389,9 @@ func RoleFromOCSPermissions(p Permissions, ri *provider.ResourceInfo) *Role {
 				return NewEditorRole(true)
 			}
 
-			return NewSpaceEditorRole()
+			if isSpaceRoot(ri) {
+				return NewSpaceEditorRole()
+			}
 		}
 
 		if p == PermissionRead && isSpaceRoot(ri) {