cozy-sharing 37.5.5 → 37.6.0

package/dist/helpers/effectiveRecipients.spec.js

@@ -0,0 +1,134 @@
+import _defineProperty from "@babel/runtime/helpers/defineProperty";
+
+function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (Object.getOwnPropertySymbols) { var symbols = Object.getOwnPropertySymbols(object); enumerableOnly && (symbols = symbols.filter(function (sym) { return Object.getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
+
+function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? ownKeys(Object(source), !0).forEach(function (key) { _defineProperty(target, key, source[key]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)) : ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } return target; }
+
+import { mapEffectiveRecipients } from './effectiveRecipients';
+
+var makeAncestorSource = function makeAncestorSource() {
+  var rootName = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : 'Parent Folder';
+  return {
+    sharing_id: 'sharing-1',
+    root_id: 'root-1',
+    root_name: rootName,
+    kind: 'ancestor',
+    member_index: 1,
+    read_only: false,
+    manageable: false
+  };
+};
+
+var makeSelfSource = function makeSelfSource() {
+  var _opts$memberIndex, _opts$readOnly, _opts$manageable;
+
+  var opts = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
+  return {
+    sharing_id: opts.sharing_id || 'sharing-2',
+    root_id: 'root-2',
+    root_name: 'Child Folder',
+    kind: 'self',
+    member_index: (_opts$memberIndex = opts.memberIndex) !== null && _opts$memberIndex !== void 0 ? _opts$memberIndex : 1,
+    read_only: (_opts$readOnly = opts.readOnly) !== null && _opts$readOnly !== void 0 ? _opts$readOnly : false,
+    manageable: (_opts$manageable = opts.manageable) !== null && _opts$manageable !== void 0 ? _opts$manageable : true
+  };
+};
+
+var makeRecipient = function makeRecipient() {
+  var overrides = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
+  return _objectSpread({
+    id: 'recip-1',
+    name: 'Alice',
+    email: 'alice@example.com',
+    instance: 'https://alice.mycozy.cloud',
+    status: 'ready',
+    read_only: false,
+    can_edit_here: false,
+    sources: overrides.sources || []
+  }, overrides);
+};
+
+describe('mapEffectiveRecipients', function () {
+  it('returns empty array for empty input', function () {
+    expect(mapEffectiveRecipients([])).toEqual([]);
+  });
+  it('returns empty array for null or undefined', function () {
+    expect(mapEffectiveRecipients(null)).toEqual([]);
+    expect(mapEffectiveRecipients(undefined)).toEqual([]);
+  });
+  it('maps a recipient with a self source using the self source', function () {
+    var recipient = makeRecipient({
+      sources: [makeSelfSource()]
+    });
+    var result = mapEffectiveRecipients([recipient]);
+    expect(result).toHaveLength(1);
+    expect(result[0].sharingId).toBe('sharing-2');
+    expect(result[0].memberIndex).toBe(1);
+    expect(result[0].avatarPath).toBe('/sharings/sharing-2/recipients/1/avatar');
+    expect(result[0].type).toBe('two-way');
+  });
+  it('maps a recipient with only ancestor sources using the ancestor source', function () {
+    var recipient = makeRecipient({
+      sources: [makeAncestorSource('Shared Parent')]
+    });
+    var result = mapEffectiveRecipients([recipient]);
+    expect(result).toHaveLength(1);
+    expect(result[0].sharingId).toBe('sharing-1');
+    expect(result[0].memberIndex).toBe(1);
+    expect(result[0].name).toBe('Alice');
+    expect(result[0].email).toBe('alice@example.com');
+  });
+  it('maps an owner (self source, index 0) correctly', function () {
+    var recipient = makeRecipient({
+      status: 'owner',
+      read_only: true,
+      sources: [makeSelfSource({
+        memberIndex: 0,
+        manageable: false
+      })]
+    });
+    var result = mapEffectiveRecipients([recipient]);
+    expect(result).toHaveLength(1);
+    expect(result[0].status).toBe('owner');
+    expect(result[0].type).toBe('one-way');
+  });
+  it('maps an owner with only ancestor sources correctly', function () {
+    var recipient = makeRecipient({
+      status: 'owner',
+      sources: [makeAncestorSource('Parent Folder')]
+    });
+    var result = mapEffectiveRecipients([recipient]);
+    expect(result).toHaveLength(1);
+    expect(result[0].status).toBe('owner');
+    expect(result[0].sharingId).toBe('sharing-1');
+    expect(result[0].memberIndex).toBe(1);
+  });
+  it('sets type to one-way for read_only recipients', function () {
+    var recipient = makeRecipient({
+      read_only: true,
+      sources: [makeSelfSource({
+        readOnly: true
+      })]
+    });
+    var result = mapEffectiveRecipients([recipient]);
+    expect(result[0].type).toBe('one-way');
+  });
+  it('handles all recipients in the same array regardless of source kind', function () {
+    var selfRecipient = makeRecipient({
+      id: 'self-1',
+      name: 'Bob',
+      email: 'bob@example.com',
+      sources: [makeSelfSource()]
+    });
+    var ancestorRecipient = makeRecipient({
+      id: 'ancestor-1',
+      name: 'Carol',
+      email: 'carol@example.com',
+      sources: [makeAncestorSource('Root Folder')]
+    });
+    var result = mapEffectiveRecipients([selfRecipient, ancestorRecipient]);
+    expect(result).toHaveLength(2);
+    expect(result[0].sharingId).toBe('sharing-2');
+    expect(result[1].sharingId).toBe('sharing-1');
+  });
+});

package/dist/state.js

@@ -1,3 +1,4 @@
+import _slicedToArray from "@babel/runtime/helpers/slicedToArray";
 import _toConsumableArray from "@babel/runtime/helpers/toConsumableArray";
 import _objectWithoutProperties from "@babel/runtime/helpers/objectWithoutProperties";
 import _defineProperty from "@babel/runtime/helpers/defineProperty";
@@ -23,6 +24,9 @@ var ADD_SHARING_LINK = 'ADD_SHARING_LINK';
 var UPDATE_SHARING_LINK = 'UPDATE_SHARING_LINK';
 var REVOKE_SHARING_LINK = 'REVOKE_SHARING_LINK';
 var RECEIVE_PATHS = 'RECEIVE_PATHS';
+var RECEIVE_EFFECTIVE_RECIPIENTS = 'RECEIVE_EFFECTIVE_RECIPIENTS';
+var UPDATE_EFFECTIVE_RECIPIENT = 'UPDATE_EFFECTIVE_RECIPIENT';
+var INVALIDATE_EFFECTIVE_RECIPIENTS = 'INVALIDATE_EFFECTIVE_RECIPIENTS';
 export var SHARING_TYPE = {
   TWO_WAY: 'two-way',
   ONE_WAY: 'one-way'
@@ -136,6 +140,27 @@ export var receivePaths = function receivePaths(paths) {
     paths: paths
   };
 };
+export var receiveEffectiveRecipients = function receiveEffectiveRecipients(docId, recipients) {
+  return {
+    type: RECEIVE_EFFECTIVE_RECIPIENTS,
+    docId: docId,
+    recipients: recipients
+  };
+};
+export var updateEffectiveRecipient = function updateEffectiveRecipient(sharingId, memberIndex, readOnly) {
+  return {
+    type: UPDATE_EFFECTIVE_RECIPIENT,
+    sharingId: sharingId,
+    memberIndex: memberIndex,
+    readOnly: readOnly
+  };
+};
+export var invalidateEffectiveRecipients = function invalidateEffectiveRecipients(docId) {
+  return {
+    type: INVALIDATE_EFFECTIVE_RECIPIENTS,
+    docId: docId
+  };
+};
 export var matchingInstanceName = function matchingInstanceName() {
   var instanceUri = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : '';
   return function (shareMember) {
@@ -373,6 +398,42 @@ var sharedPaths = function sharedPaths() {
   }
 };
 
+var effectiveRecipients = function effectiveRecipients() {
+  var state = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
+  var action = arguments.length > 1 ? arguments[1] : undefined;
+
+  switch (action.type) {
+    case RECEIVE_EFFECTIVE_RECIPIENTS:
+      return _objectSpread(_objectSpread({}, state), {}, _defineProperty({}, action.docId, action.recipients));
+
+    case UPDATE_EFFECTIVE_RECIPIENT:
+      return Object.fromEntries(Object.entries(state).map(function (_ref2) {
+        var _ref3 = _slicedToArray(_ref2, 2),
+            docId = _ref3[0],
+            recipients = _ref3[1];
+
+        return [docId, recipients.map(function (r) {
+          return r.sharingId === action.sharingId && r.memberIndex === action.memberIndex ? _objectSpread(_objectSpread({}, r), {}, {
+            read_only: action.readOnly,
+            type: action.readOnly ? 'one-way' : 'two-way'
+          }) : r;
+        })];
+      }));
+
+    case INVALIDATE_EFFECTIVE_RECIPIENTS:
+      {
+        var _action$docId2 = action.docId,
+            _ = state[_action$docId2],
+            rest = _objectWithoutProperties(state, [_action$docId2].map(_toPropertyKey));
+
+        return rest;
+      }
+
+    default:
+      return state;
+  }
+};
+
 var reducer = function reducer() {
   var state = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
   var action = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
@@ -380,7 +441,8 @@ var reducer = function reducer() {
     byDocId: byDocId(state.byDocId, action),
     sharings: sharings(state.sharings, action),
     permissions: permissions(state.permissions, action),
-    sharedPaths: sharedPaths(state.sharedPaths, action)
+    sharedPaths: sharedPaths(state.sharedPaths, action),
+    effectiveRecipients: effectiveRecipients(state.effectiveRecipients, action)
   };
 };
 
@@ -643,6 +705,11 @@ export var getSharedParentPath = function getSharedParentPath(state, documentPat
   }
 
   return null;
+};
+export var getEffectiveRecipients = function getEffectiveRecipients(state, docId) {
+  var _state$effectiveRecip;
+
+  return ((_state$effectiveRecip = state.effectiveRecipients) === null || _state$effectiveRecip === void 0 ? void 0 : _state$effectiveRecip[docId]) || [];
 }; // helpers
 
 export var getSharedDocIds = function getSharedDocIds(doc) {

package/dist/state.spec.js

@@ -13,7 +13,8 @@ describe('Sharing state', function () {
       byDocId: {},
       sharings: [],
       permissions: [],
-      sharedPaths: []
+      sharedPaths: [],
+      effectiveRecipients: {}
     });
   });
   it('should index received sharings', function () {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-sharing",
-  "version": "37.5.5",
+  "version": "37.6.0",
   "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": "^12.3.14",
+    "cozy-ui-plus": "^12.3.15",
     "jest": "30.3.0",
     "jest-environment-jsdom": "30.3.0",
     "react": "16.12.0",
@@ -85,5 +85,5 @@
   "sideEffects": [
     "*.css"
   ],
-  "gitHead": "319c4890af0fbee641e25a2cb7c1bd2501b57024"
+  "gitHead": "aa6c91c03a8b734177fddb3075aa26110516c218"
 }

package/src/SharingProvider.jsx

@@ -9,6 +9,7 @@ import {
 } from './components/ShareRestrictionModal/helpers'
 import SharingContext from './context'
 import { fetchNextPermissions } from './fetchNextPermissions'
+import { mapEffectiveRecipients } from './helpers/effectiveRecipients'
 import { fetchFilesPaths } from './helpers/files'
 import {
   getSharingObject,
@@ -28,6 +29,10 @@ import reducer, {
   revokeGroup as revokeGroupFromState,
   revokeSelf,
   receivePaths,
+  receiveEffectiveRecipients,
+  updateEffectiveRecipient,
+  invalidateEffectiveRecipients,
+  getEffectiveRecipients,
   isOwner,
   isSharedDrive,
   isOrgSharedDrive,
@@ -109,7 +114,11 @@ export class SharingProvider extends Component {
       refresh: this.fetchAllSharings,
       hasWriteAccess: this.hasWriteAccess,
       renameSharedDrive: this.renameSharedDrive,
-      updateSharingMemberType: this.updateSharingMemberType
+      updateSharingMemberType: this.updateSharingMemberType,
+      getEffectiveRecipients: docId =>
+        getEffectiveRecipients(this.state, docId),
+      fetchEffectiveRecipients: this.fetchEffectiveRecipients,
+      invalidateEffectiveRecipients: this.invalidateEffectiveRecipients
     }
     this.isPublic = props.isPublic ?? false
     this.realtime = null
@@ -261,6 +270,11 @@ export class SharingProvider extends Component {
 
       await this.state.onShared({ document, recipients, readOnlyRecipients })
 
+      await this.fetchEffectiveRecipients(
+        getDocumentId(document),
+        document.driveId
+      )
+
       return sharingResult
     }
 
@@ -284,6 +298,10 @@ export class SharingProvider extends Component {
     )
 
     await this.state.onShared({ document, recipients, readOnlyRecipients })
+    await this.fetchEffectiveRecipients(
+      getDocumentId(document),
+      document.driveId
+    )
     return data
   }
 
@@ -318,6 +336,10 @@ export class SharingProvider extends Component {
         )
       )
     })
+    await this.fetchEffectiveRecipients(
+      getDocumentId(document),
+      document.driveId
+    )
   }
 
   revoke = async (document, sharingId, recipientIndex) => {
@@ -331,6 +353,10 @@ export class SharingProvider extends Component {
         document.path || (await fetchFilesPaths(client, doctype, [document]))
       )
     )
