cozy-sharing 40.1.5 → 40.1.7

package/src/hooks/useMemberPermissionType.js

@@ -0,0 +1,110 @@
+import { useCallback } from 'react'
+
+import { useClient } from 'cozy-client'
+import flag from 'cozy-flags'
+
+import { useFetchDocumentPath } from './useFetchDocumentPath'
+import { useResolveRecipients } from './useResolveRecipients'
+import { useSharingContext } from './useSharingContext'
+import { getSharingDocIds } from '../state'
+
+/**
+ * Gives what is needed to change the permission type of the members
+ * of a document's sharing
+ *
+ * @param {object} document - Shared document
+ * @param {object} [options]
+ * @param {boolean} [options.shouldFetchPath] - Whether to fetch the document
+ * path, needed to know the shared parent
+ * @returns {{ isInsideSharedParent: boolean, sharedParentName: string|null, setMemberType: Function }}
+ * `setMemberType({ sharingId, memberIndex, type, recipient }, newType, { withAncestors })`
+ * also downgrades the member on the read-write ancestor sharings when `withAncestors` is set
+ */
+export const useMemberPermissionType = (
+  document,
+  { shouldFetchPath = true } = {}
+) => {
+  const client = useClient()
+  const {
+    hasSharedParent,
+    getSharedParentPath,
+    updateSharingMemberType,
+    getSharingById,
+    share
+  } = useSharingContext()
+  const resolveRecipients = useResolveRecipients()
+
+  const documentPath = useFetchDocumentPath(
+    client,
+    shouldFetchPath ? document : null
+  )
+  const isInsideSharedParent =
+    Boolean(documentPath) && Boolean(hasSharedParent?.(documentPath))
+  const sharedParentPath = documentPath
+    ? getSharedParentPath?.(documentPath)
+    : null
+
+  const setMemberType = useCallback(
+    async (
+      { sharingId, memberIndex, type, recipient },
+      newType,
+      { withAncestors = false } = {}
+    ) => {
+      if (withAncestors) {
+        // Ancestors first: only the target's own update refetches its
+        // effective recipients, which must already see them downgraded.
+        const ancestors = (recipient?.sources || []).filter(
+          s => s.kind === 'ancestor' && !s.read_only
+        )
+        for (const ancestor of ancestors) {
+          await updateSharingMemberType(
+            ancestor.sharing_id,
+            ancestor.member_index,
+            newType
+          )
+        }
+      }
+
+      const documentId = document?._id || document?.id
+      if (
+        flag('drive.federated-shared-folder.enabled') &&
+        type === 'one-way' &&
+        newType === 'two-way' &&
+        documentId
+      ) {
+        const sourceSharing = getSharingById(sharingId)
+        if (
+          sourceSharing &&
+          !getSharingDocIds(sourceSharing).includes(documentId)
+        ) {
+          const { recipients } = await resolveRecipients([recipient])
+          if (!recipients[0]) throw new Error('Recipient contact not found')
+          await share({
+            document: { ...document, id: documentId },
+            description: document.name,
+            recipients,
+            readOnlyRecipients: [],
+            sharedDrive: true,
+            openSharing: false
+          })
+          return
+        }
+      }
+
+      await updateSharingMemberType(sharingId, memberIndex, newType, document)
+    },
+    [
+      document,
+      getSharingById,
+      resolveRecipients,
+      share,
+      updateSharingMemberType
+    ]
+  )
+
+  return {
+    isInsideSharedParent,
+    sharedParentName: sharedParentPath?.split('/').pop() ?? null,
+    setMemberType
+  }
+}

package/src/hooks/useResolveRecipients.js

