@abaplint/core 2.113.89 → 2.113.91

package/build/abaplint.d.ts

@@ -5254,15 +5254,27 @@ declare class PrivateSection implements IStructure {
     getMatcher(): IStructureRunnable;
 }
 
-declare class ProcessAfterInput implements IStatement {
+declare class ProcessAfterInput implements IStructure {
+    getMatcher(): IStructureRunnable;
+}
+
+declare class ProcessAfterInput_2 implements IStatement {
     getMatcher(): IStatementRunnable;
 }
 
-declare class ProcessBeforeOutput implements IStatement {
+declare class ProcessBeforeOutput implements IStructure {
+    getMatcher(): IStructureRunnable;
+}
+
+declare class ProcessBeforeOutput_2 implements IStatement {
     getMatcher(): IStatementRunnable;
 }
 
-declare class ProcessOnValueRequest implements IStatement {
+declare class ProcessOnValueRequest implements IStructure {
+    getMatcher(): IStructureRunnable;
+}
+
+declare class ProcessOnValueRequest_2 implements IStatement {
     getMatcher(): IStatementRunnable;
 }
 
@@ -6496,9 +6508,9 @@ declare namespace Statements {
         DataBegin,
         DataEnd,
         TypeBegin,
-        ProcessAfterInput,
-        ProcessBeforeOutput,
-        ProcessOnValueRequest,
+        ProcessAfterInput_2 as ProcessAfterInput,
+        ProcessBeforeOutput_2 as ProcessBeforeOutput,
+        ProcessOnValueRequest_2 as ProcessOnValueRequest,
         TypeEnd,
         RaiseEvent,
         Cleanup_2 as Cleanup,
@@ -6631,10 +6643,10 @@ declare namespace Structures {
         Case,
         CatchSystemExceptions,
         Catch,
+        Chain,
         ClassData,
         ClassDefinition,
         ClassGlobal,
-        Chain,
         ClassImplementation,
         Cleanup,
         Constants,
@@ -6659,6 +6671,9 @@ declare namespace Structures {
         Normal,
         OnChange,
         PrivateSection,
+        ProcessAfterInput,
+        ProcessBeforeOutput,
+        ProcessOnValueRequest,
         ProtectedSection,
         Provide,
         PublicSection,

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

@@ -4,7 +4,7 @@ exports.SQLAliasField = void 0;
 const combi_1 = require("../combi");
 class SQLAliasField extends combi_1.Expression {
     getRunnable() {
-        return (0, combi_1.regex)(/^(\/\w+\/)?\w+~\w+$/);
+        return (0, combi_1.regex)(/^(\/\w+\/)?\w+~(\/\w+\/)?\w+$/);
     }
 }
 exports.SQLAliasField = SQLAliasField;

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

@@ -25,8 +25,12 @@ class SQLFunction extends combi_1.Expression {
         const round = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)((0, combi_1.regex)(/^round$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, commaParam, (0, combi_1.tok)(tokens_1.WParenRightW)));
         const upper = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)((0, combi_1.regex)(/^upper$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, (0, combi_1.tok)(tokens_1.WParenRightW)));
         const uuid = (0, combi_1.ver)(version_1.Version.v754, (0, combi_1.seq)((0, combi_1.regex)(/^uuid$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), (0, combi_1.tok)(tokens_1.WParenRightW)));
-        const concat_with_space = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)((0, combi_1.regex)(/^uuid$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, commaParam, commaParam, (0, combi_1.tok)(tokens_1.WParenRightW)));
-        return (0, combi_1.altPrio)(uuid, abs, ceil, floor, cast, div, mod, coalesce, concat, replace, length, lower, upper, round, concat_with_space);
+        const concat_with_space = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)((0, combi_1.regex)(/^concat_with_space$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, commaParam, commaParam, (0, combi_1.tok)(tokens_1.WParenRightW)));
+        // dunno if the version for substring is correct
+        const substring = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)((0, combi_1.regex)(/^substring$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, commaParam, commaParam, (0, combi_1.tok)(tokens_1.WParenRightW)));
+        const ltrim = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)((0, combi_1.regex)(/^ltrim$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, commaParam, (0, combi_1.tok)(tokens_1.WParenRightW)));
+        const rtrim = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)((0, combi_1.regex)(/^rtrim$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, commaParam, (0, combi_1.tok)(tokens_1.WParenRightW)));
+        return (0, combi_1.altPrio)(uuid, abs, ceil, floor, cast, div, mod, coalesce, concat, replace, length, lower, upper, round, concat_with_space, ltrim, rtrim, substring);
     }
 }
 exports.SQLFunction = SQLFunction;

package/build/src/abap/2_statements/statements/modify_line.js

