cozy-sharing 31.1.0 → 31.2.0

package/dist/components/SharedDrive/helpers.js

@@ -1,102 +0,0 @@
-import _toConsumableArray from "@babel/runtime/helpers/toConsumableArray";
-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 { models } from 'cozy-client';
-var ContactModel = models.contact;
-export var RECIPIENT_INDEX_PREFIX = 'virtual-shared-drive-sharing-';
-export var mergeAndDeduplicateRecipients = function mergeAndDeduplicateRecipients(arrays) {
-  var combinedArray = arrays.flat().map(function (item) {
-    return _objectSpread(_objectSpread({}, item), {}, {
-      id: item.id || item._id
-    });
-  });
-  var seenIds = new Set();
-  var uniqueArray = combinedArray.reduce(function (acc, item) {
-    var id = item.id || item._id;
-
-    if (id == null) {
-      // Recipients without any identifier are kept as-is to avoid silent data loss
-      acc.push(item);
-      return acc;
-    }
-
-    if (!seenIds.has(id)) {
-      seenIds.add(id); // Ensure both id and _id exist for downstream compatibility
-
-      var normalizedItem = !item.id || !item._id ? _objectSpread(_objectSpread({}, item), {}, {
-        id: item.id || item._id,
-        _id: item._id || item.id
-      }) : item;
-      acc.push(normalizedItem);
-    }
-
-    return acc;
-  }, []);
-  return uniqueArray;
-};
-export var moveRecipientToReadWrite = function moveRecipientToReadWrite(federatedRecipients, _id) {
-  var recipientToMove = federatedRecipients.readOnlyRecipients.find(function (r) {
-    return r._id === _id;
-  });
-  if (!recipientToMove) return federatedRecipients;
-  return {
-    recipients: [].concat(_toConsumableArray(federatedRecipients.recipients), [recipientToMove]),
-    readOnlyRecipients: federatedRecipients.readOnlyRecipients.filter(function (r) {
-      return r._id !== _id;
-    })
-  };
-};
-export var moveRecipientToReadOnly = function moveRecipientToReadOnly(federatedRecipients, _id) {
-  var recipientToMove = federatedRecipients.recipients.find(function (r) {
-    return r._id === _id;
-  });
-  if (!recipientToMove) return federatedRecipients;
-  return {
-    recipients: federatedRecipients.recipients.filter(function (r) {
-      return r._id !== _id;
-    }),
-    readOnlyRecipients: [].concat(_toConsumableArray(federatedRecipients.readOnlyRecipients), [recipientToMove])
-  };
-};
-export var formatRecipients = function formatRecipients(sharedDriveRecipients) {
-  var recipients = [];
-  sharedDriveRecipients.recipients.forEach(function (r) {
-    recipients.push({
-      index: "".concat(RECIPIENT_INDEX_PREFIX).concat(r._id),
-      email: ContactModel.getPrimaryEmail(r),
-      public_name: ContactModel.getDisplayName(r),
-      memberIndex: r._id,
-      type: 'two-way',
-      members: r.members
-    });
-  });
-  sharedDriveRecipients.readOnlyRecipients.forEach(function (r) {
-    recipients.push({
-      index: "".concat(RECIPIENT_INDEX_PREFIX).concat(r._id),
-      email: ContactModel.getPrimaryEmail(r),
-      public_name: ContactModel.getDisplayName(r),
-      memberIndex: r._id,
-      type: 'one-way',
-      members: r.members
-    });
-  }); // We need to sort recipients by public_name
-  // to avoid reording them every time we change
-  // them from read only to read write.
-
-  recipients.sort(function (a, b) {
-    if (a.public_name < b.public_name) {
-      return -1;
-    }
-
-    if (a.public_name > b.public_name) {
-      return 1;
-    }
-
-    return 0;
-  });
-  return recipients;
-};

package/dist/components/SharedDrive/helpers.spec.js

