more storing

Author: odeimaiz

services/static-webserver/client/source/class/osparc/conversation/AddMessage.js

@@ -213,14 +213,14 @@ qx.Class.define("osparc.conversation.AddMessage", {
         const conversationId = this.getConversationId();
         const message = this.getMessage();
         if (studyData) {
-          osparc.store.ConversationsProject.getInstance().editMessage(studyData["uuid"], conversationId, message["messageId"], content)
-            .then(data => {
-              this.fireDataEvent("messageUpdated", data);
-              commentField.getChildControl("text-area").setValue("");
-            });
+          promise = osparc.store.ConversationsProject.getInstance().editMessage(studyData["uuid"], conversationId, message["messageId"], content);
         } else {
-          // support comment editing
+          promise = osparc.store.ConversationsSupport.getInstance().editMessage(conversationId, message["messageId"], content);
         }
+        promise.then(data => {
+          this.fireDataEvent("messageUpdated", data);
+          commentField.getChildControl("text-area").setValue("");
+        });
       }
     },
 

services/static-webserver/client/source/class/osparc/store/ConversationsSupport.js

@@ -62,6 +62,10 @@ qx.Class.define("osparc.store.ConversationsSupport", {
     },
 
     getConversation: function(conversationId) {
+      if (conversationId in this.__conversationsCached) {
+        return Promise.resolve(this.__conversationsCached[conversationId]);
+      }
+
       const params = {
         url: {
           conversationId,
@@ -130,6 +134,14 @@ qx.Class.define("osparc.store.ConversationsSupport", {
     },
 
     getLastMessage: function(conversationId) {
+      if (
+        conversationId in this.__conversationsCached &&
+        this.__conversationsCached[conversationId].getMessages() &&
+        this.__conversationsCached[conversationId].getMessages().length
+      ) {
+        return Promise.resolve(this.__conversationsCached[conversationId].getMessages()[0]);
+      }
+
       const params = {
         url: {
           conversationId,
@@ -188,7 +200,9 @@ qx.Class.define("osparc.store.ConversationsSupport", {
     },
 
     __addMessageToCache: function(conversationId, messageData) {
-      this.__conversationsCached[conversationId].addMessage(messageData);
+      if (conversationId in this.__conversationsCached) {
+        this.__conversationsCached[conversationId].addMessage(messageData);
+      }
     },
   }
 });

services/static-webserver/client/source/class/osparc/support/ConversationPage.js

@@ -67,6 +67,7 @@ qx.Class.define("osparc.support.ConversationPage", {
         case "conversation-title":
           control = new qx.ui.basic.Label().set({
             alignY: "middle",
+            allowGrowX: true,
           });
           this.getChildControl("conversation-header-layout").add(control);
           break;
@@ -99,9 +100,14 @@ qx.Class.define("osparc.support.ConversationPage", {
         osparc.store.ConversationsSupport.getInstance().getConversation(conversationId)
           .then(conversation => {
             const name = conversation.getName();
-            title.set({
-              value: name && name !== "null" ? name : lastMessage.content,
-            });
+            if (name && name !== "null") {
+              title.setValue(name);
+            } else {
+              osparc.store.ConversationsSupport.getInstance().getLastMessage(conversationId)
+                .then(lastMessage => {
+                  title.setValue(lastMessage ? lastMessage.content : "");
+                });
+            }
             options.show();
           });
       } else {