@abaplint/core 2.85.21 → 2.85.24

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

@@ -1,11 +1,13 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.SQLAsName = void 0;
+const tokens_1 = require("../../1_lexer/tokens");
 const combi_1 = require("../combi");
 class SQLAsName extends combi_1.Expression {
     getRunnable() {
         // todo, below allows too much?
-        return (0, combi_1.regex)(/^[&_!]?\*?\w*(\/\w+\/)?\d*[a-zA-Z_%\$][\w\*%\$\?]*(~\w+)?$/);
+        const field = (0, combi_1.regex)(/^[&_!]?\*?\w*(\/\w+\/)?\d*[a-zA-Z_%\$][\w\*%\$\?]*(~\w+)?$/);
+        return (0, combi_1.seq)(field, (0, combi_1.starPrio)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.Dash), field)));
     }
 }
 exports.SQLAsName = SQLAsName;

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

@@ -12,7 +12,7 @@ class SQLFieldList extends combi_1.Expression {
     getRunnable() {
         const comma = (0, combi_1.optPrio)((0, combi_1.ver)(version_1.Version.v740sp05, ","));
         const abap = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), simple_field_chain_1.SimpleFieldChain));
-        const as = (0, combi_1.seq)("AS", _1.Field);
+        const as = (0, combi_1.seq)("AS", _1.SQLAsName);
         const field = (0, combi_1.altPrio)(_1.SQLAggregation, _1.SQLCase, sql_function_1.SQLFunction, sql_path_1.SQLPath, _1.SQLFieldName, abap, _1.Constant);
         const sub = (0, combi_1.plusPrio)((0, combi_1.seq)((0, combi_1.altPrio)("+", "-", "*", "/", "&&"), (0, combi_1.optPrio)((0, combi_1.tok)(tokens_1.WParenLeftW)), field, (0, combi_1.optPrio)((0, combi_1.tok)(tokens_1.WParenRightW))));
         const arith = (0, combi_1.ver)(version_1.Version.v740sp05, sub);

package/build/src/abap/5_syntax/_type_utils.js

