@abaplint/core 2.119.17 → 2.119.18

package/build/abaplint.d.ts

@@ -3487,6 +3487,8 @@ declare interface IGlobalConfig {
     useApackDependencies?: boolean;
     /** Do not report any issues for includes without main programs */
     skipIncludesWithoutMain?: boolean;
+    /** Throw an error if multiple files with the same filename are added to the registry, ignores package.devc.xml */
+    errorOnDuplicateFilenames?: boolean;
     /** list of files to exclude, these files are not added when running syntax check or any other rules, case insensitive regex
      * @uniqueItems true
      */
@@ -5769,6 +5771,8 @@ export declare class Registry implements IRegistry {
     updateFile(file: IFile): IRegistry;
     removeFile(file: IFile): IRegistry;
     private _addFiles;
+    private checkDuplicateFilename;
+    private static filenameBasename;
     addFiles(files: readonly IFile[]): IRegistry;
     addDependencies(files: readonly IFile[]): IRegistry;
     addDependency(file: IFile): IRegistry;

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

@@ -31,6 +31,12 @@ class CallFunction {
         for (const s of node.findDirectExpressions(Expressions.Source)) {
             source_1.Source.runSyntax(s, input);
         }
+        for (const d of node.findDirectExpressions(Expressions.Destination)) {
+            const source = d.findFirstExpression(Expressions.Source);
+            if (source) {
+                source_1.Source.runSyntax(source, input);
+            }
+        }
         const fp = node.findDirectExpression(Expressions.FunctionParameters);
         if (fp) {
             function_parameters_1.FunctionParameters.runSyntax(fp, input);

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

@@ -3,11 +3,15 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.FetchNextCursor = void 0;
 const Expressions = require("../../2_statements/expressions");
 const target_1 = require("../expressions/target");
+const sql_source_1 = require("../expressions/sql_source");
 class FetchNextCursor {
     runSyntax(node, input) {
         for (const t of node.findAllExpressions(Expressions.Target)) {
             target_1.Target.runSyntax(t, input);
         }
+        for (const s of node.findAllExpressions(Expressions.SQLSourceSimple)) {
+            sql_source_1.SQLSource.runSyntax(s, input);
+        }
     }
 }
 exports.FetchNextCursor = FetchNextCursor;

package/build/src/config.js

@@ -33,6 +33,7 @@ class Config {
                 skipGeneratedProxyInterfaces: false,
                 useApackDependencies: false,
                 skipIncludesWithoutMain: false,
+                errorOnDuplicateFilenames: false,
             },
             dependencies: [{
                     url: "https://github.com/abaplint/deps",
@@ -96,6 +97,9 @@ class Config {
         if (this.config.global.skipIncludesWithoutMain === undefined) {
             this.config.global.skipIncludesWithoutMain = false;
         }
+        if (this.config.global.errorOnDuplicateFilenames === undefined) {
+            this.config.global.errorOnDuplicateFilenames = false;
+        }
         this.checkVersion();
     }
     get() {

package/build/src/registry.js

@@ -74,7 +74,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.119.17";
+        return "2.119.18";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -194,10 +194,29 @@ class Registry {
                 this.removeDependency(found);
                 found = this.findOrCreate(f.getObjectName(), f.getObjectType());
             }
+            if (this.conf.getGlobal().errorOnDuplicateFilenames === true) {
+                this.checkDuplicateFilename(filename, found.getFiles());
+            }
             found.addFile(f);
         }
         return this;
     }
+    checkDuplicateFilename(filename, files) {
+        const basename = Registry.filenameBasename(filename);
+        if (basename === "PACKAGE.DEVC.XML") {
+            return;
+        }
+        for (const existingFile of files) {
+            if (Registry.filenameBasename(existingFile.getFilename()) === basename) {
+                throw new Error("Duplicate filename: " + filename + " already exists as " + existingFile.getFilename());
+            }
+        }
+    }
+    static filenameBasename(filename) {
+        const normalized = filename.replace(/\\/g, "/");
+        const last = normalized.split("/").pop();
+        return (last !== null && last !== void 0 ? last : normalized).toUpperCase();
+    }
     addFiles(files) {
         this._addFiles(files, false);
         return this;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.119.17",
+    "version": "2.119.18",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",