@adobe/acc-js-sdk 1.1.29 → 1.1.30

package/docs/changeLog.html

@@ -3,6 +3,14 @@ layout: page
 title: Change Log
 ---
 
+<section class="changelog"><h1>Version 1.1.30</h1>
+  <h2>2023/06/19</h2>
+  <li>
+    Fixed an issue causing errors when parsing some specific enumerations such as xtk:dataTransfer:decimalCount which
+    contain enumeration values without "value" attributes and whose "name" attribute contains stringified numbers.
+  </li>
+</section>
+
 <section class="changelog"><h1>Version 1.1.29</h1>
   <h2>2023/06/15</h2>
   <li>

package/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@adobe/acc-js-sdk",
-  "version": "1.1.29",
+  "version": "1.1.30",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "@adobe/acc-js-sdk",
-      "version": "1.1.29",
+      "version": "1.1.30",
       "license": "ISC",
       "dependencies": {
         "axios": "^1.2.1",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/acc-js-sdk",
-  "version": "1.1.29",
+  "version": "1.1.30",
   "description": "ACC Javascript SDK",
   "main": "src/index.js",
   "homepage": "https://github.com/adobe/acc-js-sdk#readme",

package/src/application.js

@@ -1021,7 +1021,12 @@ function XtkEnumerationValue(xml, baseType, parentTranslationId) {
      * @type {string}
      */
     this.applicableIf = EntityAccessor.getAttributeAsString(xml, "applicableIf");
-    const stringValue = EntityAccessor.getAttributeAsString(xml, "value");
+    let stringValue = EntityAccessor.getAttributeAsString(xml, "value");
+    if (stringValue == "" && XtkCaster.isNumericType(baseType)) {
+        // Some enumerations (ex: xtk:dataTransfer:decimalCount) are of numeric type but do
+        // not have a "value" defined. In this case, we try to cas the name as the value
+        stringValue = this.name;
+    }
     /**
      * The enumeration value, casted according to the enumeration type
      * @type {*}
@@ -1082,31 +1087,46 @@ class XtkEnumeration {
          * The enumerations values 
          * @type {Utils.ArrayMap<Campaign.XtkEnumerationValue>}
          */
-         this.values = new ArrayMap();
-
-         var defaultValue = EntityAccessor.getAttributeAsString(xml, "default");
-         this._localizationId = `${schemaId}__${this.name}`.replace(':','__');
+        this.values = new ArrayMap();
+
+        var defaultValue = EntityAccessor.getAttributeAsString(xml, "default");
+        this._localizationId = `${schemaId}__${this.name}`.replace(':','__');
+
+         // Determine if the enumeration can support both access by name and by index.
+         // Some enumerations, such as xtk:dataTransfer:decimalCount are ambiguous since
+         // they have names which are string reprensentation of their values (ex: name="0").
+         // In this case enumValue[0] may mean either the first enumeration value (index 0)
+         // or the enumeration value with value "0" which happens to be the second
+        let supportsIndexing = true;
+        const values = [];
+        for (var child of EntityAccessor.getChildElements(xml, "value")) {
+            const e = new XtkEnumerationValue(child, this.baseType, this._localizationId);
+            values.push(e);
+            const numericName = +e.name;
+            if (numericName === numericName) {
+                // Name is a number
+                supportsIndexing = false;
+            }
+        }
 
-         for (var child of EntityAccessor.getChildElements(xml, "value")) {
-             const e = new XtkEnumerationValue(child, this.baseType, this._localizationId);
-             this.values._push(e.name, e);
-             if (e.image != "") this.hasImage = true;
-             const stringValue = EntityAccessor.getAttributeAsString(child, "name");
-             if (defaultValue == stringValue)
-                 this.default = e;
-         }
+        for (const e of values) {
+            this.values._push(e.name, e, !supportsIndexing);
+            if (e.image != "") this.hasImage = true;
+            const stringValue = e.name;
+            if (defaultValue == stringValue)
+                this.default = e;
+        }
 
-         this.labelLocalizationId = this._localizationId + "__@label";
-         this.descriptionLocalizationId = this._localizationId + "__@desc";
-         propagateImplicitValues(this, true);
+        this.labelLocalizationId = this._localizationId + "__@label";
+        this.descriptionLocalizationId = this._localizationId + "__@desc";
+        propagateImplicitValues(this, true);
 
         /**
          * The system enumeration name, without the schema id prefix
          * @type {string}
          */
-         this.shortName = this.name;
-         this.name = `${schemaId}:${this.shortName}`;
-         
+        this.shortName = this.name;
+        this.name = `${schemaId}:${this.shortName}`;
     }
 }
 