@@ -0,0 +1,38 @@
+import { useCallback } from 'react'
+
+import { useClient } from 'cozy-client'
+
+import { getOrCreateFromArray } from '../helpers/contacts'
+import { DOCTYPE_CONTACTS } from '../models'
+
+/**
+ * Turns the recipients picked in a sharing form into contacts, creating the
+ * missing ones, and splits them by the selected sharing right.
+ *
+ * @returns {Function} `resolveRecipients(pendingRecipients, selectedOption, createContact?)`
+ * resolving to `{ recipients, readOnlyRecipients }`, ready to be passed to `share`
+ */
+export const useResolveRecipients = () => {
+  const client = useClient()
+
+  return useCallback(
+    async (
+      pendingRecipients,
+      selectedOption = 'readWrite',
+      createContact = contact => client.create(DOCTYPE_CONTACTS, contact)
+    ) => {
+      const contacts = await getOrCreateFromArray(
+        client,
+        pendingRecipients,
+        createContact
+      )
+      const isReadOnly = selectedOption === 'readOnly'
+
+      return {
+        recipients: isReadOnly ? [] : contacts,
+        readOnlyRecipients: isReadOnly ? contacts : []
+      }
+    },
+    [client]
+  )
+}

package/src/hooks/useResolveRecipients.spec.js

@@ -0,0 +1,70 @@
+import { renderHook } from '@testing-library/react-hooks'
+
+import { useResolveRecipients } from './useResolveRecipients'
+import { getOrCreateFromArray } from '../helpers/contacts'
+
+const mockClient = { create: jest.fn() }
+
+jest.mock('cozy-client', () => ({
+  useClient: () => mockClient
+}))
+
+jest.mock('../helpers/contacts', () => ({
+  getOrCreateFromArray: jest.fn()
+}))
+
+const contacts = [{ _id: 'alice' }, { _id: 'bob' }]
+
+describe('useResolveRecipients', () => {
+  beforeEach(() => {
+    jest.clearAllMocks()
+    getOrCreateFromArray.mockResolvedValue(contacts)
+  })
+
+  it('gives read-write rights by default', async () => {
+    const { result } = renderHook(() => useResolveRecipients())
+
+    const resolved = await result.current([{ email: 'alice@cozy.cc' }])
+
+    expect(resolved).toEqual({ recipients: contacts, readOnlyRecipients: [] })
+  })
+
+  it('gives read-only rights when the read-only option is selected', async () => {
+    const { result } = renderHook(() => useResolveRecipients())
+
+    const resolved = await result.current([], 'readOnly')
+
+    expect(resolved).toEqual({ recipients: [], readOnlyRecipients: contacts })
+  })
+
+  it('creates missing contacts with the client by default', async () => {
+    const { result } = renderHook(() => useResolveRecipients())
+    const pendingRecipients = [{ email: 'alice@cozy.cc' }]
+
+    await result.current(pendingRecipients)
+    const createContact = getOrCreateFromArray.mock.calls[0][2]
+    createContact({ email: [{ address: 'alice@cozy.cc' }] })
+
+    expect(getOrCreateFromArray).toHaveBeenCalledWith(
+      mockClient,
+      pendingRecipients,
+      expect.any(Function)
+    )
+    expect(mockClient.create).toHaveBeenCalledWith('io.cozy.contacts', {
+      email: [{ address: 'alice@cozy.cc' }]
+    })
+  })
+
+  it('creates missing contacts with the given function', async () => {
+    const { result } = renderHook(() => useResolveRecipients())
+    const createContact = jest.fn()
+
+    await result.current([], 'readWrite', createContact)
+
+    expect(getOrCreateFromArray).toHaveBeenCalledWith(
+      mockClient,
+      [],
+      createContact
+    )
+  })
+})

package/src/index.jsx

@@ -36,7 +36,7 @@ export { ShareButtonWithRecipients } from './ShareButtonWithRecipients'
 export { DOCUMENT_TYPE } from './helpers/documentType'
 export { getRecipientsFromSharing } from './state'
 export { default as OpenSharingLinkButton } from './OpenSharingLinkButton'
