Remove an owner

www/common/common-ui-elements.js

@@ -112,23 +112,27 @@ define([
             cb(void 0, data);
         });
     };
-    var createOwnerModal = function (common, owners) {
+    var createOwnerModal = function (common, channel, owners) {
         var friends = common.getFriends(true);
+        var sframeChan = common.getSframeChannel();
+        var priv = common.getMetadataMgr().getPrivateData();
+        var edPublic = priv.edPublic;
 
         // Remove owner column
+        var drawRemove = function () {
             var _owners = {};
-        owners.forEach(function (edPublic) {
+            owners.forEach(function (ed) {
                 var f;
                 Object.keys(friends).some(function (c) {
-                if (friends[c].edPublic === edPublic) {
+                    if (friends[c].edPublic === ed) {
                         f = friends[c];
                         return true;
                     }
                 });
-            _owners[edPublic] = f || {
+                _owners[ed] = f || {
-                displayName: 'Unknown user: '+ edPublic, // XXX
+                    displayName: 'Unknown user: '+ ed, // XXX
                     notifications: true,
-                edPublic: edPublic,
+                    edPublic: ed,
                 };
             });
             var removeCol = UIElements.getFriendsList('Remove an existing owner instantly', {
@@ -136,8 +140,6 @@ define([
                 friends: _owners,
                 noFilter: true
             }, function () {
-            // XXX onSelect...
-            //onSelect...
                 console.log(arguments);
             });
             var $div1 = $(removeCol.div);
@@ -152,11 +154,51 @@ define([
                     order = order ? 'order:'+order : '';
                     $(this).removeClass('cp-selected').attr('style', order);
                 }
-            // XXX onSelect...
             });
+            // When clicking on the remove button, we check the selected users.
+            // If you try to remove yourself, we'll display an additional warning message
+            var removeButton = h('button.no-margin', 'Remove owners'); // XXX
+            $(removeButton).click(function () {
+                // Check selection
+                var $sel = $div1.find('.cp-share-friend.cp-selected');
+                var sel = $sel.toArray();
+                var me = false;
+                var toRemove = sel.map(function (el) {
+                    var ed = $(el).attr('data-ed');
+                    if (!ed) { return; }
+                    if (ed === edPublic) { me = true; }
+                    return ed;
+                }).filter(function (x) { return x; });
+                // Send the command
+                var send = function () {
+                    sframeChan.query('Q_SET_PAD_METADATA', {
+                        channel: channel,
+                        command: 'RM_OWNERS',
+                        value: toRemove
+                    }, function (err, res) {
+                        err = err || (res && res.error);
+                        if (err) { return void UI.warn('ERROR' + err); } // XXX
+                        owners = res.owners;
+                        drawRemove().insertBefore($div1);
+                        $div1.remove();
+                        UI.log('DONE'); // XXX
+                    });
+                };
+                var msg = me ?
+                  "Are you sure? You're going to give up on your rights, this can't be undone!" :
+                  "Are you sure?"; // XXX
+                UI.confirm(msg, function (yes) {
+                    if (!yes) { return; }
+                    send();
+                });
+            });
+            $div1.append(h('p', removeButton));
+            return $div1;
+        };
 
         // Add owners column
-        var addCol = UIElements.getFriendsList('Ask a friend to be an owner', {
+        var drawAdd = function () {
+            var addCol = UIElements.getFriendsList('Ask a friend to be an owner.', {
                 common: common,
                 friends: friends
             }, function () {
@@ -177,15 +219,49 @@ define([
                 }
                 // XXX onSelect...
             });
+            // When clicking on the add button, we get the selected users.
+            var addButton = h('button.no-margin', 'Add owners'); // XXX
+            $(addButton).click(function () {
+                // Check selection
+                var $sel = $div2.find('.cp-share-friend.cp-selected');
+                var sel = $sel.toArray();
+                var toAdd = sel.map(function (el) {
+                    return $(el).attr('data-curve');
+                }).filter(function (x) { return x; });
+                // Send the command
+                var send = function () {
+                    // XXX Pinning problem....
+                    sframeChan.query('Q_SET_PAD_METADATA', {
+                        channel: channel,
+                        command: 'ADD_OWNERS',
+                        value: toAdd
+                    }, function (err, res) {
+                        err = err || (res && res.error);
+                        if (err) { return void UI.warn('ERROR' + err); } // XXX
+                        owners = res.owners;
+                        drawRemove().insertBefore($div2);
+                        $div2.remove();
+                        UI.log('DONE'); // XXX
+                    });
+                };
+                var msg = "Are you sure?"; // XXX
+                UI.confirm(msg, function (yes) {
+                    if (!yes) { return; }
+                    send();
+                });
+            });
+            //$div2.append(h('p', addButton));
+            return $div2;
+        };
 
         // Create modal
         var link = h('div.cp-share-columns', [
-            removeCol.div,
+            drawRemove()[0],
-            addCol.div
+            drawAdd()[0]
         ]);
         var linkButtons = [{
             className: 'cancel',
-            name: Messages.ok,
+            name: 'CLOSE', // XXX existing key?
             onClick: function () {},
             keys: [27]
         }];
@@ -234,7 +310,7 @@ define([
         if (owned) {
             var manageOwners = h('button.no-margin', 'Manage owners'); // XXX
             $(manageOwners).click(function () {
-                var modal = createOwnerModal(common, data.owners);
+                var modal = createOwnerModal(common, data.channel, data.owners);
                 UI.openCustomModal(modal, {
                     wide: true,
                 });
@@ -437,8 +513,6 @@ define([
     UIElements.getFriendsList = function (label, config, onSelect) {
         var common = config.common;
         var friends = config.friends;
-        var myName = common.getMetadataMgr().getUserData().name;
-        var order = [];
         if (!friends) { return; }
 
         var others = Object.keys(friends).map(function (curve, i) {
@@ -538,12 +612,15 @@ define([
         var common = config.common;
         var title = config.title;
         var friends = config.friends;
+        var myName = common.getMetadataMgr().getUserData().name;
         if (!friends) { return; }
+        var order = [];
 
         var smallCurves = Object.keys(friends).map(function (c) {
             return friends[c].curvePublic.slice(0,8);
         });
 
+        var $div;
         // Replace "copy link" by "share with friends" if at least one friend is selected
         // Also create the "share with friends" button if it doesn't exist
         var refreshButtons = function () {
@@ -606,7 +683,7 @@ define([
 
         var friendsList = UIElements.getFriendsList(Messages.share_linkFriends, config, refreshButtons);
         var div = friendsList.div;
-        var $div = $(div);
+        $div = $(div);
         var others = friendsList.others;
 
         common.getAttribute(['general', 'share-friends'], function (err, val) {

www/common/cryptpad-common.js

@@ -765,6 +765,9 @@ define([
         postMessage("GIVE_PAD_ACCESS", data, cb);
     };
 
+    common.setPadMetadata = function (data, cb) {
+        postMessage('SET_PAD_METADATA', data, cb);
+    };
     common.getPadMetadata = function (data, cb) {
         postMessage('GET_PAD_METADATA', data, cb);
     };

www/common/outer/async-store.js

@@ -1395,6 +1395,15 @@ define([
             }
             cb(channel.data || {});
         };
+        Store.setPadMetadata = function (clientId, data, cb) {
+            if (!data.channel) { return void cb({ error: 'ENOTFOUND'}); }
+            if (!data.command) { return void cb({ error: 'EINVAL' }); }
+            store.rpc.setMetadata(data, function (err, res) {
+                if (err) { return void cb({ error: err }); }
+                if (!Array.isArray(res) || !res.length) { return void cb({}); }
+                cb(res[0]);
+            });
+        };
 
         // GET_FULL_HISTORY from sframe-common-outer
         Store.getFullHistory = function (clientId, data, cb) {

www/common/outer/store-rpc.js

@@ -81,6 +81,7 @@ define([
             REQUEST_PAD_ACCESS: Store.requestPadAccess,
             GIVE_PAD_ACCESS: Store.givePadAccess,
             GET_PAD_METADATA: Store.getPadMetadata,
+            SET_PAD_METADATA: Store.setPadMetadata,
             // Drive
             DRIVE_USEROBJECT: Store.userObjectCommand,
             // Settings,

www/common/sframe-common-outer.js

@@ -1004,6 +1004,10 @@ define([
                 });
             });
 
+            sframeChan.on('Q_SET_PAD_METADATA', function (data, cb) {
+                Cryptpad.setPadMetadata(data, cb);
+            });
+
             if (cfg.messaging) {
                 Notifier.getPermission();