dcos-core-monalisav2-latam 1.1.5 → 1.1.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dcos-core-monalisav2-latam",
-  "version": "1.1.5",
+  "version": "1.1.7",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1"
   },

package/src/functions/clients/clients-suggested-product-consumer.js

@@ -25,6 +25,16 @@ const consumer = async (event) => {
 
       const { clientId, productsRegular, productsIncremental } = client;
 
+      // Insertar datos en Clientes Totales
+      await vtexApi.fetch(`/dataentities/${DATA_ENTITY_CLIENT_ACRONYM}/documents`, {
+        method: 'PATCH',
+        data: {
+          id: clientId,
+          productsRegular: JSON.stringify(productsRegular),
+          productsIncremental: JSON.stringify(productsIncremental),
+        }
+      });
+
       // Objeto de datos para el producto sugerido
       const suggestedProductData = {
         orderDate: new Date().toISOString(),
@@ -33,8 +43,8 @@ const consumer = async (event) => {
         productsRegular,
         productsIncremental,
       };
-      const email = `${Startemail}+${clientId}@${Domainemail}`;
       // Consulta del userProfileId
+      const email = `${Startemail}+${clientId}@${Domainemail}`;
       await findUserProfileId(vtexApi, email, suggestedProductData);
 
       // Insertar datos en entidad de Productos Sugeridos
@@ -43,16 +53,6 @@ const consumer = async (event) => {
         data: suggestedProductData
       });
 
-      // Insertar datos en Clientes Totales
-      await vtexApi.fetch(`/dataentities/${DATA_ENTITY_CLIENT_ACRONYM}/documents`, {
-        method: 'PATCH',
-        data: {
-          id: clientId,
-          productsRegular: JSON.stringify(productsRegular),
-          productsIncremental: JSON.stringify(productsIncremental),
-        }
-      });
-
       Logger.log("Suggested products processed successfully", { an, clientId });
     }
   } catch (error) {

package/src/functions/clients/helpers/suggested-product-mdlz.js

@@ -13,7 +13,6 @@ const {
 const suggestedProductMdlz = async (event) => {
   try {
     const { an, clients } = JSON.parse(event.body);
-    // return ApiResponse.response(200, { an, clients });
 
     const account = await AccountData.getAccountDataByAccountName(an);
     if (!account) {
@@ -33,10 +32,6 @@ const suggestedProductMdlz = async (event) => {
     if (!IndexClientRegister?.length) {
       return ApiResponse.response(400, "IndexClientRegister not configured for account");
     }
-    // Se valida que exista el índice 'an'
-    if (!IndexClientRegister.includes('an')) {
-      return ApiResponse.response(400, "IndexClientRegister must have a 'an' property");
-    }
 
     // Se obtiene las propiedades de la configuración de ProductSuggestedConfig
     const {
@@ -103,17 +98,17 @@ function validateClientsData(clients, minProductsCount, indexClientRegister, an,
 
   for (const client of clients) {
     let remain = [], clientId = '';
+    // Se recorre la lista para definir el clientId del usuario asociado
     for (let prop of indexClientRegister) {
       if (prop == 'an') {
         clientId += an;
+      } else if (client.hasOwnProperty(prop)) {
+        clientId += client[prop] ?? '';
       } else {
-        if (client.hasOwnProperty(prop)) {
-          clientId += client[prop] ?? '';
-        } else {
-          remain.push(prop);
-        }
+        remain.push(prop);
       }
     }
+    // Si no se encontró propiedades, se genera el mensaje de error
     if (remain.length) {
       client.error = `The properties ${remain.join(',')} are required`;
       notProcessedItems.push(client);
@@ -127,16 +122,16 @@ function validateClientsData(clients, minProductsCount, indexClientRegister, an,
       continue;
     }
 
-    // Se valida si la longitud de alguno de los ítems del cliente es menos al número mínimo de productos
-    if (client.productsRegular?.length < minProductsCount || client.productsIncremental?.length < minProductsCount) {
+    client.clientId = clientId;
+    const productsCounter = { 'REGULAR': 0, 'INCREMENTAL': 0 };
+    setItemInformation(client.productsRegular, 'REGULAR', skusList, productsCounter);
+    setItemInformation(client.productsIncremental, 'INCREMENTAL', skusList, productsCounter);
+
+    // Se valida si la longitud de alguno de la lista de ítems válidos del cliente es menos al número mínimo de productos
+    if (productsCounter.REGULAR.length < minProductsCount || productsCounter.INCREMENTAL.length < minProductsCount) {
       client.error = `Validate the following properties productsRegular and productsIncremental minimum ${minProductsCount} elements`;
       notProcessedItems.push(client);
-      continue;
     }
-
-    client.clientId = clientId;
-    setItemInformation(client.productsRegular, 'REGULAR', skusList);
-    setItemInformation(client.productsIncremental, 'INCREMENTAL', skusList);
   }
 
   // Si hay ítems no procesados, se retorna el mensaje de error
@@ -173,21 +168,33 @@ async function getSkuData(skusList, account) {
   do {
     // Se extrae las primeras filas de la lista para ejecutar las Promises
     let rowsData = items.splice(0, 10);
+    const skus = {};
     await Promise.allSettled(rowsData.map(sku => {
       return new Promise((resolve, reject) => {
         // Se consulta el SKU en MonalisaV2
         getSkuInformation(sku, account, skuRefUsed).then(skuId => {
-          // Se consulta en VTEX información del producto
+          // Se establece el skuId del producto, para luego consultar en VTEX
           skusList[sku].SkuId = skuId;
-          vtexApi.getDetailSkuById([skuId]).then(res => {
-            const { IsActive, ProductIsVisible } = res[0];
-            skusList[sku].IsActive = IsActive;
-            skusList[sku].ProductIsVisible = ProductIsVisible;
-            resolve(sku);
-          }).catch(reject);
+          skus[skuId] = sku;
+          resolve(sku);
         }).catch(reject);
       });
     })).catch(Logger.error);
+
+    // Se consulta el detalle de los SKU en VTEX
+    if (Object.keys(skus).length) {
+      const skusData = await vtexApi.getDetailSkuById(Object.keys(skus)).catch(Logger.error);
+      if (skusData?.length) {
+        for (let skuInfo of skusData) {
+          const { Id, IsActive, ProductIsVisible } = skuInfo;
+          if (skus.hasOwnProperty(Id)) {
+            const { [Id]: sku } = skus;
+            skusList[sku].IsActive = IsActive;
+            skusList[sku].ProductIsVisible = ProductIsVisible;
+          }
+        }
+      }
+    }
   } while (items.length);
 }
 
@@ -249,22 +256,27 @@ function removeDuplicatesSku(items, skusList) {
 
 /**
  * Organiza la información de los SKU en cada una de las listas de productos del cliente.
- * @param {Object[]} list Lista de ítems de producto sugerido.
- * @param {Object} skusList Objeto donde se consolida la información de los SKU.
+ * @param {Object[]} list Lista de ítems enviados en la petición.
+ * @param {String} productType Tipo de producto ('REGULAR' o 'INCREMENTAL').
  * @param {Object} skusList Objeto donde se consolida la información de los SKU que no fueron encontrados.
+ * @param {Object} productsCounter Objeto donde se consolida el conteo de productos válidos por el tipo de producto `productType`.
  */
-function setItemInformation(list, productType, skusList) {
+function setItemInformation(list, productType, skusList, productsCounter) {
   for (let item of list) {
-    item.type = productType;
     const { sku } = item;
-    // Se elimina la propiedad "sku", para luego insertar este valor en la entidad de productos sugeridos
+    item.type = productType;
+
+    // Se elimina la propiedad "sku" del ítem de la lista, ya que su valor se estableció en la propiedad "refId"
+    item.refId = sku;
     delete item.sku;
-    if (skusList?.[sku]) {
+
+    if (skusList?.[sku]?.hasOwnProperty('SkuId')) {
       const { SkuId, IsActive, ProductIsVisible } = skusList[sku];
-      item.refId = sku;
       item.skuId = SkuId;
       if (IsActive) {
         item.status = ProductIsVisible ? 'active' : 'not_visible';
+        // Se incrementa el contador de productos válidos asociados al tipo "productType"
+        productsCounter[productType] += 1;
       } else {
         item.status = 'inactive';
       }