@abaplint/core 2.95.14 → 2.95.16

package/build/abaplint.d.ts

@@ -5927,6 +5927,7 @@ declare class TableType extends AbstractObject {
     };
     getDescription(): string | undefined;
     setDirty(): void;
+    private buildTableOptions;
     parseType(reg: IRegistry): AbstractType;
     private parseXML;
 }

package/build/src/abap/2_statements/expressions/sql_field_list.js

@@ -6,8 +6,9 @@ const _1 = require(".");
 const version_1 = require("../../../version");
 class SQLFieldList extends combi_1.Expression {
     getRunnable() {
-        const comma = (0, combi_1.optPrio)((0, combi_1.ver)(version_1.Version.v740sp05, ","));
-        return (0, combi_1.altPrio)("*", _1.Dynamic, (0, combi_1.plusPrio)((0, combi_1.seq)(_1.SQLField, comma)));
+        const nev = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.starPrio)((0, combi_1.seq)(",", _1.SQLField)));
+        const old = (0, combi_1.starPrio)(_1.SQLField);
+        return (0, combi_1.altPrio)("*", _1.Dynamic, (0, combi_1.seq)(_1.SQLField, (0, combi_1.alt)(nev, old)));
     }
 }
 exports.SQLFieldList = SQLFieldList;

package/build/src/abap/5_syntax/statements/create_object.js