package/src/util.js

@@ -194,14 +194,15 @@ class ArrayMap {
       });
   }
 
-  _push(key, value) {
+  _push(key, value, withoutIndexing) {
       let isNumKey = false;
       if (key) {
         // reserved keyworkds
         const isReserved = key === "_items" || key === "length" || key === "_push" || key === "forEach" || key === "map" || key === "_map" || key === "get" || key === "find" || key === "flatMap" || key === "filter";
 
         // already a child with the name => there's a problem with the schema
-        if (!isReserved && this[key]) throw new Error(`Failed to add element '${key}' to ArrayMap. There's already an item with the same name`);
+        if (!isReserved && this[key]) 
+          throw new Error(`Failed to add element '${key}' to ArrayMap. There's already an item with the same name`);
 
         // Set key as a enumerable property, so that elements can be accessed by key, 
         // but also iterated on with a for ... in loop
@@ -218,7 +219,7 @@ class ArrayMap {
         }
       }
 
-      if (!isNumKey) {
+      if (!withoutIndexing && !isNumKey) {
         // Set the index property so that items can be accessed by array index.
         // However, make it non-enumerable to make sure indexes do not show up in a for .. in loop
         Object.defineProperty(this, this._items.length, {

package/test/application.test.js

@@ -700,6 +700,43 @@ describe('Application', () => {
                     expect(enumeration).toBeFalsy();
                 });
             });
+
+            it("Should support numeric enumeration with implicit value", async () => {
+                // Example in xtk:dataTransfer:decimalCount
+                const client = await Mock.makeClient();
+                client._transport.mockReturnValueOnce(Mock.LOGON_RESPONSE);
+                await client.NLWS.xtkSession.logon();
+                client._transport.mockReturnValueOnce(Promise.resolve(`<?xml version='1.0'?>
+                <SOAP-ENV:Envelope xmlns:xsd='http://www.w3.org/2001/XMLSchema' xmlns:xsi='http://www.w3.org/2001/XMLSchema-instance' xmlns:ns='urn:wpp:default' xmlns:SOAP-ENV='http://schemas.xmlsoap.org/soap/envelope/'>
+                <SOAP-ENV:Body>
+                    <GetEntityIfMoreRecentResponse xmlns='urn:wpp:default' SOAP-ENV:encodingStyle='http://schemas.xmlsoap.org/soap/encoding/'>
+                        <pdomDoc xsi:type='ns:Element' SOAP-ENV:encodingStyle='http://xml.apache.org/xml-soap/literalxml'>
+                            <schema name="dataTransfer" namespace="xtk" xtkschema="xtk:schema">
+                                <enumeration name="decimalCount" basetype="short">
+                                    <value value="-1" name="all" label="All"/>
+                                    <value name="0"/>
+                                    <value name="1"/>
+                                    <value name="2"/>
+                                    <value name="3"/>
+                                    <value name="4"/>
+                                    <value name="5"/>
+                                    <value name="6"/>
+                                    <value name="7"/>
+                                    <value name="8"/>
+                                    <value name="9"/>
+                                </enumeration>
+                            </schema>
+                        </pdomDoc>
+                    </GetEntityIfMoreRecentResponse>
+                </SOAP-ENV:Body>
+                </SOAP-ENV:Envelope>`));
+                const enumeration = await client.application.getSysEnum("decimalCount", "xtk:dataTransfer");
+                expect(enumeration.label).toBe("DecimalCount");
+                const values = enumeration.values;
+                expect(values["all"].value).toBe(-1)
+                expect(values["0"].value).toBe(0)
+                expect(values["1"].value).toBe(1)
+            });
         });
 
         describe("Keys", () => {