@abaplint/cli 2.119.37 → 2.119.39

package/build/cli.js

@@ -26933,16 +26933,16 @@ class BasicTypes {
             options.primaryKey.type = basic_1.TableAccessType.standard;
             return new Types.TableType(structure, options);
         }
-        else if (typename && (text.startsWith("TYPE TABLE FOR CREATE ")
-            || text.startsWith("TYPE TABLE FOR READ ")
-            || text.startsWith("TYPE TABLE FOR DELETE ")
-            || text.startsWith("TYPE TABLE FOR UPDATE "))) {
+        else if (typename && this.isRAPTableFor(text)) {
             const name = typename.concatTokens();
             const ddlsName = this.getRAPBaseEntityName(name);
             const type = (_d = this.input.scope.getDDIC().lookupDDLS(ddlsName)) === null || _d === void 0 ? void 0 : _d.type;
             if (type) {
                 return new Types.TableType(basic_1.VoidType.get("RAP-TODO"), options);
             }
+            else if (this.isRAPDerivedEntityName(name)) {
+                return Types.VoidType.get(name);
+            }
             else if (this.input.scope.getDDIC().inErrorNamespace(ddlsName)) {
                 return new Types.UnknownType(`DDLS ${ddlsName} not found`);
             }
@@ -26954,7 +26954,7 @@ class BasicTypes {
         return this.parseType(node, name);
     }
     parseType(node, qualifiedName) {
-        var _a, _b, _c, _d, _e, _f, _g, _h;
+        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k;
         const typeName = node.findFirstExpression(Expressions.TypeName);
         let text = (_a = node.findFirstExpression(Expressions.Type)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase();
         if (text === undefined) {
@@ -26967,7 +26967,10 @@ class BasicTypes {
             }
         }
         if (text === undefined) {
-            text = (_d = node.findFirstExpression(Expressions.FormParamType)) === null || _d === void 0 ? void 0 : _d.concatTokens().toUpperCase();
+            text = (_d = node.findFirstExpression(Expressions.TypeStructure)) === null || _d === void 0 ? void 0 : _d.concatTokens().toUpperCase();
+        }
+        if (text === undefined) {
+            text = (_e = node.findFirstExpression(Expressions.FormParamType)) === null || _e === void 0 ? void 0 : _e.concatTokens().toUpperCase();
         }
         if (text === undefined
             && node.get() instanceof Statements.Parameter
@@ -26978,8 +26981,11 @@ class BasicTypes {
             text = "TYPE";
         }
         let found = undefined;
-        if (text.startsWith("LIKE LINE OF ")) {
-            const name = (_e = node.findFirstExpression(Expressions.FieldChain)) === null || _e === void 0 ? void 0 : _e.concatTokens();
+        if (this.isRAPTypeStructure(text)) {
+            return Types.VoidType.get((_f = node.findFirstExpression(Expressions.TypeStructure)) === null || _f === void 0 ? void 0 : _f.concatTokens());
+        }
+        else if (text.startsWith("LIKE LINE OF ")) {
+            const name = (_g = node.findFirstExpression(Expressions.FieldChain)) === null || _g === void 0 ? void 0 : _g.concatTokens();
             let e = node.findFirstExpression(Expressions.Type);
             if (e === undefined) {
                 e = node.findFirstExpression(Expressions.FormParamType);
@@ -27002,7 +27008,7 @@ class BasicTypes {
             }
         }
         else if (text.startsWith("LIKE REF TO ")) {
-            const name = (_f = node.findFirstExpression(Expressions.FieldChain)) === null || _f === void 0 ? void 0 : _f.concatTokens();
+            const name = (_h = node.findFirstExpression(Expressions.FieldChain)) === null || _h === void 0 ? void 0 : _h.concatTokens();
             const type = this.resolveLikeName(node.findFirstExpression(Expressions.Type), false);
             if (type === undefined) {
                 return new Types.UnknownType("Type error, could not resolve \"" + name + "\", parseType");
@@ -27093,7 +27099,7 @@ class BasicTypes {
             }
         }
         if (text.includes(" WITH INDICATORS ")) {
-            const componentName = (_h = (_g = node.findFirstExpression(Expressions.Type)) === null || _g === void 0 ? void 0 : _g.findDirectExpression(Expressions.ComponentName)) === null || _h === void 0 ? void 0 : _h.concatTokens().toUpperCase();
+            const componentName = (_k = (_j = node.findFirstExpression(Expressions.Type)) === null || _j === void 0 ? void 0 : _j.findDirectExpression(Expressions.ComponentName)) === null || _k === void 0 ? void 0 : _k.concatTokens().toUpperCase();
             if (componentName === undefined) {
                 throw new Error("parseType, componentName expected");
             }
@@ -27119,12 +27125,38 @@ class BasicTypes {
     getRAPBaseEntityName(name) {
         const association = name.indexOf("\\_");
         const path = name.indexOf("\\\\");
-        if (association === -1 && path === -1) {
+        const action = name.indexOf("~");
+        if (association === -1 && path === -1 && action === -1) {
             return name;
         }
-        const splitAt = association === -1 ? path : path === -1 ? association : Math.min(association, path);
+        const candidates = [association, path, action].filter(i => i !== -1);
+        const splitAt = Math.min(...candidates);
         return name.substring(0, splitAt);
     }
+    isRAPDerivedEntityName(name) {
+        return name.includes("\\") || name.includes("~");
+    }
+    isRAPTableFor(text) {
+        return text.startsWith("TYPE TABLE FOR ACTION IMPORT ")
+            || text.startsWith("TYPE TABLE FOR ACTION RESULT ")
+            || text.startsWith("TYPE TABLE FOR CREATE ")
+            || text.startsWith("TYPE TABLE FOR DELETE ")
+            || text.startsWith("TYPE TABLE FOR DETERMINATION ")
+            || text.startsWith("TYPE TABLE FOR EVENT ")
+            || text.startsWith("TYPE TABLE FOR FAILED ")
+            || text.startsWith("TYPE TABLE FOR FAILED EARLY ")
+            || text.startsWith("TYPE TABLE FOR LOCK ")
+            || text.startsWith("TYPE TABLE FOR READ ")
+            || text.startsWith("TYPE TABLE FOR READ IMPORT ")
+            || text.startsWith("TYPE TABLE FOR READ RESULT ")
+            || text.startsWith("TYPE TABLE FOR REPORTED EARLY ")
+            || text.startsWith("TYPE TABLE FOR UPDATE ");
+    }
+    isRAPTypeStructure(text) {
+        return text.startsWith("TYPE STRUCTURE FOR ")
+            || text.startsWith("TYPE RESPONSE FOR ")
+            || text.startsWith("TYPE REQUEST FOR CHANGE ");
+    }
     // todo, rewrite this method
     resolveTypeChain(expr) {
         var _a, _b, _c, _d;
@@ -37383,6 +37415,7 @@ const source_1 = __webpack_require__(/*! ../expressions/source */ "../core/build
 const component_compare_1 = __webpack_require__(/*! ../expressions/component_compare */ "../core/build/src/abap/5_syntax/expressions/component_compare.js");
 const component_cond_1 = __webpack_require__(/*! ../expressions/component_cond */ "../core/build/src/abap/5_syntax/expressions/component_cond.js");
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "../core/build/src/abap/types/basic/index.js");
+const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
 class DeleteInternal {
     runSyntax(node, input) {
         var _a;
@@ -37393,7 +37426,8 @@ class DeleteInternal {
         const target = node.findDirectExpression(Expressions.Target);
         if (target) {
             let tabl = undefined;
-            if (node.getChildren().length === 5 && node.getChildren()[2].concatTokens().toUpperCase() === "FROM") {
+            const localVariable = input.scope.findVariable(target.concatTokens());
+            if (localVariable === undefined && node.getChildren().length === 5 && node.getChildren()[2].concatTokens().toUpperCase() === "FROM") {
                 // it might be a database table
                 tabl = (_a = input.scope.getDDIC()) === null || _a === void 0 ? void 0 : _a.lookupTableOrView(target.concatTokens());
                 if (tabl) {
@@ -37402,6 +37436,14 @@ class DeleteInternal {
             }
             if (tabl === undefined) {
                 targetType = target_1.Target.runSyntax(target, input);
+                if (node.findDirectTokenByText("TABLE") === undefined
+                    && node.findDirectTokenByText("FROM")
+                    && targetType instanceof basic_1.TableType
+                    && targetType.getAccessType() === basic_1.TableAccessType.hashed) {
+                    const message = "Implicit or explicit index operation on hashed table is not possible";
+                    input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+                    return;
+                }
                 if (targetType instanceof basic_1.TableType) {
                     targetType = targetType.getRowType();
                 }
@@ -45846,6 +45888,10 @@ class SyntaxLogic {
             this.scope.addList(values);
             return true;
         }
+        else if (stru instanceof Structures.TestInjection) {
+            // todo: skipped for now
+            return true;
+        }
         return false;
     }
     updateScopeStatement(node) {
@@ -66531,7 +66577,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.119.37";
+        return "2.119.39";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -89398,6 +89444,9 @@ class SelectionScreenTextsMissing {
             key: "selection_screen_texts_missing",
             title: "Selection screen texts missing",
             shortDescription: `Checks that selection screen parameters and select-options have selection texts`,
+            extendedInformation: `Excludes parameters and select-options that:
+* are inside a "SELECTION-SCREEN BEGIN OF LINE" block
+* have the addition "NO-DISPLAY"`,
         };
     }
     getConfig() {
@@ -89423,7 +89472,7 @@ class SelectionScreenTextsMissing {
         this.checkFile(obj.getMainABAPFile(), selTexts, output, checked);
         return output;
     }
-    checkFile(file, selTexts, output, checked) {
+    checkFile(file, selTexts, output, checked, mainProgName = undefined) {
         if (file === undefined) {
             return;
         }
@@ -89431,26 +89480,44 @@ class SelectionScreenTextsMissing {
             return;
         }
         checked.add(file.getFilename());
+        let inLine = false;
         for (const stat of file.getStatements()) {
             const s = stat.get();
-            if (s instanceof statements_1.Parameter || s instanceof statements_1.SelectOption) {
-                const fieldNode = stat.findFirstExpression(expressions_1.FieldSub);
-                if (fieldNode) {
-                    const fieldName = fieldNode.getFirstToken().getStr().toUpperCase();
-                    if (selTexts[fieldName] === undefined) {
-                        output.push(issue_1.Issue.atToken(file, fieldNode.getFirstToken(), `Selection text missing for "${fieldName}"`, this.getMetadata().key, this.conf.severity));
-                    }
+            if (s instanceof statements_1.SelectionScreen) {
+                const tokens = stat.concatTokens().toUpperCase();
+                if (tokens.includes("BEGIN OF LINE")) {
+                    inLine = true;
                 }
+                else if (tokens.includes("END OF LINE")) {
+                    // known issue: doesn't support BEGIN and END OF LINE span across several includes (not seen a lot in the wild)
+                    inLine = false;
+                }
+                continue;
             }
-            else if (s instanceof statements_1.Include) {
+            if (s instanceof statements_1.Include) {
                 const nameNode = stat.findFirstExpression(expressions_1.IncludeName);
                 if (nameNode) {
                     const inclName = nameNode.getFirstToken().getStr().toUpperCase();
                     const inclObj = this.reg.getObject("PROG", inclName);
                     if (inclObj) {
-                        this.checkFile(inclObj.getMainABAPFile(), selTexts, output, checked);
+                        this.checkFile(inclObj.getMainABAPFile(), selTexts, output, checked, mainProgName !== null && mainProgName !== void 0 ? mainProgName : file.getFilename());
                     }
                 }
+                continue;
+            }
+            if (inLine || !(s instanceof statements_1.Parameter || s instanceof statements_1.SelectOption)) {
+                continue;
+            }
+            if (stat.concatTokens().toUpperCase().includes("NO-DISPLAY")) {
+                continue;
+            }
+            const fieldNode = stat.findFirstExpression(expressions_1.FieldSub);
+            if (fieldNode) {
+                const fieldName = fieldNode.getFirstToken().getStr().toUpperCase();
+                if (selTexts[fieldName] === undefined) {
+                    const suffix = mainProgName ? `, ${mainProgName}` : ``;
+                    output.push(issue_1.Issue.atToken(file, fieldNode.getFirstToken(), `Selection text missing for "${fieldName}"${suffix}`, this.getMetadata().key, this.conf.severity));
+                }
             }
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.119.37",
+    "version": "2.119.39",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.119.37",
+        "@abaplint/core": "^2.119.39",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",