@abaplint/core 2.97.4 → 2.97.6

package/build/abaplint.d.ts

@@ -1262,6 +1262,7 @@ export declare class CurrentScope {
     addListPrefix(identifiers: readonly TypedIdentifier[], prefix: string): void;
     addList(identifiers: readonly TypedIdentifier[]): void;
     addReference(usage: Token | undefined, referencing: Identifier | undefined, type: ReferenceType | undefined, filename: string, extra?: IReferenceExtras): void;
+    findFunctionModule(name: string | undefined): FunctionModuleDefinition | undefined;
     findObjectDefinition(name: string | undefined): IClassDefinition | IInterfaceDefinition | undefined;
     isBadiDef(name: string): boolean;
     existsObject(name: string | undefined): {

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

@@ -6,7 +6,7 @@ const expressions_1 = require("../expressions");
 const version_1 = require("../../../version");
 class CallFunction {
     getMatcher() {
-        const starting = (0, combi_1.seq)("STARTING NEW TASK", expressions_1.SimpleSource2);
+        const starting = (0, combi_1.verNot)(version_1.Version.Cloud, (0, combi_1.seq)("STARTING NEW TASK", expressions_1.SimpleSource2));
         const update = (0, combi_1.verNot)(version_1.Version.Cloud, (0, combi_1.str)("IN UPDATE TASK"));
         const unit = (0, combi_1.seq)("UNIT", expressions_1.Source);
         const background = (0, combi_1.verNot)(version_1.Version.Cloud, (0, combi_1.seq)("IN BACKGROUND", (0, combi_1.altPrio)("TASK", unit)));

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

@@ -138,6 +138,18 @@ class CurrentScope {
         (_a = this.current) === null || _a === void 0 ? void 0 : _a.getData().references.push({ position, resolved: referencing, referenceType: type, extra });
     }
     ///////////////////////////
+    findFunctionModule(name) {
+        if (name === undefined) {
+            return undefined;
+        }
+        for (const fugr of this.reg.getObjectsByType("FUGR")) {
+            const func = fugr.getModule(name);
+            if (func !== undefined) {
+                return func;
+            }
+        }
+        return undefined;
+    }
     findObjectDefinition(name) {
         if (name === undefined) {
             return undefined;

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

@@ -14,9 +14,10 @@ const tokens_1 = require("../../1_lexer/tokens");
 const attribute_name_1 = require("./attribute_name");
 const component_name_1 = require("./component_name");
 const types_1 = require("../../types");
+const version_1 = require("../../../version");
 class MethodSource {
     runSyntax(node, scope, filename) {
-        var _a;
+        var _a, _b, _c;
         const helper = new _object_oriented_1.ObjectOriented(scope);
         const children = node.getChildren().slice();
         const first = children.shift();
@@ -28,6 +29,15 @@ class MethodSource {
             context = (_a = scope.findVariable("me")) === null || _a === void 0 ? void 0 : _a.getType();
             children.unshift(first);
         }
+        if (scope.getVersion() === version_1.Version.Cloud
+            && first.get() instanceof Expressions.Dynamic
+            && first instanceof nodes_1.ExpressionNode
+            && ((_b = children[0]) === null || _b === void 0 ? void 0 : _b.concatTokens()) === "=>") {
+            const name = (_c = first.findDirectExpression(Expressions.Constant)) === null || _c === void 0 ? void 0 : _c.concatTokens().replace(/'/g, "");
+            if (name !== undefined && scope.findClassDefinition(name) === undefined) {
+                throw new Error(`Class "${name}" not found/released`);
+            }
+        }
         if (context instanceof basic_1.VoidType) {
             // todo, if there are more dynamic with variables, the references for the variables are not added?
             return context;
@@ -58,7 +68,7 @@ class MethodSource {
                         continue;
                     }
                 }
-                catch (_b) {
+                catch (_d) {
                     // ignore
                 }
                 // try looking for method name

package/build/src/abap/5_syntax/statements/call_function.js

@@ -6,6 +6,7 @@ const source_1 = require("../expressions/source");
 const target_1 = require("../expressions/target");
 const field_chain_1 = require("../expressions/field_chain");
 const _reference_1 = require("../_reference");
+const version_1 = require("../../../version");
 class CallFunction {
     runSyntax(node, scope, filename) {
         // todo, lots of work here, similar to receive.ts
@@ -14,6 +15,13 @@ class CallFunction {
         if (chain) {
             new field_chain_1.FieldChain().runSyntax(chain, scope, filename, _reference_1.ReferenceType.DataReadReference);
         }
+        else if (scope.getVersion() === version_1.Version.Cloud
+            && node.findDirectExpression(Expressions.Destination) === undefined) {
+            const functionName = name === null || name === void 0 ? void 0 : name.concatTokens().replace(/'/, "");
+            if (scope.findFunctionModule(functionName) === undefined) {
+                throw new Error(`Function module "${functionName}" not found/released`);
+            }
+        }
         // just recurse
         for (const s of node.findAllExpressions(Expressions.Source)) {
             new source_1.Source().runSyntax(s, scope, filename);

package/build/src/abap/5_syntax/statements/create_object.js

@@ -56,7 +56,10 @@ class CreateObject {
                 }
                 else if (found instanceof basic_1.ObjectReferenceType) {
                     const id = found.getIdentifier();
-                    if (id instanceof types_1.ClassDefinition && cdef === undefined) {
+                    if (id instanceof types_1.InterfaceDefinition && cdef === undefined) {
+                        throw new Error("Interface reference, cannot be instantiated");
+                    }
+                    else if (id instanceof types_1.ClassDefinition && cdef === undefined) {
                         cdef = id;
                     }
                     if (type === undefined && id instanceof types_1.ClassDefinition && id.isAbstract() === true) {

package/build/src/lsp/language_server.js

@@ -49,10 +49,15 @@ class LanguageServer {
         }
         const text = new pretty_printer_1.PrettyPrinter(file, this.reg.getConfig()).run();
         const rows = file.getRawRows();
-        return [{
-                range: LServer.Range.create(0, 0, rows.length, rows[rows.length - 1].length + 1),
-                newText: text,
-            }];
+        if (text === file.getRaw()) {
+            return [];
+        }
+        else {
+            return [{
+                    range: LServer.Range.create(0, 0, rows.length, rows[rows.length - 1].length + 1),
+                    newText: text,
+                }];
+        }
     }
     // https://microsoft.github.io/language-server-protocol/specifications/specification-3-14/#textDocument_publishDiagnostics
     diagnostics(textDocument) {

package/build/src/pretty_printer/pretty_printer.js

@@ -4,7 +4,6 @@ exports.PrettyPrinter = void 0;
 const _statement_1 = require("../abap/2_statements/statements/_statement");
 const fix_keyword_case_1 = require("./fix_keyword_case");
 const indent_1 = require("./indent");
-const remove_sequential_blanks_1 = require("./remove_sequential_blanks");
 const position_1 = require("../position");
 const indentation_1 = require("../rules/indentation");
 class PrettyPrinter {
@@ -34,8 +33,6 @@ class PrettyPrinter {
         }
         const indentation = new indent_1.Indent(this.options);
         this.result = indentation.execute(this.file, this.result);
-        const removeBlanks = new remove_sequential_blanks_1.RemoveSequentialBlanks(this.config);
-        this.result = removeBlanks.execute(this.file, this.result);
         return this.result;
     }
 }

package/build/src/registry.js

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

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.97.4",
+    "version": "2.97.6",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -52,15 +52,15 @@
         "@types/mocha": "^10.0.1",
         "@types/node": "^18.15.11",
         "chai": "^4.3.7",
-        "eslint": "^8.37.0",
+        "eslint": "^8.38.0",
         "mocha": "^10.2.0",
         "c8": "^7.13.0",
         "source-map-support": "^0.5.21",
         "ts-json-schema-generator": "^1.2.0",
-        "typescript": "^5.0.3"
+        "typescript": "^5.0.4"
     },
     "dependencies": {
-        "fast-xml-parser": "^4.1.3",
+        "fast-xml-parser": "^4.1.4",
         "json5": "^2.2.3",
         "vscode-languageserver-types": "^3.17.3"
     }

package/build/src/pretty_printer/remove_sequential_blanks.js

@@ -1,50 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.RemoveSequentialBlanks = void 0;
-const rules_1 = require("../rules");
-class RemoveSequentialBlanks {
-    constructor(config) {
-        this.config = config;
-    }
-    execute(file, modified) {
-        const sequentialBlankConfig = this.getSequentialBlankConfig();
-        if (sequentialBlankConfig) {
-            return this.withoutSequentialBlanks(file, modified, sequentialBlankConfig.lines);
-        }
-        return modified;
-    }
-    withoutSequentialBlanks(file, modified, threshold) {
-        const rows = file.getRawRows();
-        let blanks = 0;
-        const rowsToRemove = [];
-        const newBlankCount = (current, row) => {
-            return rules_1.SequentialBlank.isBlankOrWhitespace(row) ? current + 1 : 0;
-        };
-        for (let i = 0; i < rows.length; i++) {
-            blanks = newBlankCount(blanks, rows[i]);
-            if (blanks === threshold) {
-                // count additional blanks
-                for (let j = i; j < rows.length; j++) {
-                    if (rules_1.SequentialBlank.isBlankOrWhitespace(rows[j])) {
-                        rowsToRemove.push(j);
-                    }
-                    else {
-                        break;
-                    }
-                }
-            }
-        }
-        return this.removeRows(modified.split("\n"), rowsToRemove);
-    }
-    removeRows(lines, rowsToRemove) {
-        const withoutRemoved = lines.filter((_, idx) => {
-            return rowsToRemove.indexOf(idx) === -1;
-        });
-        return withoutRemoved.join("\n").trim();
-    }
-    getSequentialBlankConfig() {
-        return this.config.readByRule(new rules_1.SequentialBlank().getMetadata().key);
-    }
-}
-exports.RemoveSequentialBlanks = RemoveSequentialBlanks;
-//# sourceMappingURL=remove_sequential_blanks.js.map