@@ -1,227 +0,0 @@
-import { mergeAndDeduplicateRecipients, moveRecipientToReadWrite, moveRecipientToReadOnly, formatRecipients, RECIPIENT_INDEX_PREFIX } from './helpers';
-describe('mergeAndDeduplicateRecipients', function () {
-  it('should merge multiple arrays into one', function () {
-    var result = mergeAndDeduplicateRecipients([[{
-      id: '1',
-      name: 'Alice'
-    }], [{
-      id: '2',
-      name: 'Bob'
-    }]]);
-    expect(result).toEqual([{
-      id: '1',
-      _id: '1',
-      name: 'Alice'
-    }, {
-      id: '2',
-      _id: '2',
-      name: 'Bob'
-    }]);
-  });
-  it('should deduplicate recipients with the same id', function () {
-    var result = mergeAndDeduplicateRecipients([[{
-      id: '1',
-      name: 'Alice'
-    }], [{
-      id: '1',
-      name: 'Alice'
-    }]]);
-    expect(result).toEqual([{
-      id: '1',
-      _id: '1',
-      name: 'Alice'
-    }]);
-  });
-  it('should normalize _id to id for deduplication', function () {
-    var result = mergeAndDeduplicateRecipients([[{
-      _id: '1',
-      name: 'Alice'
-    }], [{
-      _id: '2',
-      name: 'Bob'
-    }]]);
-    expect(result).toEqual([{
-      _id: '1',
-      id: '1',
-      name: 'Alice'
-    }, {
-      _id: '2',
-      id: '2',
-      name: 'Bob'
-    }]);
-  });
-  it('should deduplicate recipients when one has id and the other has _id', function () {
-    var result = mergeAndDeduplicateRecipients([[{
-      id: '1',
-      name: 'Alice'
-    }], [{
-      _id: '1',
-      name: 'Alice'
-    }]]);
-    expect(result).toEqual([{
-      id: '1',
-      _id: '1',
-      name: 'Alice'
-    }]);
-  });
-  it('should prefer existing id over _id when both are present', function () {
-    var result = mergeAndDeduplicateRecipients([[{
-      id: 'real-id',
-      _id: 'other-id',
-      name: 'Alice'
-    }]]);
-    expect(result).toEqual([{
-      id: 'real-id',
-      _id: 'other-id',
-      name: 'Alice'
-    }]);
-  });
-  it('should handle empty arrays', function () {
-    var result = mergeAndDeduplicateRecipients([[], []]);
-    expect(result).toEqual([]);
-  });
-});
-describe('moveRecipientToReadWrite', function () {
-  it('should move a recipient from readOnly to readWrite', function () {
-    var state = {
-      recipients: [{
-        _id: '1',
-        name: 'Alice'
-      }],
-      readOnlyRecipients: [{
-        _id: '2',
-        name: 'Bob'
-      }]
-    };
-    var result = moveRecipientToReadWrite(state, '2');
-    expect(result.recipients).toEqual([{
-      _id: '1',
-      name: 'Alice'
-    }, {
-      _id: '2',
-      name: 'Bob'
-    }]);
-    expect(result.readOnlyRecipients).toEqual([]);
-  });
-  it('should return unchanged state if recipient not found', function () {
-    var state = {
-      recipients: [{
-        _id: '1',
-        name: 'Alice'
-      }],
-      readOnlyRecipients: []
-    };
-    var result = moveRecipientToReadWrite(state, '999');
-    expect(result).toBe(state);
-  });
-});
-describe('moveRecipientToReadOnly', function () {
-  it('should move a recipient from readWrite to readOnly', function () {
-    var state = {
-      recipients: [{
-        _id: '1',
-        name: 'Alice'
-      }],
-      readOnlyRecipients: [{
-        _id: '2',
-        name: 'Bob'
-      }]
-    };
-    var result = moveRecipientToReadOnly(state, '1');
-    expect(result.recipients).toEqual([]);
-    expect(result.readOnlyRecipients).toEqual([{
-      _id: '2',
-      name: 'Bob'
-    }, {
-      _id: '1',
-      name: 'Alice'
-    }]);
-  });
-  it('should return unchanged state if recipient not found', function () {
-    var state = {
-      recipients: [],
-      readOnlyRecipients: [{
-        _id: '1',
-        name: 'Alice'
-      }]
-    };
-    var result = moveRecipientToReadOnly(state, '999');
-    expect(result).toBe(state);
-  });
-});
-describe('formatRecipients', function () {
-  it('should format readWrite and readOnly recipients with correct types', function () {
-    var input = {
-      recipients: [{
-        _id: '1',
-        displayName: 'Alice',
-        email: [{
-          address: 'alice@example.com',
-          primary: true
-        }]
-      }],
-      readOnlyRecipients: [{
-        _id: '2',
-        displayName: 'Bob',
-        email: [{
-          address: 'bob@example.com',
-          primary: true
-        }]
-      }]
-    };
-    var result = formatRecipients(input);
-    expect(result).toHaveLength(2);
-    expect(result[0]).toMatchObject({
-      index: "".concat(RECIPIENT_INDEX_PREFIX, "1"),
-      type: 'two-way',
-      public_name: 'Alice'
-    });
-    expect(result[1]).toMatchObject({
-      index: "".concat(RECIPIENT_INDEX_PREFIX, "2"),
-      type: 'one-way',
-      public_name: 'Bob'
-    });
-  });
-  it('should flatten an object-shaped contact name into a string public_name', function () {
-    var input = {
-      recipients: [{
-        _id: '3',
-        name: {
-          givenName: 'Arya',
-          familyName: 'Stark'
-        },
-        email: [{
-          address: 'arya@example.com',
-          primary: true
-        }]
-      }],
-      readOnlyRecipients: []
-    };
-    var result = formatRecipients(input);
-    expect(typeof result[0].public_name).toBe('string');
-    expect(result[0].public_name).toBe('Arya Stark');
-  });
-  it('should sort recipients by public_name', function () {
-    var input = {
-      recipients: [{
-        _id: '1',
-        displayName: 'Zara',
-        email: [{
-          address: 'zara@example.com',
-          primary: true
-        }]
-      }, {
-        _id: '2',
-        displayName: 'Alice',
-        email: [{
-          address: 'alice@example.com',
-          primary: true
-        }]
-      }],
-      readOnlyRecipients: []
-    };
-    var result = formatRecipients(input);
-    expect(result[0].public_name).toBe('Alice');
-    expect(result[1].public_name).toBe('Zara');
-  });
-});