-export { PermissionTypeMenu } from './components/Recipient/PermissionTypeMenu'
+export { default as PermissionTypeMenu } from './components/Recipient/PermissionTypeMenuWrapper'
 export { default as OpenSharingLinkFabButton } from './OpenSharingLinkFabButton'
 export {
   NativeFileSharingProvider,

package/dist/components/Recipient/OwnerRecipientDefault.js

@@ -1,27 +0,0 @@
-import React from 'react';
-import { useClient, useQuery, hasQueryBeenLoaded } from 'cozy-client';
-import MemberRecipient from './MemberRecipient';
-import { buildInstanceSettingsQuery } from '../../queries/queries'; // When the owner is not part of a sharing's members (eg. the folder only has
-// a link), there is no per-sharing avatar to reference. The instance's public
-// avatar still holds the owner's picture. fallback=initials makes the stack
-// serve a generated initials avatar (instead of the app icon) when no picture
-// has been uploaded, matching how avatars are rendered everywhere else.
-
-var OWNER_PUBLIC_AVATAR_PATH = '/public/avatar?fallback=initials';
-
-var OwnerRecipientDefault = function OwnerRecipientDefault() {
-  var _instanceSettingsResu;
-
-  var client = useClient();
-  var instanceSettingsQuery = buildInstanceSettingsQuery();
-  var instanceSettingsResult = useQuery(instanceSettingsQuery.definition, instanceSettingsQuery.options);
-  return hasQueryBeenLoaded(instanceSettingsResult) && /*#__PURE__*/React.createElement(MemberRecipient, {
-    isOwner: true,
-    status: "owner",
-    instance: client.options.uri,
-    avatarPath: OWNER_PUBLIC_AVATAR_PATH,
-    public_name: instanceSettingsResult === null || instanceSettingsResult === void 0 || (_instanceSettingsResu = instanceSettingsResult.data) === null || _instanceSettingsResu === void 0 || (_instanceSettingsResu = _instanceSettingsResu.attributes) === null || _instanceSettingsResu === void 0 ? void 0 : _instanceSettingsResu.public_name
-  });
-};
-
-export default OwnerRecipientDefault;

package/dist/components/Recipient/OwnerRecipientDefault.stories.js

@@ -1,10 +0,0 @@
-import OwnerRecipientDefault from './OwnerRecipientDefault';
-var meta = {
-  component: OwnerRecipientDefault,
-  args: {}
-};
-export default meta;
-export var Default = {
-  name: 'Default',
-  args: {}
-};

package/src/components/Recipient/OwnerRecipientDefault.jsx

@@ -1,37 +0,0 @@
-import React from 'react'
-
-import { useClient, useQuery, hasQueryBeenLoaded } from 'cozy-client'
-
-import MemberRecipient from './MemberRecipient'
-import { buildInstanceSettingsQuery } from '../../queries/queries'
-
-// When the owner is not part of a sharing's members (eg. the folder only has
-// a link), there is no per-sharing avatar to reference. The instance's public
-// avatar still holds the owner's picture. fallback=initials makes the stack
-// serve a generated initials avatar (instead of the app icon) when no picture
-// has been uploaded, matching how avatars are rendered everywhere else.
-const OWNER_PUBLIC_AVATAR_PATH = '/public/avatar?fallback=initials'
-
-const OwnerRecipientDefault = () => {
-  const client = useClient()
-
-  const instanceSettingsQuery = buildInstanceSettingsQuery()
-  const instanceSettingsResult = useQuery(
-    instanceSettingsQuery.definition,
-    instanceSettingsQuery.options
-  )
-
-  return (
-    hasQueryBeenLoaded(instanceSettingsResult) && (
-      <MemberRecipient
-        isOwner={true}
-        status="owner"
-        instance={client.options.uri}
-        avatarPath={OWNER_PUBLIC_AVATAR_PATH}
-        public_name={instanceSettingsResult?.data?.attributes?.public_name}
-      />
-    )
-  )
-}
-
-export default OwnerRecipientDefault

package/src/components/Recipient/OwnerRecipientDefault.stories.jsx

@@ -1,13 +0,0 @@
-import OwnerRecipientDefault from './OwnerRecipientDefault'
-
-const meta = {
-  component: OwnerRecipientDefault,
-  args: {}
-}
-
-export default meta
-
-export const Default = {
-  name: 'Default',
-  args: {}
-}