fix: share create permission naming

owncloud · Nov 13, 2023 · ef4766b · ef4766b
1 parent d51b377
commit ef4766b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/packages/web-runtime/src/services/auth/abilities.ts b/packages/web-runtime/src/services/auth/abilities.ts
@@ -37,7 +37,10 @@ export const getAbilities = (
       { action: 'read-all', subject: 'Role' },
       { action: 'update-all', subject: 'Role' }
     ],
-    'Share.Create.all': [{ action: 'create-all', subject: 'Share' }],
+    'Shares.Write.all': [
+      { action: 'create-all', subject: 'Share' },
+      { action: 'update-all', subject: 'Share' }
+    ],
     'Settings.ReadWrite.all': [
       { action: 'read-all', subject: 'Setting' },
       { action: 'update-all', subject: 'Setting' }

diff --git a/packages/web-runtime/tests/unit/services/auth/abilities.spec.ts b/packages/web-runtime/tests/unit/services/auth/abilities.spec.ts
@@ -45,8 +45,8 @@ describe('getAbilities', () => {
     expect(abilities).toEqual(expectedActions.map((action) => ({ action, subject: 'PublicLink' })))
   })
   it('gets correct abilities for subject "Share"', function () {
-    const abilities = getAbilities(['Share.Create.all'])
-    const expectedActions = ['create-all']
+    const abilities = getAbilities(['Shares.Write.all'])
+    const expectedActions = ['create-all', 'update-all']
     expect(abilities).toEqual(expectedActions.map((action) => ({ action, subject: 'Share' })))
   })
   it('gets correct abilities for subject "Setting"', function () {