@contrail/flexplm 1.1.23 → 1.1.25

package/lib/publish/base-process-publish-assortment.js

@@ -401,16 +401,16 @@ class BaseProcessPublishAssortment {
         const assortmentItemsArray = fullChange['assortmentItems'];
         for (const aItem of assortmentItemsArray) {
             const meetsCriteria = this.meetsCriteria(aItem);
-            const item = aItem['item'];
-            const itemId = item['id'];
-            const itemFamilyId = item['itemFamilyId'];
-            console.debug('itemId: ' + item['id']);
-            if (meetsCriteria) {
+            const item = aItem?.item;
+            const itemId = item?.id;
+            const itemFamilyId = item?.itemFamilyId;
+            console.debug('itemId: ' + item?.id);
+            if (item && meetsCriteria) {
                 console.debug('adding item to releasedForDevelopmentItemIds');
                 console.debug('itemFamilyId: ' + item['itemFamilyId']);
                 releasedForDevelopmentItemIds.push(itemId);
                 if (!itemFamilySet.has(itemFamilyId) && itemId !== itemFamilyId) {
-                    const familyItem = item['itemFamily'];
+                    const familyItem = item?.itemFamily;
                     const familyItemMeetsCriteria = this.meetsCriteria({ item: familyItem });
                     if (familyItemMeetsCriteria) {
                         releasedForDevelopmentItemIds.push(itemFamilyId);
@@ -421,14 +421,14 @@ class BaseProcessPublishAssortment {
         }
         for (const dItem of deleteChanges) {
             const meetsCriteria = this.meetsCriteria(dItem);
-            const item = dItem['item'];
-            const itemId = item['id'];
-            const itemFamilyId = item['itemFamilyId'];
-            console.debug('itemId: ' + item['id']);
-            if (meetsCriteria) {
+            const item = dItem?.item;
+            const itemId = item?.id;
+            const itemFamilyId = item?.itemFamilyId;
+            console.debug('itemId: ' + item?.id);
+            if (item && meetsCriteria) {
                 releasedForDevelopmentItemIds.push(itemId);
                 if (!itemFamilySet.has(itemFamilyId) && itemId !== itemFamilyId) {
-                    const familyItem = item['itemFamily'];
+                    const familyItem = item?.itemFamily;
                     if (familyItem) {
                         const familyItemMeetsCriteria = this.meetsCriteria({ item: familyItem });
                         if (familyItemMeetsCriteria) {
@@ -445,8 +445,7 @@ class BaseProcessPublishAssortment {
         return releasedForDevelopmentItemIds;
     }
     meetsCriteria(aItem) {
-        const item = aItem['item'];
-        const lifecycleStage = item['lifecycleStage'];
+        const lifecycleStage = aItem?.item?.lifecycleStage;
         return lifecycleStage && !this.config.itemPreDevelopmentLifecycleStages.includes(lifecycleStage);
     }
     async processPublish(pcd, changeDetail, fullChange, deleteChanges) {
@@ -563,7 +562,8 @@ class BaseProcessPublishAssortment {
         const updateIds = updates.map(item => item.id);
         const familyItemsRemovedIds = familyItemsRemoved.map(item => item.itemId);
         for (const [itemId, aItem] of assortmentItemFullChangeMap) {
-            const { item: { itemFamilyId }, projectItem } = aItem;
+            const projectItem = aItem?.projectItem;
+            const itemFamilyId = aItem?.item?.itemFamilyId;
             if (!pcd.releasedForDevelopmentItemIds.includes(itemFamilyId) || !pcd.releasedForDevelopmentItemIds.includes(itemId)) {
                 continue;
             }
@@ -707,7 +707,7 @@ class BaseProcessPublishAssortment {
             let item = {};
             if (itemFamilyChanges.assortmentItemFullChangeMap.has(entityReference)) {
                 const fullAi = itemFamilyChanges.assortmentItemFullChangeMap.get(entityReference);
-                item = fullAi['item'];
+                item = fullAi?.item;
             }
             const projectItem = this.getProjectItem(itemFamilyChanges, entityReference);
             let seasonalData = await this.getSeasonalData(projectItem);

package/lib/publish/base-process-publish-assortment.spec.js

@@ -1051,3 +1051,46 @@ describe('getCarriedFromSeason', () => {
         });
     });
 });
+describe('meetsCriteria', () => {
+    const config = {
+        identifierAtts: {
+            LCSSeason: ['flexPLMSeasonName']
+        },
+        itemPreDevelopmentLifecycleStages: ['concept']
+    };
+    const mapFileUtil = new transform_data_1.MapFileUtil(new sdk_1.Entities());
+    const dc = new data_converter_1.DataConverter(config, mapFileUtil);
+    const bppa = new base_process_publish_assortment_1.BaseProcessPublishAssortment(config, dc, mapFileUtil);
+    it('no item', () => {
+        const aItem = {};
+        const results = bppa.meetsCriteria(aItem);
+        expect(results).toBeFalsy();
+    });
+    it('item - no lifecycle stage', () => {
+        const aItem = {
+            item: {
+                lifecycleStage: ''
+            }
+        };
+        const results = bppa.meetsCriteria(aItem);
+        expect(results).toBeFalsy();
+    });
+    it('item - concept lifecycle stage', () => {
+        const aItem = {
+            item: {
+                lifecycleStage: 'concept'
+            }
+        };
+        const results = bppa.meetsCriteria(aItem);
+        expect(results).toBeFalsy();
+    });
+    it('item - delevopment lifecycle stage', () => {
+        const aItem = {
+            item: {
+                lifecycleStage: 'delevopment'
+            }
+        };
+        const results = bppa.meetsCriteria(aItem);
+        expect(results).toBeTruthy();
+    });
+});

package/lib/util/data-converter.js

@@ -327,13 +327,17 @@ class DataConverter {
         if (!nd) {
             return objectReferenceId;
         }
+        if (this.transformMapFile) {
+            const mapKey = await type_conversion_utils_1.TypeConversionUtils.getMapKeyFromObject(this.transformMapFile, this.mapFileUtil, nd, type_conversion_utils_1.TypeConversionUtils.FLEX2VIBE_DIRECTION);
+            nd = await map_utils_1.MapUtil.applyTransformMap(this.transformMapFile, this.mapFileUtil, nd, mapKey, type_conversion_utils_1.TypeConversionUtils.FLEX2VIBE_DIRECTION);
+        }
         const entityType = prop['referencedTypeRootSlug'];
         const entityTypePath = prop['referencedTypePath'];
         const entityKeys = Object.keys(nd);
         const identifierKeys = await type_conversion_utils_1.TypeConversionUtils.getIdentifierPropertiesFromObject(this.transformMapFile, this.mapFileUtil, nd);
         const hasAllIdentifiers = identifierKeys.every(key => entityKeys.includes(key));
         if (identifierKeys.length == 0 || !hasAllIdentifiers) {
-            console.warn(`The inbound ${entityType} doesnt have all "identifier properties, so there is no processing. Identifier properties: ${identifierKeys}`);
+            console.warn(`The inbound ${entityType} for prop '${prop['slug']}' doesnt have all "identifier" properties, so there is no processing. Identifier properties: ${identifierKeys}`);
             return objectReferenceId;
         }
         const rootType = await this.typeUtils.getByRootAndPath({ root: entityType });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contrail/flexplm",
-  "version": "1.1.23",
+  "version": "1.1.25",
   "description": "Library used for integration with flexplm.",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",

package/src/publish/base-process-publish-assortment.spec.ts

@@ -1237,3 +1237,54 @@ describe('getCarriedFromSeason', () =>{
 
 });
 
+describe('meetsCriteria', () =>{
+  const config = {
+    identifierAtts: {
+      LCSSeason: ['flexPLMSeasonName']
+    },
+    itemPreDevelopmentLifecycleStages: ['concept']
+  } as unknown as FCConfig;
+  const mapFileUtil = new MapFileUtil(new Entities());
+  const dc = new DataConverter(config, mapFileUtil);
+  const bppa = new BaseProcessPublishAssortment(config, dc, mapFileUtil);
+  it('no item', () =>{
+    const aItem = {};
+
+    const results = bppa.meetsCriteria(aItem);
+    expect(results).toBeFalsy();
+  });
+
+  it('item - no lifecycle stage', () =>{
+    const aItem = {
+      item: {
+        lifecycleStage: ''
+      }
+    };
+
+    const results = bppa.meetsCriteria(aItem);
+    expect(results).toBeFalsy();
+  });
+
+  it('item - concept lifecycle stage', () =>{
+    const aItem = {
+      item: {
+        lifecycleStage: 'concept'
+      }
+    };
+
+    const results = bppa.meetsCriteria(aItem);
+    expect(results).toBeFalsy();
+  });
+
+  it('item - delevopment lifecycle stage', () =>{
+    const aItem = {
+      item: {
+        lifecycleStage: 'delevopment'
+      }
+    };
+
+    const results = bppa.meetsCriteria(aItem);
+    expect(results).toBeTruthy();
+  });
+
+});

package/src/publish/base-process-publish-assortment.ts

@@ -497,16 +497,16 @@ export class BaseProcessPublishAssortment {
 
     for (const aItem of assortmentItemsArray) {
       const meetsCriteria= this.meetsCriteria(aItem);
-      const item = aItem['item'];
-      const itemId = item['id'];
-      const itemFamilyId = item['itemFamilyId'];
-      console.debug('itemId: ' + item['id']);
-      if(meetsCriteria){
+      const item = aItem?.item;
+      const itemId = item?.id;
+      const itemFamilyId = item?.itemFamilyId;
+      console.debug('itemId: ' + item?.id);
+      if(item && meetsCriteria){
         console.debug('adding item to releasedForDevelopmentItemIds');
         console.debug('itemFamilyId: ' + item['itemFamilyId']);
         releasedForDevelopmentItemIds.push(itemId);
         if(!itemFamilySet.has(itemFamilyId) && itemId !== itemFamilyId){
-          const familyItem = item['itemFamily'];
+          const familyItem = item?.itemFamily;
           const familyItemMeetsCriteria = this.meetsCriteria({item: familyItem});
           if(familyItemMeetsCriteria){
             releasedForDevelopmentItemIds.push(itemFamilyId);
@@ -518,14 +518,14 @@ export class BaseProcessPublishAssortment {
 
     for(const dItem of deleteChanges){
       const meetsCriteria = this.meetsCriteria(dItem);
-      const item = dItem['item'];
-      const itemId = item['id'];
-      const itemFamilyId = item['itemFamilyId'];
-      console.debug('itemId: ' + item['id']);
-      if(meetsCriteria) {
+      const item = dItem?.item;
+      const itemId = item?.id;
+      const itemFamilyId = item?.itemFamilyId;
+      console.debug('itemId: ' + item?.id);
+      if(item && meetsCriteria) {
         releasedForDevelopmentItemIds.push(itemId);
         if(!itemFamilySet.has(itemFamilyId) && itemId !== itemFamilyId){
-          const familyItem = item['itemFamily'];
+          const familyItem = item?.itemFamily;
           if(familyItem){
             const familyItemMeetsCriteria = this.meetsCriteria({item: familyItem});
             if(familyItemMeetsCriteria){
@@ -542,8 +542,7 @@ export class BaseProcessPublishAssortment {
   }
 
   meetsCriteria(aItem): boolean {
-    const item = aItem['item'];
-    const lifecycleStage = item['lifecycleStage'];
+    const lifecycleStage = aItem?.item?.lifecycleStage;
 
     return lifecycleStage && !this.config.itemPreDevelopmentLifecycleStages.includes(lifecycleStage);
   }
@@ -678,7 +677,8 @@ export class BaseProcessPublishAssortment {
     const familyItemsRemovedIds = familyItemsRemoved.map(item => item.itemId);
 
     for (const [itemId, aItem] of assortmentItemFullChangeMap) {
-      const { item: { itemFamilyId }, projectItem } = aItem;
+      const projectItem = aItem?.projectItem;
+      const itemFamilyId = aItem?.item?.itemFamilyId;
 
       if (!pcd.releasedForDevelopmentItemIds.includes(itemFamilyId) || !pcd.releasedForDevelopmentItemIds.includes(itemId)) {
         continue;
@@ -855,7 +855,7 @@ export class BaseProcessPublishAssortment {
       let item = {};
       if (itemFamilyChanges.assortmentItemFullChangeMap.has(entityReference)) {
         const fullAi = itemFamilyChanges.assortmentItemFullChangeMap.get(entityReference);
-        item = fullAi['item'];
+        item = fullAi?.item;
       }
       const projectItem = this.getProjectItem(itemFamilyChanges, entityReference);
       let seasonalData = await this.getSeasonalData(projectItem);

package/src/util/data-converter.ts

@@ -405,6 +405,10 @@ export class DataConverter {
     if(!nd) {
       return objectReferenceId;
     }
+    if(this.transformMapFile){
+      const mapKey = await TypeConversionUtils.getMapKeyFromObject(this.transformMapFile, this.mapFileUtil, nd, TypeConversionUtils.FLEX2VIBE_DIRECTION);
+      nd = await MapUtil.applyTransformMap(this.transformMapFile, this.mapFileUtil, nd, mapKey, TypeConversionUtils.FLEX2VIBE_DIRECTION)
+    }
 
     const entityType = prop['referencedTypeRootSlug'];
     const entityTypePath = prop['referencedTypePath'];
@@ -412,7 +416,7 @@ export class DataConverter {
     const identifierKeys = await TypeConversionUtils.getIdentifierPropertiesFromObject(this.transformMapFile, this.mapFileUtil, nd)
     const hasAllIdentifiers = identifierKeys.every(key => entityKeys.includes(key));
     if(identifierKeys.length == 0 || !hasAllIdentifiers){
-      console.warn(`The inbound ${entityType} doesnt have all "identifier properties, so there is no processing. Identifier properties: ${identifierKeys}`);
+      console.warn(`The inbound ${entityType} for prop '${prop['slug']}' doesnt have all "identifier" properties, so there is no processing. Identifier properties: ${identifierKeys}`);
       return objectReferenceId;
     }