@abaplint/cli 2.102.5 → 2.102.7

package/build/cli.js

@@ -27133,7 +27133,7 @@ class Assign {
         const theSource = sources[sources.length - 1];
         let sourceType = new source_1.Source().runSyntax(theSource, scope, filename);
         if (sourceType === undefined || ((_b = node.findDirectExpression(Expressions.AssignSource)) === null || _b === void 0 ? void 0 : _b.findDirectExpression(Expressions.Dynamic))) {
-            sourceType = new basic_1.VoidType("DynamicAssign");
+            sourceType = new basic_1.AnyType();
         }
         for (const d of ((_c = node.findDirectExpression(Expressions.AssignSource)) === null || _c === void 0 ? void 0 : _c.findAllExpressions(Expressions.Dynamic)) || []) {
             new dynamic_1.Dynamic().runSyntax(d, scope, filename);
@@ -28181,16 +28181,16 @@ const data_definition_1 = __webpack_require__(/*! ../expressions/data_definition
 const unknown_type_1 = __webpack_require__(/*! ../../types/basic/unknown_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/unknown_type.js");
 class Data {
     runSyntax(node, scope, filename) {
+        const name = node.findFirstExpression(Expressions.DefinitionName);
         const dd = node.findFirstExpression(Expressions.DataDefinition);
         if (dd) {
             const id = new data_definition_1.DataDefinition().runSyntax(dd, scope, filename);
             if ((id === null || id === void 0 ? void 0 : id.getType().isGeneric()) === true
                 && (id === null || id === void 0 ? void 0 : id.getType().containsVoid()) === false) {
-                throw new Error("DATA definition cannot be generic");
+                throw new Error("DATA definition cannot be generic, " + name);
             }
             return id;
         }
-        const name = node.findFirstExpression(Expressions.DefinitionName);
         if (name) {
             return new _typed_identifier_1.TypedIdentifier(name.getFirstToken(), filename, new unknown_type_1.UnknownType("data, fallback"));
         }
@@ -47353,42 +47353,45 @@ class TableType extends _abstract_object_1.AbstractObject {
         this.parsedXML = undefined;
         super.setDirty();
     }
-    buildTableOptions() {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j;
-        let primaryKey = undefined;
-        if (((_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.accessmode) === "S") {
-            primaryKey = {
-                isUnique: ((_b = this.parsedXML) === null || _b === void 0 ? void 0 : _b.keykind) === "U",
-                type: basic_1.TableAccessType.sorted,
-                keyFields: [],
-                name: "primary_key",
-            };
+    buildPrimaryKey() {
+        var _a, _b, _c, _d, _e;
+        const primaryKey = {
+            isUnique: ((_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.keykind) === "U",
+            type: basic_1.TableAccessType.standard,
+            keyFields: [],
+            name: "primary_key",
+        };
+        if (((_b = this.parsedXML) === null || _b === void 0 ? void 0 : _b.accessmode) === "S") {
+            primaryKey.type = basic_1.TableAccessType.sorted;
         }
         else if (((_c = this.parsedXML) === null || _c === void 0 ? void 0 : _c.accessmode) === "H") {
-            primaryKey = {
-                isUnique: ((_d = this.parsedXML) === null || _d === void 0 ? void 0 : _d.keykind) === "U",
-                type: basic_1.TableAccessType.hashed,
-                keyFields: [],
-                name: "primary_key",
-            };
+            primaryKey.type = basic_1.TableAccessType.hashed;
         }
-        for (const f of ((_e = this.parsedXML) === null || _e === void 0 ? void 0 : _e.dd42v) || []) {
+        for (const f of ((_d = this.parsedXML) === null || _d === void 0 ? void 0 : _d.dd42v) || []) {
             if (f.keyname === "") {
-                primaryKey === null || primaryKey === void 0 ? void 0 : primaryKey.keyFields.push(f.keyfield);
+                primaryKey.keyFields.push(f.keyfield);
             }
         }
-        if (((_f = this.parsedXML) === null || _f === void 0 ? void 0 : _f.keydef) === "T" && (primaryKey === null || primaryKey === void 0 ? void 0 : primaryKey.keyFields.length) === 0) {
-            primaryKey === null || primaryKey === void 0 ? void 0 : primaryKey.keyFields.push("table_line");
+        if (((_e = this.parsedXML) === null || _e === void 0 ? void 0 : _e.keydef) === "T" && primaryKey.keyFields.length === 0) {
+            primaryKey.keyFields.push("table_line");
+        }
+        return primaryKey;
+    }
+    buildTableOptions() {
+        var _a, _b, _c, _d;
+        let keyType = Types.TableKeyType.user;
+        if (((_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.keydef) === "D") {
+            keyType = Types.TableKeyType.default;
         }
         const tableOptions = {
             withHeader: false,
-            keyType: Types.TableKeyType.user,
-            primaryKey: primaryKey,
+            keyType: keyType,
+            primaryKey: this.buildPrimaryKey(),
             secondary: [],
         };
-        for (const k of ((_g = this.parsedXML) === null || _g === void 0 ? void 0 : _g.dd43v) || []) {
+        for (const k of ((_b = this.parsedXML) === null || _b === void 0 ? void 0 : _b.dd43v) || []) {
             const fields = [];
-            for (const f of ((_h = this.parsedXML) === null || _h === void 0 ? void 0 : _h.dd42v) || []) {
+            for (const f of ((_c = this.parsedXML) === null || _c === void 0 ? void 0 : _c.dd42v) || []) {
                 if (f.keyname === k.keyname) {
                     fields.push(f.keyfield);
                 }
@@ -47404,7 +47407,7 @@ class TableType extends _abstract_object_1.AbstractObject {
                 default:
                     break;
             }
-            (_j = tableOptions.secondary) === null || _j === void 0 ? void 0 : _j.push({
+            (_d = tableOptions.secondary) === null || _d === void 0 ? void 0 : _d.push({
                 name: k.keyname,
                 type: accessType,
                 keyFields: fields,
@@ -48535,7 +48538,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.5";
+        return "2.102.7";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.102.5",
+    "version": "2.102.7",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,12 +38,12 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.102.5",
+        "@abaplint/core": "^2.102.7",
         "@types/chai": "^4.3.5",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.2",
         "@types/mocha": "^10.0.1",
-        "@types/node": "^20.4.4",
+        "@types/node": "^20.4.5",
         "@types/progress": "^2.0.5",
         "chai": "^4.3.7",
         "chalk": "^5.3.0",