Bug 1129868 - [Loop] Delete button is always shown also when the user deleted the room

Author: crdlc

app/js/helpers/room/room_controller.js

@@ -133,22 +133,29 @@
 
   function onInvalidToken(params) {
     var token = params.token;
-    var options = new OptionMenu({
-      section: navigator.mozL10n.get('invalidRoomToken'),
-      type: 'confirm',
-      items: [
-        {
-          name: 'Cancel',
-          l10nId: 'cancel',
-          method: (token) => {
-            RoomsDB.get(token).then(room => {
-              room.noLongerAvailable = true;
-              Controller.onRoomUpdated(room);
-            });
-          },
-          params: [token]
-        },
-        {
+
+    RoomsDB.get(token).then(room => {
+      var parameters = {
+        type: 'confirm',
+        items: [
+          {
+            name: 'Cancel',
+            l10nId: 'cancel'
+          }
+        ]
+      };
+
+      var title;
+
+      if (room) {
+        title = 'invalidRoomTokenAndDelete';
+
+        parameters.items[0].method = () => {
+          room.noLongerAvailable = true;
+          Controller.onRoomUpdated(room);
+        };
+
+        parameters.items.push({
           name: 'Delete',
           class: 'danger',
           l10nId: 'delete',
@@ -160,8 +167,19 @@
             Rooms.delete(token).then(deleteFromDB, deleteFromDB);
           },
           params: [token]
-        }
-      ]
+        });
+      } else {
+        title = 'invalidRoomToken';
+
+        var item = parameters.items[0];
+        item.name = 'OK';
+        item.l10nId = 'ok';
+        item.class = 'full';
+      }
+
+      parameters.section = navigator.mozL10n.get(title);
+
+      var options = new OptionMenu(parameters);
     });
   }
 
@@ -550,18 +568,19 @@
             currentToken = null;
             isConnected = false;
             removeHeadPhonesChangeHandler();
-            RoomUI.hide();
-            // See https://docs.services.mozilla.com/loop/apis.html
-            if (error) {
-              if (error.code === 400 && error.errno === 202) {
-                // Room full
-                return removeMyselfFromRoom(params);
-              } else if (error.code === 404) {
-                return onInvalidToken(params);
+            RoomUI.hide(() => {
+              // See https://docs.services.mozilla.com/loop/apis.html
+              if (error) {
+                if (error.code === 400 && error.errno === 202) {
+                  // Room full
+                  return removeMyselfFromRoom(params);
+                } else if (error.code === 404) {
+                  return onInvalidToken(params);
+                }
               }
-            }
 
-            showError();
+              showError();
+            });
           });
         }, () => {
           // The user cancels

app/js/screens/room/room_ui.js

@@ -168,14 +168,15 @@
     participantName.textContent = '';
   }
 
