@abaplint/transpiler-cli 2.7.158 → 2.7.160

package/build/bundle.js

@@ -6145,7 +6145,7 @@ const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/co
 const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 class RaiseWith extends combi_1.Expression {
     getRunnable() {
-        const wit = (0, combi_1.seq)("WITH", _1.SimpleSource3, (0, combi_1.opt)(_1.SimpleSource3), (0, combi_1.opt)(_1.SimpleSource3), (0, combi_1.opt)(_1.SimpleSource3));
+        const wit = (0, combi_1.seq)("WITH", _1.SimpleSource1, (0, combi_1.opt)(_1.SimpleSource1), (0, combi_1.opt)(_1.SimpleSource1), (0, combi_1.opt)(_1.SimpleSource1));
         return wit;
     }
 }
@@ -21267,6 +21267,7 @@ class TypeUtils {
         }
         else if (type instanceof basic_1.StringType
             || type instanceof basic_1.AnyType
+            || type instanceof basic_1.DataType
             || type instanceof basic_1.CharacterType
             || type instanceof basic_1.SimpleType
             || type instanceof cgeneric_type_1.CGenericType
@@ -21315,6 +21316,7 @@ class TypeUtils {
             || type instanceof basic_1.CSequenceType
             || type instanceof cgeneric_type_1.CGenericType
             || type instanceof basic_1.DateType
+            || type instanceof basic_1.DataType
             || type instanceof basic_1.CLikeType
             || type instanceof basic_1.PackedType
             || type instanceof basic_1.TimeType) {
@@ -21545,7 +21547,9 @@ class TypeUtils {
             else if (target instanceof basic_1.CLikeType) {
                 return this.isCharLikeStrict(source);
             }
-            else if (target instanceof basic_1.VoidType || target instanceof basic_1.AnyType) {
+            else if (target instanceof basic_1.VoidType
+                || target instanceof basic_1.AnyType
+                || target instanceof basic_1.DataType) {
                 return true;
             }
             return false;
@@ -21607,6 +21611,7 @@ class TypeUtils {
             }
             if (source instanceof basic_1.VoidType
                 || source instanceof basic_1.AnyType
+                || source instanceof basic_1.DataType
                 || source instanceof basic_1.UnknownType) {
                 return true;
             }
@@ -21651,6 +21656,7 @@ class TypeUtils {
             if (source instanceof basic_1.ObjectReferenceType
                 || source instanceof basic_1.GenericObjectReferenceType
                 || source instanceof basic_1.VoidType
+                || source instanceof basic_1.DataType
                 || source instanceof basic_1.AnyType
                 || source instanceof basic_1.UnknownType) {
                 return true;
@@ -21660,6 +21666,7 @@ class TypeUtils {
         else if (target instanceof basic_1.DataReference) {
             if (source instanceof basic_1.DataReference
                 || source instanceof basic_1.VoidType
+                || source instanceof basic_1.DataType
                 || source instanceof basic_1.AnyType
                 || source instanceof basic_1.UnknownType) {
                 return true;
@@ -21672,6 +21679,7 @@ class TypeUtils {
             }
             else if (source instanceof basic_1.VoidType
                 || source instanceof basic_1.AnyType
+                || source instanceof basic_1.DataType
                 || source instanceof basic_1.UnknownType) {
                 return true;
             }
@@ -22568,7 +22576,7 @@ class BasicTypes {
             return new Types.DataReference(found);
         }
         else if (chain.concatTokens().toUpperCase() === "DATA") {
-            return new Types.DataReference(new Types.AnyType());
+            return new Types.DataReference(new Types.DataType());
         }
         if (this.scope.isBadiDef(name) === true) {
             return new Types.VoidType(name);
@@ -23466,6 +23474,7 @@ class Dereference {
     runSyntax(type) {
         if (type instanceof basic_1.VoidType
             || type instanceof basic_1.AnyType
+            || type instanceof basic_1.DataType
             || type === undefined
             || type instanceof basic_1.UnknownType) {
             return type;
@@ -25215,7 +25224,7 @@ class NewObject {
             new method_parameters_1.MethodParameters().checkExporting(parameters, scope, method, filename);
         }
         else if (requiredParameters.length > 0) {
-            throw new Error(`constructor parameter "${requiredParameters[0].getName()}" must be supplied` + name);
+            throw new Error(`constructor parameter "${requiredParameters[0].getName()}" must be supplied, ` + name);
         }
     }
     defaultImportingType(method) {
@@ -25251,7 +25260,7 @@ const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ ".
 const source_1 = __webpack_require__(/*! ./source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
 class RaiseWith {
     runSyntax(node, scope, filename) {
-        for (const f of node.findDirectExpressions(Expressions.SimpleSource3)) {
+        for (const f of node.findDirectExpressions(Expressions.SimpleSource1)) {
             new source_1.Source().runSyntax(f, scope, filename);
         }
     }
@@ -28336,6 +28345,7 @@ class CreateObject {
                 }
                 else if (!(found instanceof basic_1.ObjectReferenceType)
                     && !(found instanceof basic_1.AnyType)
+                    && !(found instanceof basic_1.DataType)
                     && !(found instanceof basic_1.GenericObjectReferenceType)) {
                     throw new Error("Target must be an object reference, " + t.concatTokens());
                 }
@@ -29657,6 +29667,7 @@ class Loop {
         }
         else if (!(sourceType instanceof basic_1.TableType)
             && !(sourceType instanceof basic_1.AnyType)
+            && !(sourceType instanceof basic_1.DataType)
             && !(sourceType instanceof basic_1.VoidType)
             && concat.startsWith("LOOP AT GROUP ") === false) {
             throw new Error("Loop, not a table type");
@@ -34874,6 +34885,7 @@ exports.CSequenceType = CSequenceType;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.DataReference = void 0;
 const _abstract_type_1 = __webpack_require__(/*! ./_abstract_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/_abstract_type.js");
+const any_type_1 = __webpack_require__(/*! ./any_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/any_type.js");
 class DataReference extends _abstract_type_1.AbstractType {
     constructor(type, qualifiedName) {
         super({ qualifiedName: qualifiedName });
@@ -34893,7 +34905,9 @@ class DataReference extends _abstract_type_1.AbstractType {
         return "REF TO " + this.type.toABAP();
     }
     isGeneric() {
-        // a DATA definition can be "REF TO data", so its not generic
+        if (this.type instanceof any_type_1.AnyType) {
+            return true;
+        }
         return false;
     }
     containsVoid() {
@@ -34908,6 +34922,39 @@ exports.DataReference = DataReference;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/types/basic/data_type.js":
+/*!*****************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/types/basic/data_type.js ***!
+  \*****************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.DataType = void 0;
+const _abstract_type_1 = __webpack_require__(/*! ./_abstract_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/_abstract_type.js");
+class DataType extends _abstract_type_1.AbstractType {
+    toText() {
+        return "```data```";
+    }
+    toABAP() {
+        return "data";
+    }
+    isGeneric() {
+        return true;
+    }
+    containsVoid() {
+        return false;
+    }
+    toCDS() {
+        return "abap.TODO_DATA";
+    }
+}
+exports.DataType = DataType;
+//# sourceMappingURL=data_type.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/types/basic/date_type.js":
 /*!*****************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/types/basic/date_type.js ***!
@@ -35258,6 +35305,7 @@ __exportStar(__webpack_require__(/*! ./character_type */ "./node_modules/@abapli
 __exportStar(__webpack_require__(/*! ./clike_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/clike_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./csequence_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/csequence_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./data_reference_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/data_reference_type.js"), exports);
+__exportStar(__webpack_require__(/*! ./data_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/data_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./date_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/date_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./decfloat_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/decfloat_type.js"), exports);
 __exportStar(__webpack_require__(/*! ./decfloat16_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/decfloat16_type.js"), exports);
@@ -38899,7 +38947,7 @@ class DDIC {
             case "TABLE":
                 return new Types.TableType(new Types.AnyType(), { withHeader: false, keyType: Types.TableKeyType.default });
             case "DATA":
-                return new Types.AnyType({ qualifiedName: qualifiedName });
+                return new Types.DataType({ qualifiedName: qualifiedName });
             case "NUMERIC":
                 return new Types.NumericGenericType({ qualifiedName: qualifiedName });
             case "UTCLONG": // todo, take version into account
@@ -49126,7 +49174,7 @@ class TableType extends _abstract_object_1.AbstractObject {
             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()), tableOptions, this.getName());
+            type = new Types.TableType(new basic_1.DataReference(new Types.DataType()), tableOptions, this.getName());
         }
         else if (this.parsedXML.rowkind === "R" && this.parsedXML.rowtype !== undefined) {
             const lookup = ddic.lookupObject(this.parsedXML.rowtype);
@@ -50198,7 +50246,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.105.18";
+        return "2.105.21";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -56008,7 +56056,7 @@ ${indentation}CATCH ${className} INTO ${targetName}.`;
             }
             startToken = node.getFirstToken();
         }
-        const withs = ((_f = node.findDirectExpression(Expressions.RaiseWith)) === null || _f === void 0 ? void 0 : _f.findDirectExpressions(Expressions.SimpleSource3)) || [];
+        const withs = ((_f = node.findDirectExpression(Expressions.RaiseWith)) === null || _f === void 0 ? void 0 : _f.findDirectExpressions(Expressions.SimpleSource1)) || [];
         const className = ((_g = node.findDirectExpression(Expressions.ClassName)) === null || _g === void 0 ? void 0 : _g.concatTokens()) || "ERROR";
         const uniqueName1 = this.uniqueName(node.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
         const uniqueName2 = this.uniqueName(node.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
@@ -58096,6 +58144,7 @@ const Structures = __webpack_require__(/*! ../abap/3_structures/structures */ ".
 const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
 class EmptyStructureConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
@@ -58150,6 +58199,11 @@ class EmptyStructure extends _abap_rule_1.ABAPRule {
         if (stru === undefined) {
             return [];
         }
+        for (const statement of file.getStatements()) {
+            if (statement.get() instanceof _statement_1.Unknown) {
+                return []; // contains parser errors
+            }
+        }
         const candidates = [];
         if (this.getConfig().loop === true) {
             candidates.push(...stru.findAllStructuresRecursive(Structures.Loop));
@@ -59559,6 +59613,7 @@ const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./
 const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
 const nodes_1 = __webpack_require__(/*! ../abap/nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
+const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
 class IdenticalContentsConf extends _basic_rule_config_1.BasicRuleConfig {
 }
 exports.IdenticalContentsConf = IdenticalContentsConf;
@@ -59604,6 +59659,11 @@ WRITE 'world'.`,
         if (structure === undefined) {
             return [];
         }
+        for (const statement of file.getStatements()) {
+            if (statement.get() instanceof _statement_1.Unknown) {
+                return []; // contains parser errors
+            }
+        }
         for (const i of structure.findAllStructuresRecursive(Structures.If)) {
             issues = issues.concat(this.analyzeIf(file, i));
         }
@@ -60989,6 +61049,11 @@ class KeepSingleParameterCallsOnOneLine extends _abap_rule_1.ABAPRule {
     }
     isSingleParameter(c) {
         if (c.findDirectExpression(Expressions.Source)) {
+            for (const params of c.findAllExpressions(Expressions.ParameterListS)) {
+                if (params.getChildren().length > 1) {
+                    return false;
+                }
+            }
             return true;
         }
         const list = c.findDirectExpression(Expressions.ParameterListS);
@@ -83900,7 +83965,8 @@ class TranspileTypes {
             // if not supplied its a Character(1)
             resolved = "Character";
         }
-        else if (type instanceof abaplint.BasicTypes.AnyType) {
+        else if (type instanceof abaplint.BasicTypes.AnyType
+            || type instanceof abaplint.BasicTypes.DataType) {
             // if not supplied its a Character(4)
             resolved = "Character";
             extra = "4";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.7.158",
+  "version": "2.7.160",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -26,14 +26,14 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/transpiler": "^2.7.158",
+    "@abaplint/transpiler": "^2.7.160",
     "@types/glob": "^8.1.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.7",
-    "@types/node": "^20.11.19",
-    "@abaplint/core": "^2.105.18",
+    "@types/node": "^20.11.20",
+    "@abaplint/core": "^2.105.21",
     "progress": "^2.0.3",
-    "webpack": "^5.90.2",
+    "webpack": "^5.90.3",
     "webpack-cli": "^5.1.4",
     "typescript": "^5.3.3"
   }