@abaplint/transpiler-cli 2.8.20 → 2.8.21

package/build/bundle.js

@@ -14730,9 +14730,11 @@ exports.Retry = Retry;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.Return = void 0;
 const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
+const expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 class Return {
     getMatcher() {
-        return (0, combi_1.str)("RETURN");
+        return (0, combi_1.seq)((0, combi_1.str)("RETURN"), (0, combi_1.optPrio)((0, combi_1.ver)(version_1.Version.v758, expressions_1.Source)));
     }
 }
 exports.Return = Return;
@@ -21366,6 +21368,10 @@ class TypeUtils {
         if (tname === sname) {
             return true;
         }
+        /*
+            console.dir(sid);
+            console.dir(tid);
+        */
         if (!(sid instanceof types_1.ClassDefinition || sid instanceof types_1.InterfaceDefinition)) {
             const found = this.scope.findObjectDefinition(sid.getName());
             if (found) {
@@ -21585,7 +21591,11 @@ class TypeUtils {
             }
         }
         else if (source instanceof basic_1.GenericObjectReferenceType) {
-            if (target instanceof basic_1.ObjectReferenceType) {
+            if (target instanceof basic_1.ObjectReferenceType
+                || target instanceof basic_1.StringType
+                || target instanceof basic_1.CharacterType
+                || target instanceof basic_1.TableType
+                || target instanceof basic_1.CLikeType) {
                 return false;
             }
         }
@@ -23063,6 +23073,7 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.ComponentCompareSimple = void 0;
 const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const nodes_1 = __webpack_require__(/*! ../../nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
+const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_type_utils.js");
 const component_chain_1 = __webpack_require__(/*! ./component_chain */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/component_chain.js");
 const source_1 = __webpack_require__(/*! ./source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
 class ComponentCompareSimple {
@@ -23077,7 +23088,10 @@ class ComponentCompareSimple {
                     targetType = undefined;
                 }
                 else if (c.get() instanceof Expressions.Source) {
-                    new source_1.Source().runSyntax(c, scope, filename, targetType);
+                    const sourceType = new source_1.Source().runSyntax(c, scope, filename, targetType);
+                    if (targetType && new _type_utils_1.TypeUtils(scope).isAssignable(sourceType, targetType) === false) {
+                        throw new Error("ComponentCompareSimple, incompatible types");
+                    }
                 }
                 else {
                     throw "ComponentCompareSimple, unexpected node";
@@ -24839,6 +24853,9 @@ class MethodParameters {
     checkChanging(node, scope, method, filename) {
         var _a;
         for (const item of this.parameterListT(node, scope, filename)) {
+            if (item.target.findFirstExpression(Expressions.InlineData) !== undefined) {
+                throw new Error("CHANGING cannot be inlined");
+            }
             let parameterType = undefined;
             if (method instanceof basic_1.VoidType) {
                 parameterType = method;
@@ -26625,6 +26642,7 @@ const field_assignment_1 = __webpack_require__(/*! ./field_assignment */ "./node
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
 class ValueBody {
     runSyntax(node, scope, filename, targetType) {
+        var _a;
         if (node === undefined) {
             return targetType;
         }
@@ -26640,8 +26658,16 @@ class ValueBody {
                 forScopes++;
             }
         }
+        const fields = new Set();
         for (const s of node.findDirectExpressions(Expressions.FieldAssignment)) {
             new field_assignment_1.FieldAssignment().runSyntax(s, scope, filename, targetType);
+            const fieldname = (_a = s.findDirectExpression(Expressions.FieldSub)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase();
+            if (fieldname) {
+                if (fields.has(fieldname)) {
+                    throw new Error("Duplicate field assignment");
+                }
+                fields.add(fieldname);
+            }
         }
         let type = undefined; // todo, this is only correct if there is a single source in the body
         for (const s of node.findDirectExpressions(Expressions.Source)) {
@@ -28774,11 +28800,21 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.Do = void 0;
 const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const source_1 = __webpack_require__(/*! ../expressions/source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
+const target_1 = __webpack_require__(/*! ../expressions/target */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/target.js");
+const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_type_utils.js");
+const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
 class Do {
     runSyntax(node, scope, filename) {
-        // just recurse
+        const afterDo = node.findExpressionAfterToken("DO");
         for (const s of node.findDirectExpressions(Expressions.Source)) {
-            new source_1.Source().runSyntax(s, scope, filename);
+            const type = new source_1.Source().runSyntax(s, scope, filename);
+            if (s === afterDo
+                && new _type_utils_1.TypeUtils(scope).isAssignable(type, basic_1.IntegerType.get()) === false) {
+                throw new Error("DO TIMES must be numeric");
+            }
+        }
+        for (const t of node.findDirectExpressions(Expressions.Target)) {
+            new target_1.Target().runSyntax(t, scope, filename);
         }
     }
 }
@@ -29540,7 +29576,14 @@ class InsertInternal {
         if (t) {
             targetType = new target_1.Target().runSyntax(t, scope, filename);
         }
-        if (targetType instanceof basic_1.TableType
+        if (!(targetType instanceof basic_1.TableType)
+            && !(targetType instanceof basic_1.VoidType)
+            && !(targetType instanceof basic_1.AnyType)
+            && !(targetType instanceof basic_1.UnknownType)
+            && targetType !== undefined) {
+            throw new Error("INSERT target must be a table");
+        }
+        else if (targetType instanceof basic_1.TableType
             && node.findDirectTokenByText("LINES") === undefined) {
             targetType = targetType.getRowType();
         }
@@ -29549,6 +29592,13 @@ class InsertInternal {
             source = node.findDirectExpression(Expressions.Source);
         }
         const sourceType = source ? new source_1.Source().runSyntax(source, scope, filename, targetType) : targetType;
+        if (targetType === undefined
+            && !(sourceType instanceof basic_1.TableType)
+            && !(sourceType instanceof basic_1.VoidType)
+            && !(sourceType instanceof basic_1.AnyType)
+            && !(sourceType instanceof basic_1.UnknownType)) {
+            throw new Error("INSERT target must be a table");
+        }
         const afterAssigning = node.findExpressionAfterToken("ASSIGNING");
         if ((afterAssigning === null || afterAssigning === void 0 ? void 0 : afterAssigning.get()) instanceof Expressions.FSTarget) {
             const inlinefs = afterAssigning === null || afterAssigning === void 0 ? void 0 : afterAssigning.findDirectExpression(Expressions.InlineFS);
@@ -29559,9 +29609,14 @@ class InsertInternal {
                 new fstarget_1.FSTarget().runSyntax(afterAssigning, scope, filename, sourceType);
             }
         }
-        if (node.findDirectTokenByText("INITIAL") === undefined
-            && new _type_utils_1.TypeUtils(scope).isAssignableStrict(sourceType, targetType) === false) {
-            throw new Error("Types not compatible");
+        if (node.findDirectTokenByText("INITIAL") === undefined) {
+            if (new _type_utils_1.TypeUtils(scope).isAssignableStrict(sourceType, targetType) === false) {
+                throw new Error("Types not compatible");
+            }
+            else if (sourceType instanceof basic_1.CharacterType && targetType instanceof basic_1.StringType) {
+                // yea, well, INSERT doesnt convert the values automatically, like everything else?
+                throw new Error("Types not compatible");
+            }
         }
         const afterInto = node.findExpressionAfterToken("INTO");
         if ((afterInto === null || afterInto === void 0 ? void 0 : afterInto.get()) instanceof Expressions.Target && sourceType) {
@@ -30663,11 +30718,17 @@ class ReadTable {
                 throw new Error("READ TABLE, FROM must be compatible");
             }
         }
+        const afterKey = node.findExpressionAfterToken("KEY");
         for (const s of sources) {
             if (s === firstSource || s === indexSource || s === fromSource) {
                 continue;
             }
-            new source_1.Source().runSyntax(s, scope, filename);
+            const type = new source_1.Source().runSyntax(s, scope, filename);
+            if (s === afterKey) {
+                if (type instanceof basic_1.StringType || type instanceof basic_1.TableType || type instanceof basic_1.ObjectReferenceType) {
+                    throw new Error("Key cannot be string or table or reference");
+                }
+            }
         }
         const target = node.findDirectExpression(Expressions.ReadTableTarget);
         if (target) {
@@ -50322,7 +50383,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.106.3";
+        return "2.106.5";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -51741,6 +51802,7 @@ const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./
 const Structures = __webpack_require__(/*! ../abap/3_structures/structures */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/index.js");
 const Expressions = __webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
 const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
 class BeginEndNamesConf extends _basic_rule_config_1.BasicRuleConfig {
@@ -51777,6 +51839,10 @@ class BeginEndNames extends _abap_rule_1.ABAPRule {
         if (struc === undefined) {
             return [];
         }
+        const containsUnknown = file.getStatements().some(s => s.get() instanceof _statement_1.Unknown);
+        if (containsUnknown === true) {
+            return [];
+        }
         output.push(...this.test(struc, Structures.Data, Statements.DataBegin, Statements.DataEnd, file));
         output.push(...this.test(struc, Structures.ClassData, Statements.ClassDataBegin, Statements.ClassDataEnd, file));
         output.push(...this.test(struc, Structures.Constants, Statements.ConstantBegin, Statements.ConstantEnd, file));
@@ -69978,6 +70044,7 @@ const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@ab
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
 const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
 class UnnecessaryPragmaConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
@@ -70092,7 +70159,8 @@ DATA: BEGIN OF blah ##NEEDED,
         if (statement.findFirstExpression(Expressions.ConstantString) === undefined
             && statement.findFirstExpression(Expressions.StringTemplate) === undefined) {
             const message = "There is no text, NO_TEXT can be removed";
-            return [issue_1.Issue.atToken(file, p, message, this.getMetadata().key, this.getConfig().severity)];
+            const fix = edit_helper_1.EditHelper.deleteToken(file, p);
+            return [issue_1.Issue.atToken(file, p, message, this.getMetadata().key, this.getConfig().severity, fix)];
         }
         return [];
     }
@@ -70104,7 +70172,8 @@ DATA: BEGIN OF blah ##NEEDED,
         const concat = next.concatTokens().toUpperCase();
         if (concat.includes(" SY-SUBRC")) {
             const message = "SUBRC_OK can be removed as sy-subrc is checked";
-            return [issue_1.Issue.atToken(file, p, message, this.getMetadata().key, this.getConfig().severity)];
+            const fix = edit_helper_1.EditHelper.deleteToken(file, p);
+            return [issue_1.Issue.atToken(file, p, message, this.getMetadata().key, this.getConfig().severity, fix)];
         }
         return [];
     }
@@ -70131,7 +70200,8 @@ DATA: BEGIN OF blah ##NEEDED,
             && !(statement.get() instanceof Statements.MethodDef)
             && statement.findFirstExpression(Expressions.InlineFS) === undefined) {
             const message = "There is no data definition, NEEDED can be removed";
-            return [issue_1.Issue.atToken(file, p, message, this.getMetadata().key, this.getConfig().severity)];
+            const fix = edit_helper_1.EditHelper.deleteToken(file, p);
+            return [issue_1.Issue.atToken(file, p, message, this.getMetadata().key, this.getConfig().severity, fix)];
         }
         return [];
     }
@@ -72599,9 +72669,10 @@ var Version;
     Version["v755"] = "v755";
     Version["v756"] = "v756";
     Version["v757"] = "v757";
+    Version["v758"] = "v758";
     Version["Cloud"] = "Cloud";
 })(Version || (exports.Version = Version = {}));
-exports.defaultVersion = Version.v757;
+exports.defaultVersion = Version.v758;
 function getPreviousVersion(v) {
     if (v === Version.OpenABAP) {
         return Version.v702;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.8.20",
+  "version": "2.8.21",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -26,12 +26,12 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/transpiler": "^2.8.20",
+    "@abaplint/transpiler": "^2.8.21",
     "@types/glob": "^8.1.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.7",
     "@types/node": "^20.11.30",
-    "@abaplint/core": "^2.106.3",
+    "@abaplint/core": "^2.106.5",
     "progress": "^2.0.3",
     "webpack": "^5.91.0",
     "webpack-cli": "^5.1.4",