From b3244215340f2268cfec5687c1319c50f3b3c0e0 Mon Sep 17 00:00:00 2001
From: sachaaaaa <sacha@loki.network>
Date: Fri, 1 Nov 2019 16:16:44 +1100
Subject: [PATCH] Allow primary device to assign local aliases to paired
 devices

---
 _locales/en/messages.json              |  3 +++
 background.html                        |  5 +++-
 js/views/device_pairing_dialog_view.js | 32 ++++++++++++++++++++++----
 3 files changed, 35 insertions(+), 5 deletions(-)

diff --git a/_locales/en/messages.json b/_locales/en/messages.json
index 1f4aa04cc..279269fe7 100644
--- a/_locales/en/messages.json
+++ b/_locales/en/messages.json
@@ -955,6 +955,9 @@
   "allowPairing": {
     "message": "Allow Pairing"
   },
+  "provideDeviceAlias": {
+    "message": "Please provide an alias for this paired device"
+  },
   "clear": {
     "message": "Clear"
   },
diff --git a/background.html b/background.html
index b859fae6b..0024c07ea 100644
--- a/background.html
+++ b/background.html
@@ -285,7 +285,10 @@
         <h4>{{ requestAcceptedTitle }}</h4>
         <p class="secretWords"></p>
         <p class="transmissionStatus">Please be patient...</p>
-        <div class='buttons'>
+        <div id="deviceAliasView" style="display: none;" >
+          <input id="deviceAlias" type="text" placeholder="Device Alias" required>
+        </div>
+          <div class='buttons'>
           <button class="ok" style="display: none;">{{ okText }}</button>
         </div>
       </div>
diff --git a/js/views/device_pairing_dialog_view.js b/js/views/device_pairing_dialog_view.js
index c4e339c11..c5af9c002 100644
--- a/js/views/device_pairing_dialog_view.js
+++ b/js/views/device_pairing_dialog_view.js
@@ -1,4 +1,4 @@
-/* global Whisper, i18n, libloki, textsecure */
+/* global Whisper, i18n, libloki, textsecure, ConversationController */
 
 // eslint-disable-next-line func-names
 (function() {
@@ -19,6 +19,7 @@
       this.pubKey = null;
       this.accepted = false;
       this.isListening = false;
+      this.success = false;
     },
     events: {
       'click #startPairing': 'startReceivingRequests',
@@ -48,6 +49,13 @@
       this.showView();
     },
     stopReceivingRequests() {
+      if (this.success) {
+        const deviceAlias = this.$('#deviceAlias')[0].value.trim();
+        const conv = ConversationController.get(this.pubKey);
+        if (conv) {
+          conv.setNickname(deviceAlias);
+        }
+      }
       this.trigger('stopReceivingRequests');
       this.reset();
       this.showView();
@@ -69,11 +77,22 @@
     },
     transmisssionCB(errors) {
       if (!errors) {
-        this.$('.transmissionStatus').text(i18n('sent'));
+        this.$('.transmissionStatus').text(i18n('provideDeviceAlias'));
+        this.$('#deviceAliasView').show();
+        this.$('#deviceAlias').on('keydown', (e) => {
+          if (e.target.value.trim()) {
+            this.$('.requestAcceptedView .ok').removeAttr('disabled');
+          } else {
+            this.$('.requestAcceptedView .ok').attr('disabled', true);
+          }
+        });
+        this.$('.requestAcceptedView .ok').show();
+        this.$('.requestAcceptedView .ok').attr('disabled', true);
+        this.success = true;
       } else {
         this.$('.transmissionStatus').text(errors);
+        this.$('.requestAcceptedView .ok').show();
       }
-      this.$('.requestAcceptedView .ok').show();
     },
     skipDevice() {
       this.trigger('devicePairingRequestRejected', this.pubKey);
@@ -99,8 +118,13 @@
         if (pubKeys && pubKeys.length > 0) {
           this.$('#pairedPubKeys').empty();
           pubKeys.forEach(x => {
+            let deviceAlias = 'Paired Device';
             const secretWords = window.mnemonic.pubkey_to_secret_words(x);
-            this.$('#pairedPubKeys').append(`<li>(${secretWords})</li>`);
+            const conv = ConversationController.get(x);
+            if (conv) {
+              deviceAlias = conv.getNickname();
+            }
+            this.$('#pairedPubKeys').append(`<li>${deviceAlias} (${secretWords})</li>`);
           });
         }
       } else if (this.accepted) {