apply fixes regarding message templates

Signed-off-by: Thomas Jäckle <thomas.jaeckle@beyonnex.io>
eclipse-ditto · Oct 15, 2023 · c21b94e · c21b94e
1 parent 7846f78
commit c21b94e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/ui/modules/things/featureMessages.ts b/ui/modules/things/featureMessages.ts
@@ -67,6 +67,7 @@ export async function ready() {
   dom.buttonMessageFavorite.onclick = () => {
     const templateName = dom.inputMessageTemplate.value;
     const featureId = theFeatureId;
+    const payload = acePayload.getValue();
     Utils.assert(featureId, 'Please select a Feature', dom.tableValidationFeature);
     Utils.assert(templateName, 'Please give a name for the template', dom.inputMessageTemplate);
     Environments.current().messageTemplates[featureId] = Environments.current().messageTemplates[featureId] || {};
@@ -79,7 +80,7 @@ export async function ready() {
       Environments.current().messageTemplates[featureId][templateName] = {
         subject: dom.inputMessageSubject.value,
         timeout: dom.inputMessageTimeout.value,
-        payload: JSON.parse(acePayload.getValue()),
+        ...(payload) && {payload: JSON.parse(payload)},
       };
       acePayload.session.getUndoManager().markClean();
     }

diff --git a/ui/modules/things/thingMessages.ts b/ui/modules/things/thingMessages.ts
@@ -60,6 +60,7 @@ export async function ready() {
 
   dom.buttonThingMessageFavorite.onclick = () => {
     const templateName = dom.inputThingMessageTemplate.value;
+    const payload = acePayload.getValue();
     Utils.assert(templateName, 'Please give a name for the template', dom.inputThingMessageTemplate);
     Environments.current().messageTemplates['/'] = Environments.current().messageTemplates['/'] || {};
     if (Object.keys(Environments.current().messageTemplates['/']).includes(templateName) &&
@@ -71,7 +72,7 @@ export async function ready() {
       Environments.current().messageTemplates['/'][templateName] = {
         subject: dom.inputThingMessageSubject.value,
         timeout: dom.inputThingMessageTimeout.value,
-        payload: JSON.parse(acePayload.getValue()),
+        ...(payload) && {payload: JSON.parse(payload)},
       };
       acePayload.session.getUndoManager().markClean();
     }
@@ -132,9 +133,7 @@ function onEnvironmentChanged(modifiedField) {
   if (!modifiedField) {
     clearAllFields();
   }
-  if (modifiedField === 'messageTemplates') {
-    refillTemplates();
-  }
+  refillTemplates();
 }
 
 function clearAllFields() {