@abaplint/cli 2.102.16 → 2.102.18

package/build/cli.js

@@ -19387,16 +19387,16 @@ class BuiltIn {
     get(extras) {
         const ret = this.buildSY();
         ret.push(this.buildVariable("screen")); // todo, add structure, or alternatively make native Statements
-        ret.push(this.buildConstant("%_BACKSPACE"));
-        ret.push(this.buildConstant("%_CHARSIZE"));
-        ret.push(this.buildConstant("%_CR_LF"));
         ret.push(this.buildConstant("%_ENDIAN"));
-        ret.push(this.buildConstant("%_FORMFEED"));
-        ret.push(this.buildConstant("%_HORIZONTAL_TAB"));
-        ret.push(this.buildConstant("%_MAXCHAR"));
-        ret.push(this.buildConstant("%_MINCHAR"));
-        ret.push(this.buildConstant("%_NEWLINE"));
-        ret.push(this.buildConstant("%_VERTICAL_TAB"));
+        ret.push(this.buildConstant("%_CHARSIZE"));
+        ret.push(this.buildConstant("%_BACKSPACE", new basic_1.CharacterType(1), "\b"));
+        ret.push(this.buildConstant("%_CR_LF", new basic_1.CharacterType(2), "\r\n"));
+        ret.push(this.buildConstant("%_FORMFEED", new basic_1.CharacterType(1), "\f"));
+        ret.push(this.buildConstant("%_HORIZONTAL_TAB", new basic_1.CharacterType(1), "\t"));
+        ret.push(this.buildConstant("%_MAXCHAR", new basic_1.CharacterType(1), Buffer.from("FDFF", "hex").toString()));
+        ret.push(this.buildConstant("%_MINCHAR", new basic_1.CharacterType(1), Buffer.from("0000", "hex").toString()));
+        ret.push(this.buildConstant("%_NEWLINE", new basic_1.CharacterType(1), "\n"));
+        ret.push(this.buildConstant("%_VERTICAL_TAB", new basic_1.CharacterType(1), "\v"));
         ret.push(this.buildConstant("abap_false", new basic_1.CharacterType(1, { qualifiedName: "ABAP_BOOL", ddicName: "ABAP_BOOL" }), "' '"));
         ret.push(this.buildConstant("abap_true", new basic_1.CharacterType(1, { qualifiedName: "ABAP_BOOL", ddicName: "ABAP_BOOL" }), "'X'"));
         ret.push(this.buildConstant("abap_undefined", new basic_1.CharacterType(1, { qualifiedName: "ABAP_BOOL", ddicName: "ABAP_BOOL" }), "'-'"));
@@ -21896,6 +21896,12 @@ class TypeUtils {
             }
             return true;
         }
+        else if (target instanceof basic_1.SimpleType) {
+            if (source instanceof basic_1.ObjectReferenceType
+                || source instanceof basic_1.TableType) {
+                return false;
+            }
+        }
         return true;
     }
 }
@@ -27573,6 +27579,10 @@ class ClassData {
             if (found === undefined) {
                 return undefined;
             }
+            if ((found === null || found === void 0 ? void 0 : found.getType().isGeneric()) === true
+                && (found === null || found === void 0 ? void 0 : found.getType().containsVoid()) === false) {
+                throw new Error("DATA definition cannot be generic, " + found.getName());
+            }
             const meta = [...found.getMeta(), "static" /* IdentifierMeta.Static */];
             return new _typed_identifier_1.TypedIdentifier(found.getToken(), filename, found.getType(), meta, found.getValue());
         }
@@ -31345,6 +31355,10 @@ class Type {
         }
         const found = new basic_types_1.BasicTypes(filename, scope).simpleType(node, qualifiedNamePrefix);
         if (found) {
+            if ((found === null || found === void 0 ? void 0 : found.getType().isGeneric()) === true
+                && (found === null || found === void 0 ? void 0 : found.getType().containsVoid()) === false) {
+                throw new Error("TYPES definition cannot be generic, " + found.getName());
+            }
             return found;
         }
         const fallback = node.findFirstExpression(Expressions.NamespaceSimpleName);