package/dist/components/Sharesubmit.js

@@ -1,28 +0,0 @@
-import PropTypes from 'prop-types';
-import React from 'react';
-import Button from 'cozy-ui/transpiled/react/Buttons';
-
-var ShareSubmit = function ShareSubmit(props) {
-  return /*#__PURE__*/React.createElement(Button, {
-    onClick: function onClick() {
-      props.onSubmit();
-    },
-    size: "large",
-    busy: props.loading,
-    label: props.label,
-    disabled: props.disabled,
-    variant: "ghost"
-  });
-};
-
-ShareSubmit.propTypes = {
-  onSubmit: PropTypes.func.isRequired,
-  label: PropTypes.string,
-  loading: PropTypes.bool,
-  disabled: PropTypes.bool
-};
-ShareSubmit.defaultProps = {
-  label: 'Submit',
-  loading: false
-};
-export default ShareSubmit;

package/src/components/SharedDrive/helpers.js

@@ -1,108 +0,0 @@
-import { models } from 'cozy-client'
-
-const ContactModel = models.contact
-
-export const RECIPIENT_INDEX_PREFIX = 'virtual-shared-drive-sharing-'
-
-export const mergeAndDeduplicateRecipients = arrays => {
-  const combinedArray = arrays.flat().map(item => ({
-    ...item,
-    id: item.id || item._id
-  }))
-
-  const seenIds = new Set()
-
-  const uniqueArray = combinedArray.reduce((acc, item) => {
-    const id = item.id || item._id
-
-    if (id == null) {
-      // Recipients without any identifier are kept as-is to avoid silent data loss
-      acc.push(item)
-      return acc
-    }
-
-    if (!seenIds.has(id)) {
-      seenIds.add(id)
-      // Ensure both id and _id exist for downstream compatibility
-      const normalizedItem =
-        !item.id || !item._id
-          ? { ...item, id: item.id || item._id, _id: item._id || item.id }
-          : item
-      acc.push(normalizedItem)
-    }
-
-    return acc
-  }, [])
-
-  return uniqueArray
-}
-
-export const moveRecipientToReadWrite = (federatedRecipients, _id) => {
-  const recipientToMove = federatedRecipients.readOnlyRecipients.find(
-    r => r._id === _id
-  )
-  if (!recipientToMove) return federatedRecipients
-
-  return {
-    recipients: [...federatedRecipients.recipients, recipientToMove],
-    readOnlyRecipients: federatedRecipients.readOnlyRecipients.filter(
-      r => r._id !== _id
-    )
-  }
-}
-
-export const moveRecipientToReadOnly = (federatedRecipients, _id) => {
-  const recipientToMove = federatedRecipients.recipients.find(
-    r => r._id === _id
-  )
-  if (!recipientToMove) return federatedRecipients
-
-  return {
-    recipients: federatedRecipients.recipients.filter(r => r._id !== _id),
-    readOnlyRecipients: [
-      ...federatedRecipients.readOnlyRecipients,
-      recipientToMove
-    ]
-  }
-}
-
-export const formatRecipients = sharedDriveRecipients => {
-  const recipients = []
-
-  sharedDriveRecipients.recipients.forEach(r => {
-    recipients.push({
-      index: `${RECIPIENT_INDEX_PREFIX}${r._id}`,
-      email: ContactModel.getPrimaryEmail(r),
-      public_name: ContactModel.getDisplayName(r),
-      memberIndex: r._id,
-      type: 'two-way',
-      members: r.members
-    })
-  })
-
-  sharedDriveRecipients.readOnlyRecipients.forEach(r => {
-    recipients.push({
-      index: `${RECIPIENT_INDEX_PREFIX}${r._id}`,
-      email: ContactModel.getPrimaryEmail(r),
-      public_name: ContactModel.getDisplayName(r),
-      memberIndex: r._id,
-      type: 'one-way',
-      members: r.members
-    })
-  })
-
-  // We need to sort recipients by public_name
-  // to avoid reording them every time we change
-  // them from read only to read write.
-  recipients.sort((a, b) => {
-    if (a.public_name < b.public_name) {
-      return -1
-    }
-    if (a.public_name > b.public_name) {
-      return 1
-    }
-    return 0
-  })
-
-  return recipients
-}

