@abaplint/core 2.91.36 → 2.91.39

package/build/abaplint.d.ts

@@ -150,6 +150,15 @@ declare class AnyType extends AbstractType {
     toCDS(): string;
 }
 
+declare class APIReleaseState extends AbstractObject {
+    getType(): string;
+    getAllowedNaming(): {
+        maxLength: number;
+        allowNamespace: boolean;
+    };
+    getDescription(): string | undefined;
+}
+
 declare class Append implements IStatement {
     getMatcher(): IStatementRunnable;
 }
@@ -615,6 +624,10 @@ declare class CDSDefineAbstract extends Expression {
     getRunnable(): IStatementRunnable;
 }
 
+declare class CDSDefineProjection extends Expression {
+    getRunnable(): IStatementRunnable;
+}
+
 declare class CDSDefineView extends Expression {
     getRunnable(): IStatementRunnable;
 }
@@ -2014,6 +2027,7 @@ declare namespace ExpressionsCDS {
         CDSComposition,
         CDSCondition,
         CDSDefineAbstract,
+        CDSDefineProjection,
         CDSDefineView,
         CDSElement,
         CDSFunction,
@@ -3208,6 +3222,7 @@ export declare interface IRegistry {
     parseAsync(input?: IRunInput): Promise<IRegistry>;
     addDependencies(files: readonly IFile[]): IRegistry;
     addDependency(file: IFile): IRegistry;
+    removeDependency(obj: IObject): void;
     isDependency(obj: IObject): boolean;
     isFileDependency(filename: string): boolean;
     findIssues(input?: IRunInput): readonly Issue[];
@@ -3953,6 +3968,7 @@ declare class ObjectReferenceType extends AbstractType {
 declare namespace Objects {
     export {
         ActivationVariant,
+        APIReleaseState,
         AssignmentServiceToAuthorizationGroup,
         AuthorizationCheckField,
         AuthorizationGroup,
@@ -4450,7 +4466,6 @@ declare class RefreshControl implements IStatement {
 export declare class Registry implements IRegistry {
     private readonly objects;
     private readonly objectsByType;
-    /** object containing filenames of dependencies */
     private readonly dependencies;
     private readonly references;
     private conf;
@@ -4471,9 +4486,11 @@ export declare class Registry implements IRegistry {
     addFile(file: IFile): IRegistry;
     updateFile(file: IFile): IRegistry;
     removeFile(file: IFile): IRegistry;
+    private _addFiles;
     addFiles(files: readonly IFile[]): IRegistry;
     addDependencies(files: readonly IFile[]): IRegistry;
     addDependency(file: IFile): IRegistry;
+    removeDependency(obj: IObject): void;
     isDependency(obj: IObject): boolean;
     isFileDependency(filename: string): boolean;
     findObjectForFile(file: IFile): IObject | undefined;

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

@@ -7,7 +7,8 @@ const tokens_1 = require("../../1_lexer/tokens");
 const _1 = require(".");
 class SQLTarget extends combi_1.Expression {
     getRunnable() {
-        const at = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.altPrio)((0, combi_1.tok)(tokens_1.WAt), (0, combi_1.tok)(tokens_1.At)), _1.Target));
+        const n = (0, combi_1.ver)(version_1.Version.v754, "NEW");
+        const at = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.opt)(n), (0, combi_1.altPrio)((0, combi_1.tok)(tokens_1.WAt), (0, combi_1.tok)(tokens_1.At)), _1.Target));
         return (0, combi_1.altPrio)(at, _1.Target);
     }
 }

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

@@ -26,7 +26,9 @@ class MethodCallParam {
                 }
             }
         }