+    await this.fetchEffectiveRecipients(
+      getDocumentId(document),
+      document.driveId
+    )
   }
 
   revokeGroup = async (document, sharingId, recipientIndex) => {
@@ -350,6 +376,10 @@ export class SharingProvider extends Component {
     const sharing = getSharingForSelf(this.state, document.id)
     await this.sharingCol.revokeSelf(sharing)
     this.dispatch(revokeSelf(sharing))
+    await this.fetchEffectiveRecipients(
+      getDocumentId(document),
+      document.driveId
+    )
   }
 
   /**
@@ -386,6 +416,9 @@ export class SharingProvider extends Component {
     }
 
     this.dispatch(updateSharing(optimisticSharing))
+    this.dispatch(
+      updateEffectiveRecipient(sharingId, memberIndex, makeReadOnly)
+    )
 
     try {
       if (makeReadOnly) {
@@ -411,12 +444,39 @@ export class SharingProvider extends Component {
       }
 
       this.dispatch(updateSharing(rollbackSharing))
+      this.dispatch(
+        updateEffectiveRecipient(sharingId, memberIndex, !makeReadOnly)
+      )
       log.error(
         `Failed to change member ${member.email} permission type to ${newType}`,
         error
       )
       throw error
     }
+
+    const docId = Object.keys(this.state.byDocId).find(id =>
+      this.state.byDocId[id].sharings?.includes(sharingId)
+    )
+    if (docId) {
+      const driveId = this.state.byDocId[docId].driveId
+      await this.fetchEffectiveRecipients(docId, driveId)
+    }
+  }
+
+  fetchEffectiveRecipients = async (fileId, driveId) => {
+    const resp = await this.sharingCol.fetchEffectiveRecipients(fileId, {
+      driveId
+    })
+    this.dispatch(
+      receiveEffectiveRecipients(
+        fileId,
+        mapEffectiveRecipients(resp?.data || [])
+      )
+    )
+  }
+
+  invalidateEffectiveRecipients = docId => {
+    this.dispatch(invalidateEffectiveRecipients(docId))
   }
 
   shareByLink = async (document, options) => {

package/src/SharingProvider.spec.jsx

@@ -533,7 +533,8 @@ describe('shared drive recipient revocation', () => {
     sharingCol = {
       addRecipients: jest.fn(),
       create: jest.fn().mockResolvedValue({ data: NEW_SHARING }),
-      revokeRecipient: jest.fn().mockResolvedValue({})
+      revokeRecipient: jest.fn().mockResolvedValue({}),
+      fetchEffectiveRecipients: jest.fn().mockResolvedValue({ data: [] })
     }
     provider.sharingCol = sharingCol
   })
@@ -792,7 +793,8 @@ describe('updateSharingMemberType', () => {
     )
     instance.sharingCol = {
       setReadOnly: jest.fn().mockResolvedValue({}),
-      setReadWrite: jest.fn().mockResolvedValue({})
+      setReadWrite: jest.fn().mockResolvedValue({}),
+      fetchEffectiveRecipients: jest.fn().mockResolvedValue({ data: [] })
     }
   })
 
@@ -913,6 +915,12 @@ describe('updateSharingMemberType', () => {
       }
     })
     expect(instance.dispatch).toHaveBeenNthCalledWith(2, {
+      type: 'UPDATE_EFFECTIVE_RECIPIENT',
+      sharingId: 'sharing-123',
+      memberIndex: 1,
+      readOnly: true
+    })
+    expect(instance.dispatch).toHaveBeenNthCalledWith(3, {
       type: 'UPDATE_SHARING',
       sharing: {
         ...mockSharing,
@@ -928,6 +936,12 @@ describe('updateSharingMemberType', () => {
         }
       }
     })
+    expect(instance.dispatch).toHaveBeenNthCalledWith(4, {
+      type: 'UPDATE_EFFECTIVE_RECIPIENT',
+      sharingId: 'sharing-123',
+      memberIndex: 1,
+      readOnly: false
+    })
   })
 
   it('should preserve realtime changes when rolling back an API failure', async () => {
@@ -964,6 +978,12 @@ describe('updateSharingMemberType', () => {
     ).rejects.toThrow('Network error')
 
     expect(instance.dispatch).toHaveBeenNthCalledWith(2, {
+      type: 'UPDATE_EFFECTIVE_RECIPIENT',
+      sharingId: 'sharing-123',
+      memberIndex: 1,
+      readOnly: true
+    })
+    expect(instance.dispatch).toHaveBeenNthCalledWith(3, {
       type: 'UPDATE_SHARING',
       sharing: {
         ...realtimeSharing,
@@ -979,6 +999,12 @@ describe('updateSharingMemberType', () => {
         }
       }
     })
+    expect(instance.dispatch).toHaveBeenNthCalledWith(4, {
+      type: 'UPDATE_EFFECTIVE_RECIPIENT',
+      sharingId: 'sharing-123',
+      memberIndex: 1,
+      readOnly: false
+    })
   })
 })
 

package/src/components/FederatedFolder/FederatedFolderModal.jsx

@@ -16,7 +16,6 @@ import { getOrCreateFromArray } from '../../helpers/contacts'
 import withLocales from '../../hoc/withLocales'
 import { usePendingRecipients } from '../../hooks/usePendingRecipients'
 import { useSharingContext } from '../../hooks/useSharingContext'
-import { getRecipientsFromSharing } from '../../state'
 import styles from '../../styles/share.styl'
 import AntivirusAlert from '../AntivirusAlert'
 import { default as DumbShareByEmail } from '../ShareByEmail'
@@ -29,6 +28,7 @@ const FederatedFolderModalContent = ({
   onClose,
   onRevokeSuccess,
   document: existingDocument,
+  recipients,
   autoOpenShareRestriction,
   showGenerateLinkButton
 }) => {
@@ -48,10 +48,6 @@ const FederatedFolderModalContent = ({
     getFederatedShareLink,
     getDocumentPermissions,
     fetchSharedDriveSharingLinks,
-    getRecipients,
-    getSharedParentPath,
-    hasSharedChild,
-    hasSharedParent,
     isOwner,
     revoke,
     revokeSelf
@@ -70,11 +66,9 @@ const FederatedFolderModalContent = ({
     useState(null)
   const [frozenDoc, setFrozenDoc] = useState(null)
   const [frozenRecipients, setFrozenRecipients] = useState(null)
-  const [sharedParentFolder, setSharedParentFolder] = useState(null)
 
   const documentId = existingDocument?._id || existingDocument?.id
   const folderName = existingDocument?.name || ''
-  const documentPath = existingDocument?.path
   const sharedDriveSharing = existingDocument?.driveId
     ? getSharingById(existingDocument.driveId)
     : null
@@ -83,25 +77,7 @@ const FederatedFolderModalContent = ({
       (ids, rule) => ids.concat(rule.values || []),
       []
     ) || []
-  // Owner case: when document is inside a shared folder (via path) but has no driveId,
-  // find the parent shared folder to use its _id for recipient/permission lookups.
-  const hasSharedParentByPath = Boolean(
-    documentPath && hasSharedParent(documentPath)
-  )
-  const sharedParentPath =
-    !sharedDriveSharing && hasSharedParentByPath && documentPath
-      ? getSharedParentPath(documentPath)
-      : null
-  const sharingReferenceId =
-    !sharedDriveSharing && sharedParentFolder?._id
-      ? sharedParentFolder._id
-      : documentId
-  const existingRecipients =
-    sharedDriveSharing?.attributes?.members && documentId
-      ? getRecipientsFromSharing(sharedDriveSharing, documentId)
-      : sharingReferenceId
-        ? getRecipients(sharingReferenceId)
-        : []
+
   const documentPermissions = documentId
     ? getDocumentPermissions(documentId)
     : []
@@ -124,11 +100,6 @@ const FederatedFolderModalContent = ({
       ? isOwner(documentId)
       : false
   const isMemberReadOnly = isInsideSharedDrive && !isCurrentUserOwner
-  const hasParentRestriction = isInsideSharedDrive || hasSharedParentByPath
-  const hasChildRestriction = Boolean(
-    documentPath && hasSharedChild(documentPath)
-  )
-  const canShareByEmail = !hasParentRestriction && !hasChildRestriction
   const canManageSharing =
     isCurrentUserOwner || (documentId ? canReshare(documentId) : false)
 
@@ -181,7 +152,7 @@ const FederatedFolderModalContent = ({
 
     setIsSending(true)
     setFrozenDoc(existingDocument)
-    setFrozenRecipients(existingRecipients)
+    setFrozenRecipients(recipients)
 
     try {
       const contacts = await getOrCreateFromArray(
@@ -219,21 +190,6 @@ const FederatedFolderModalContent = ({
     }
   }
 
-  useEffect(() => {
-    if (!sharedParentPath) return
-    const fetchParentFolder = async () => {
-      try {
-        const res = await client
-          .collection('io.cozy.files')
-          .statByPath(sharedParentPath)
-        if (res.data) setSharedParentFolder(res.data)
-      } catch (err) {
-        log.error('Failed to fetch shared parent folder', err)
-      }
-    }
-    fetchParentFolder()
-  }, [sharedParentPath, client])
-
   useEffect(() => {
     if (!existingDocument?.driveId) return
     if (!documentId) return
@@ -283,7 +239,7 @@ const FederatedFolderModalContent = ({
             <AntivirusAlert
               document={isSending ? frozenDoc : existingDocument}
             />
-            {!canShareByEmail && (
+            {isInsideSharedDrive ? (
               <div className={styles['share-byemail-onlybylink']}>
                 {t('Files.share.shareByEmail.onlyByLink', {
                   type: t(
@@ -292,26 +248,15 @@ const FederatedFolderModalContent = ({
                     }`
                   )
                 })}{' '}
-                <strong>
-                  {t(
-                    `Files.share.shareByEmail.${
-                      hasParentRestriction
-                        ? 'hasSharedParent'
-                        : 'hasSharedChild'
-                    }`
-                  )}
-                </strong>
+                <strong>{t('Files.share.shareByEmail.hasSharedParent')}</strong>
               </div>
-            )}
-            {canShareByEmail && (
+            ) : (
               <>
                 <Typography variant="h6" className="u-mt-1-half u-mb-half">
                   {t('Share.contacts.addUsers')}
                 </Typography>
                 <DumbShareByEmail
-                  currentRecipients={
-                    isSending ? frozenRecipients : existingRecipients
-                  }
+                  currentRecipients={isSending ? frozenRecipients : recipients}
                   document={isSending ? frozenDoc : existingDocument}
                   documentType="Files"
                   pendingRecipients={pendingRecipients}
@@ -329,7 +274,7 @@ const FederatedFolderModalContent = ({
             isSharedDrive
             canManageMembers={!isMemberReadOnly}
             canManageLink={true}
-            recipients={isSending ? frozenRecipients : existingRecipients}
+            recipients={isSending ? frozenRecipients : recipients}
             document={isSending ? frozenDoc : existingDocument}
             documentType="Files"
             className={styles['share-dialog-members']}
@@ -348,7 +293,7 @@ const FederatedFolderModalContent = ({
             showGenerateLinkButton={showGenerateLinkButton}
             autoOpenShareRestriction={autoOpenShareRestriction}
           />
-          {canShareByEmail && (
+          {!isInsideSharedDrive && (
             <Button
               variant="primary"
               label={t('FederatedFolder.share')}
@@ -362,9 +307,49 @@ const FederatedFolderModalContent = ({
   )
 }
 
+FederatedFolderModalContent.propTypes = {
+  onClose: PropTypes.func.isRequired,
+  onRevokeSuccess: PropTypes.func,
+  document: PropTypes.object.isRequired,
+  recipients: PropTypes.array,
+  autoOpenShareRestriction: PropTypes.bool,
+  showGenerateLinkButton: PropTypes.bool
+}
+
+const FederatedFolderModalWrapper = ({ document, ...props }) => {
+  const fileId = document._id || document.id
+  const {
+    getEffectiveRecipients,
+    fetchEffectiveRecipients,
+    invalidateEffectiveRecipients
+  } = useSharingContext()
+
+  useEffect(() => {
+    fetchEffectiveRecipients(fileId, document.driveId).catch(log.error)
+    return () => {
+      invalidateEffectiveRecipients(fileId)
+    }
+  }, [
+    fileId,
+    document.driveId,
+    fetchEffectiveRecipients,
+    invalidateEffectiveRecipients
+  ])
+
+  const recipients = getEffectiveRecipients(fileId)
+
+  return (
+    <FederatedFolderModalContent
+      document={document}
+      recipients={recipients}
+      {...props}
+    />
+  )
+}
+
 export const FederatedFolderModal = withLocales(props => (
   <ConfirmDialogProvider>
-    <FederatedFolderModalContent {...props} />
+    <FederatedFolderModalWrapper {...props} />
   </ConfirmDialogProvider>
 ))