apn-app-manager 1.3.2 → 1.3.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "apn-app-manager",
-  "version": "1.3.2",
+  "version": "1.3.3",
   "description": "Appian App Manager",
   "license": "Apache-2.0",
   "preferGlobal": true,

package/src/handlers/objectMapsUpdateHandler.js

@@ -68,24 +68,28 @@ function renameObjectProp(curVal, haulField, curNamespace, newNamespace) {
   if (util.isBlank(curVal)) {
     return "";
   } else {
-    let replacementRegexMap = buildNamespaceReplacementRegex(curNamespace, newNamespace, "gi");
+    let replacementRegexMap;
     let nampespaceReplacement;
     switch (haulField.type) {
       // UUIDs - just append the new namespace
       case obProps.propTypes.UUID:
+        replacementRegexMap = buildNamespaceReplacementRegex(curNamespace, newNamespace, "i");
         return `${curVal}-${util.toLower(newNamespace)}`;
       // URL_STUBs - attempt a replacement, but fall back to prefixing if unchanged to ensure they are unique
       case obProps.propTypes.URL_STUB:
+        replacementRegexMap = buildNamespaceReplacementRegex(curNamespace, newNamespace, "i");
         nampespaceReplacement = curVal.replace(replacementRegexMap.from, util.toLower(replacementRegexMap.to));
         return nampespaceReplacement != curVal ? nampespaceReplacement : `${util.toLower(newNamespace)}-${nampespaceReplacement}`;
       // NAMEs - attempt a replacement, but fall back to prefixing if unchanged to ensure they are unique
       case obProps.propTypes.NAME:
+        replacementRegexMap = buildNamespaceReplacementRegex(curNamespace, newNamespace, "i");
         nampespaceReplacement = curVal.replace(replacementRegexMap.from, replacementRegexMap.to);
         return nampespaceReplacement != curVal
           ? nampespaceReplacement
           : `${newNamespace}${util.firstMatch(curVal, new RegExp(cons.namespaceDelimiterRegex, "g"), "_")}${nampespaceReplacement}`;
       // TEXTs - just do a replacement, they do not need to be unique
       case obProps.propTypes.TEXT:
+        replacementRegexMap = buildNamespaceReplacementRegex(curNamespace, newNamespace, "g");
         return curVal.replace(replacementRegexMap.from, replacementRegexMap.to);
     }
   }
@@ -99,6 +103,11 @@ function removeFoundUuids(uuidCollector, haulField, uuids) {
 }
 
 // Creates the regex replacement for a namespace
+// Note that:
+// - UUID/URL_STUB/NAME calls of this pass "i" for flag because we can be case insensitive but we only want to replace the first match
+//   - e.g. Changing prefix COM to NEW, example endpoint change: com-write-comment -> new-write-comment
+// - TEXT calls of this pass "g" for flag because we want to be case sensitive here and can change all instances
+//   - e.g. Changing prefix COM to NEW, example description change: "Creates COM_Comment and COM_CommentThread" -> "Creates NEW_Comment and NEW_CommentThread"
 function buildNamespaceReplacementRegex(curNamespace, newNamespace, regexTags) {
   if (curNamespace == cons.noNamespace) {
     return { from: "", to: "" };

package/src/objectProps.js

@@ -118,7 +118,7 @@ const haulTypes = [
       {
         fieldName: fieldNames.DB_TABLE,
         xpath: `xsd:schema/xsd:complexType/xsd:annotation/xsd:appinfo[not(ns2:Metadata)]/node()`,
-        type: propTypes.TEXT,
+        type: propTypes.NAME,
         usage: [usageTypes.CLONE, usageTypes.BUILD, usageTypes.RENAME],
         extractRegex: /(?<=")[^"]+(?=")/g,
       },
@@ -274,7 +274,7 @@ const haulTypes = [
       {
         fieldName: fieldNames.DB_TABLE,
         xpath: `recordTypeHaul/recordType/a:sourceConfiguration/friendlyName/node()`,
-        type: propTypes.TEXT,
+        type: propTypes.NAME,
         usage: [usageTypes.CLONE, usageTypes.BUILD, usageTypes.RENAME],
       },
       {