@@ -40471,11 +40485,13 @@ const LServer = __webpack_require__(/*! vscode-languageserver-types */ "./node_m
 const _lsp_utils_1 = __webpack_require__(/*! ./_lsp_utils */ "./node_modules/@abaplint/core/build/src/lsp/_lsp_utils.js");
 const syntax_1 = __webpack_require__(/*! ../abap/5_syntax/syntax */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/syntax.js");
 const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
+const _reference_1 = __webpack_require__(/*! ../abap/5_syntax/_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
+const types_1 = __webpack_require__(/*! ../abap/types */ "./node_modules/@abaplint/core/build/src/abap/types/index.js");
 class CodeLens {
     constructor(reg) {
         this.reg = reg;
     }
-    list(textDocument, settings = { messageText: true }) {
+    list(textDocument, settings = { messageText: true, dynamicExceptions: true }) {
         var _a;
         const file = _lsp_utils_1.LSPUtils.getABAPFile(this.reg, textDocument.uri);
         if (file === undefined) {
@@ -40485,7 +40501,7 @@ class CodeLens {
         if (obj === undefined || !(obj instanceof _abap_object_1.ABAPObject)) {
             return [];
         }
-        new syntax_1.SyntaxLogic(this.reg, obj).run();
+        const top = new syntax_1.SyntaxLogic(this.reg, obj).run().spaghetti.getTop();
         const ret = [];
         if (settings.messageText === true) {
             const list = this.reg.getMSAGReferences().listByFilename(file.getFilename());
@@ -40500,10 +40516,59 @@ class CodeLens {
                 }
                 ret.push({
                     range: _lsp_utils_1.LSPUtils.tokenToRange(l.token),
-                    command: LServer.Command.create(text, ""),
+                    command: LServer.Command.create(text, "")
                 });
             }
         }
+        if (settings.dynamicExceptions === true) {
+            for (const ref of this.findMethodReferences(top)) {
+                if (!(ref.resolved instanceof types_1.MethodDefinition)) {
+                    continue;
+                }
+                let text = "";
+                for (const e of ref.resolved.getExceptions()) {
+                    if (this.isDynamicException(e, top)) {
+                        if (text === "") {
+                            text = "Dynamic Exceptions: ";
+                        }
+                        else {
+                            text += " & ";
+                        }
+                        text += e.toUpperCase();
+                    }
+                }
+                if (text !== "") {
+                    ret.push({
+                        range: _lsp_utils_1.LSPUtils.tokenToRange(ref.resolved.getToken()),
+                        command: LServer.Command.create(text, "")
+                    });
+                }
+            }
+        }
+        return ret;
+    }
+    isDynamicException(name, top) {
+        var _a;
+        // todo: this method only works with global exceptions?
+        let current = name;
+        while (current !== undefined) {
+            if (current.toUpperCase() === "CX_DYNAMIC_CHECK") {
+                return true;
+            }
+            current = (_a = top.findClassDefinition(current)) === null || _a === void 0 ? void 0 : _a.getSuperClass();
+        }
+        return false;
+    }
+    findMethodReferences(node) {
+        const ret = [];
+        for (const r of node.getData().references) {
+            if (r.referenceType === _reference_1.ReferenceType.MethodReference) {
+                ret.push(r);
+            }
+        }
+        for (const c of node.getChildren()) {
+            ret.push(...this.findMethodReferences(c));
+        }
         return ret;
     }
 }
@@ -48624,7 +48689,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.16";
+        return "2.102.18";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -57338,6 +57403,8 @@ class FunctionModuleRecommendationsConf extends _basic_rule_config_1.BasicRuleCo
         this.recommendations = [
             { name: "CALCULATE_HASH_FOR_RAW", replace: "use CL_ABAP_HMAC or CL_ABAP_MESSAGE_DIGEST" },
             { name: "CCU_TIMESTAMP_DIFFERENCE", replace: "use CL_ABAP_TSTMP" },
+            { name: "CONVERT_DATE_TO_EXTERNAL", replace: "use CL_ABAP_DATFM" },
+            { name: "CONVERT_TIME_INPUT", replace: "use CL_ABAP_TIMEFM" },
             { name: "ECATT_CONV_XSTRING_TO_STRING", replace: "use CL_BINARY_CONVERT" },
             { name: "F4_FILENAME", replace: "use CL_GUI_FRONTEND_SERVICES" },
             { name: "FUNCTION_EXISTS", replace: "surround with try-catch CX_SY_DYN_CALL_ILLEGAL_METHOD instead" },
@@ -65144,7 +65211,9 @@ class SelectPerformanceConf extends _basic_rule_config_1.BasicRuleConfig {
         this.endSelect = true;
         /** Detects SELECT * */
         this.selectStar = true;
-        /** "SELECT" * is considered okay if the table is less than X columns, the table must be known to the linter */
+        /** "SELECT" * is considered okay if the table is less than X columns, the table must be known to the linter
+         * @default 10
+         */
         this.starOkayIfFewColumns = DEFAULT_COLUMNS;
     }
 }
@@ -67102,6 +67171,11 @@ const syntax_1 = __webpack_require__(/*! ../abap/5_syntax/syntax */ "./node_modu
 const _reference_1 = __webpack_require__(/*! ../abap/5_syntax/_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 const types_1 = __webpack_require__(/*! ../abap/types */ "./node_modules/@abaplint/core/build/src/abap/types/index.js");
 class UncaughtExceptionConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        this.reportDynamic = false;
+        this.reportNoCheck = false;
+    }
 }
 exports.UncaughtExceptionConf = UncaughtExceptionConf;
 class UncaughtException extends _abap_rule_1.ABAPRule {
@@ -67268,11 +67342,17 @@ class UncaughtException extends _abap_rule_1.ABAPRule {
             return true;
         }
         const sup = this.globalExceptions[name.toUpperCase()];
-        if (sup === "CX_DYNAMIC_CHECK" || sup === "CX_NO_CHECK") {
+        if (sup === "CX_DYNAMIC_CHECK" && this.getConfig().reportDynamic !== true) {
+            return true;
+        }
+        if (sup === "CX_NO_CHECK" && this.getConfig().reportNoCheck !== true) {
             return true;
         }
         const lsup = this.localExceptions[name.toUpperCase()];
-        if (lsup === "CX_DYNAMIC_CHECK" || lsup === "CX_NO_CHECK") {
+        if (lsup === "CX_DYNAMIC_CHECK" && this.getConfig().reportDynamic !== true) {
+            return true;
+        }
+        if (lsup === "CX_NO_CHECK" && this.getConfig().reportNoCheck !== true) {
             return true;
         }
         return this.sinked.some(a => a.toUpperCase() === name.toUpperCase())

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.102.16",
+    "version": "2.102.18",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,16 +38,16 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.102.16",
+        "@abaplint/core": "^2.102.18",
         "@types/chai": "^4.3.5",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.2",
         "@types/mocha": "^10.0.1",
-        "@types/node": "^20.4.8",
+        "@types/node": "^20.5.0",
         "@types/progress": "^2.0.5",
         "chai": "^4.3.7",
         "chalk": "^5.3.0",
-        "eslint": "^8.46.0",
+        "eslint": "^8.47.0",
         "glob": "^7.2.3",
         "json5": "^2.2.3",
         "memfs": "^4.2.1",