cozy-sharing 28.4.4 → 28.6.0

package/CHANGELOG.md

@@ -3,6 +3,24 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [28.6.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@28.5.0...cozy-sharing@28.6.0) (2026-03-11)
+
+### Features
+
+- Add ability to change member permission type in sharing ([76d0dce](https://github.com/cozy/cozy-libs/commit/76d0dce150d8ea7ac7cb12c6e00b340756b6512a))
+- Hide link text in link recipient ([209857f](https://github.com/cozy/cozy-libs/commit/209857fcf01b71e584e49ef8ab3da4a897d408ed))
+- Remove link prop from LinkRecipient component ([7bbf831](https://github.com/cozy/cozy-libs/commit/7bbf831594d7c33357ade79665f4308fd35e2b8e))
+
+# [28.5.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@28.4.4...cozy-sharing@28.5.0) (2026-03-10)
+
+### Bug Fixes
+
+- Make sharingDesc prop optional with default empty string ([538eaa6](https://github.com/cozy/cozy-libs/commit/538eaa61988405be4225a6147fad17805b218e30))
+
+### Features
+
+- Allow to share a shared drive file by link ([a78e796](https://github.com/cozy/cozy-libs/commit/a78e7963af05b6815856246a722542acee2d900f))
+
 ## [28.4.4](https://github.com/cozy/cozy-libs/compare/cozy-sharing@28.4.3...cozy-sharing@28.4.4) (2026-03-09)
 
 **Note:** Version bump only for package cozy-sharing

package/dist/SharingProvider.js

@@ -33,13 +33,15 @@ function _callSuper(_this, derived, args) {
 import React, { Component } from 'react';
 import { withClient } from 'cozy-client';
 import minilog from 'cozy-minilog';
+import { generateShareLinkFromFile } from './components/ShareRestrictionModal/helpers';
 import SharingContext from './context';
 import { fetchNextPermissions } from './fetchNextPermissions';
 import { fetchFilesPaths } from './helpers/files';
 import { getSharingObject, createSharingInStore, updateSharingInStore } from './helpers/sharings';
+import { getShortcode } from './helpers/shortcodes';
 import { SynchronousJobQueue } from './helpers/synchronousJobQueue';
 import { fetchApps } from './queries/queries';
-import reducer, { receiveSharings, addSharing, updateSharing, addSharingLink, updateSharingLink, revokeSharingLink, revokeRecipient, revokeGroup as revokeGroupFromState, revokeSelf, receivePaths, isOwner as _isOwner, isSharedDrive as _isSharedDrive, canReshare as _canReshare, getOwner as _getOwner, getRecipients as _getRecipients, getSharingById, getExternalSharingIds, getSharingForSelf as _getSharingForSelf, getSharingType as _getSharingType, getSharingLink as _getSharingLink, getSharedDocIdsBySharings, getDocumentSharing, getDocumentPermissions as _getDocumentPermissions, hasSharedParent as _hasSharedParent, hasSharedChild as _hasSharedChild, getSharedParentPath as _getSharedParentPath, getSharedDriveSharingType, SHARING_TYPE } from './state';
+import reducer, { receiveSharings, addSharing, updateSharing, addSharingLink, updateSharingLink, revokeSharingLink, revokeRecipient, revokeGroup as revokeGroupFromState, revokeSelf, receivePaths, isOwner as _isOwner, isSharedDrive as _isSharedDrive, canReshare as _canReshare, getOwner as _getOwner, getRecipients as _getRecipients, getSharingById as _getSharingById, getExternalSharingIds, getSharingForSelf as _getSharingForSelf, getSharingType as _getSharingType, getSharingLink as _getSharingLink, getSharedDocIdsBySharings, getDocumentSharing, getDocumentPermissions as _getDocumentPermissions, hasSharedParent as _hasSharedParent, hasSharedChild as _hasSharedChild, getSharedParentPath as _getSharedParentPath, getSharedDriveSharingType, SHARING_TYPE } from './state';
 var log = minilog('SharingProvider');
 var SHARING_DOCTYPE = 'io.cozy.sharings';
 var PERMISSION_DOCTYPE = 'io.cozy.permissions';
@@ -87,7 +89,7 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
           while (1) switch (_context.prev = _context.next) {
             case 0:
               _this2$props = _this2.props, client = _this2$props.client, doctype = _this2$props.doctype;
-              internalSharing = getSharingById(_this2.state, sharing._id);
+              internalSharing = _getSharingById(_this2.state, sharing._id);
               newSharing = getSharingObject(internalSharing, sharing);
 
               if (internalSharing) {
@@ -348,7 +350,7 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
           while (1) switch (_context7.prev = _context7.next) {
             case 0:
               _this2$props4 = _this2.props, client = _this2$props4.client, doctype = _this2$props4.doctype;
-              sharing = getSharingById(_this2.state, sharingId);
+              sharing = _getSharingById(_this2.state, sharingId);
               _context7.next = 4;
               return _this2.sharingCol.revokeRecipient(sharing, recipientIndex);
 
@@ -396,7 +398,7 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
           while (1) switch (_context8.prev = _context8.next) {
             case 0:
               _this2$props5 = _this2.props, client = _this2$props5.client, doctype = _this2$props5.doctype;
-              sharing = getSharingById(_this2.state, sharingId);
+              sharing = _getSharingById(_this2.state, sharingId);
               _context8.next = 4;
               return _this2.sharingCol.revokeGroup(sharing, recipientIndex);
 
@@ -461,31 +463,230 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
       };
     }());
 
-    _defineProperty(_this2, "shareByLink", /*#__PURE__*/function () {
-      var _ref12 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10(document, options) {
-        var resp;
+    /**
+     * Update a sharing member's read_only field to change permission type
+     * @param {string} sharingId - The ID of the sharing
+     * @param {number} memberIndex - The index of the member in the sharing members array
+     * @param {string} newType - 'one-way' for read-only, 'two-way' for read-write
+     */
+    _defineProperty(_this2, "updateSharingMemberType", /*#__PURE__*/function () {
+      var _ref12 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10(sharingId, memberIndex, newType) {
+        var sharing, member, matchedContacts, contact, readOnly;
         return _regeneratorRuntime.wrap(function _callee10$(_context10) {
           while (1) switch (_context10.prev = _context10.next) {
             case 0:
-              _context10.next = 2;
+              sharing = _getSharingById(_this2.state, sharingId);
+
+              if (sharing) {
+                _context10.next = 3;
+                break;
+              }
+
+              throw new Error('Sharing not found');
+
+            case 3:
+              member = sharing.attributes.members[memberIndex];
+
+              if (member) {
+                _context10.next = 6;
+                break;
+              }
+
+              throw new Error('Member not found');
+
+            case 6:
+              _context10.next = 8;
+              return _this2.props.client.collection('io.cozy.contacts').find({
+                email: {
+                  $elemMatch: {
+                    address: member.email
+                  }
+                },
+                _id: {
+                  $gt: null
+                }
+              }, {
+                indexedFields: ['_id']
+              });
+
+            case 8:
+              matchedContacts = _context10.sent;
+
+              if (matchedContacts.data.length) {
+                _context10.next = 11;
+                break;
+              }
+
+              throw new Error("Contact not found for member ".concat(member.email));
+
+            case 11:
+              contact = matchedContacts.data[0]; // Revoke (API call only, no state dispatch to avoid UI flickering)
+
+              _context10.next = 14;
+              return _this2.sharingCol.revokeRecipient(sharing, memberIndex);
+
+            case 14:
+              // Re-add with the new permission type
+              readOnly = newType === 'one-way';
+              _context10.prev = 15;
+              _context10.next = 18;
+              return _this2.addRecipients({
+                document: sharing,
+                recipients: readOnly ? [] : [contact],
+                readOnlyRecipients: readOnly ? [contact] : []
+              });
+
+            case 18:
+              _context10.next = 24;
+              break;
+
+            case 20:
+              _context10.prev = 20;
+              _context10.t0 = _context10["catch"](15);
+              log.error("Failed to re-add member ".concat(member.email, " after revocation. The member has been revoked but could not be re-added with the new permission type."), _context10.t0);
+              throw _context10.t0;
+
+            case 24:
+            case "end":
+              return _context10.stop();
+          }
+        }, _callee10, null, [[15, 20]]);
+      }));
+
+      return function (_x16, _x17, _x18) {
+        return _ref12.apply(this, arguments);
+      };
+    }());
+
+    _defineProperty(_this2, "shareByLink", /*#__PURE__*/function () {
+      var _ref13 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11(document, options) {
+        var resp;
+        return _regeneratorRuntime.wrap(function _callee11$(_context11) {
+          while (1) switch (_context11.prev = _context11.next) {
+            case 0:
+              if (!document.driveId) {
+                _context11.next = 6;
+                break;
+              }
+
+              _context11.next = 3;
+              return _this2.createSharedDriveSharingLink(document, options);
+
+            case 3:
+              _context11.t0 = _context11.sent;
+              _context11.next = 9;
+              break;
+
+            case 6:
+              _context11.next = 8;
               return _this2.permissionCol.createSharingLink(document, options);
 
-            case 2:
-              resp = _context10.sent;
+            case 8:
+              _context11.t0 = _context11.sent;
+
+            case 9:
+              resp = _context11.t0;
 
               _this2.dispatch(addSharingLink(resp.data));
 
-              return _context10.abrupt("return", resp);
+              return _context11.abrupt("return", resp);
+
+            case 12:
+            case "end":
+              return _context11.stop();
+          }
+        }, _callee11);
+      }));
+
+      return function (_x19, _x20) {
+        return _ref13.apply(this, arguments);
+      };
+    }());
+
+    _defineProperty(_this2, "getFederatedShareLink", function (document) {
+      if (!document.driveId) return null;
+
+      var permissions = _getDocumentPermissions(_this2.state, document._id);
+
+      var perm = permissions.find(function (p) {
+        return getShortcode(p);
+      });
+      if (!perm) return null;
+      var code = getShortcode(perm);
+      if (!code) return null;
+      var client = _this2.props.client;
+      return generateShareLinkFromFile({
+        client: client,
+        file: document,
+        sharecode: code,
+        getOwner: _this2.state.getOwner,
+        getSharingById: _this2.state.getSharingById,
+        documentType: _this2.state.documentType
+      });
+    });
+
+    _defineProperty(_this2, "createSharedDriveSharingLink", /*#__PURE__*/function () {
+      var _ref14 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12(document, options) {
+        var client, drivePermissionCollection, resp;
+        return _regeneratorRuntime.wrap(function _callee12$(_context12) {
+          while (1) switch (_context12.prev = _context12.next) {
+            case 0:
+              client = _this2.props.client;
+              drivePermissionCollection = client.collection('io.cozy.permissions', {
+                driveId: document.driveId
+              });
+              _context12.next = 4;
+              return drivePermissionCollection.createSharingLink(document, options);
+
+            case 4:
+              resp = _context12.sent;
+              return _context12.abrupt("return", resp);
+
+            case 6:
+            case "end":
+              return _context12.stop();
+          }
+        }, _callee12);
+      }));
+
+      return function (_x21, _x22) {
+        return _ref14.apply(this, arguments);
+      };
+    }());
+
+    _defineProperty(_this2, "updateSharedDrivePermissions", /*#__PURE__*/function () {
+      var _ref15 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13(document, permissionDocument, updatedPermissions, options) {
+        var client, expiresAt, password, drivePermissionCollection, resp;
+        return _regeneratorRuntime.wrap(function _callee13$(_context13) {
+          while (1) switch (_context13.prev = _context13.next) {
+            case 0:
+              client = _this2.props.client;
+              expiresAt = options.expiresAt, password = options.password;
+              drivePermissionCollection = client.collection('io.cozy.permissions', {
+                driveId: document.driveId
+              });
+              _context13.next = 5;
+              return drivePermissionCollection.add(permissionDocument, updatedPermissions, {
+                expiresAt: expiresAt,
+                password: password
+              });
 
             case 5:
+              resp = _context13.sent;
+
+              _this2.dispatch(updateSharingLink(resp.data));
+
+              return _context13.abrupt("return", resp);
+
+            case 8:
             case "end":
-              return _context10.stop();
+              return _context13.stop();
           }
-        }, _callee10);
+        }, _callee13);
       }));
 
-      return function (_x16, _x17) {
-        return _ref12.apply(this, arguments);
+      return function (_x23, _x24, _x25, _x26) {
+        return _ref15.apply(this, arguments);
       };
     }());
 
@@ -501,75 +702,99 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
      * @return {Array}
      */
     _defineProperty(_this2, "updateDocumentPermissions", /*#__PURE__*/function () {
-      var _ref13 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12(document, options) {
+      var _ref16 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15(document, options) {
         var verbs, expiresAt, password, permissions, responses;
-        return _regeneratorRuntime.wrap(function _callee12$(_context12) {
-          while (1) switch (_context12.prev = _context12.next) {
+        return _regeneratorRuntime.wrap(function _callee15$(_context15) {
+          while (1) switch (_context15.prev = _context15.next) {
             case 0:
               verbs = options.verbs, expiresAt = options.expiresAt, password = options.password;
               permissions = _getDocumentPermissions(_this2.state, document.id);
-              _context12.next = 4;
+
+              if (!(!permissions || permissions.length === 0)) {
+                _context15.next = 4;
+                break;
+              }
+
+              return _context15.abrupt("return", []);
+
+            case 4:
+              _context15.next = 6;
               return Promise.all(permissions.map( /*#__PURE__*/function () {
-                var _ref14 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11(permissionDocument) {
+                var _ref17 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14(permissionDocument) {
                   var updatedPermissions, resp;
-                  return _regeneratorRuntime.wrap(function _callee11$(_context11) {
-                    while (1) switch (_context11.prev = _context11.next) {
+                  return _regeneratorRuntime.wrap(function _callee14$(_context14) {
+                    while (1) switch (_context14.prev = _context14.next) {
                       case 0:
                         updatedPermissions = permissionDocument.attributes.permissions;
-                        Object.keys(updatedPermissions).map(function (permType) {
+                        Object.keys(updatedPermissions).forEach(function (permType) {
                           updatedPermissions[permType].verbs = verbs;
                         });
-                        _context11.next = 4;
+
+                        if (!document.driveId) {
+                          _context14.next = 8;
+                          break;
+                        }
+
+                        _context14.next = 5;
+                        return _this2.updateSharedDrivePermissions(document, permissionDocument, updatedPermissions, options);
+
+                      case 5:
+                        _context14.t0 = _context14.sent;
+                        _context14.next = 11;
+                        break;
+
+                      case 8:
+                        _context14.next = 10;
                         return _this2.permissionCol.add(permissionDocument, updatedPermissions, {
                           expiresAt: expiresAt,
                           password: password
                         });
 
-                      case 4:
-                        resp = _context11.sent;
-
-                        _this2.dispatch(updateSharingLink(resp.data));
+                      case 10:
+                        _context14.t0 = _context14.sent;
 
-                        return _context11.abrupt("return", resp);
+                      case 11:
+                        resp = _context14.t0;
+                        return _context14.abrupt("return", resp);
 
-                      case 7:
+                      case 13:
                       case "end":
-                        return _context11.stop();
+                        return _context14.stop();
                     }
-                  }, _callee11);
+                  }, _callee14);
                 }));
 
-                return function (_x20) {
-                  return _ref14.apply(this, arguments);
+                return function (_x29) {
+                  return _ref17.apply(this, arguments);
                 };
               }()));
 
-            case 4:
-              responses = _context12.sent;
-              return _context12.abrupt("return", responses);
-
             case 6:
+              responses = _context15.sent;
+              return _context15.abrupt("return", responses);
+
+            case 8:
             case "end":
-              return _context12.stop();
+              return _context15.stop();
           }
-        }, _callee12);
+        }, _callee15);
       }));
 
-      return function (_x18, _x19) {
-        return _ref13.apply(this, arguments);
+      return function (_x27, _x28) {
+        return _ref16.apply(this, arguments);
       };
     }());
 
     _defineProperty(_this2, "revokeSharingLink", /*#__PURE__*/function () {
-      var _ref15 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13(document) {
+      var _ref18 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16(document) {
         var perms;
-        return _regeneratorRuntime.wrap(function _callee13$(_context13) {
-          while (1) switch (_context13.prev = _context13.next) {
+        return _regeneratorRuntime.wrap(function _callee16$(_context16) {
+          while (1) switch (_context16.prev = _context16.next) {
             case 0:
               // Because some duplicate links have been created in the past, we must ensure
               // we revoke all of them
               perms = _getDocumentPermissions(_this2.state, document.id);
-              _context13.next = 3;
+              _context16.next = 3;
               return Promise.all(perms.map(function (p) {
                 return _this2.permissionCol.destroy(p);
               }));
@@ -579,13 +804,13 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
 
             case 4:
             case "end":
-              return _context13.stop();
+              return _context16.stop();
           }
-        }, _callee13);
+        }, _callee16);
       }));
 
-      return function (_x21) {
-        return _ref15.apply(this, arguments);
+      return function (_x30) {
+        return _ref18.apply(this, arguments);
       };
     }());
 
@@ -634,6 +859,9 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
       getSharingType: function getSharingType(docId) {
         return _getSharingType(_this2.state, docId, _instanceUri);
       },
+      getSharingById: function getSharingById(docId) {
+        return _getSharingById(_this2.state, docId);
+      },
       getSharingForSelf: function getSharingForSelf(docId) {
         return _getSharingForSelf(_this2.state, docId);
       },
@@ -661,6 +889,7 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
       revokeGroup: _this2.revokeGroup,
       revokeSelf: _this2.revokeSelf,
       shareByLink: _this2.shareByLink,
+      getFederatedShareLink: _this2.getFederatedShareLink,
       updateDocumentPermissions: _this2.updateDocumentPermissions,
       revokeSharingLink: _this2.revokeSharingLink,
       hasLoadedAtLeastOnePage: false,
@@ -668,7 +897,8 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
       revokeAllRecipients: _this2.revokeAllRecipients,
       refresh: _this2.fetchAllSharings,
       hasWriteAccess: _this2.hasWriteAccess,
-      renameSharedDrive: _this2.renameSharedDrive
+      renameSharedDrive: _this2.renameSharedDrive,
+      updateSharingMemberType: _this2.updateSharingMemberType
     };
     _this2.isPublic = (_props$isPublic = props.isPublic) !== null && _props$isPublic !== void 0 ? _props$isPublic : false;
     _this2.realtime = null;
@@ -704,14 +934,14 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
   }, {
     key: "fetchAllSharings",
     value: function () {
-      var _fetchAllSharings = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14() {
+      var _fetchAllSharings = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee17() {
         var _this$props, doctype, client, _yield$Promise$all, _yield$Promise$all2, sharings, permissions, apps, sharedDocIds, resp, folderPaths, filePaths;
 
-        return _regeneratorRuntime.wrap(function _callee14$(_context14) {
-          while (1) switch (_context14.prev = _context14.next) {
+        return _regeneratorRuntime.wrap(function _callee17$(_context17) {
+          while (1) switch (_context17.prev = _context17.next) {
             case 0:
               if (!this.isPublic) {
-                _context14.next = 4;
+                _context17.next = 4;
                 break;
               }
 
@@ -720,17 +950,17 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
               this.setState({
                 allLoaded: true
               });
-              return _context14.abrupt("return");
+              return _context17.abrupt("return");
 
             case 4:
               _this$props = this.props, doctype = _this$props.doctype, client = _this$props.client;
-              _context14.next = 7;
+              _context17.next = 7;
               return Promise.all([this.sharingCol.findByDoctype(doctype, {
                 withSharedDocs: false
               }), this.permissionCol.findLinksByDoctype(doctype), client.fetchQueryAndGetFromState(fetchApps())]);
 
             case 7:
-              _yield$Promise$all = _context14.sent;
+              _yield$Promise$all = _context17.sent;
               _yield$Promise$all2 = _slicedToArray(_yield$Promise$all, 3);
               sharings = _yield$Promise$all2[0];
               permissions = _yield$Promise$all2[1];
@@ -745,35 +975,35 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
                 hasLoadedAtLeastOnePage: true
               }); // eslint-disable-next-line promise/catch-or-return
 
-              _context14.next = 16;
+              _context17.next = 16;
               return fetchNextPermissions(permissions, this.dispatch, this.permissionCol);
 
             case 16:
               if (!(doctype === 'io.cozy.files')) {
-                _context14.next = 26;
+                _context17.next = 26;
                 break;
               }
 
               sharedDocIds = getSharedDocIdsBySharings(sharings);
-              _context14.next = 20;
+              _context17.next = 20;
               return client.collection(doctype).all({
                 keys: sharedDocIds
               });
 
             case 20:
-              resp = _context14.sent;
+              resp = _context17.sent;
               folderPaths = resp.data.filter(function (f) {
                 return f.type === 'directory' && !f.trashed;
               }).map(function (f) {
                 return f.path;
               });
-              _context14.next = 24;
+              _context17.next = 24;
               return fetchFilesPaths(client, doctype, resp.data.filter(function (f) {
                 return f.type !== 'directory' && !f.trashed;
               }));
 
             case 24:
-              filePaths = _context14.sent;
+              filePaths = _context17.sent;
               this.dispatch(receivePaths([].concat(_toConsumableArray(folderPaths), _toConsumableArray(filePaths))));
 
             case 26:
@@ -783,9 +1013,9 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
 
             case 27:
             case "end":
-              return _context14.stop();
+              return _context17.stop();
           }
-        }, _callee14, this);
+        }, _callee17, this);
       }));
 
       function fetchAllSharings() {