@abaplint/cli 2.105.18 → 2.105.20

package/build/cli.js

@@ -7196,7 +7196,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;
     }
 }
@@ -22366,6 +22366,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) {
@@ -22596,7 +22597,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;
@@ -22658,6 +22661,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;
             }
@@ -22702,6 +22706,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;
@@ -22711,6 +22716,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;
@@ -22723,6 +22729,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;
             }
@@ -23619,7 +23626,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);
@@ -24517,6 +24524,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;
@@ -26302,7 +26310,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);
         }
     }
@@ -30708,6 +30716,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");
@@ -35925,6 +35934,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 });
@@ -35944,7 +35954,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() {
@@ -35959,6 +35971,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 ***!
@@ -36309,6 +36354,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);
@@ -39950,7 +39996,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
@@ -50177,7 +50223,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);
@@ -51249,7 +51295,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.105.18";
+        return "2.105.20";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -57059,7 +57105,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);
@@ -59147,6 +59193,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);
@@ -59201,6 +59248,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));
@@ -60610,6 +60662,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;
@@ -60655,6 +60708,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));
         }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.105.18",
+    "version": "2.105.20",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.105.18",
+        "@abaplint/core": "^2.105.20",
         "@types/chai": "^4.3.11",
         "@types/glob": "^8.1.0",
         "@types/minimist": "^1.2.5",
@@ -55,7 +55,7 @@
         "mocha": "^10.3.0",
         "progress": "^2.0.3",
         "typescript": "^5.3.3",
-        "webpack": "^5.90.2",
+        "webpack": "^5.90.3",
         "webpack-cli": "^5.1.4",
         "xml-js": "^1.6.11"
     },