@@ -8,7 +8,7 @@ class ModifyLine {
     getMatcher() {
         const onOff = (0, combi_1.alt)("ON", "OFF");
         const eq = (0, combi_1.seq)("=", expressions_1.Source);
-        const form = (0, combi_1.seq)((0, combi_1.alt)("INVERSE", "INPUT", "COLOR", "HOTSPOT"), (0, combi_1.alt)(eq, onOff));
+        const form = (0, combi_1.seq)((0, combi_1.alt)("INVERSE", "INPUT", "COLOR", "HOTSPOT"), (0, combi_1.opt)((0, combi_1.alt)(eq, onOff)));
         const from = (0, combi_1.seq)("FROM", expressions_1.Source);
         const value = (0, combi_1.seq)("FIELD VALUE", (0, combi_1.plus)((0, combi_1.seq)(expressions_1.Source, (0, combi_1.optPrio)(from))));
         const format = (0, combi_1.seq)("FIELD FORMAT", expressions_1.Source, (0, combi_1.opt)(form));

package/build/src/abap/2_statements/statements/verification_message.js

@@ -6,7 +6,8 @@ const expressions_1 = require("../expressions");
 const version_1 = require("../../../version");
 class VerificationMessage {
     getMatcher() {
-        const ret = (0, combi_1.seq)("VERIFICATION-MESSAGE", expressions_1.Source, expressions_1.Source);
+        const priority = (0, combi_1.seq)("PRIORITY", expressions_1.Source);
+        const ret = (0, combi_1.seq)("VERIFICATION-MESSAGE", expressions_1.Source, expressions_1.Source, (0, combi_1.opt)(priority));
         return (0, combi_1.verNot)(version_1.Version.Cloud, ret);
     }
 }

package/build/src/abap/3_structures/structures/dynpro_logic.js

@@ -1,15 +1,11 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.DynproLogic = void 0;
-const Statements = require("../../2_statements/statements");
+const Structures = require("./");
 const _combi_1 = require("./_combi");
-const chain_1 = require("./chain");
 class DynproLogic {
     getMatcher() {
-        const pai = (0, _combi_1.star)((0, _combi_1.alt)((0, _combi_1.sta)(Statements.Module), (0, _combi_1.sta)(Statements.Field), (0, _combi_1.sta)(Statements.CallSubscreen), (0, _combi_1.sub)(chain_1.Chain)));
-        const pbo = (0, _combi_1.star)((0, _combi_1.alt)((0, _combi_1.sta)(Statements.Module), (0, _combi_1.sta)(Statements.Field), (0, _combi_1.sta)(Statements.CallSubscreen)));
-        const pov = (0, _combi_1.star)((0, _combi_1.sta)(Statements.Field));
-        return (0, _combi_1.seq)((0, _combi_1.sta)(Statements.ProcessBeforeOutput), pbo, (0, _combi_1.sta)(Statements.ProcessAfterInput), pai, (0, _combi_1.opt)((0, _combi_1.seq)((0, _combi_1.sta)(Statements.ProcessOnValueRequest), pov)));
+        return (0, _combi_1.seq)((0, _combi_1.sub)(Structures.ProcessBeforeOutput), (0, _combi_1.sub)(Structures.ProcessAfterInput), (0, _combi_1.opt)((0, _combi_1.sub)(Structures.ProcessOnValueRequest)));
     }
 }
 exports.DynproLogic = DynproLogic;

package/build/src/abap/3_structures/structures/index.js

@@ -23,10 +23,10 @@ __exportStar(require("./case_type"), exports);
 __exportStar(require("./case"), exports);
 __exportStar(require("./catch_system_exceptions"), exports);
 __exportStar(require("./catch"), exports);
+__exportStar(require("./chain"), exports);
 __exportStar(require("./class_data"), exports);
 __exportStar(require("./class_definition"), exports);
 __exportStar(require("./class_global"), exports);
-__exportStar(require("./chain"), exports);
 __exportStar(require("./class_implementation"), exports);
 __exportStar(require("./cleanup"), exports);
 __exportStar(require("./constants"), exports);
@@ -51,6 +51,9 @@ __exportStar(require("./module"), exports);
 __exportStar(require("./normal"), exports);
 __exportStar(require("./on_change"), exports);
 __exportStar(require("./private_section"), exports);
+__exportStar(require("./process_after_input"), exports);
+__exportStar(require("./process_before_output"), exports);
+__exportStar(require("./process_on_value_request"), exports);
 __exportStar(require("./protected_section"), exports);
 __exportStar(require("./provide"), exports);
 __exportStar(require("./public_section"), exports);

package/build/src/abap/3_structures/structures/process_after_input.js

@@ -0,0 +1,14 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ProcessAfterInput = void 0;
+const Statements = require("../../2_statements/statements");
+const _combi_1 = require("./_combi");
+const chain_1 = require("./chain");
+class ProcessAfterInput {
+    getMatcher() {
+        const pai = (0, _combi_1.star)((0, _combi_1.alt)((0, _combi_1.sta)(Statements.Module), (0, _combi_1.sta)(Statements.Field), (0, _combi_1.sta)(Statements.CallSubscreen), (0, _combi_1.sub)(chain_1.Chain)));
+        return (0, _combi_1.seq)((0, _combi_1.sta)(Statements.ProcessAfterInput), pai);
+    }
+}
+exports.ProcessAfterInput = ProcessAfterInput;
+//# sourceMappingURL=process_after_input.js.map

package/build/src/abap/3_structures/structures/process_before_output.js

@@ -0,0 +1,13 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ProcessBeforeOutput = void 0;
+const Statements = require("../../2_statements/statements");
+const _combi_1 = require("./_combi");
+class ProcessBeforeOutput {
+    getMatcher() {
+        const pbo = (0, _combi_1.star)((0, _combi_1.alt)((0, _combi_1.sta)(Statements.Module), (0, _combi_1.sta)(Statements.Field), (0, _combi_1.sta)(Statements.CallSubscreen)));
+        return (0, _combi_1.seq)((0, _combi_1.sta)(Statements.ProcessBeforeOutput), pbo);
+    }
+}
+exports.ProcessBeforeOutput = ProcessBeforeOutput;
+//# sourceMappingURL=process_before_output.js.map

package/build/src/abap/3_structures/structures/process_on_value_request.js

@@ -0,0 +1,13 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ProcessOnValueRequest = void 0;
+const Statements = require("../../2_statements/statements");
+const _combi_1 = require("./_combi");
+class ProcessOnValueRequest {
+    getMatcher() {
+        const pov = (0, _combi_1.star)((0, _combi_1.sta)(Statements.Field));
+        return (0, _combi_1.seq)((0, _combi_1.sta)(Statements.ProcessOnValueRequest), pov);
+    }
+}
+exports.ProcessOnValueRequest = ProcessOnValueRequest;
+//# sourceMappingURL=process_on_value_request.js.map

package/build/src/cds/cds_determine_types.js

@@ -28,6 +28,13 @@ class CDSDetermineTypes {
                         continue;
                     }
                     if ((source === null || source === void 0 ? void 0 : source.name) === undefined) {
+                        if (prefixUpper.startsWith("_")) {
+                            components.push({
+                                name: f.name,
+                                type: new basic_1.VoidType("DDLS:association"),
+                            });
+                            continue;
+                        }
                         components.push({
                             name: f.name,
                             type: new basic_1.UnknownType("CDS parser error, unknown source, " + ddlsName),

package/build/src/lsp/help.js

@@ -7,15 +7,22 @@ const syntax_1 = require("../abap/5_syntax/syntax");
 const _abap_object_1 = require("../objects/_abap_object");
 const dump_scope_1 = require("./dump_scope");
 const virtual_position_1 = require("../virtual_position");
+const objects_1 = require("../objects");
 class Help {
     static find(reg, textDocument, position) {
-        const file = _lsp_utils_1.LSPUtils.getABAPFile(reg, textDocument.uri);
+        const abapFile = _lsp_utils_1.LSPUtils.getABAPFile(reg, textDocument.uri);
+        if (abapFile !== undefined) {
+            return this.dumpABAP(abapFile, reg, textDocument, position);
+        }
+        const file = reg.getFileByName(textDocument.uri);
         if (file === undefined) {
-            return "file not found";
+            return "File not found: " + textDocument.uri;
         }
-        else {
-            return this.dumpABAP(file, reg, textDocument, position);
+        const obj = reg.findObjectForFile(file);
+        if (obj instanceof objects_1.DataDefinition) {
+            return "Data definition, dump todo";
         }
+        return "Unhandled object type: " + obj.getType();
     }
     /////////////////////////////////////////////////
     static dumpABAP(file, reg, textDocument, position) {

package/build/src/registry.js

@@ -67,7 +67,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.89";
+        return "2.113.91";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/build/src/rules/local_testclass_consistency.js

@@ -30,7 +30,7 @@ class LocalTestclassConsistency extends _abap_rule_1.ABAPRule {
         this.conf = conf;
     }
     runParsed(file, obj) {
-        var _a;
+        var _a, _b, _c;
         const issues = [];
         if (this.reg.getConfig().getVersion() === version_1.Version.v700) {
             // 700 does not have testclass includes
@@ -56,6 +56,16 @@ class LocalTestclassConsistency extends _abap_rule_1.ABAPRule {
                 issues.push(issue);
             }
         }
+        if (file.getFilename() === ((_b = obj.getMainABAPFile()) === null || _b === void 0 ? void 0 : _b.getFilename())
+            && obj.getTestclassFile() === undefined
+            && ((_c = obj.getXML()) === null || _c === void 0 ? void 0 : _c.includes("<WITH_UNIT_TESTS>X</WITH_UNIT_TESTS>")) === true) {
+            const id = obj.getIdentifier();
+            if (id) {
+                const message = "Has <WITH_UNIT_TESTS> set in XML, but no testclasses";
+                const issue = issue_1.Issue.atIdentifier(id, message, this.getMetadata().key, this.conf.severity);
+                issues.push(issue);
+            }
+        }
         return issues;
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.113.89",
+    "version": "2.113.91",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -53,7 +53,7 @@
         "@microsoft/api-extractor": "^7.49.1",
         "@types/chai": "^4.3.20",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^22.10.5",
+        "@types/node": "^22.10.7",
         "chai": "^4.5.0",
         "eslint": "^9.18.0",
         "mocha": "^11.0.1",