cozy-sharing 28.0.1 → 28.0.2

package/CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [28.0.2](https://github.com/cozy/cozy-libs/compare/cozy-sharing@28.0.1...cozy-sharing@28.0.2) (2025-12-08)
+
+### Bug Fixes
+
+- Use sharing provider to rename shared drive ([6208d72](https://github.com/cozy/cozy-libs/commit/6208d72bf6ebb7330dff82e90fb207754b135671))
+
 ## [28.0.1](https://github.com/cozy/cozy-libs/compare/cozy-sharing@28.0.0...cozy-sharing@28.0.1) (2025-12-05)
 
 ### Bug Fixes

package/dist/SharingProvider.js

@@ -247,176 +247,203 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
       };
     }());
 
+    _defineProperty(_this2, "renameSharedDrive", /*#__PURE__*/function () {
+      var _ref6 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee4(document, newName) {
+        var sharing;
+        return _regeneratorRuntime.wrap(function _callee4$(_context4) {
+          while (1) switch (_context4.prev = _context4.next) {
+            case 0:
+              sharing = getDocumentSharing(_this2.state, document.id);
+              _context4.next = 3;
+              return _this2.sharingCol.renameSharedDrive(sharing, newName);
+
+            case 3:
+              sharing.description = newName;
+
+              _this2.dispatch(updateSharing(sharing));
+
+            case 5:
+            case "end":
+              return _context4.stop();
+          }
+        }, _callee4);
+      }));
+
+      return function (_x4, _x5) {
+        return _ref6.apply(this, arguments);
+      };
+    }());
+
     _defineProperty(_this2, "revokeAllRecipients", /*#__PURE__*/function () {
-      var _ref6 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee5(document) {
+      var _ref7 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee6(document) {
         var _this2$props3, client, doctype, recipients, sharing;
 
-        return _regeneratorRuntime.wrap(function _callee5$(_context5) {
-          while (1) switch (_context5.prev = _context5.next) {
+        return _regeneratorRuntime.wrap(function _callee6$(_context6) {
+          while (1) switch (_context6.prev = _context6.next) {
             case 0:
               _this2$props3 = _this2.props, client = _this2$props3.client, doctype = _this2$props3.doctype;
               recipients = _getRecipients(_this2.state, document.id);
               sharing = getDocumentSharing(_this2.state, document.id);
-              _context5.next = 5;
+              _context6.next = 5;
               return _this2.sharingCol.revokeAllRecipients(sharing);
 
             case 5:
               recipients.map( /*#__PURE__*/function () {
-                var _ref7 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee4(recipient, recipientIndex) {
-                  return _regeneratorRuntime.wrap(function _callee4$(_context4) {
-                    while (1) switch (_context4.prev = _context4.next) {
+                var _ref8 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee5(recipient, recipientIndex) {
+                  return _regeneratorRuntime.wrap(function _callee5$(_context5) {
+                    while (1) switch (_context5.prev = _context5.next) {
                       case 0:
-                        _context4.t0 = _this2;
-                        _context4.t1 = revokeRecipient;
-                        _context4.t2 = sharing;
-                        _context4.t3 = recipientIndex;
-                        _context4.t4 = document.path;
-
-                        if (_context4.t4) {
-                          _context4.next = 9;
+                        _context5.t0 = _this2;
+                        _context5.t1 = revokeRecipient;
+                        _context5.t2 = sharing;
+                        _context5.t3 = recipientIndex;
+                        _context5.t4 = document.path;
+
+                        if (_context5.t4) {
+                          _context5.next = 9;
                           break;
                         }
 
-                        _context4.next = 8;
+                        _context5.next = 8;
                         return fetchFilesPaths(client, doctype, [document]);
 
                       case 8:
-                        _context4.t4 = _context4.sent;
+                        _context5.t4 = _context5.sent;
 
                       case 9:
-                        _context4.t5 = _context4.t4;
-                        _context4.t6 = (0, _context4.t1)(_context4.t2, _context4.t3, _context4.t5);
+                        _context5.t5 = _context5.t4;
+                        _context5.t6 = (0, _context5.t1)(_context5.t2, _context5.t3, _context5.t5);
 
-                        _context4.t0.dispatch.call(_context4.t0, _context4.t6);
+                        _context5.t0.dispatch.call(_context5.t0, _context5.t6);
 
                       case 12:
                       case "end":
-                        return _context4.stop();
+                        return _context5.stop();
                     }
-                  }, _callee4);
+                  }, _callee5);
                 }));
 
-                return function (_x5, _x6) {
-                  return _ref7.apply(this, arguments);
+                return function (_x7, _x8) {
+                  return _ref8.apply(this, arguments);
                 };
               }());
 
             case 6:
             case "end":
-              return _context5.stop();
+              return _context6.stop();
           }
-        }, _callee5);
+        }, _callee6);
       }));
 
-      return function (_x4) {
-        return _ref6.apply(this, arguments);
+      return function (_x6) {
+        return _ref7.apply(this, arguments);
       };
     }());
 
     _defineProperty(_this2, "revoke", /*#__PURE__*/function () {
-      var _ref8 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee6(document, sharingId, recipientIndex) {
+      var _ref9 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee7(document, sharingId, recipientIndex) {
         var _this2$props4, client, doctype, sharing;
 
-        return _regeneratorRuntime.wrap(function _callee6$(_context6) {
-          while (1) switch (_context6.prev = _context6.next) {
+        return _regeneratorRuntime.wrap(function _callee7$(_context7) {
+          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);
-              _context6.next = 4;
+              _context7.next = 4;
               return _this2.sharingCol.revokeRecipient(sharing, recipientIndex);
 
             case 4:
-              _context6.t0 = _this2;
-              _context6.t1 = revokeRecipient;
-              _context6.t2 = sharing;
-              _context6.t3 = recipientIndex;
-              _context6.t4 = document.path;
-
-              if (_context6.t4) {
-                _context6.next = 13;
+              _context7.t0 = _this2;
+              _context7.t1 = revokeRecipient;
+              _context7.t2 = sharing;
+              _context7.t3 = recipientIndex;
+              _context7.t4 = document.path;
+
+              if (_context7.t4) {
+                _context7.next = 13;
                 break;
               }
 
-              _context6.next = 12;
+              _context7.next = 12;
               return fetchFilesPaths(client, doctype, [document]);
 
             case 12:
-              _context6.t4 = _context6.sent;
+              _context7.t4 = _context7.sent;
 
             case 13:
-              _context6.t5 = _context6.t4;
-              _context6.t6 = (0, _context6.t1)(_context6.t2, _context6.t3, _context6.t5);
+              _context7.t5 = _context7.t4;
+              _context7.t6 = (0, _context7.t1)(_context7.t2, _context7.t3, _context7.t5);
 
-              _context6.t0.dispatch.call(_context6.t0, _context6.t6);
+              _context7.t0.dispatch.call(_context7.t0, _context7.t6);
 
             case 16:
             case "end":
-              return _context6.stop();
+              return _context7.stop();
           }
-        }, _callee6);
+        }, _callee7);
       }));
 
-      return function (_x7, _x8, _x9) {
-        return _ref8.apply(this, arguments);
+      return function (_x9, _x10, _x11) {
+        return _ref9.apply(this, arguments);
       };
     }());
 
     _defineProperty(_this2, "revokeGroup", /*#__PURE__*/function () {
-      var _ref9 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee7(document, sharingId, recipientIndex) {
+      var _ref10 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee8(document, sharingId, recipientIndex) {
         var _this2$props5, client, doctype, sharing;
 
-        return _regeneratorRuntime.wrap(function _callee7$(_context7) {
-          while (1) switch (_context7.prev = _context7.next) {
+        return _regeneratorRuntime.wrap(function _callee8$(_context8) {
+          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);
-              _context7.next = 4;
+              _context8.next = 4;
               return _this2.sharingCol.revokeGroup(sharing, recipientIndex);
 
             case 4:
-              _context7.t0 = _this2;
-              _context7.t1 = revokeGroupFromState;
-              _context7.t2 = sharing;
-              _context7.t3 = recipientIndex;
-              _context7.t4 = document.path;
-
-              if (_context7.t4) {
-                _context7.next = 13;
+              _context8.t0 = _this2;
+              _context8.t1 = revokeGroupFromState;
+              _context8.t2 = sharing;
+              _context8.t3 = recipientIndex;
+              _context8.t4 = document.path;
+
+              if (_context8.t4) {
+                _context8.next = 13;
                 break;
               }
 
-              _context7.next = 12;
+              _context8.next = 12;
               return fetchFilesPaths(client, doctype, [document]);
 
             case 12:
-              _context7.t4 = _context7.sent;
+              _context8.t4 = _context8.sent;
 
             case 13:
-              _context7.t5 = _context7.t4;
-              _context7.t6 = (0, _context7.t1)(_context7.t2, _context7.t3, _context7.t5);
+              _context8.t5 = _context8.t4;
+              _context8.t6 = (0, _context8.t1)(_context8.t2, _context8.t3, _context8.t5);
 
-              _context7.t0.dispatch.call(_context7.t0, _context7.t6);
+              _context8.t0.dispatch.call(_context8.t0, _context8.t6);
 
             case 16:
             case "end":
-              return _context7.stop();
+              return _context8.stop();
           }
-        }, _callee7);
+        }, _callee8);
       }));
 
-      return function (_x10, _x11, _x12) {
-        return _ref9.apply(this, arguments);
+      return function (_x12, _x13, _x14) {
+        return _ref10.apply(this, arguments);
       };
     }());
 
     _defineProperty(_this2, "revokeSelf", /*#__PURE__*/function () {
-      var _ref10 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee8(document) {
+      var _ref11 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee9(document) {
         var sharing;
-        return _regeneratorRuntime.wrap(function _callee8$(_context8) {
-          while (1) switch (_context8.prev = _context8.next) {
+        return _regeneratorRuntime.wrap(function _callee9$(_context9) {
+          while (1) switch (_context9.prev = _context9.next) {
             case 0:
               sharing = _getSharingForSelf(_this2.state, document.id);
-              _context8.next = 3;
+              _context9.next = 3;
               return _this2.sharingCol.revokeSelf(sharing);
 
             case 3:
@@ -424,41 +451,41 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
 
             case 4:
             case "end":
-              return _context8.stop();
+              return _context9.stop();
           }
-        }, _callee8);
+        }, _callee9);
       }));
 
-      return function (_x13) {
-        return _ref10.apply(this, arguments);
+      return function (_x15) {
+        return _ref11.apply(this, arguments);
       };
     }());
 
     _defineProperty(_this2, "shareByLink", /*#__PURE__*/function () {
-      var _ref11 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee9(document, options) {
+      var _ref12 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10(document, options) {
         var resp;
-        return _regeneratorRuntime.wrap(function _callee9$(_context9) {
-          while (1) switch (_context9.prev = _context9.next) {
+        return _regeneratorRuntime.wrap(function _callee10$(_context10) {
+          while (1) switch (_context10.prev = _context10.next) {
             case 0:
-              _context9.next = 2;
+              _context10.next = 2;
               return _this2.permissionCol.createSharingLink(document, options);
 
             case 2:
-              resp = _context9.sent;
+              resp = _context10.sent;
 
               _this2.dispatch(addSharingLink(resp.data));
 
-              return _context9.abrupt("return", resp);
+              return _context10.abrupt("return", resp);
 
             case 5:
             case "end":
-              return _context9.stop();
+              return _context10.stop();
           }
-        }, _callee9);
+        }, _callee10);
       }));
 
-      return function (_x14, _x15) {
-        return _ref11.apply(this, arguments);
+      return function (_x16, _x17) {
+        return _ref12.apply(this, arguments);
       };
     }());
 
@@ -474,75 +501,75 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
      * @return {Array}
      */
     _defineProperty(_this2, "updateDocumentPermissions", /*#__PURE__*/function () {
-      var _ref12 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11(document, options) {
+      var _ref13 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12(document, options) {
         var verbs, expiresAt, password, permissions, responses;
-        return _regeneratorRuntime.wrap(function _callee11$(_context11) {
-          while (1) switch (_context11.prev = _context11.next) {
+        return _regeneratorRuntime.wrap(function _callee12$(_context12) {
+          while (1) switch (_context12.prev = _context12.next) {
             case 0:
               verbs = options.verbs, expiresAt = options.expiresAt, password = options.password;
               permissions = _getDocumentPermissions(_this2.state, document.id);
-              _context11.next = 4;
+              _context12.next = 4;
               return Promise.all(permissions.map( /*#__PURE__*/function () {
-                var _ref13 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10(permissionDocument) {
+                var _ref14 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11(permissionDocument) {
                   var updatedPermissions, resp;
-                  return _regeneratorRuntime.wrap(function _callee10$(_context10) {
-                    while (1) switch (_context10.prev = _context10.next) {
+                  return _regeneratorRuntime.wrap(function _callee11$(_context11) {
+                    while (1) switch (_context11.prev = _context11.next) {
                       case 0:
                         updatedPermissions = permissionDocument.attributes.permissions;
                         Object.keys(updatedPermissions).map(function (permType) {
                           updatedPermissions[permType].verbs = verbs;
                         });
-                        _context10.next = 4;
+                        _context11.next = 4;
                         return _this2.permissionCol.add(permissionDocument, updatedPermissions, {
                           expiresAt: expiresAt,
                           password: password
                         });
 
                       case 4:
-                        resp = _context10.sent;
+                        resp = _context11.sent;
 
                         _this2.dispatch(updateSharingLink(resp.data));
 
-                        return _context10.abrupt("return", resp);
+                        return _context11.abrupt("return", resp);
 
                       case 7:
                       case "end":
-                        return _context10.stop();
+                        return _context11.stop();
                     }
-                  }, _callee10);
+                  }, _callee11);
                 }));
 
-                return function (_x18) {
-                  return _ref13.apply(this, arguments);
+                return function (_x20) {
+                  return _ref14.apply(this, arguments);
                 };
               }()));
 
             case 4:
-              responses = _context11.sent;
-              return _context11.abrupt("return", responses);
+              responses = _context12.sent;
+              return _context12.abrupt("return", responses);
 
             case 6:
             case "end":
-              return _context11.stop();
+              return _context12.stop();
           }
-        }, _callee11);
+        }, _callee12);
       }));
 
-      return function (_x16, _x17) {
-        return _ref12.apply(this, arguments);
+      return function (_x18, _x19) {
+        return _ref13.apply(this, arguments);
       };
     }());
 
     _defineProperty(_this2, "revokeSharingLink", /*#__PURE__*/function () {
-      var _ref14 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12(document) {
+      var _ref15 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13(document) {
         var perms;
-        return _regeneratorRuntime.wrap(function _callee12$(_context12) {
-          while (1) switch (_context12.prev = _context12.next) {
+        return _regeneratorRuntime.wrap(function _callee13$(_context13) {
+          while (1) switch (_context13.prev = _context13.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);
-              _context12.next = 3;
+              _context13.next = 3;
               return Promise.all(perms.map(function (p) {
                 return _this2.permissionCol.destroy(p);
               }));
@@ -552,13 +579,13 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
 
             case 4:
             case "end":
-              return _context12.stop();
+              return _context13.stop();
           }
-        }, _callee12);
+        }, _callee13);
       }));
 
-      return function (_x19) {
-        return _ref14.apply(this, arguments);
+      return function (_x21) {
+        return _ref15.apply(this, arguments);
       };
     }());
 
@@ -640,7 +667,8 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
       allLoaded: false,
       revokeAllRecipients: _this2.revokeAllRecipients,
       refresh: _this2.fetchAllSharings,
-      hasWriteAccess: _this2.hasWriteAccess
+      hasWriteAccess: _this2.hasWriteAccess,
+      renameSharedDrive: _this2.renameSharedDrive
     };
     _this2.isPublic = (_props$isPublic = props.isPublic) !== null && _props$isPublic !== void 0 ? _props$isPublic : false;
     _this2.realtime = null;
@@ -676,14 +704,14 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
   }, {
     key: "fetchAllSharings",
     value: function () {
-      var _fetchAllSharings = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13() {
+      var _fetchAllSharings = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14() {
         var _this$props, doctype, client, _yield$Promise$all, _yield$Promise$all2, sharings, permissions, apps, sharedDocIds, resp, folderPaths, filePaths;
 
-        return _regeneratorRuntime.wrap(function _callee13$(_context13) {
-          while (1) switch (_context13.prev = _context13.next) {
+        return _regeneratorRuntime.wrap(function _callee14$(_context14) {
+          while (1) switch (_context14.prev = _context14.next) {
             case 0:
               if (!this.isPublic) {
-                _context13.next = 4;
+                _context14.next = 4;
                 break;
               }
 
@@ -692,17 +720,17 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
               this.setState({
                 allLoaded: true
               });
-              return _context13.abrupt("return");
+              return _context14.abrupt("return");
 
             case 4:
               _this$props = this.props, doctype = _this$props.doctype, client = _this$props.client;
-              _context13.next = 7;
+              _context14.next = 7;
               return Promise.all([this.sharingCol.findByDoctype(doctype, {
                 withSharedDocs: false
               }), this.permissionCol.findLinksByDoctype(doctype), client.fetchQueryAndGetFromState(fetchApps())]);
 
             case 7:
-              _yield$Promise$all = _context13.sent;
+              _yield$Promise$all = _context14.sent;
               _yield$Promise$all2 = _slicedToArray(_yield$Promise$all, 3);
               sharings = _yield$Promise$all2[0];
               permissions = _yield$Promise$all2[1];
@@ -717,35 +745,35 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
                 hasLoadedAtLeastOnePage: true
               }); // eslint-disable-next-line promise/catch-or-return
 
-              _context13.next = 16;
+              _context14.next = 16;
               return fetchNextPermissions(permissions, this.dispatch, this.permissionCol);
 
             case 16:
               if (!(doctype === 'io.cozy.files')) {
-                _context13.next = 26;
+                _context14.next = 26;
                 break;
               }
 
               sharedDocIds = getSharedDocIdsBySharings(sharings);
-              _context13.next = 20;
+              _context14.next = 20;
               return client.collection(doctype).all({
                 keys: sharedDocIds
               });
 
             case 20:
-              resp = _context13.sent;
+              resp = _context14.sent;
               folderPaths = resp.data.filter(function (f) {
                 return f.type === 'directory' && !f.trashed;
               }).map(function (f) {
                 return f.path;
               });
-              _context13.next = 24;
+              _context14.next = 24;
               return fetchFilesPaths(client, doctype, resp.data.filter(function (f) {
                 return f.type !== 'directory' && !f.trashed;
               }));
 
             case 24:
-              filePaths = _context13.sent;
+              filePaths = _context14.sent;
               this.dispatch(receivePaths([].concat(_toConsumableArray(folderPaths), _toConsumableArray(filePaths))));
 
             case 26:
@@ -755,9 +783,9 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
 
             case 27:
             case "end":
-              return _context13.stop();
+              return _context14.stop();
           }
-        }, _callee13, this);
+        }, _callee14, this);
       }));
 
       function fetchAllSharings() {

package/dist/components/SharedDrive/SharedDriveEditModal.js

@@ -8,6 +8,7 @@ import { useClient } from 'cozy-client';
 import { useAlert } from 'cozy-ui/transpiled/react/providers/Alert';
 import { DumbSharedDriveModal } from './DumbSharedDriveModal';
 import withLocales from '../../hoc/withLocales';
+import { useSharingContext } from '../../hooks/useSharingContext';
 import { Contact } from '../../models';
 export var SharedDriveEditModal = withLocales(function (_ref) {
   var document = _ref.document,
@@ -24,6 +25,9 @@ export var SharedDriveEditModal = withLocales(function (_ref) {
   var _useAlert = useAlert(),
       showAlert = _useAlert.showAlert;
 
+  var _useSharingContext = useSharingContext(),
+      renameSharedDrive = _useSharingContext.renameSharedDrive;
+
   var createContact = function createContact(contact) {
     return client.create(Contact.doctype, contact);
   };
@@ -44,7 +48,7 @@ export var SharedDriveEditModal = withLocales(function (_ref) {
           case 0:
             _context.prev = 0;
             _context.next = 3;
-            return client.collection('io.cozy.sharings').renameSharedDrive(sharing, value);
+            return renameSharedDrive(document, value);
 
           case 3:
             showAlert({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-sharing",
-  "version": "28.0.1",
+  "version": "28.0.2",
   "description": "Provides sharing login for React applications.",
   "main": "dist/index.js",
   "author": "Cozy",
@@ -83,5 +83,5 @@
   "sideEffects": [
     "*.css"
   ],
-  "gitHead": "bb5068da547bbf4b1b6512be5c46166f4d0e3a03"
+  "gitHead": "cc29fb457750a8823c3a79441efd34de2bcfa5b9"
 }

package/src/SharingProvider.jsx

@@ -89,7 +89,8 @@ export class SharingProvider extends Component {
       allLoaded: false,
       revokeAllRecipients: this.revokeAllRecipients,
       refresh: this.fetchAllSharings,
-      hasWriteAccess: this.hasWriteAccess
+      hasWriteAccess: this.hasWriteAccess,
+      renameSharedDrive: this.renameSharedDrive
     }
     this.isPublic = props.isPublic ?? false
     this.realtime = null
@@ -278,6 +279,13 @@ export class SharingProvider extends Component {
     this.dispatch(updateSharing(resp.data))
   }
 
+  renameSharedDrive = async (document, newName) => {
+    const sharing = getDocumentSharing(this.state, document.id)
+    await this.sharingCol.renameSharedDrive(sharing, newName)
+    sharing.description = newName
+    this.dispatch(updateSharing(sharing))
+  }
+
   revokeAllRecipients = async document => {
     const { client, doctype } = this.props
     const recipients = getRecipients(this.state, document.id)

package/src/components/SharedDrive/SharedDriveEditModal.jsx

@@ -7,6 +7,7 @@ import { useAlert } from 'cozy-ui/transpiled/react/providers/Alert'
 
 import { DumbSharedDriveModal } from './DumbSharedDriveModal'
 import withLocales from '../../hoc/withLocales'
+import { useSharingContext } from '../../hooks/useSharingContext'
 import { Contact } from '../../models'
 
 export const SharedDriveEditModal = withLocales(
@@ -14,6 +15,7 @@ export const SharedDriveEditModal = withLocales(
     const client = useClient()
     const { t } = useI18n()
     const { showAlert } = useAlert()
+    const { renameSharedDrive } = useSharingContext()
 
     const createContact = contact => client.create(Contact.doctype, contact)
     const [name, setName] = useState(sharing?.description)
@@ -23,9 +25,7 @@ export const SharedDriveEditModal = withLocales(
     }
     const onRename = async value => {
       try {
-        await client
-          .collection('io.cozy.sharings')
-          .renameSharedDrive(sharing, value)
+        await renameSharedDrive(document, value)
         showAlert({
           message: t('SharedDrive.sharedDriveModal.successNotificationUpdate'),
           severity: 'success'