@@ -67,6 +67,12 @@ class TypeUtils {
     }
     isOOAssignable(source, target) {
         let sid = source.getIdentifier();
+        let tid = target.getIdentifier();
+        const tname = tid.getName().toUpperCase();
+        const sname = sid.getName().toUpperCase();
+        if (tname === sname) {
+            return true;
+        }
         if (!(sid instanceof types_1.ClassDefinition || sid instanceof types_1.InterfaceDefinition)) {
             const found = this.scope.findObjectDefinition(sid.getName());
             if (found) {
@@ -76,7 +82,6 @@ class TypeUtils {
                 return false;
             }
         }
-        let tid = target.getIdentifier();
         if (!(tid instanceof types_1.ClassDefinition || tid instanceof types_1.InterfaceDefinition)) {
             const found = this.scope.findObjectDefinition(tid.getName());
             if (found) {
@@ -86,9 +91,8 @@ class TypeUtils {
                 return false;
             }
         }
-        const tname = tid.getName().toUpperCase();
         if (sid instanceof types_1.ClassDefinition && tid instanceof types_1.ClassDefinition) {
-            if (sid.getName().toUpperCase() === tname) {
+            if (sname === tname) {
                 return true;
             }
             const slist = this.listAllSupers(sid);
@@ -106,7 +110,7 @@ class TypeUtils {
             }
         }
         else if (sid instanceof types_1.InterfaceDefinition && tid instanceof types_1.InterfaceDefinition) {
-            if (sid.getName().toUpperCase() === tname) {
+            if (sname === tname) {
                 return true;
             }
             if (sid.getImplementing().some(i => i.name === tname)) {

package/build/src/abap/5_syntax/expressions/method_source.js

@@ -11,6 +11,7 @@ const _reference_1 = require("../_reference");
 const _object_oriented_1 = require("../_object_oriented");
 class MethodSource {
     runSyntax(node, scope, filename) {
+        // todo, rewrite the context finding, and/or restructure the expression?
         const context = new method_call_chain_1.MethodCallChain().runSyntax(node, scope, filename);
         const last = node.getLastChild();
         const first = node.getFirstChild();
@@ -19,8 +20,11 @@ class MethodSource {
         }
         else if (last instanceof nodes_1.ExpressionNode && last.get() instanceof Expressions.MethodName) {
             if (context instanceof basic_1.ObjectReferenceType) {
-                const id = context.getIdentifier();
-                if (id instanceof types_1.ClassDefinition) {
+                let id = context.getIdentifier();
+                if (!(id instanceof types_1.ClassDefinition)) {
+                    id = scope.findObjectDefinition(id.getName());
+                }
+                if (id instanceof types_1.ClassDefinition) { // todo || id instanceof InterfaceDefinition) {
                     const methodName = last.concatTokens().toUpperCase();
                     const helper = new _object_oriented_1.ObjectOriented(scope);
                     const { method: foundMethod } = helper.searchMethodName(id, methodName);

package/build/src/abap/5_syntax/expressions/source.js

@@ -72,8 +72,11 @@ class Source {
                 case "SWITCH":
                     {
                         const foundType = this.determineType(node, scope, filename, targetType);
-                        new switch_body_1.SwitchBody().runSyntax(node.findDirectExpression(Expressions.SwitchBody), scope, filename);
-                        return foundType;
+                        const bodyType = new switch_body_1.SwitchBody().runSyntax(node.findDirectExpression(Expressions.SwitchBody), scope, filename);
+                        if (foundType === undefined) {
+                            this.addIfInferred(node, scope, filename, bodyType);
+                        }
+                        return foundType ? foundType : bodyType;
                     }
                 case "COND":
                     {

package/build/src/abap/5_syntax/expressions/sql_for_all_entries.js

@@ -18,7 +18,7 @@ class SQLForAllEntries {
             if (!(type instanceof basic_1.TableType)) {
                 throw new Error("FAE parameter must be table type");
             }
-            const name = s.concatTokens();
+            const name = s.concatTokens().replace("[]", "");
             scope.setAllowHeaderUse(name);
         }
     }

package/build/src/abap/5_syntax/expressions/switch_body.js

@@ -8,9 +8,18 @@ class SwitchBody {
         if (node === undefined) {
             return;
         }
+        const thenSource = node.findExpressionAfterToken("THEN");
+        if (!((thenSource === null || thenSource === void 0 ? void 0 : thenSource.get()) instanceof Expressions.Source)) {
+            throw new Error("SwitchBody, unexpected");
+        }
+        const type = new source_1.Source().runSyntax(thenSource, scope, filename);
         for (const s of node.findDirectExpressions(Expressions.Source)) {
+            if (s === thenSource) {
+                continue;
+            }
             new source_1.Source().runSyntax(s, scope, filename);
         }
+        return type;
     }
 }
 exports.SwitchBody = SwitchBody;

package/build/src/registry.js

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

package/build/src/rules/downport.js

@@ -56,7 +56,7 @@ Current rules:
 * PARTIALLY IMPLEMENTED removed, it can be quick fixed via rule implement_methods
 * RAISE EXCEPTION ... MESSAGE
 * APPEND expression is outlined
-* Moving with +=, -=, /=, *=, &&=
+* Moving with +=, -=, /=, *=, &&= is expanded
 
 Only one transformation is applied to a statement at a time, so multiple steps might be required to do the full downport.`,
             tags: [_irule_1.RuleTag.Experimental, _irule_1.RuleTag.Downport, _irule_1.RuleTag.Quickfix],
@@ -224,7 +224,11 @@ Only one transformation is applied to a statement at a time, so multiple steps m
         if (found) {
             return found;
         }
-        // todo, line_exists() should be replaced before this call
+        found = this.replaceLineExists(high, lowFile, highSyntax);
+        if (found) {
+            return found;
+        }
+        // note: line_exists() must be replaced before this call
         found = this.replaceTableExpression(high, lowFile, highSyntax);
         if (found) {
             return found;
@@ -233,7 +237,6 @@ Only one transformation is applied to a statement at a time, so multiple steps m
         if (found) {
             return found;
         }
-        // todo, add more rules here
         return undefined;
     }
     //////////////////////////////////////////
@@ -1089,6 +1092,53 @@ ${indentation}    output = ${topTarget}.`;
         }
         return undefined;
     }
+    findMethodCallExpression(node, token) {
+        var _a;
+        for (const m of node.findAllExpressions(Expressions.MethodCall)) {
+            if ((_a = m.findDirectExpression(Expressions.MethodName)) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStart().equals(token.getStart())) {
+                return m;
+            }
+        }
+        return undefined;
+    }
+    replaceLineExists(node, lowFile, highSyntax) {
+        var _a, _b;
+        const spag = highSyntax.spaghetti.lookupPosition(node.getFirstToken().getStart(), lowFile.getFilename());
+        for (const r of (spag === null || spag === void 0 ? void 0 : spag.getData().references) || []) {
+            if (r.referenceType === _reference_1.ReferenceType.BuiltinMethodReference
+                && r.position.getName().toUpperCase() === "LINE_EXISTS") {
+                const token = r.position.getToken();
+                const expression = this.findMethodCallExpression(node, token);
+                if (expression === undefined) {
+                    continue;
+                }
+                let condition = "";
+                for (const c of ((_a = expression === null || expression === void 0 ? void 0 : expression.findFirstExpression(Expressions.TableExpression)) === null || _a === void 0 ? void 0 : _a.getChildren()) || []) {
+                    if (c.getFirstToken().getStr() === "[" || c.getFirstToken().getStr() === "]") {
+                        continue;
+                    }
+                    else if (c.get() instanceof Expressions.ComponentChainSimple && condition === "") {
+                        condition = "WITH KEY ";
+                    }
+                    condition += c.concatTokens() + " ";
+                }
+                const tableName = (_b = expression === null || expression === void 0 ? void 0 : expression.findFirstExpression(Expressions.SourceField)) === null || _b === void 0 ? void 0 : _b.concatTokens();
+                const uniqueName = this.uniqueName(node.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
+                const indentation = " ".repeat(node.getFirstToken().getStart().getCol() - 1);
+                const code = `DATA ${uniqueName} LIKE sy-subrc.\n` +
+                    indentation + `READ TABLE ${tableName} ${condition}TRANSPORTING NO FIELDS.\n` +
+                    indentation + uniqueName + " = sy-subrc.\n" +
+                    indentation;
+                const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, node.getFirstToken().getStart(), code);
+                const start = expression.getFirstToken().getStart();
+                const end = expression.getLastToken().getEnd();
+                const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, start, end, uniqueName + " = 0");
+                const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
+                return issue_1.Issue.atToken(lowFile, token, "Use BOOLC", this.getMetadata().key, this.conf.severity, fix);
+            }
+        }
+        return undefined;
+    }
     newToCreateObject(node, lowFile, highSyntax) {
         const source = node.findDirectExpression(Expressions.Source);
         let fix = undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.85.21",
+    "version": "2.85.24",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -48,7 +48,7 @@
         "@microsoft/api-extractor": "^7.19.4",
         "@types/chai": "^4.3.0",
         "@types/mocha": "^9.1.0",
-        "@types/node": "^17.0.19",
+        "@types/node": "^17.0.21",
         "chai": "^4.3.6",
         "eslint": "^8.9.0",
         "mocha": "^9.2.1",