package/src/components/SharedDrive/helpers.spec.js

@@ -1,177 +0,0 @@
-import {
-  mergeAndDeduplicateRecipients,
-  moveRecipientToReadWrite,
-  moveRecipientToReadOnly,
-  formatRecipients,
-  RECIPIENT_INDEX_PREFIX
-} from './helpers'
-
-describe('mergeAndDeduplicateRecipients', () => {
-  it('should merge multiple arrays into one', () => {
-    const result = mergeAndDeduplicateRecipients([
-      [{ id: '1', name: 'Alice' }],
-      [{ id: '2', name: 'Bob' }]
-    ])
-    expect(result).toEqual([
-      { id: '1', _id: '1', name: 'Alice' },
-      { id: '2', _id: '2', name: 'Bob' }
-    ])
-  })
-
-  it('should deduplicate recipients with the same id', () => {
-    const result = mergeAndDeduplicateRecipients([
-      [{ id: '1', name: 'Alice' }],
-      [{ id: '1', name: 'Alice' }]
-    ])
-    expect(result).toEqual([{ id: '1', _id: '1', name: 'Alice' }])
-  })
-
-  it('should normalize _id to id for deduplication', () => {
-    const result = mergeAndDeduplicateRecipients([
-      [{ _id: '1', name: 'Alice' }],
-      [{ _id: '2', name: 'Bob' }]
-    ])
-    expect(result).toEqual([
-      { _id: '1', id: '1', name: 'Alice' },
-      { _id: '2', id: '2', name: 'Bob' }
-    ])
-  })
-
-  it('should deduplicate recipients when one has id and the other has _id', () => {
-    const result = mergeAndDeduplicateRecipients([
-      [{ id: '1', name: 'Alice' }],
-      [{ _id: '1', name: 'Alice' }]
-    ])
-    expect(result).toEqual([{ id: '1', _id: '1', name: 'Alice' }])
-  })
-
-  it('should prefer existing id over _id when both are present', () => {
-    const result = mergeAndDeduplicateRecipients([
-      [{ id: 'real-id', _id: 'other-id', name: 'Alice' }]
-    ])
-    expect(result).toEqual([{ id: 'real-id', _id: 'other-id', name: 'Alice' }])
-  })
-
-  it('should handle empty arrays', () => {
-    const result = mergeAndDeduplicateRecipients([[], []])
-    expect(result).toEqual([])
-  })
-})
-
-describe('moveRecipientToReadWrite', () => {
-  it('should move a recipient from readOnly to readWrite', () => {
-    const state = {
-      recipients: [{ _id: '1', name: 'Alice' }],
-      readOnlyRecipients: [{ _id: '2', name: 'Bob' }]
-    }
-    const result = moveRecipientToReadWrite(state, '2')
-    expect(result.recipients).toEqual([
-      { _id: '1', name: 'Alice' },
-      { _id: '2', name: 'Bob' }
-    ])
-    expect(result.readOnlyRecipients).toEqual([])
-  })
-
-  it('should return unchanged state if recipient not found', () => {
-    const state = {
-      recipients: [{ _id: '1', name: 'Alice' }],
-      readOnlyRecipients: []
-    }
-    const result = moveRecipientToReadWrite(state, '999')
-    expect(result).toBe(state)
-  })
-})
-
-describe('moveRecipientToReadOnly', () => {
-  it('should move a recipient from readWrite to readOnly', () => {
-    const state = {
-      recipients: [{ _id: '1', name: 'Alice' }],
-      readOnlyRecipients: [{ _id: '2', name: 'Bob' }]
-    }
-    const result = moveRecipientToReadOnly(state, '1')
-    expect(result.recipients).toEqual([])
-    expect(result.readOnlyRecipients).toEqual([
-      { _id: '2', name: 'Bob' },
-      { _id: '1', name: 'Alice' }
-    ])
-  })
-
-  it('should return unchanged state if recipient not found', () => {
-    const state = {
-      recipients: [],
-      readOnlyRecipients: [{ _id: '1', name: 'Alice' }]
-    }
-    const result = moveRecipientToReadOnly(state, '999')
-    expect(result).toBe(state)
-  })
-})
-
-describe('formatRecipients', () => {
-  it('should format readWrite and readOnly recipients with correct types', () => {
-    const input = {
-      recipients: [
-        {
-          _id: '1',
-          displayName: 'Alice',
-          email: [{ address: 'alice@example.com', primary: true }]
-        }
-      ],
-      readOnlyRecipients: [
-        {
-          _id: '2',
-          displayName: 'Bob',
-          email: [{ address: 'bob@example.com', primary: true }]
-        }
-      ]
-    }
-    const result = formatRecipients(input)
-    expect(result).toHaveLength(2)
-    expect(result[0]).toMatchObject({
-      index: `${RECIPIENT_INDEX_PREFIX}1`,
-      type: 'two-way',
-      public_name: 'Alice'
-    })
-    expect(result[1]).toMatchObject({
-      index: `${RECIPIENT_INDEX_PREFIX}2`,
-      type: 'one-way',
-      public_name: 'Bob'
-    })
-  })
-
-  it('should flatten an object-shaped contact name into a string public_name', () => {
-    const input = {
-      recipients: [
-        {
-          _id: '3',
-          name: { givenName: 'Arya', familyName: 'Stark' },
-          email: [{ address: 'arya@example.com', primary: true }]
-        }
-      ],
-      readOnlyRecipients: []
-    }
-    const result = formatRecipients(input)
-    expect(typeof result[0].public_name).toBe('string')
-    expect(result[0].public_name).toBe('Arya Stark')
-  })
-
-  it('should sort recipients by public_name', () => {
-    const input = {
-      recipients: [
-        {
-          _id: '1',
-          displayName: 'Zara',
-          email: [{ address: 'zara@example.com', primary: true }]
-        },
-        {
-          _id: '2',
-          displayName: 'Alice',
-          email: [{ address: 'alice@example.com', primary: true }]
-        }
-      ],
-      readOnlyRecipients: []
-    }
-    const result = formatRecipients(input)
-    expect(result[0].public_name).toBe('Alice')
-    expect(result[1].public_name).toBe('Zara')
-  })
-})

package/src/components/Sharesubmit.jsx

@@ -1,31 +0,0 @@
-import PropTypes from 'prop-types'
-import React from 'react'
-
-import Button from 'cozy-ui/transpiled/react/Buttons'
-
-const ShareSubmit = props => (
-  <Button
-    onClick={() => {
-      props.onSubmit()
-    }}
-    size="large"
-    busy={props.loading}
-    label={props.label}
-    disabled={props.disabled}
-    variant="ghost"
-  />
-)
-
-ShareSubmit.propTypes = {
-  onSubmit: PropTypes.func.isRequired,
-  label: PropTypes.string,
-  loading: PropTypes.bool,
-  disabled: PropTypes.bool
-}
-
-ShareSubmit.defaultProps = {
-  label: 'Submit',
-  loading: false
-}
-
-export default ShareSubmit