#1522: support double-array of markets in marketLists / buildDefiniti…

…onBulk
Accenture · Aug 28, 2024 · 978daef · 978daef
1 parent acc06b9
commit 978daef
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 17 deletions.
diff --git a/@types/lib/Builder.d.ts.map b/@types/lib/Builder.d.ts.map
diff --git a/lib/Builder.js b/lib/Builder.js
@@ -249,25 +249,28 @@ saved
                 continue;
             }
             i++;
+            /** @type {string | string[] | string[][]} */
             const market = properties.marketList[listName][businessUnit];
-            let marketList = [];
-            if ('string' === typeof market) {
-                marketList.push(market);
-            } else {
-                marketList = market;
-            }
+            const marketList = 'string' === typeof market ? [market] : market;
+
             for (const market of marketList) {
-                if (Util.checkMarket(market, properties)) {
-                    Util.logger.info(`Executing for '${businessUnit}': '${market}'`);
-                    // omitting "await" to speed up creation
-                    responseObj[businessUnit] ||= {};
-                    responseObj[businessUnit][market] = await this.buildDefinition(
-                        businessUnit,
-                        type,
-                        nameArr,
-                        market
-                    );
+                // one can now send multiple markets to buildTemplate/buildDefinition and hence that also needs to work for marketLists
+                const marketArr = 'string' === typeof market ? [market] : market;
+                for (const market of marketArr) {
+                    if (!Util.checkMarket(market, properties)) {
+                        return;
+                    }
                 }
+
+                Util.logger.info(`Executing for '${businessUnit}': '${marketArr.join('-')}'`);
+                // omitting "await" to speed up creation
+                responseObj[businessUnit] ||= {};
+                responseObj[businessUnit][marketArr.join('-')] = await this.buildDefinition(
+                    businessUnit,
+                    type,
+                    nameArr,
+                    marketArr
+                );
             }
         }