-        else if (child instanceof nodes_1.ExpressionNode && child.get() instanceof Expressions.Source) {
+        else if (child instanceof nodes_1.ExpressionNode
+            && (child.get() instanceof Expressions.Source
+                || child.get() instanceof Expressions.ConstantString)) {
             if (!(method instanceof basic_1.VoidType) && method.getParameters().getImporting().length === 0) {
                 throw new Error("Method \"" + method.getName() + "\" has no importing parameters");
             }
@@ -46,7 +48,10 @@ class MethodCallParam {
             else {
                 targetType = method;
             }
-            const sourceType = new source_1.Source().runSyntax(child, scope, filename, targetType);
+            let sourceType = new basic_1.StringType();
+            if (child.get() instanceof Expressions.Source) {
+                sourceType = new source_1.Source().runSyntax(child, scope, filename, targetType);
+            }
             if (sourceType === undefined) {
                 throw new Error("No source type determined, method source");
             }

package/build/src/cds/cds_parser.js

@@ -18,6 +18,9 @@ class CDSParser {
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
             res = combi_1.Combi.run(new Expressions.CDSDefineAbstract(), tokens, version_1.defaultVersion);
         }
+        if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
+            res = combi_1.Combi.run(new Expressions.CDSDefineProjection(), tokens, version_1.defaultVersion);
+        }
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
             return undefined;
         }

package/build/src/cds/expressions/cds_aggregate.js

@@ -5,7 +5,8 @@ const _1 = require(".");
 const combi_1 = require("../../abap/2_statements/combi");
 class CDSAggregate extends combi_1.Expression {
     getRunnable() {
-        const value = (0, combi_1.alt)(_1.CDSName, _1.CDSCast, _1.CDSCase);
+        const name = (0, combi_1.seq)(_1.CDSName, (0, combi_1.opt)((0, combi_1.seq)(".", _1.CDSName)));
+        const value = (0, combi_1.alt)(name, _1.CDSCast, _1.CDSCase);
         return (0, combi_1.seq)((0, combi_1.alt)("MAX", "MIN", "SUM", "AVG", "COUNT"), "(", (0, combi_1.opt)("DISTINCT"), value, ")");
     }
 }

package/build/src/cds/expressions/cds_define_abstract.js

@@ -6,7 +6,7 @@ const combi_1 = require("../../abap/2_statements/combi");
 const cds_name_1 = require("./cds_name");
 class CDSDefineAbstract extends combi_1.Expression {
     getRunnable() {
-        const field = (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.str)("KEY"), cds_name_1.CDSName, ":", cds_name_1.CDSName, ";");
+        const field = (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.opt)((0, combi_1.str)("KEY")), cds_name_1.CDSName, ":", _1.CDSType, ";");
         return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.str)("DEFINE ABSTRACT ENTITY"), cds_name_1.CDSName, (0, combi_1.str)("{"), (0, combi_1.plus)(field), (0, combi_1.str)("}"), (0, combi_1.opt)(";"));
     }
 }

package/build/src/cds/expressions/cds_define_projection.js

@@ -0,0 +1,13 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.CDSDefineProjection = void 0;
+const _1 = require(".");
+const __1 = require("../..");
+const combi_1 = require("../../abap/2_statements/combi");
+class CDSDefineProjection extends combi_1.Expression {
+    getRunnable() {
+        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), "DEFINE", (0, combi_1.opt)("ROOT"), "VIEW", (0, combi_1.ver)(__1.Version.v755, (0, combi_1.opt)("ENTITY")), _1.CDSName, "PROVIDER CONTRACT", _1.CDSName, "AS PROJECTION ON", _1.CDSName, (0, combi_1.str)("{"), (0, combi_1.plus)(_1.CDSElement), (0, combi_1.star)((0, combi_1.seq)(",", _1.CDSElement)), (0, combi_1.str)("}"), (0, combi_1.opt)(";"));
+    }
+}
+exports.CDSDefineProjection = CDSDefineProjection;
+//# sourceMappingURL=cds_define_projection.js.map

package/build/src/cds/expressions/index.js

@@ -29,6 +29,7 @@ __exportStar(require("./cds_cast"), exports);
 __exportStar(require("./cds_composition"), exports);
 __exportStar(require("./cds_condition"), exports);
 __exportStar(require("./cds_define_abstract"), exports);
+__exportStar(require("./cds_define_projection"), exports);
 __exportStar(require("./cds_define_view"), exports);
 __exportStar(require("./cds_element"), exports);
 __exportStar(require("./cds_function"), exports);

package/build/src/objects/api_release_state.js

@@ -0,0 +1,21 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.APIReleaseState = void 0;
+const _abstract_object_1 = require("./_abstract_object");
+class APIReleaseState extends _abstract_object_1.AbstractObject {
+    getType() {
+        return "APIS";
+    }
+    getAllowedNaming() {
+        return {
+            maxLength: 200,
+            allowNamespace: true,
+        };
+    }
+    getDescription() {
+        // todo
+        return undefined;
+    }
+}
+exports.APIReleaseState = APIReleaseState;
+//# sourceMappingURL=api_release_state.js.map

package/build/src/objects/index.js

@@ -15,6 +15,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 __exportStar(require("./activation_variant"), exports);
+__exportStar(require("./api_release_state"), exports);
 __exportStar(require("./assignment_service_to_authorization_group"), exports);
 __exportStar(require("./authorization_check_field"), exports);
 __exportStar(require("./authorization_group"), exports);

package/build/src/registry.js