-  function hide() {
+  function hide(cb) {
     panel.dataset.status = panel.dataset.participants = '';
     removeFakeVideo();
     removeHandlers();
     Navigation.to('calllog-panel', 'bottom').then(() => {
       cleanUI();
       Countdown.stop();
       Countdown.reset();
+      (typeof cb === 'function') && cb();
     });
   }
 

app/locales/loop.de.properties

@@ -176,7 +176,8 @@ noConnection = Bitte überprüfen Sie Ihre Internetverbindung.
 notLoggedIn = Sie müssen angemeldet sein, bevor Sie einen Anruf mit {{serviceName}} führen können.
 pickActivityFail = Dieser Kontakt hat weder eine Telefonnummer noch eine E-Mail-Adresse.
 genericServerError1 = Es tut uns leid, wir haben Probleme. Bitte versuchen Sie es erneut.
-invalidRoomToken = Sie können den Raum nicht betreten, da er nicht mehr verfügbar ist. Er könnte abgelaufen oder gelöscht worden sein. Soll er aus Ihrer Gesprächsliste gelöscht werden?
+invalidRoomTokenAndDelete = Sie können den Raum nicht betreten, da er nicht mehr verfügbar ist. Er könnte abgelaufen oder gelöscht worden sein. Soll er aus Ihrer Gesprächsliste gelöscht werden?
+invalidRoomToken = Sie können den Raum nicht betreten, da er nicht mehr verfügbar ist. Er könnte abgelaufen oder gelöscht worden sein.
 cameraPermission = Wir können leider nicht auf Ihr Mikrofon und Ihre Kamera zugreifen, da Sie uns nicht die Rechte dafür eingeräumt haben. Bitte erteilen Sie uns die Berechtigung in "Einstellungen" > "App-Berechtigungen" > "{{serviceName}}", um {{serviceName}} zu verwenden.
 notCompatibleDevice = Es tut uns leid, Ihr Gerät ist mit dieser App nicht kompatibel.
 oldOSVersion = Es tut uns leid, diese App benötigt Firefox OS 2.0 oder neuer.

app/locales/loop.en-US.properties

@@ -174,7 +174,8 @@ noConnection = Please check your Internet connection
 notLoggedIn = You need to be logged in to use {{serviceName}}
 pickActivityFail = This contact does not have either a phone number or an email address.
 genericServerError1 = Sorry, we have problems on our side. Please try again.
-invalidRoomToken = Oops... You can’t join this room because it is not longer available. It may be expired or deleted. Do you want to delete it from your room list?
+invalidRoomTokenAndDelete = Oops... You can’t join this room because it is not longer available. It may be expired or deleted. Do you want to delete it from your room list?
+invalidRoomToken = Oops... You can’t join this room because it is not longer available. It may be expired or deleted.
 cameraPermission = We can't access your microphone and camera because you didn't give us permission. To use {{serviceName}}, please grant them in Settings > App permissions > {{serviceName}}
 notCompatibleDevice = Sorry, your device isn't compatible with this app
 oldOSVersion = Sorry, this app requires Firefox OS 2.0 or later

app/locales/loop.es.properties

@@ -176,7 +176,8 @@ noConnection = Por favor, comprueba tu conexión a internet
 notLoggedIn = Necesitas iniciar sesión para usar {{serviceName}}
 pickActivityFail = Este contacto no tiene ni número de teléfono ni email.
 genericServerError1 = Ups, esto no ha funcionado como esperábamos. ¿Lo intentamos otra vez?
-invalidRoomToken = Ups... No puedes unirte a la sala porque no está disponible. Pudo haber expirado o haber sido borrada. ¿Quieres borrarla de tu lista de salas?
+invalidRoomTokenAndDelete = Ups... No puedes unirte a la sala porque no está disponible. Pudo haber expirado o haber sido borrada. ¿Quieres borrarla de tu lista de salas?
+invalidRoomToken = Ups... No puedes unirte a la sala porque no está disponible. Pudo haber expirado o haber sido borrada.
 cameraPermission = No tenemos acceso al micrófono ni la cámara porque no nos diste permiso. Para usar {{serviceName}}, por favor cambia estos permisos en Ajustes > Permisos de aplicaciones > {{serviceName}}
 notCompatibleDevice = Lo sentimos, tu dispositivo no es compatible con esta aplicación
 oldOSVersion = Lo sentimos, esta aplicación requiere Firefox OS 2.0 o posterior

app/locales/loop.it.properties

@@ -134,7 +134,8 @@ noConnection = Verificare la connessione a Internet
 notLoggedIn = Bisogna effettuare l’accesso per utilizzare {{serviceName}}
 pickActivityFail = Questo contatto non ha un numero di telefono o un indirizzo email.
 genericServerError1 = Siamo spiacenti, si è verificato un problema sul server. Riprovare.
-invalidRoomToken = Oops… Non è possibile utilizzare questa stanza in quanto non è più disponibile. Potrebbe essere scaduta o essere stata eliminata. Rimuoverla dall’elenco delle stanze?
+invalidRoomTokenAndDelete = Oops… Non è possibile utilizzare questa stanza in quanto non è più disponibile. Potrebbe essere scaduta o essere stata eliminata. Rimuoverla dall’elenco delle stanze?
+invalidRoomToken = Oops… Non è possibile utilizzare questa stanza in quanto non è più disponibile. Potrebbe essere scaduta o essere stata eliminata.
 cameraPermission = Non è possibile accedere a microfono e videocamera per un problema di permessi. Per utilizzare {{serviceName}}, garantire i permessi in Impostazioni > Permessi applicazioni > {{serviceName}}
 notCompatibleDevice = Siamo spiacenti, il dispositivo in uso non è compatibile con questa app
 oldOSVersion = Siamo spiacenti, questa app richiede Firefox OS 2.0 o versioni successive