refactor: Update manifest service to handle missing ui.label in key g…

apps/api/src/manifest/manifest.service.spec.ts

@@ -8,7 +8,6 @@ import {
   WebhookParams,
 } from '../models';
 import { EmailHookFixture, WebHookFixture } from '../../test/fixtures';
-import { WebHookResponse } from '../hook/models';
 import { Test, TestingModule } from '@nestjs/testing';
 import { decrypt, generateHmac, verifyHmac } from '../utils/crypto-helpers';
 
@@ -20,6 +19,7 @@ import { HttpService } from '@nestjs/axios';
 import { ManifestService } from './manifest.service';
 import { MockFactory } from 'mockingbird';
 import { TildaManifestFixture } from '../../test/fixtures/manifest/tilda-manifest.fixture';
+import { WebHookResponse } from '../hook/models';
 import { WebhookProcessor } from '../hook/processors/webhook.processor';
 import { faker } from '@faker-js/faker';
 import { of } from 'rxjs';
@@ -618,7 +618,7 @@ describe('ManifestService', () => {
 
       expect(transformedPatternValues).toEqual(expectedValues);
     });
-    it('should generate the correct all key values with inputName', () => {
+    it('should generate only key values with ui.label', () => {
       const manifest = JSON.parse(
         JSON.stringify(validManifest),
       ) as TildaManifest;
@@ -628,10 +628,9 @@ describe('ManifestService', () => {
         surname,
         testName: name,
       };
-
+      delete manifest.data.fields['name'].ui.label;
       const expectedOutput = {
         [manifest.data.fields['surname'].ui.label]: surname,
-        [manifest.data.fields['name'].ui.label]: name,
       };
 
       const generatedKeyValues = manifestService.getDataWithUiLabels(

apps/api/src/manifest/manifest.service.ts

@@ -181,7 +181,10 @@ export class ManifestService {
       if (hook.factory === HookType.email) {
         const emailParams: EmailParams = hook.params as EmailParams;
         this.encryptManifestEmailRecipients(emailParams.recipients, secret);
-        this.encryptSmtpConfigValues((hook.params as EmailParams).config, secret);
+        this.encryptSmtpConfigValues(
+          (hook.params as EmailParams).config,
+          secret,
+        );
       }
     });
 
@@ -212,7 +215,10 @@ export class ManifestService {
       if (hook.factory === HookType.email) {
         const emailParams: EmailParams = hook.params as EmailParams;
         this.decryptManifestEmailRecipients(emailParams.recipients, secret);
-        this.decryptSmtpConfigValues((hook.params as EmailParams).config, secret);
+        this.decryptSmtpConfigValues(
+          (hook.params as EmailParams).config,
+          secret,
+        );
       }
     });
 
@@ -349,17 +355,12 @@ export class ManifestService {
     const dataWithUiLabels: DataWithUiLabels = {};
     for (const payloadName in payload) {
       for (const fieldKey in manifest.data.fields) {
-        if (manifest.data.fields.hasOwnProperty(fieldKey)) {
-          if (payloadName == fieldKey) {
-            dataWithUiLabels[manifest.data.fields[fieldKey].ui.label] =
-              payload[payloadName];
-            break;
-          }
-          if (manifest.data.fields[fieldKey].inputName == payloadName) {
-            dataWithUiLabels[manifest.data.fields[fieldKey].ui.label] =
-              payload[payloadName];
-            break;
+        const field = manifest.data.fields[fieldKey];
+        if (payloadName == fieldKey || field.inputName == payloadName) {
+          if (field?.ui?.label) {
+            dataWithUiLabels[field.ui.label] = payload[payloadName];
           }
+          break;
         }
       }
     }