@@ -60,7 +60,6 @@ class Registry {
     constructor(conf) {
         this.objects = {};
         this.objectsByType = {};
-        /** object containing filenames of dependencies */
         this.dependencies = {};
         this.issues = [];
         this.conf = conf ? conf : config_1.Config.getDefault();
@@ -68,7 +67,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.91.36";
+        return "2.91.39";
     }
     getDDICReferences() {
         return this.references;
@@ -143,6 +142,7 @@ class Registry {
         return this;
     }
     inErrorNamespace(name) {
+        // todo: performance? cache regexp?
         const reg = new RegExp(this.getConfig().getSyntaxSetttings().errorNamespace, "i");
         return reg.test(name);
     }
@@ -163,7 +163,7 @@ class Registry {
         }
         return this;
     }
-    addFiles(files) {
+    _addFiles(files, dependency) {
         var _a;
         const globalExclude = ((_a = this.conf.getGlobal().exclude) !== null && _a !== void 0 ? _a : [])
             .map(pattern => new RegExp(pattern, "i"));
@@ -173,28 +173,60 @@ class Registry {
             if (isNotAbapgitFile || excludeHelper_1.ExcludeHelper.isExcluded(filename, globalExclude)) {
                 continue;
             }
-            const found = this.findOrCreate(f.getObjectName(), f.getObjectType());
+            let found = this.findOrCreate(f.getObjectName(), f.getObjectType());
+            if (dependency === false && found && this.isDependency(found)) {
+                this.removeDependency(found);
+                found = this.findOrCreate(f.getObjectName(), f.getObjectType());
+            }
             found.addFile(f);
         }
         return this;
     }
+    addFiles(files) {
+        this._addFiles(files, false);
+        return this;
+    }
     addDependencies(files) {
         for (const f of files) {
-            this.dependencies[f.getFilename().toUpperCase()] = true;
+            this.addDependency(f);
         }
-        return this.addFiles(files);
+        return this;
     }
     addDependency(file) {
-        this.dependencies[file.getFilename().toUpperCase()] = true;
-        this.addFile(file);
+        var _a;
+        const type = (_a = file.getObjectType()) === null || _a === void 0 ? void 0 : _a.toUpperCase();
+        if (type === undefined) {
+            return this;
+        }
+        const name = file.getObjectName().toUpperCase();
+        if (this.dependencies[type] === undefined) {
+            this.dependencies[type] = {};
+        }
+        this.dependencies[type][name] = true;
+        this._addFiles([file], true);
         return this;
     }
+    removeDependency(obj) {
+        var _a;
+        (_a = this.dependencies[obj.getType()]) === null || _a === void 0 ? true : delete _a[obj.getName()];
+        this.removeObject(obj);
+    }
     isDependency(obj) {
-        const filename = obj.getFiles()[0].getFilename().toUpperCase();
-        return this.dependencies[filename] === true;
+        var _a;
+        return ((_a = this.dependencies[obj.getType()]) === null || _a === void 0 ? void 0 : _a[obj.getName()]) === true;
     }
     isFileDependency(filename) {
-        return this.dependencies[filename.toUpperCase()] === true;
+        var _a, _b;
+        const f = this.getFileByName(filename);
+        if (f === undefined) {
+            return false;
+        }
+        const type = (_a = f.getObjectType()) === null || _a === void 0 ? void 0 : _a.toUpperCase();
+        if (type === undefined) {
+            return false;
+        }
+        const name = f.getObjectName().toUpperCase();
+        return ((_b = this.dependencies[type]) === null || _b === void 0 ? void 0 : _b[name]) === true;
     }
     // assumption: the file is already in the registry
     findObjectForFile(file) {

package/build/src/rules/rfc_error_handling.js

@@ -36,7 +36,7 @@ CALL FUNCTION 'ZRFC'
         };
     }
     getMessage() {
-        return "RFC error handling: At least one unhandled exception from SYSTEM_FAILURE, COMMUNICATION_FAILURE, RESOURCE_FAILURE.";
+        return "RFC error handling: At least one unhandled exception from SYSTEM_FAILURE, COMMUNICATION_FAILURE, RESOURCE_FAILURE";
     }
     getConfig() {
         return this.conf;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.91.36",
+    "version": "2.91.39",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -45,10 +45,10 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@microsoft/api-extractor": "^7.29.3",
+        "@microsoft/api-extractor": "^7.29.5",
         "@types/chai": "^4.3.3",
         "@types/mocha": "^9.1.1",
-        "@types/node": "^18.7.11",
+        "@types/node": "^18.7.13",
         "chai": "^4.3.6",
         "eslint": "^8.22.0",
         "mocha": "^10.0.0",