cozy-sharing 17.0.0 → 19.0.0

package/CHANGELOG.md

@@ -3,6 +3,55 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [19.0.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@18.0.0...cozy-sharing@19.0.0) (2025-01-08)
+
+
+### Bug Fixes
+
+* **sharing:** Update propTypes ([0474d9c](https://github.com/cozy/cozy-libs/commit/0474d9c46344b1f7abf65fc6e419a25aef33d2ed))
+
+
+### Features
+
+* **sharing:** Add 3 actions to use the new action menu ([1af8fa7](https://github.com/cozy/cozy-libs/commit/1af8fa7dbffad63d117149d14e09399e9d71a8a6))
+* **sharing:** Add missing devDeps ([5221b40](https://github.com/cozy/cozy-libs/commit/5221b40397935538e95dcec82c939fb22196c991))
+* **sharing:** Add needed packages ([36a2b67](https://github.com/cozy/cozy-libs/commit/36a2b6771275a35b3ab484b22a18b6d9c7b61496))
+* **sharing:** Add ShareRestrictionModal component ([df5ed28](https://github.com/cozy/cozy-libs/commit/df5ed28c7c7f5dbb339bf26d563e8ffd825a686f))
+* **sharing:** Add the 3 boxes of the new link-sharing modal ([a297d20](https://github.com/cozy/cozy-libs/commit/a297d20c46966c82447491a6fefe2d96509c45a9))
+* **sharing:** Move revoke action to a secondary button ([83d3738](https://github.com/cozy/cozy-libs/commit/83d3738220e2c02ed749a08a47f92ca08ce9feb7))
+* **sharing:** Remove EditLinkPermissionDialog ([1b8bd56](https://github.com/cozy/cozy-libs/commit/1b8bd560f4e72b885817f1f17a83156b6c1a7d58))
+* **sharing:** Remove useless props ([cc3f816](https://github.com/cozy/cozy-libs/commit/cc3f816b6d35deb619488495bb767e1d76b557de))
+* **sharing:** Replace old sharing link menu with the new modal ([de1a1c2](https://github.com/cozy/cozy-libs/commit/de1a1c27a5747c0b8bf091ed1856994f65cdcdb2))
+* **sharing:** Upgrade packages ([9a1312d](https://github.com/cozy/cozy-libs/commit/9a1312d43cba1dcd521e8d6502865ac2071535cf))
+* **sharing:** Use ShareRestrictionModal when creating share link ([248c3b4](https://github.com/cozy/cozy-libs/commit/248c3b4904649e17e935afdef1bd2a0b87082dc7))
+
+
+### BREAKING CHANGES
+
+* **sharing:** You muse have `cozy-client >= 52.1.0` and `cozy-ui >= 116.0.0`
+
+
+
+
+
+# [18.0.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@17.0.0...cozy-sharing@18.0.0) (2024-12-23)
+
+
+### Features
+
+* **cozy-sharing:** Remove Enzyme tests ([0c678a1](https://github.com/cozy/cozy-libs/commit/0c678a1a096966db9baf5311b99964fc458ce8dc))
+* **cozy-sharing:** Replace jsdom-sixteen package ([b75bc51](https://github.com/cozy/cozy-libs/commit/b75bc51a8d2cbfa0300e5c9fc5f0d839dea24ef2))
+* **cozy-sharing:** Upgrade cozy-ui to 115.0.2 ([f25a351](https://github.com/cozy/cozy-libs/commit/f25a351df58ed0517f35ec4fef7c9044ceac6d56))
+
+
+### BREAKING CHANGES
+
+* **cozy-sharing:** You must have `cozy-ui >= 115.0.2` & `cozy-client >= 51.6.1`
+
+
+
+
+
 # [17.0.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@16.17.0...cozy-sharing@17.0.0) (2024-12-18)
 
 

package/dist/SharingProvider.spec.js

@@ -1,12 +1,11 @@
 import _asyncToGenerator from "@babel/runtime/helpers/asyncToGenerator";
 import _regeneratorRuntime from "@babel/runtime/regenerator";
 import { act, render, screen } from '@testing-library/react';
-import { mount } from 'enzyme';
 import React from 'react';
 import { createMockClient } from 'cozy-client';
 import { SharingProvider } from './SharingProvider';
-import SharingContext from './context';
-import { receiveSharings } from './state';
+import SharingContext from './context'; // import { receiveSharings } from './state'
+
 import AppLike from '../test/AppLike';
 
 var AppWrapper = function AppWrapper(_ref) {
@@ -102,7 +101,7 @@ describe('SharingProvider', function () {
     return jest.resetAllMocks();
   });
   it('loads data on mount', function () {
-    mount( /*#__PURE__*/React.createElement(AppWrapper, {
+    render( /*#__PURE__*/React.createElement(AppWrapper, {
       client: client
     }));
     expect(client.plugins.realtime.subscribe).toHaveBeenCalled();
@@ -110,7 +109,7 @@ describe('SharingProvider', function () {
   });
   it('loads nothing when the client is not logged in', function () {
     client.isLogged = false;
-    mount( /*#__PURE__*/React.createElement(AppWrapper, {
+    render( /*#__PURE__*/React.createElement(AppWrapper, {
       client: client
     }));
     expect(client.plugins.realtime.subscribe).not.toHaveBeenCalled();
@@ -131,78 +130,83 @@ describe('SharingProvider', function () {
     expect(client.collection().findLinksByDoctype).not.toHaveBeenCalled();
     expect(client.collection().findApps).not.toHaveBeenCalled();
   });
-});
-describe('hasWriteAccess', function () {
-  it('tells if a doc is writtable', function () {
-    var client = createMockClient({});
-    client.stackClient.uri = 'http://cozy.tools:8080';
-    var component = mount( /*#__PURE__*/React.createElement(AppWrapper, {
-      client: client
-    }, /*#__PURE__*/React.createElement(SharingContext.Consumer, null, function (_ref5) {
-      var hasWriteAccess = _ref5.hasWriteAccess;
-      return /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement("div", {
-        "data-id": "no-sharing"
-      }, hasWriteAccess('no-sharing') ? 'yes' : 'no'), /*#__PURE__*/React.createElement("div", {
-        "data-id": "owner-doc"
-      }, hasWriteAccess('owner-doc') ? 'yes' : 'no'), /*#__PURE__*/React.createElement("div", {
-        "data-id": "synced-doc"
-      }, hasWriteAccess('synced-doc') ? 'yes' : 'no'), /*#__PURE__*/React.createElement("div", {
-        "data-id": "read-only-doc"
-      }, hasWriteAccess('read-only-doc') ? 'yes' : 'no'));
-    })));
-    expect(component.find('div[data-id="no-sharing"]').text()).toBe('yes');
-    expect(component.find('div[data-id="owner-doc"]').text()).toBe('yes');
-    expect(component.find('div[data-id="synced-doc"]').text()).toBe('yes');
-    expect(component.find('div[data-id="read-only-doc"]').text()).toBe('yes');
-    var provider = component.find(SharingProvider);
-    provider.instance().dispatch(receiveSharings({
-      sharings: [{
-        id: '123',
-        type: 'io.cozy.sharings',
-        attributes: {
-          owner: true,
-          members: [{
-            read_only: false,
-            instance: 'http://cozy.tools:8080'
-          }],
-          rules: [{
-            values: ['owner-doc']
-          }]
-        }
-      }, {
-        id: '456',
-        type: 'io.cozy.sharings',
-        attributes: {
-          owner: false,
-          members: [{
-            read_only: false,
-            instance: 'http://cozy.tools:8080'
-          }],
-          rules: [{
-            values: ['synced-doc'],
-            update: 'sync',
-            remove: 'sync'
-          }]
-        }
-      }, {
-        id: '789',
-        type: 'io.cozy.sharings',
-        attributes: {
-          owner: false,
-          members: [{
-            read_only: true,
-            instance: 'http://cozy.tools:8080'
-          }],
-          rules: [{
-            values: ['read-only-doc']
-          }]
-        }
-      }]
-    }));
-    component.update();
-    expect(component.find('div[data-id="no-sharing"]').text()).toBe('yes');
-    expect(component.find('div[data-id="owner-doc"]').text()).toBe('yes');
-    expect(component.find('div[data-id="synced-doc"]').text()).toBe('yes');
-    expect(component.find('div[data-id="read-only-doc"]').text()).toBe('no');
-  });
-});
+}); // TODO Convert with react-testing-library
+// describe('hasWriteAccess', () => {
+//   it('tells if a doc is writtable', () => {
+//     const client = createMockClient({})
+//     client.stackClient.uri = 'http://cozy.tools:8080'
+//     const component = mount(
+//       <AppWrapper client={client}>
+//         <SharingContext.Consumer>
+//           {({ hasWriteAccess }) => (
+//             <>
+//               <div data-id="no-sharing">
+//                 {hasWriteAccess('no-sharing') ? 'yes' : 'no'}
+//               </div>
+//               <div data-id="owner-doc">
+//                 {hasWriteAccess('owner-doc') ? 'yes' : 'no'}
+//               </div>
+//               <div data-id="synced-doc">
+//                 {hasWriteAccess('synced-doc') ? 'yes' : 'no'}
+//               </div>
+//               <div data-id="read-only-doc">
+//                 {hasWriteAccess('read-only-doc') ? 'yes' : 'no'}
+//               </div>
+//             </>
+//           )}
+//         </SharingContext.Consumer>
+//       </AppWrapper>
+//     )
+//     expect(component.find('div[data-id="no-sharing"]').text()).toBe('yes')
+//     expect(component.find('div[data-id="owner-doc"]').text()).toBe('yes')
+//     expect(component.find('div[data-id="synced-doc"]').text()).toBe('yes')
+//     expect(component.find('div[data-id="read-only-doc"]').text()).toBe('yes')
+//     const provider = component.find(SharingProvider)
+//     provider.instance().dispatch(
+//       receiveSharings({
+//         sharings: [
+//           {
+//             id: '123',
+//             type: 'io.cozy.sharings',
+//             attributes: {
+//               owner: true,
+//               members: [
+//                 { read_only: false, instance: 'http://cozy.tools:8080' }
+//               ],
+//               rules: [{ values: ['owner-doc'] }]
+//             }
+//           },
+//           {
+//             id: '456',
+//             type: 'io.cozy.sharings',
+//             attributes: {
+//               owner: false,
+//               members: [
+//                 { read_only: false, instance: 'http://cozy.tools:8080' }
+//               ],
+//               rules: [
+//                 { values: ['synced-doc'], update: 'sync', remove: 'sync' }
+//               ]
+//             }
+//           },
+//           {
+//             id: '789',
+//             type: 'io.cozy.sharings',
+//             attributes: {
+//               owner: false,
+//               members: [
+//                 { read_only: true, instance: 'http://cozy.tools:8080' }
+//               ],
+//               rules: [{ values: ['read-only-doc'] }]
+//             }
+//           }
+//         ]
+//       })
+//     )
+//     component.update()
+//     expect(component.find('div[data-id="no-sharing"]').text()).toBe('yes')
+//     expect(component.find('div[data-id="owner-doc"]').text()).toBe('yes')
+//     expect(component.find('div[data-id="synced-doc"]').text()).toBe('yes')
+//     expect(component.find('div[data-id="read-only-doc"]').text()).toBe('no')
+//   })
+// })

package/dist/components/Recipient/LinkRecipientPermissions.js

@@ -1,26 +1,15 @@
-import _asyncToGenerator from "@babel/runtime/helpers/asyncToGenerator";
 import _slicedToArray from "@babel/runtime/helpers/slicedToArray";
-import _regeneratorRuntime from "@babel/runtime/regenerator";
+import PropTypes from 'prop-types';
 import React, { useState, useRef } from 'react';
 import DropdownButton from 'cozy-ui/transpiled/react/DropdownButton';
-import Icon from 'cozy-ui/transpiled/react/Icon';
-import TrashIcon from 'cozy-ui/transpiled/react/Icons/Trash';
-import Spinner from 'cozy-ui/transpiled/react/Spinner';
-import Typography from 'cozy-ui/transpiled/react/Typography';
-import ActionMenu, { ActionMenuItem, ActionMenuRadio } from 'cozy-ui/transpiled/react/deprecated/ActionMenu';
-import Alerter from 'cozy-ui/transpiled/react/deprecated/Alerter';
 import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n';
-import { isOnlyReadOnlyLinkAllowed } from '../../helpers/link';
 import { checkIsReadOnlyPermissions } from '../../helpers/permissions';
-import logger from '../../logger';
+import { ShareRestrictionModal } from '../ShareRestrictionModal/ShareRestrictionModal';
 
 var LinkRecipientPermissions = function LinkRecipientPermissions(_ref) {
   var className = _ref.className,
       document = _ref.document,
-      documentType = _ref.documentType,
-      permissions = _ref.permissions,
-      onChangePermissions = _ref.onChangePermissions,
-      onDisable = _ref.onDisable;
+      permissions = _ref.permissions;
 
   var _useI18n = useI18n(),
       t = _useI18n.t;
@@ -29,129 +18,29 @@ var LinkRecipientPermissions = function LinkRecipientPermissions(_ref) {
 
   var _useState = useState(false),
       _useState2 = _slicedToArray(_useState, 2),
-      menuIsOpen = _useState2[0],
-      setMenuIsOpen = _useState2[1];
-
-  var _useState3 = useState(false),
-      _useState4 = _slicedToArray(_useState3, 2),
-      loading = _useState4[0],
-      setLoading = _useState4[1];
+      openShareRestrictionModal = _useState2[0],
+      setOpenShareRestrictionModal = _useState2[1];
 
   var isReadOnlyPermissions = checkIsReadOnlyPermissions(permissions);
-
-  var toggleMenu = function toggleMenu() {
-    setMenuIsOpen(!menuIsOpen);
-  };
-
-  var deleteShareLink = /*#__PURE__*/function () {
-    var _ref2 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee() {
-      return _regeneratorRuntime.wrap(function _callee$(_context) {
-        while (1) switch (_context.prev = _context.next) {
-          case 0:
-            _context.prev = 0;
-            setLoading(true);
-            _context.next = 4;
-            return onDisable(document);
-
-          case 4:
-            _context.next = 10;
-            break;
-
-          case 6:
-            _context.prev = 6;
-            _context.t0 = _context["catch"](0);
-            Alerter.error(t("".concat(documentType, ".share.error.revoke")));
-            logger.log(_context.t0);
-
-          case 10:
-            _context.prev = 10;
-            setLoading(false);
-            return _context.finish(10);
-
-          case 13:
-          case "end":
-            return _context.stop();
-        }
-      }, _callee, null, [[0, 6, 10, 13]]);
-    }));
-
-    return function deleteShareLink() {
-      return _ref2.apply(this, arguments);
-    };
-  }();
-
-  var updateLinkPermissions = function updateLinkPermissions(_ref3) {
-    var isReadOnly = _ref3.isReadOnly;
-    var verbs = isReadOnly ? ['GET'] : ['GET', 'POST', 'PUT', 'PATCH'];
-
-    try {
-      onChangePermissions(document, verbs);
-    } catch (err) {
-      Alerter.error(t("".concat(documentType, ".share.shareByLink.permserror")));
-      logger.log(err);
-    }
-  };
-
   return /*#__PURE__*/React.createElement("div", {
     className: className
-  }, loading && /*#__PURE__*/React.createElement(Spinner, null), !loading && /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(DropdownButton, {
-    onClick: toggleMenu,
+  }, /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(DropdownButton, {
     ref: buttonRef,
-    textVariant: "body2"
-  }, t("Share.type.".concat(isReadOnlyPermissions ? 'one-way' : 'two-way')).toLowerCase()), menuIsOpen && /*#__PURE__*/React.createElement(ActionMenu, {
-    onClose: toggleMenu,
-    popperOptions: {
-      placement: 'bottom-end',
-      strategy: 'fixed'
-    },
-    anchorElRef: buttonRef
-  }, /*#__PURE__*/React.createElement(ActionMenuItem, {
-    left: /*#__PURE__*/React.createElement(ActionMenuRadio, {
-      name: "permissionLinkMenu",
-      value: "true",
-      checked: isReadOnlyPermissions
-    }),
+    textVariant: "body2",
     onClick: function onClick() {
-      toggleMenu();
-      updateLinkPermissions({
-        isReadOnly: true
-      });
+      return setOpenShareRestrictionModal(true);
     }
-  }, /*#__PURE__*/React.createElement(React.Fragment, null, (document === null || document === void 0 ? void 0 : document.type) === 'directory' ? t("Share.permissionLink.seeFolder") : t("Share.permissionLink.seeFile"), /*#__PURE__*/React.createElement(Typography, {
-    className: "u-mt-half",
-    variant: "caption",
-    color: "textSecondary"
-  }, t('Share.permissionLink.readDescription')))), !isOnlyReadOnlyLinkAllowed({
-    documentType: documentType
-  }) && /*#__PURE__*/React.createElement(ActionMenuItem, {
-    left: /*#__PURE__*/React.createElement(ActionMenuRadio, {
-      name: "permissionLinkMenu",
-      value: "false",
-      checked: !isReadOnlyPermissions
-    }),
-    onClick: function onClick() {
-      toggleMenu();
-      updateLinkPermissions({
-        isReadOnly: false
-      });
-    }
-  }, /*#__PURE__*/React.createElement(React.Fragment, null, (document === null || document === void 0 ? void 0 : document.type) === 'directory' ? t("Share.permissionLink.modifyFolder") : t("Share.permissionLink.modifyFile"), /*#__PURE__*/React.createElement(Typography, {
-    className: "u-mt-half",
-    variant: "caption",
-    color: "textSecondary"
-  }, t('Share.permissionLink.writeDescription')))), /*#__PURE__*/React.createElement("hr", null), /*#__PURE__*/React.createElement(ActionMenuItem, {
-    left: /*#__PURE__*/React.createElement(Icon, {
-      icon: TrashIcon,
-      color: "var(--errorColor)"
-    }),
-    onClick: function onClick() {
-      toggleMenu();
-      deleteShareLink();
+  }, t("Share.type.".concat(isReadOnlyPermissions ? 'one-way' : 'two-way')).toLowerCase()), openShareRestrictionModal && /*#__PURE__*/React.createElement(ShareRestrictionModal, {
+    file: document,
+    onClose: function onClose() {
+      return setOpenShareRestrictionModal(false);
     }
-  }, /*#__PURE__*/React.createElement(Typography, {
-    className: "u-error",
-    variant: "body1"
-  }, t('Share.permissionLink.deactivate'))))));
+  })));
 };
 
+LinkRecipientPermissions.propTypes = {
+  className: PropTypes.string,
+  document: PropTypes.object.isRequired,
+  permissions: PropTypes.arrayOf(PropTypes.object).isRequired
+};
 export default LinkRecipientPermissions;

package/dist/components/Recipient/actions/editPermissionLink.js

@@ -0,0 +1,71 @@
+import _extends from "@babel/runtime/helpers/extends";
+import React, { forwardRef } from 'react';
+import ActionsMenuItem from 'cozy-ui/transpiled/react/ActionsMenu/ActionsMenuItem';
+import ListItemIcon from 'cozy-ui/transpiled/react/ListItemIcon';
+import ListItemText from 'cozy-ui/transpiled/react/ListItemText';
+import Radios from 'cozy-ui/transpiled/react/Radios';
+import { isOnlyReadOnlyLinkAllowed } from '../../../helpers/link';
+import { useSharingContext } from '../../../hooks/useSharingContext';
+
+var makeComponent = function makeComponent(_ref) {
+  var label = _ref.label,
+      desc = _ref.desc,
+      editingRights = _ref.editingRights;
+  var Component = /*#__PURE__*/forwardRef(function (props, ref) {
+    var _useSharingContext = useSharingContext(),
+        updateDocumentPermissions = _useSharingContext.updateDocumentPermissions;
+
+    return /*#__PURE__*/React.createElement(ActionsMenuItem, _extends({}, props, {
+      ref: ref,
+      button: false,
+      onClick: function onClick() {
+        return props.onClick({
+          updateDocumentPermissions: updateDocumentPermissions
+        });
+      }
+    }), /*#__PURE__*/React.createElement(ListItemIcon, null, /*#__PURE__*/React.createElement(Radios, {
+      checked: editingRights === 'write'
+    })), /*#__PURE__*/React.createElement(ListItemText, {
+      primary: label,
+      secondary: desc
+    }));
+  });
+  Component.displayName = 'EditPermissionLink';
+  return Component;
+};
+/**
+ * @param {boolean} isDirectory - true if the document is a directory
+ * @param {function} t - translation function
+ * @param {string} editingRights - current editing rights
+ * @param {function} setEditingRights - function to set editing rights
+ * @param {string} documentType - type of the document
+ */
+
+
+export var editPermissionLink = function editPermissionLink(_ref2) {
+  var isDirectory = _ref2.isDirectory,
+      t = _ref2.t,
+      editingRights = _ref2.editingRights,
+      setEditingRights = _ref2.setEditingRights,
+      documentType = _ref2.documentType;
+  var label = isDirectory ? t('Share.permissionLink.modifyFolder') : t('Share.permissionLink.modifyFile');
+  var desc = t('Share.permissionLink.writeDescription');
+  return {
+    name: 'editPermissionLink',
+    label: label,
+    icon: null,
+    displayCondition: function displayCondition() {
+      return !isOnlyReadOnlyLinkAllowed({
+        documentType: documentType
+      });
+    },
+    action: function action() {
+      setEditingRights('write');
+    },
+    Component: makeComponent({
+      label: label,
+      desc: desc,
+      editingRights: editingRights
+    })
+  };
+};

package/dist/components/Recipient/actions/readOnlyPermissionLink.js

@@ -0,0 +1,63 @@
+import _extends from "@babel/runtime/helpers/extends";
+import React, { forwardRef } from 'react';
+import ActionsMenuItem from 'cozy-ui/transpiled/react/ActionsMenu/ActionsMenuItem';
+import ListItemIcon from 'cozy-ui/transpiled/react/ListItemIcon';
+import ListItemText from 'cozy-ui/transpiled/react/ListItemText';
+import Radios from 'cozy-ui/transpiled/react/Radios';
+import { useSharingContext } from '../../../hooks/useSharingContext';
+
+var makeComponent = function makeComponent(_ref) {
+  var label = _ref.label,
+      desc = _ref.desc,
+      editingRights = _ref.editingRights;
+  var Component = /*#__PURE__*/forwardRef(function (props, ref) {
+    var _useSharingContext = useSharingContext(),
+        updateDocumentPermissions = _useSharingContext.updateDocumentPermissions;
+
+    return /*#__PURE__*/React.createElement(ActionsMenuItem, _extends({}, props, {
+      ref: ref,
+      button: false,
+      onClick: function onClick() {
+        return props.onClick({
+          updateDocumentPermissions: updateDocumentPermissions
+        });
+      }
+    }), /*#__PURE__*/React.createElement(ListItemIcon, null, /*#__PURE__*/React.createElement(Radios, {
+      checked: editingRights === 'readOnly'
+    })), /*#__PURE__*/React.createElement(ListItemText, {
+      primary: label,
+      secondary: desc
+    }));
+  });
+  Component.displayName = 'ReadOnlyPermissionLink';
+  return Component;
+};
+/**
+ * @param {boolean} isDirectory - true if the document is a directory
+ * @param {function} t - translation function
+ * @param {string} editingRights - current editing rights
+ * @param {function} setEditingRights - function to set editing rights
+ */
+
+
+export var readOnlyPermissionLink = function readOnlyPermissionLink(_ref2) {
+  var isDirectory = _ref2.isDirectory,
+      t = _ref2.t,
+      editingRights = _ref2.editingRights,
+      setEditingRights = _ref2.setEditingRights;
+  var label = isDirectory ? t('Share.permissionLink.seeFolder') : t('Share.permissionLink.seeFile');
+  var desc = t('Share.permissionLink.readDescription');
+  return {
+    name: 'readOnlyPermissionLink',
+    label: label,
+    icon: null,
+    action: function action() {
+      setEditingRights('readOnly');
+    },
+    Component: makeComponent({
+      label: label,
+      desc: desc,
+      editingRights: editingRights
+    })
+  };
+};