@@ -43,10 +43,13 @@ class CreateObject {
                 if (found instanceof basic_1.VoidType) {
                     continue;
                 }
+                else if (found instanceof basic_1.UnknownType) {
+                    throw new Error("Target type unknown, " + t.concatTokens());
+                }
                 else if (!(found instanceof basic_1.ObjectReferenceType)
                     && !(found instanceof basic_1.AnyType)
                     && !(found instanceof basic_1.GenericObjectReferenceType)) {
-                    throw new Error("Target must be an object reference");
+                    throw new Error("Target must be an object reference, " + t.concatTokens());
                 }
                 else if (found instanceof basic_1.GenericObjectReferenceType && type === undefined) {
                     throw new Error("Generic type, cannot be instantiated");

package/build/src/objects/table_type.js

@@ -5,6 +5,7 @@ const _abstract_object_1 = require("./_abstract_object");
 const Types = require("../abap/types/basic");
 const ddic_1 = require("../ddic");
 const basic_1 = require("../abap/types/basic");
+const xml_utils_1 = require("../xml_utils");
 class TableType extends _abstract_object_1.AbstractObject {
     constructor() {
         super(...arguments);
@@ -27,44 +28,78 @@ class TableType extends _abstract_object_1.AbstractObject {
         this.parsedXML = undefined;
         super.setDirty();
     }
+    buildTableOptions() {
+        var _a, _b, _c;
+        const tableOptions = {
+            withHeader: false,
+            secondary: [],
+        };
+        for (const k of ((_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.dd43v) || []) {
+            const fields = [];
+            for (const f of ((_b = this.parsedXML) === null || _b === void 0 ? void 0 : _b.dd42v) || []) {
+                if (f.keyname === k.keyname) {
+                    fields.push(f.keyfield);
+                }
+            }
+            let accessType = basic_1.TableAccessType.standard;
+            switch (k.accessmode) {
+                case "S":
+                    accessType = basic_1.TableAccessType.sorted;
+                    break;
+                case "H":
+                    accessType = basic_1.TableAccessType.hashed;
+                    break;
+                default:
+                    break;
+            }
+            (_c = tableOptions.secondary) === null || _c === void 0 ? void 0 : _c.push({
+                name: k.keyname,
+                type: accessType,
+                keyFields: fields,
+                isUnique: k.unique,
+            });
+        }
+        return tableOptions;
+    }
     parseType(reg) {
         this.parseXML();
         const ddic = new ddic_1.DDIC(reg);
         const references = [];
         let type;
+        const tableOptions = this.buildTableOptions();
         if (this.parsedXML === undefined) {
             type = new Types.UnknownType("Table Type, parser error", this.getName());
         }
         else if (this.parsedXML.rowkind === "S") {
             const lookup = ddic.lookupTableOrView(this.parsedXML.rowtype);
-            type = new Types.TableType(lookup.type, { withHeader: false }, this.getName());
+            type = new Types.TableType(lookup.type, tableOptions, this.getName());
             if (lookup.object) {
                 references.push({ object: lookup.object });
             }
         }
         else if (this.parsedXML.rowkind === "E") {
             const lookup = ddic.lookupDataElement(this.parsedXML.rowtype);
-            type = new Types.TableType(lookup.type, { withHeader: false }, this.getName());
+            type = new Types.TableType(lookup.type, tableOptions, this.getName());
             if (lookup.object) {
                 references.push({ object: lookup.object });
             }
         }
         else if (this.parsedXML.rowkind === "L") {
             const lookup = ddic.lookupTableType(this.parsedXML.rowtype);
-            type = new Types.TableType(lookup.type, { withHeader: false }, this.getName());
+            type = new Types.TableType(lookup.type, tableOptions, this.getName());
             if (lookup.object) {
                 references.push({ object: lookup.object });
             }
         }
         else if (this.parsedXML.rowkind === "R" && this.parsedXML.rowtype === "OBJECT") {
-            type = new Types.TableType(new basic_1.GenericObjectReferenceType(), { withHeader: false }, this.getName());
+            type = new Types.TableType(new basic_1.GenericObjectReferenceType(), tableOptions, this.getName());
         }
         else if (this.parsedXML.rowkind === "R" && this.parsedXML.rowtype === "DATA") {
-            type = new Types.TableType(new basic_1.DataReference(new basic_1.AnyType()), { withHeader: false }, this.getName());
+            type = new Types.TableType(new basic_1.DataReference(new basic_1.AnyType()), tableOptions, this.getName());
         }
         else if (this.parsedXML.rowkind === "R" && this.parsedXML.rowtype !== undefined) {
             const lookup = ddic.lookupObject(this.parsedXML.rowtype);
-            type = new Types.TableType(lookup.type, { withHeader: false }, this.getName());
+            type = new Types.TableType(lookup.type, tableOptions, this.getName());
             if (lookup.object) {
                 references.push({ object: lookup.object });
             }
@@ -75,7 +110,7 @@ class TableType extends _abstract_object_1.AbstractObject {
             }
             else {
                 const row = ddic.textToType(this.parsedXML.datatype, this.parsedXML.leng, this.parsedXML.decimals, this.getName());
-                type = new Types.TableType(row, { withHeader: false }, this.getName());
+                type = new Types.TableType(row, tableOptions, this.getName());
             }
         }
         else {
@@ -86,20 +121,39 @@ class TableType extends _abstract_object_1.AbstractObject {
     }
     ////////////////////
     parseXML() {
+        var _a, _b;
         if (this.parsedXML !== undefined) {
             return;
         }
-        this.parsedXML = {};
+        this.parsedXML = {
+            dd42v: [],
+            dd43v: [],
+        };
         const parsed = super.parseRaw2();
         if (parsed === undefined || parsed.abapGit === undefined) {
             return;
         }
-        const dd40v = parsed.abapGit["asx:abap"]["asx:values"].DD40V;
+        const values = parsed.abapGit["asx:abap"]["asx:values"];
+        const dd40v = values.DD40V;
         this.parsedXML.rowtype = dd40v.ROWTYPE ? dd40v.ROWTYPE : "";
         this.parsedXML.rowkind = dd40v.ROWKIND ? dd40v.ROWKIND : "";
         this.parsedXML.datatype = dd40v.DATATYPE;
         this.parsedXML.leng = dd40v.LENG;
         this.parsedXML.decimals = dd40v.DECIMALS;
+        for (const x of (0, xml_utils_1.xmlToArray)((_a = values.DD42V) === null || _a === void 0 ? void 0 : _a.DD42V)) {
+            this.parsedXML.dd42v.push({
+                keyname: x.SECKEYNAME || "",
+                keyfield: x.KEYFIELD || "",
+            });
+        }
+        for (const x of (0, xml_utils_1.xmlToArray)((_b = values.DD43V) === null || _b === void 0 ? void 0 : _b.DD43V)) {
+            this.parsedXML.dd43v.push({
+                keyname: x.SECKEYNAME || "",
+                accessmode: x.ACCESSMODE || "",
+                kind: x.KIND || "",
+                unique: x.SECKEYUNIQUE === "X",
+            });
+        }
     }
 }
 exports.TableType = TableType;

package/build/src/registry.js

@@ -63,7 +63,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.95.14";
+        return "2.95.16";
     }
     getDDICReferences() {
         return this.references;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.95.14",
+    "version": "2.95.16",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -47,12 +47,12 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@microsoft/api-extractor": "^7.34.3",
+        "@microsoft/api-extractor": "^7.34.4",
         "@types/chai": "^4.3.4",
         "@types/mocha": "^10.0.1",
         "@types/node": "^18.13.0",
         "chai": "^4.3.7",
-        "eslint": "^8.33.0",
+        "eslint": "^8.34.0",
         "mocha": "^10.2.0",
         "c8": "^7.12.0",
         "source-map-support": "^0.5.21",
@@ -60,7 +60,7 @@
         "typescript": "^4.9.5"
     },
     "dependencies": {
-        "fast-xml-parser": "^4.1.1",
+        "fast-xml-parser": "^4.1.2",
         "json5": "^2.2.3",
         "vscode-languageserver-types": "^3.17.2"
     }