cozy-sharing 38.1.0 → 38.1.1

package/CHANGELOG.md

@@ -3,6 +3,13 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [38.1.1](https://github.com/cozy/cozy-libs/compare/cozy-sharing@38.1.0...cozy-sharing@38.1.1) (2026-09-21)
+
+### Bug Fixes
+
+- **sharing:** Keep full share modal for nested-folder editors ([014c848](https://github.com/cozy/cozy-libs/commit/014c848bb46699cee47eb4baf8bb7a365edaf2f1))
+- **sharing:** Show editable share modal for members with write access ([cba3a13](https://github.com/cozy/cozy-libs/commit/cba3a13185d1fe088136c3a758ec94f0d3472b69))
+
 # [38.1.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@38.0.0...cozy-sharing@38.1.0) (2026-09-16)
 
 ### Features

package/dist/components/FederatedFolder/FederatedFolderModal.js

@@ -64,6 +64,7 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
 
   var _useSharingContext = useSharingContext(),
       canReshare = _useSharingContext.canReshare,
+      hasWriteAccess = _useSharingContext.hasWriteAccess,
       share = _useSharingContext.share,
       getSharingById = _useSharingContext.getSharingById,
       getSharingLink = _useSharingContext.getSharingLink,
@@ -125,7 +126,7 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
   }));
   var isInsideSharedDrive = Boolean((existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.driveId) && !isSharedDriveRoot);
   var isCurrentUserOwner = existingDocument !== null && existingDocument !== void 0 && existingDocument.driveId ? Boolean(sharedDriveSharing === null || sharedDriveSharing === void 0 || (_sharedDriveSharing$a2 = sharedDriveSharing.attributes) === null || _sharedDriveSharing$a2 === void 0 ? void 0 : _sharedDriveSharing$a2.owner) : documentId ? isOwner(documentId) : false;
-  var isMemberReadOnly = isInsideSharedDrive && !isCurrentUserOwner;
+  var isMemberReadOnly = isInsideSharedDrive && !isCurrentUserOwner && !hasWriteAccess(documentId, existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.driveId);
   var canManageSharing = isCurrentUserOwner || (documentId ? canReshare(documentId) : false);
 
   var handleRevokeSelf = /*#__PURE__*/function () {
@@ -318,7 +319,7 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
       className: "u-ph-2"
     }, /*#__PURE__*/React.createElement(AntivirusAlert, {
       document: isSending ? frozenDoc : existingDocument
-    }), isInsideSharedDrive ? /*#__PURE__*/React.createElement("div", {
+    }), isMemberReadOnly ? /*#__PURE__*/React.createElement("div", {
       className: styles['share-byemail-onlybylink']
     }, t('Files.share.shareByEmail.onlyByLink', {
       type: t("Files.share.shareByEmail.type.".concat((existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.type) === 'file' ? 'file' : 'folder'))
@@ -354,7 +355,7 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
       documentType: "Files",
       showGenerateLinkButton: showGenerateLinkButton,
       autoOpenShareRestriction: autoOpenShareRestriction
-    }), !isInsideSharedDrive && /*#__PURE__*/React.createElement(Button, {
+    }), !isMemberReadOnly && /*#__PURE__*/React.createElement(Button, {
       variant: "primary",
       label: t('FederatedFolder.share'),
       busy: isSending,

package/dist/components/FederatedFolder/FederatedFolderModal.spec.js

@@ -20,6 +20,7 @@ var mockRevoke = jest.fn();
 var mockRevokeSelf = jest.fn();
 var mockIsOwner = jest.fn();
 var mockCanReshare = jest.fn();
+var mockHasWriteAccess = jest.fn();
 var mockGetSharingLink = jest.fn();
 var mockGetFederatedShareLink = jest.fn();
 var mockGetSharingById = jest.fn();
@@ -42,6 +43,7 @@ jest.mock('../../hooks/useSharingContext', function () {
         revokeSelf: mockRevokeSelf,
         isOwner: mockIsOwner,
         canReshare: mockCanReshare,
+        hasWriteAccess: mockHasWriteAccess,
         getSharingLink: mockGetSharingLink,
         getFederatedShareLink: mockGetFederatedShareLink,
         getDocumentPermissions: mockGetDocumentPermissions,
@@ -142,6 +144,7 @@ describe('FederatedFolderModal', function () {
     mockRevokeSelf.mockResolvedValue();
     mockIsOwner.mockReturnValue(false);
     mockCanReshare.mockReturnValue(false);
+    mockHasWriteAccess.mockReturnValue(false);
     mockGetDocumentPermissions.mockReturnValue([]);
     mockFetchSharedDriveSharingLinks.mockResolvedValue([]);
     mockGetSharingLink.mockReturnValue('https://example.com/share/abc123');
@@ -319,11 +322,40 @@ describe('FederatedFolderModal', function () {
         }
       }, _callee6);
     })));
-    it('should allow share by email when document is the federated shared folder root', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee7() {
+    it('should allow share by email when the current user has write access, even inside a federated shared folder', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee7() {
       var _setup7, findByText, queryByText;
 
       return _regeneratorRuntime.wrap(function _callee7$(_context7) {
         while (1) switch (_context7.prev = _context7.next) {
+          case 0:
+            mockHasWriteAccess.mockReturnValue(true);
+            _setup7 = setup({
+              document: _objectSpread(_objectSpread({}, mockDocument), {}, {
+                driveId: 'federated-folder-id',
+                type: 'directory'
+              })
+            }), findByText = _setup7.findByText, queryByText = _setup7.queryByText;
+            _context7.next = 4;
+            return findByText('Add users');
+
+          case 4:
+            _context7.next = 6;
+            return findByText('Done');
+
+          case 6:
+            expect(queryByText('This folder can only be shared by link, because')).toBe(null);
+
+          case 7:
+          case "end":
+            return _context7.stop();
+        }
+      }, _callee7);
+    })));
+    it('should allow share by email when document is the federated shared folder root', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee8() {
+      var _setup8, findByText, queryByText;
+
+      return _regeneratorRuntime.wrap(function _callee8$(_context8) {
+        while (1) switch (_context8.prev = _context8.next) {
           case 0:
             mockGetSharingById.mockReturnValue({
               attributes: {
@@ -334,13 +366,13 @@ describe('FederatedFolderModal', function () {
                 }]
               }
             });
-            _setup7 = setup({
+            _setup8 = setup({
               document: _objectSpread(_objectSpread({}, mockDocument), {}, {
                 driveId: 'federated-folder-id',
                 type: 'directory'
               })
-            }), findByText = _setup7.findByText, queryByText = _setup7.queryByText;
-            _context7.next = 4;
+            }), findByText = _setup8.findByText, queryByText = _setup8.queryByText;
+            _context8.next = 4;
             return findByText('Add users');
 
           case 4:
@@ -349,15 +381,15 @@ describe('FederatedFolderModal', function () {
 
           case 6:
           case "end":
-            return _context7.stop();
+            return _context8.stop();
         }
-      }, _callee7);
+      }, _callee8);
     })));
   });
   describe('shared drive link fetch on mount', function () {
-    it('should fetch shared drive sharing links when document has a driveId and no permissions yet', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee8() {
-      return _regeneratorRuntime.wrap(function _callee8$(_context8) {
-        while (1) switch (_context8.prev = _context8.next) {
+    it('should fetch shared drive sharing links when document has a driveId and no permissions yet', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee9() {
+      return _regeneratorRuntime.wrap(function _callee9$(_context9) {
+        while (1) switch (_context9.prev = _context9.next) {
           case 0:
             mockGetDocumentPermissions.mockReturnValue([]);
             setup({
@@ -365,7 +397,7 @@ describe('FederatedFolderModal', function () {
                 driveId: 'federated-folder-id'
               })
             });
-            _context8.next = 4;
+            _context9.next = 4;
             return waitFor(function () {
               expect(mockFetchSharedDriveSharingLinks).toHaveBeenCalledTimes(1);
             });
@@ -378,14 +410,14 @@ describe('FederatedFolderModal', function () {
 
           case 5:
           case "end":
-            return _context8.stop();
+            return _context9.stop();
         }
-      }, _callee8);
+      }, _callee9);
     })));
-    it('should use document.id when _id is missing', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee9() {
+    it('should use document.id when _id is missing', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10() {
       var document;
-      return _regeneratorRuntime.wrap(function _callee9$(_context9) {
-        while (1) switch (_context9.prev = _context9.next) {
+      return _regeneratorRuntime.wrap(function _callee10$(_context10) {
+        while (1) switch (_context10.prev = _context10.next) {
           case 0:
             document = {
               id: 'folder-id-only',
@@ -396,7 +428,7 @@ describe('FederatedFolderModal', function () {
             setup({
               document: document
             });
-            _context9.next = 4;
+            _context10.next = 4;
             return waitFor(function () {
               expect(mockFetchSharedDriveSharingLinks).toHaveBeenCalledWith(document);
             });
@@ -406,20 +438,20 @@ describe('FederatedFolderModal', function () {
 
           case 5:
           case "end":
-            return _context9.stop();
+            return _context10.stop();
         }
-      }, _callee9);
+      }, _callee10);
     })));
-    it('should not fetch shared drive sharing links when document has no driveId', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10() {
-      var _setup8, findByText;
+    it('should not fetch shared drive sharing links when document has no driveId', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11() {
+      var _setup9, findByText;
 
-      return _regeneratorRuntime.wrap(function _callee10$(_context10) {
-        while (1) switch (_context10.prev = _context10.next) {
+      return _regeneratorRuntime.wrap(function _callee11$(_context11) {
+        while (1) switch (_context11.prev = _context11.next) {
           case 0:
-            _setup8 = setup({
+            _setup9 = setup({
               document: mockDocument
-            }), findByText = _setup8.findByText;
-            _context10.next = 3;
+            }), findByText = _setup9.findByText;
+            _context11.next = 3;
             return findByText('Copy link');
 
           case 3:
@@ -427,23 +459,23 @@ describe('FederatedFolderModal', function () {
 
           case 4:
           case "end":
-            return _context10.stop();
+            return _context11.stop();
         }
-      }, _callee10);
+      }, _callee11);
     })));
-    it('should not refetch after one attempt when permissions stay empty', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11() {
-      var document, _setup9, rerender;
+    it('should not refetch after one attempt when permissions stay empty', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12() {
+      var document, _setup10, rerender;
 
-      return _regeneratorRuntime.wrap(function _callee11$(_context11) {
-        while (1) switch (_context11.prev = _context11.next) {
+      return _regeneratorRuntime.wrap(function _callee12$(_context12) {
+        while (1) switch (_context12.prev = _context12.next) {
           case 0:
             document = _objectSpread(_objectSpread({}, mockDocument), {}, {
               driveId: 'federated-folder-id'
             });
-            _setup9 = setup({
+            _setup10 = setup({
               document: document
-            }), rerender = _setup9.rerender;
-            _context11.next = 4;
+            }), rerender = _setup10.rerender;
+            _context12.next = 4;
             return waitFor(function () {
               expect(mockFetchSharedDriveSharingLinks).toHaveBeenCalledTimes(1);
             });
@@ -460,15 +492,15 @@ describe('FederatedFolderModal', function () {
 
           case 6:
           case "end":
-            return _context11.stop();
+            return _context12.stop();
         }
-      }, _callee11);
+      }, _callee12);
     })));
-    it('should not fetch shared drive sharing links when permissions are already loaded', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12() {
-      var _setup10, findByText;
+    it('should not fetch shared drive sharing links when permissions are already loaded', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13() {
+      var _setup11, findByText;
 
-      return _regeneratorRuntime.wrap(function _callee12$(_context12) {
-        while (1) switch (_context12.prev = _context12.next) {
+      return _regeneratorRuntime.wrap(function _callee13$(_context13) {
+        while (1) switch (_context13.prev = _context13.next) {
           case 0:
             mockGetDocumentPermissions.mockReturnValue([{
               id: 'perm-1',
@@ -479,12 +511,12 @@ describe('FederatedFolderModal', function () {
                 }
               }
             }]);
-            _setup10 = setup({
+            _setup11 = setup({
               document: _objectSpread(_objectSpread({}, mockDocument), {}, {
                 driveId: 'federated-folder-id'
               })
-            }), findByText = _setup10.findByText;
-            _context12.next = 4;
+            }), findByText = _setup11.findByText;
+            _context13.next = 4;
             return findByText('Copy link');
 
           case 4:
@@ -492,9 +524,9 @@ describe('FederatedFolderModal', function () {
 
           case 5:
           case "end":
-            return _context12.stop();
+            return _context13.stop();
         }
-      }, _callee12);
+      }, _callee13);
     })));
   });
   describe('onSend callback', function () {
@@ -502,21 +534,21 @@ describe('FederatedFolderModal', function () {
       name: 'Alice',
       email: 'alice@example.com'
     };
-    it('should call onClose and skip share when there are no pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13() {
-      var _setup11, findByText, sendButton;
+    it('should call onClose and skip share when there are no pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14() {
+      var _setup12, findByText, sendButton;
 
-      return _regeneratorRuntime.wrap(function _callee13$(_context13) {
-        while (1) switch (_context13.prev = _context13.next) {
+      return _regeneratorRuntime.wrap(function _callee14$(_context14) {
+        while (1) switch (_context14.prev = _context14.next) {
           case 0:
             // pendingRecipients defaults to [] via beforeEach
-            _setup11 = setup(), findByText = _setup11.findByText;
-            _context13.next = 3;
+            _setup12 = setup(), findByText = _setup12.findByText;
+            _context14.next = 3;
             return findByText('Done');
 
           case 3:
-            sendButton = _context13.sent;
+            sendButton = _context14.sent;
             fireEvent.click(sendButton);
-            _context13.next = 7;
+            _context14.next = 7;
             return waitFor(function () {
               expect(mockOnClose).toHaveBeenCalled();
               expect(mockShare).not.toHaveBeenCalled();
@@ -524,15 +556,15 @@ describe('FederatedFolderModal', function () {
 
           case 7:
           case "end":
-            return _context13.stop();
+            return _context14.stop();
         }
-      }, _callee13);
+      }, _callee14);
     })));
-    it('should call share with correct parameters', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14() {
-      var _setup12, findByText, sendButton;
+    it('should call share with correct parameters', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15() {
+      var _setup13, findByText, sendButton;
 
-      return _regeneratorRuntime.wrap(function _callee14$(_context14) {
-        while (1) switch (_context14.prev = _context14.next) {
+      return _regeneratorRuntime.wrap(function _callee15$(_context15) {
+        while (1) switch (_context15.prev = _context15.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [pendingRecipient],
@@ -542,14 +574,14 @@ describe('FederatedFolderModal', function () {
             });
             getOrCreateFromArray.mockResolvedValue([pendingRecipient]);
             mockShare.mockResolvedValueOnce({});
-            _setup12 = setup(), findByText = _setup12.findByText;
-            _context14.next = 6;
+            _setup13 = setup(), findByText = _setup13.findByText;
+            _context15.next = 6;
             return findByText('Done');
 
           case 6:
-            sendButton = _context14.sent;
+            sendButton = _context15.sent;
             fireEvent.click(sendButton);
-            _context14.next = 10;
+            _context15.next = 10;
             return waitFor(function () {
               expect(mockShare).toHaveBeenCalledWith({
                 description: 'My Test Folder',
@@ -563,15 +595,15 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context14.stop();
+            return _context15.stop();
         }
-      }, _callee14);
+      }, _callee15);
     })));
-    it('should show success alert and close modal on successful share', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15() {
-      var _setup13, findByText, sendButton;
+    it('should show success alert and close modal on successful share', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16() {
+      var _setup14, findByText, sendButton;
 
-      return _regeneratorRuntime.wrap(function _callee15$(_context15) {
-        while (1) switch (_context15.prev = _context15.next) {
+      return _regeneratorRuntime.wrap(function _callee16$(_context16) {
+        while (1) switch (_context16.prev = _context16.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [pendingRecipient],
@@ -581,14 +613,14 @@ describe('FederatedFolderModal', function () {
             });
             getOrCreateFromArray.mockResolvedValue([pendingRecipient]);
             mockShare.mockResolvedValueOnce({});
-            _setup13 = setup(), findByText = _setup13.findByText;
-            _context15.next = 6;
+            _setup14 = setup(), findByText = _setup14.findByText;
+            _context16.next = 6;
             return findByText('Done');
 
           case 6:
-            sendButton = _context15.sent;
+            sendButton = _context16.sent;
             fireEvent.click(sendButton);
-            _context15.next = 10;
+            _context16.next = 10;
             return waitFor(function () {
               expect(mockShowAlert).toHaveBeenCalledWith({
                 message: 'Folder has been shared',
@@ -600,15 +632,15 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context15.stop();
+            return _context16.stop();
         }
-      }, _callee15);
+      }, _callee16);
     })));
-    it('should show error alert when share fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16() {
-      var _setup14, findByText, sendButton;
+    it('should show error alert when share fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee17() {
+      var _setup15, findByText, sendButton;
 
-      return _regeneratorRuntime.wrap(function _callee16$(_context16) {
-        while (1) switch (_context16.prev = _context16.next) {
+      return _regeneratorRuntime.wrap(function _callee17$(_context17) {
+        while (1) switch (_context17.prev = _context17.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [pendingRecipient],
@@ -618,14 +650,14 @@ describe('FederatedFolderModal', function () {
             });
             getOrCreateFromArray.mockResolvedValue([pendingRecipient]);
             mockShare.mockRejectedValueOnce(new Error('Share failed'));
-            _setup14 = setup(), findByText = _setup14.findByText;
-            _context16.next = 6;
+            _setup15 = setup(), findByText = _setup15.findByText;
+            _context17.next = 6;
             return findByText('Done');
 
           case 6:
-            sendButton = _context16.sent;
+            sendButton = _context17.sent;
             fireEvent.click(sendButton);
-            _context16.next = 10;
+            _context17.next = 10;
             return waitFor(function () {
               expect(mockShowAlert).toHaveBeenCalledWith({
                 message: 'Error while sharing folder',
@@ -637,15 +669,15 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context16.stop();
+            return _context17.stop();
         }
-      }, _callee16);
+      }, _callee17);
     })));
-    it('should show progress icon on Done button while share is in progress', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee18() {
-      var resolveShare, _setup15, findByText, getByRole, queryByRole, sendButton;
+    it('should show progress icon on Done button while share is in progress', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee19() {
+      var resolveShare, _setup16, findByText, getByRole, queryByRole, sendButton;
 
-      return _regeneratorRuntime.wrap(function _callee18$(_context18) {
-        while (1) switch (_context18.prev = _context18.next) {
+      return _regeneratorRuntime.wrap(function _callee19$(_context19) {
+        while (1) switch (_context19.prev = _context19.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [pendingRecipient],
@@ -657,14 +689,14 @@ describe('FederatedFolderModal', function () {
             mockShare.mockReturnValueOnce(new Promise(function (resolve) {
               resolveShare = resolve;
             }));
-            _setup15 = setup(), findByText = _setup15.findByText, getByRole = _setup15.getByRole, queryByRole = _setup15.queryByRole;
-            _context18.next = 6;
+            _setup16 = setup(), findByText = _setup16.findByText, getByRole = _setup16.getByRole, queryByRole = _setup16.queryByRole;
+            _context19.next = 6;
             return findByText('Done');
 
           case 6:
-            sendButton = _context18.sent;
+            sendButton = _context19.sent;
             fireEvent.click(sendButton);
-            _context18.next = 10;
+            _context19.next = 10;
             return waitFor(function () {
               expect(getByRole('button', {
                 name: 'Done'
@@ -673,25 +705,25 @@ describe('FederatedFolderModal', function () {
             });
 
           case 10:
-            _context18.next = 12;
-            return act( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee17() {
-              return _regeneratorRuntime.wrap(function _callee17$(_context17) {
-                while (1) switch (_context17.prev = _context17.next) {
+            _context19.next = 12;
+            return act( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee18() {
+              return _regeneratorRuntime.wrap(function _callee18$(_context18) {
+                while (1) switch (_context18.prev = _context18.next) {
                   case 0:
                     resolveShare({});
 
                   case 1:
                   case "end":
-                    return _context17.stop();
+                    return _context18.stop();
                 }
-              }, _callee17);
+              }, _callee18);
             })));
 
           case 12:
           case "end":
-            return _context18.stop();
+            return _context19.stop();
         }
-      }, _callee18);
+      }, _callee19);
     })));
   });
   describe('onRevoke callback', function () {
@@ -704,27 +736,27 @@ describe('FederatedFolderModal', function () {
       name: 'Me',
       email: 'me@test.com'
     };
-    it('should call onRevokeSelf and onRevokeSuccess when the current user is revoked', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee19() {
-      var onRevokeSuccess, _setup16, findByLabelText;
+    it('should call onRevokeSelf and onRevokeSuccess when the current user is revoked', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee20() {
+      var onRevokeSuccess, _setup17, findByLabelText;
 
-      return _regeneratorRuntime.wrap(function _callee19$(_context19) {
-        while (1) switch (_context19.prev = _context19.next) {
+      return _regeneratorRuntime.wrap(function _callee20$(_context20) {
+        while (1) switch (_context20.prev = _context20.next) {
           case 0:
             onRevokeSuccess = jest.fn();
             mockGetEffectiveRecipients.mockReturnValue([currentUserRecipient]);
-            _setup16 = setup({
+            _setup17 = setup({
               onRevokeSuccess: onRevokeSuccess
-            }), findByLabelText = _setup16.findByLabelText;
-            _context19.t0 = fireEvent;
-            _context19.next = 6;
+            }), findByLabelText = _setup17.findByLabelText;
+            _context20.t0 = fireEvent;
+            _context20.next = 6;
             return findByLabelText('Remove from sharing');
 
           case 6:
-            _context19.t1 = _context19.sent;
+            _context20.t1 = _context20.sent;
 
-            _context19.t0.click.call(_context19.t0, _context19.t1);
+            _context20.t0.click.call(_context20.t0, _context20.t1);
 
-            _context19.next = 10;
+            _context20.next = 10;
             return waitFor(function () {
               expect(mockRevokeSelf).toHaveBeenCalledWith(mockDocument);
               expect(mockRevoke).not.toHaveBeenCalled();
@@ -733,15 +765,15 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context19.stop();
+            return _context20.stop();
         }
-      }, _callee19);
+      }, _callee20);
     })));
-    it('should allow a write recipient to revoke another recipient', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee20() {
-      var otherRecipient, _setup17, findByLabelText;
+    it('should allow a write recipient to revoke another recipient', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee21() {
+      var otherRecipient, _setup18, findByLabelText;
 
-      return _regeneratorRuntime.wrap(function _callee20$(_context20) {
-        while (1) switch (_context20.prev = _context20.next) {
+      return _regeneratorRuntime.wrap(function _callee21$(_context21) {
+        while (1) switch (_context21.prev = _context21.next) {
           case 0:
             otherRecipient = {
               instance: 'https://other.mycozy.cloud',
@@ -754,17 +786,17 @@ describe('FederatedFolderModal', function () {
             };
             mockCanReshare.mockReturnValue(true);
             mockGetEffectiveRecipients.mockReturnValue([otherRecipient]);
-            _setup17 = setup(), findByLabelText = _setup17.findByLabelText;
-            _context20.t0 = fireEvent;
-            _context20.next = 7;
+            _setup18 = setup(), findByLabelText = _setup18.findByLabelText;
+            _context21.t0 = fireEvent;
+            _context21.next = 7;
             return findByLabelText('Remove from sharing');
 
           case 7:
-            _context20.t1 = _context20.sent;
+            _context21.t1 = _context21.sent;
 
-            _context20.t0.click.call(_context20.t0, _context20.t1);
+            _context21.t0.click.call(_context21.t0, _context21.t1);
 
-            _context20.next = 11;
+            _context21.next = 11;
             return waitFor(function () {
               expect(mockCanReshare).toHaveBeenCalledWith(mockDocument._id);
               expect(mockRevoke).toHaveBeenCalledWith(mockDocument, otherRecipient.sharingId, otherRecipient.memberIndex);
@@ -773,40 +805,40 @@ describe('FederatedFolderModal', function () {
 
           case 11:
           case "end":
-            return _context20.stop();
+            return _context21.stop();
         }
-      }, _callee20);
+      }, _callee21);
     })));
   });
   describe('onClose callback', function () {
-    it('should call onClose when close button is clicked', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee21() {
-      var _setup18, findByRole, closeButton;
+    it('should call onClose when close button is clicked', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee22() {
+      var _setup19, findByRole, closeButton;
 
-      return _regeneratorRuntime.wrap(function _callee21$(_context21) {
-        while (1) switch (_context21.prev = _context21.next) {
+      return _regeneratorRuntime.wrap(function _callee22$(_context22) {
+        while (1) switch (_context22.prev = _context22.next) {
           case 0:
-            _setup18 = setup(), findByRole = _setup18.findByRole;
-            _context21.next = 3;
+            _setup19 = setup(), findByRole = _setup19.findByRole;
+            _context22.next = 3;
             return findByRole('button', {
               name: /close/i
             });
 
           case 3:
-            closeButton = _context21.sent;
+            closeButton = _context22.sent;
             fireEvent.click(closeButton);
             expect(mockOnClose).toHaveBeenCalled();
 
           case 6:
           case "end":
-            return _context21.stop();
+            return _context22.stop();
         }
-      }, _callee21);
+      }, _callee22);
     })));
-    it('should ask confirmation when close button is clicked with pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee22() {
-      var _setup19, getByRole, findByRole, getByText, closeButton;
+    it('should ask confirmation when close button is clicked with pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee23() {
+      var _setup20, getByRole, findByRole, getByText, closeButton;
 
-      return _regeneratorRuntime.wrap(function _callee22$(_context22) {
-        while (1) switch (_context22.prev = _context22.next) {
+      return _regeneratorRuntime.wrap(function _callee23$(_context23) {
+        while (1) switch (_context23.prev = _context23.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [{
@@ -817,14 +849,14 @@ describe('FederatedFolderModal', function () {
               selectedOption: 'readWrite',
               setSelectedOption: jest.fn()
             });
-            _setup19 = setup(), getByRole = _setup19.getByRole, findByRole = _setup19.findByRole, getByText = _setup19.getByText;
-            _context22.next = 4;
+            _setup20 = setup(), getByRole = _setup20.getByRole, findByRole = _setup20.findByRole, getByText = _setup20.getByText;
+            _context23.next = 4;
             return findByRole('button', {
               name: /close/i
             });
 
           case 4:
-            closeButton = _context22.sent;
+            closeButton = _context23.sent;
             fireEvent.click(closeButton);
             expect(getByText("Discard the changes you haven't saved?")).toBeTruthy();
             expect(mockOnClose).not.toHaveBeenCalled();
@@ -835,15 +867,15 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context22.stop();
+            return _context23.stop();
         }
-      }, _callee22);
+      }, _callee23);
     })));
-    it('should close after discard confirmation', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee23() {
-      var _setup20, getByRole, findByRole, getByText, closeButton;
+    it('should close after discard confirmation', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee24() {
+      var _setup21, getByRole, findByRole, getByText, closeButton;
 
-      return _regeneratorRuntime.wrap(function _callee23$(_context23) {
-        while (1) switch (_context23.prev = _context23.next) {
+      return _regeneratorRuntime.wrap(function _callee24$(_context24) {
+        while (1) switch (_context24.prev = _context24.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [{
@@ -854,14 +886,14 @@ describe('FederatedFolderModal', function () {
               selectedOption: 'readWrite',
               setSelectedOption: jest.fn()
             });
-            _setup20 = setup(), getByRole = _setup20.getByRole, findByRole = _setup20.findByRole, getByText = _setup20.getByText;
-            _context23.next = 4;
+            _setup21 = setup(), getByRole = _setup21.getByRole, findByRole = _setup21.findByRole, getByText = _setup21.getByText;
+            _context24.next = 4;
             return findByRole('button', {
               name: /close/i
             });
 
           case 4:
-            closeButton = _context23.sent;
+            closeButton = _context24.sent;
             fireEvent.click(closeButton);
             expect(getByText("Discard the changes you haven't saved?")).toBeTruthy();
             fireEvent.click(getByRole('button', {
@@ -871,9 +903,9 @@ describe('FederatedFolderModal', function () {
 
           case 9:
           case "end":
-            return _context23.stop();
+            return _context24.stop();
         }
-      }, _callee23);
+      }, _callee24);
     })));
   });
 });

package/dist/components/ShareModal/ShareModal.js

@@ -27,7 +27,7 @@ export var ShareModal = withLocales(function (props) {
   var _useSharingContext = useSharingContext(),
       byDocId = _useSharingContext.byDocId,
       isOwner = _useSharingContext.isOwner,
-      canReshare = _useSharingContext.canReshare,
+      hasWriteAccess = _useSharingContext.hasWriteAccess,
       documentType = _useSharingContext.documentType,
       getRecipients = _useSharingContext.getRecipients,
       revokeSelf = _useSharingContext.revokeSelf,
@@ -56,7 +56,7 @@ export var ShareModal = withLocales(function (props) {
     };
   }();
 
-  var isEditable = !byDocId[document.id] || isOwner(document.id) || canReshare(document.id);
+  var isEditable = !byDocId[document.id] || isOwner(document.id) || hasWriteAccess(document.id, document.driveId);
 
   if (isEditable) {
     var isFederatedMode = flag('drive.federated-shared-folder.enabled');

package/dist/components/ShareModal/ShareModal.spec.js

@@ -0,0 +1,79 @@
+import { render } from '@testing-library/react';
+import React from 'react';
+import { MemoryRouter } from 'react-router';
+import { ShareModal } from './ShareModal';
+var mockIsOwner = jest.fn();
+var mockHasWriteAccess = jest.fn();
+jest.mock('cozy-flags');
+jest.mock('../../hoc/withLocales', function () {
+  return function (Component) {
+    return Component;
+  };
+});
+jest.mock('../../hooks/useSharingContext', function () {
+  return {
+    useSharingContext: function useSharingContext() {
+      return {
+        byDocId: {
+          'doc-1': {
+            sharings: ['sharing-1']
+          }
+        },
+        isOwner: mockIsOwner,
+        hasWriteAccess: mockHasWriteAccess,
+        documentType: 'Files',
+        getRecipients: jest.fn().mockReturnValue([]),
+        revokeSelf: jest.fn(),
+        allLoaded: false
+      };
+    }
+  };
+});
+jest.mock('./EditableSharingModal', function () {
+  return function () {
+    return /*#__PURE__*/React.createElement("div", null, "editable-modal");
+  };
+});
+jest.mock('./SharingDetailsModal', function () {
+  return function () {
+    return /*#__PURE__*/React.createElement("div", null, "details-modal");
+  };
+});
+jest.mock('../FederatedFolder/FederatedFolderModal', function () {
+  return function () {
+    return /*#__PURE__*/React.createElement("div", null, "federated-modal");
+  };
+});
+var mockDocument = {
+  id: 'doc-1',
+  driveId: undefined
+};
+describe('ShareModal dispatcher', function () {
+  beforeEach(function () {
+    mockIsOwner.mockReset();
+    mockHasWriteAccess.mockReset();
+  });
+  it('renders the editable modal when the current user has write access, even without reshare rights', function () {
+    mockIsOwner.mockReturnValue(false);
+    mockHasWriteAccess.mockReturnValue(true);
+
+    var _render = render( /*#__PURE__*/React.createElement(MemoryRouter, null, /*#__PURE__*/React.createElement(ShareModal, {
+      document: mockDocument
+    }))),
+        getByText = _render.getByText;
+
+    expect(getByText('editable-modal')).toBeTruthy();
+    expect(mockHasWriteAccess).toHaveBeenCalledWith('doc-1', undefined);
+  });
+  it('renders the restricted details modal when the current user has read-only access', function () {
+    mockIsOwner.mockReturnValue(false);
+    mockHasWriteAccess.mockReturnValue(false);
+
+    var _render2 = render( /*#__PURE__*/React.createElement(MemoryRouter, null, /*#__PURE__*/React.createElement(ShareModal, {
+      document: mockDocument
+    }))),
+        getByText = _render2.getByText;
+
+    expect(getByText('details-modal')).toBeTruthy();
+  });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-sharing",
-  "version": "38.1.0",
+  "version": "38.1.1",
   "description": "Provides sharing login for React applications.",
   "main": "dist/index.js",
   "author": "Cozy",
@@ -59,7 +59,7 @@
     "cozy-intent": "^2.31.1",
     "cozy-minilog": "^3.10.1",
     "cozy-ui": "^140.5.0",
-    "cozy-ui-plus": "^13.0.4",
+    "cozy-ui-plus": "^13.0.5",
     "jest": "30.3.0",
     "jest-environment-jsdom": "30.3.0",
     "react": "16.12.0",
@@ -85,5 +85,5 @@
   "sideEffects": [
     "*.css"
   ],
-  "gitHead": "96598a165eae6c8eef3da4b6451af1dfdb8801a0"
+  "gitHead": "368b1e85eb1f6a6a0731681ef2a38a7bbc3fe779"
 }

package/src/components/FederatedFolder/FederatedFolderModal.jsx

@@ -42,6 +42,7 @@ const FederatedFolderModalContent = ({
   const { t } = useI18n()
   const {
     canReshare,
+    hasWriteAccess,
     share,
     getSharingById,
     getSharingLink,
@@ -99,7 +100,10 @@ const FederatedFolderModalContent = ({
     : documentId
       ? isOwner(documentId)
       : false
-  const isMemberReadOnly = isInsideSharedDrive && !isCurrentUserOwner
+  const isMemberReadOnly =
+    isInsideSharedDrive &&
+    !isCurrentUserOwner &&
+    !hasWriteAccess(documentId, existingDocument?.driveId)
   const canManageSharing =
     isCurrentUserOwner || (documentId ? canReshare(documentId) : false)
 
@@ -239,7 +243,7 @@ const FederatedFolderModalContent = ({
             <AntivirusAlert
               document={isSending ? frozenDoc : existingDocument}
             />
-            {isInsideSharedDrive ? (
+            {isMemberReadOnly ? (
               <div className={styles['share-byemail-onlybylink']}>
                 {t('Files.share.shareByEmail.onlyByLink', {
                   type: t(
@@ -293,7 +297,7 @@ const FederatedFolderModalContent = ({
             showGenerateLinkButton={showGenerateLinkButton}
             autoOpenShareRestriction={autoOpenShareRestriction}
           />
-          {!isInsideSharedDrive && (
+          {!isMemberReadOnly && (
             <Button
               variant="primary"
               label={t('FederatedFolder.share')}

package/src/components/FederatedFolder/FederatedFolderModal.spec.jsx

@@ -14,6 +14,7 @@ const mockRevoke = jest.fn()
 const mockRevokeSelf = jest.fn()
 const mockIsOwner = jest.fn()
 const mockCanReshare = jest.fn()
+const mockHasWriteAccess = jest.fn()
 const mockGetSharingLink = jest.fn()
 const mockGetFederatedShareLink = jest.fn()
 const mockGetSharingById = jest.fn()
@@ -34,6 +35,7 @@ jest.mock('../../hooks/useSharingContext', () => ({
     revokeSelf: mockRevokeSelf,
     isOwner: mockIsOwner,
     canReshare: mockCanReshare,
+    hasWriteAccess: mockHasWriteAccess,
     getSharingLink: mockGetSharingLink,
     getFederatedShareLink: mockGetFederatedShareLink,
     getDocumentPermissions: mockGetDocumentPermissions,
@@ -115,6 +117,7 @@ describe('FederatedFolderModal', () => {
     mockRevokeSelf.mockResolvedValue()
     mockIsOwner.mockReturnValue(false)
     mockCanReshare.mockReturnValue(false)
+    mockHasWriteAccess.mockReturnValue(false)
     mockGetDocumentPermissions.mockReturnValue([])
     mockFetchSharedDriveSharingLinks.mockResolvedValue([])
     mockGetSharingLink.mockReturnValue('https://example.com/share/abc123')
@@ -224,6 +227,25 @@ describe('FederatedFolderModal', () => {
       await findByText('it has a shared parent')
     })
 
+    it('should allow share by email when the current user has write access, even inside a federated shared folder', async () => {
+      mockHasWriteAccess.mockReturnValue(true)
+
+      const { findByText, queryByText } = setup({
+        document: {
+          ...mockDocument,
+          driveId: 'federated-folder-id',
+          type: 'directory'
+        }
+      })
+
+      await findByText('Add users')
+      await findByText('Done')
+
+      expect(
+        queryByText('This folder can only be shared by link, because')
+      ).toBe(null)
+    })
+
     it('should allow share by email when document is the federated shared folder root', async () => {
       mockGetSharingById.mockReturnValue({
         attributes: {

package/src/components/ShareModal/ShareModal.jsx

@@ -20,7 +20,7 @@ export const ShareModal = withLocales(props => {
   const {
     byDocId,
     isOwner,
-    canReshare,
+    hasWriteAccess,
     documentType,
     getRecipients,
     revokeSelf,
@@ -33,7 +33,9 @@ export const ShareModal = withLocales(props => {
   }
 
   const isEditable =
-    !byDocId[document.id] || isOwner(document.id) || canReshare(document.id)
+    !byDocId[document.id] ||
+    isOwner(document.id) ||
+    hasWriteAccess(document.id, document.driveId)
 
   if (isEditable) {
     const isFederatedMode = flag('drive.federated-shared-folder.enabled')

package/src/components/ShareModal/ShareModal.spec.jsx

@@ -0,0 +1,66 @@
+import { render } from '@testing-library/react'
+import React from 'react'
+import { MemoryRouter } from 'react-router'
+
+import { ShareModal } from './ShareModal'
+
+const mockIsOwner = jest.fn()
+const mockHasWriteAccess = jest.fn()
+
+jest.mock('cozy-flags')
+
+jest.mock('../../hoc/withLocales', () => Component => Component)
+
+jest.mock('../../hooks/useSharingContext', () => ({
+  useSharingContext: () => ({
+    byDocId: { 'doc-1': { sharings: ['sharing-1'] } },
+    isOwner: mockIsOwner,
+    hasWriteAccess: mockHasWriteAccess,
+    documentType: 'Files',
+    getRecipients: jest.fn().mockReturnValue([]),
+    revokeSelf: jest.fn(),
+    allLoaded: false
+  })
+}))
+
+jest.mock('./EditableSharingModal', () => () => <div>editable-modal</div>)
+jest.mock('./SharingDetailsModal', () => () => <div>details-modal</div>)
+jest.mock('../FederatedFolder/FederatedFolderModal', () => () => (
+  <div>federated-modal</div>
+))
+
+const mockDocument = { id: 'doc-1', driveId: undefined }
+
+describe('ShareModal dispatcher', () => {
+  beforeEach(() => {
+    mockIsOwner.mockReset()
+    mockHasWriteAccess.mockReset()
+  })
+
+  it('renders the editable modal when the current user has write access, even without reshare rights', () => {
+    mockIsOwner.mockReturnValue(false)
+    mockHasWriteAccess.mockReturnValue(true)
+
+    const { getByText } = render(
+      <MemoryRouter>
+        <ShareModal document={mockDocument} />
+      </MemoryRouter>
+    )
+
+    expect(getByText('editable-modal')).toBeTruthy()
+    expect(mockHasWriteAccess).toHaveBeenCalledWith('doc-1', undefined)
+  })
+
+  it('renders the restricted details modal when the current user has read-only access', () => {
+    mockIsOwner.mockReturnValue(false)
+    mockHasWriteAccess.mockReturnValue(false)
+
+    const { getByText } = render(
+      <MemoryRouter>
+        <ShareModal document={mockDocument} />
+      </MemoryRouter>
+    )
+
+    expect(getByText('details-modal')).toBeTruthy()
+  })
+})