@abaplint/core 2.113.109 → 2.113.111

package/build/abaplint.d.ts

@@ -192,6 +192,15 @@ declare class Append implements IStatement {
     getMatcher(): IStatementRunnable;
 }
 
+declare class ApplicationDescriptorsFiori extends AbstractObject {
+    getType(): string;
+    getAllowedNaming(): {
+        maxLength: number;
+        allowNamespace: boolean;
+    };
+    getDescription(): string | undefined;
+}
+
 declare class ApplicationJobCatalogEntry extends AbstractObject {
     getType(): string;
     getAllowedNaming(): {
@@ -545,6 +554,15 @@ declare class BreakId implements IStatement {
     getMatcher(): IStatementRunnable;
 }
 
+declare class BRFPlusSystemApplication extends AbstractObject {
+    getType(): string;
+    getAllowedNaming(): {
+        maxLength: number;
+        allowNamespace: boolean;
+    };
+    getDescription(): string | undefined;
+}
+
 declare class BSPApplication extends AbstractObject {
     getType(): string;
     getAllowedNaming(): {
@@ -2594,48 +2612,6 @@ declare class FloatType extends AbstractType {
     toCDS(): string;
 }
 
-export declare enum FLOW_EDGE_TYPE {
-    true = "true",
-    false = "false",
-    undefined = "undefined"
-}
-
-export declare class FlowGraph {
-    private edges;
-    private readonly startNode;
-    private readonly endNode;
-    private label;
-    constructor(counter: number);
-    getStart(): string;
-    getLabel(): string;
-    getEnd(): string;
-    addEdge(from: string, to: string, type: FLOW_EDGE_TYPE): void;
-    removeEdge(from: string, to: string): void;
-    listEdges(): {
-        from: string;
-        to: string;
-        type: FLOW_EDGE_TYPE;
-    }[];
-    listNodes(): string[];
-    private hasEdges;
-    /** return value: end node of to graph */
-    addGraph(from: string, to: FlowGraph, type: FLOW_EDGE_TYPE): string;
-    toJSON(): string;
-    toTextEdges(): string;
-    setLabel(label: string): void;
-    toDigraph(): string;
-    listSources(node: string): {
-        name: string;
-        type: FLOW_EDGE_TYPE;
-    }[];
-    listTargets(node: string): {
-        name: string;
-        type: FLOW_EDGE_TYPE;
-    }[];
-    /** removes all nodes containing "#" that have one in-going and one out-going edge */
-    reduce(): this;
-}
-
 declare class For extends Expression {
     getRunnable(): IStatementRunnable;
 }
@@ -3803,14 +3779,14 @@ declare interface IRange {
     end: Position;
 }
 
-export declare interface IReference {
+declare interface IReference {
     position: Identifier;
     resolved: Identifier | undefined;
     referenceType: ReferenceType;
     extra?: IReferenceExtras;
 }
 
-export declare interface IReferenceExtras {
+declare interface IReferenceExtras {
     ooName?: string;
     ooType?: "CLAS" | "INTF" | "Void";
 }
@@ -4875,11 +4851,13 @@ declare namespace Objects {
         ATCCheckObject,
         ATCCheckVariant,
         AuthorizationCheckField,
+        ApplicationDescriptorsFiori,
         AuthorizationGroup,
         AuthorizationObjectClass,
         AuthorizationObjectExtension,
         AuthorizationObject,
         BehaviorDefinition,
+        BRFPlusSystemApplication,
         BSPApplication,
         BusinessAddInImplementation,
         BusinessCatalogAppAssignment,
@@ -5439,7 +5417,7 @@ declare class ReduceNext extends Expression {
     getRunnable(): IStatementRunnable;
 }
 
-export declare enum ReferenceType {
+declare enum ReferenceType {
     /** for classes and interface references */
     ObjectOrientedReference = "Object",
     ObjectOrientedVoidReference = "Object (Void)",
@@ -6176,17 +6154,6 @@ declare class StartOfSelection implements IStatement {
     getMatcher(): IStatementRunnable;
 }
 
-export declare class StatementFlow {
-    private counter;
-    build(stru: StructureNode, obj: IObject): FlowGraph[];
-    private runEvent;
-    private run;
-    private findBody;
-    static buildName(statement: StatementNode): string;
-    private traverseBody;
-    private traverseStructure;
-}
-
 declare class StatementNode extends AbstractNode<ExpressionNode | TokenNode> {
     private readonly statement;
     private readonly colon;

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

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

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

@@ -4,7 +4,7 @@ exports.NamespaceSimpleName = void 0;
 const combi_1 = require("../combi");
 class NamespaceSimpleName extends combi_1.Expression {
     getRunnable() {
-        return (0, combi_1.regex)(/^((\w*\/\w+\/)|(\w*\/\w+\/)?[\w\*$%#]+)$/);
+        return (0, combi_1.regex)(/^!?((\w*\/\w+\/)|(\w*\/\w+\/)*[\w\*$%#]+)$/);
     }
 }
 exports.NamespaceSimpleName = NamespaceSimpleName;

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

@@ -164,7 +164,7 @@ class MethodParameters {
             if (parameter === undefined) {
                 const message = "Method importing parameter \"" + item.name + "\" does not exist";
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
-                return;
+                continue;
             }
             else if (new _type_utils_1.TypeUtils(input.scope).isAssignableStrict(item.sourceType, parameter.getType(), calculated) === false) {
                 const message = "Method parameter type not compatible, " + item.name;

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

@@ -22,6 +22,8 @@ class SelectOption {
         }
         for (const d of node.findDirectExpressions(Expressions.Dynamic)) {
             new dynamic_1.Dynamic().runSyntax(d, input);
+            input.scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(nameToken, input.filename, new basic_1.VoidType("DYNAMIC_SELECT_OPTION")));
+            return;
         }
         const nameExpression = node.findFirstExpression(Expressions.FieldChain);
         let found = new basic_types_1.BasicTypes(input).resolveLikeName(nameExpression);

package/build/src/abap/types/class_definition.js

@@ -110,7 +110,9 @@ class ClassDefinition extends _identifier_1.Identifier {
                 if (m.getVisibility() === visibility_1.Visibility.Private) {
                     continue;
                 }
-                else if (name === "CONSTRUCTOR" || name === "CLASS_CONSTRUCTOR") {
+                else if (name === "CONSTRUCTOR"
+                    || name === "DESTRUCTOR"
+                    || name === "CLASS_CONSTRUCTOR") {
                     continue;
                 }
                 names.add(name);
@@ -121,7 +123,8 @@ class ClassDefinition extends _identifier_1.Identifier {
             sup = cdef === null || cdef === void 0 ? void 0 : cdef.getSuperClass();
         }
         for (const m of this.getMethodDefinitions().getAll()) {
-            if (names.has(m.getName().toUpperCase()) && m.isRedefinition() === false) {
+            if (names.has(m.getName().toUpperCase())
+                && m.isRedefinition() === false) {
                 throw new Error(`${m.getName().toUpperCase()} already declared in superclass`);
             }
         }

package/build/src/index.js

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.ABAPFile = exports.RulesRunner = exports.SpaghettiScope = exports.SyntaxLogic = exports.ABAPObject = exports.Tokens = exports.ExpressionsCDS = exports.CDSParser = exports.LanguageServerTypes = exports.DDLParser = exports.FLOW_EDGE_TYPE = exports.FlowGraph = exports.StatementFlow = exports.NativeSQL = exports.MacroContent = exports.MacroCall = exports.applyEditList = exports.applyEditSingle = exports.SpaghettiScopeNode = exports.AbstractFile = exports.Token = exports.ScopeType = exports.BasicTypes = exports.TypedIdentifier = exports.AbstractType = exports.VirtualPosition = exports.Comment = exports.Unknown = exports.Empty = exports.Identifier = exports.Nodes = exports.Types = exports.Expressions = exports.Statements = exports.Structures = exports.SkipLogic = exports.Objects = exports.ArtifactsRules = exports.ArtifactsObjects = exports.ArtifactsABAP = exports.BuiltIn = exports.MethodLengthStats = exports.LanguageServer = exports.Registry = exports.CyclomaticComplexityStats = exports.ReferenceType = exports.Version = exports.Config = exports.Issue = exports.MemoryFile = void 0;
-exports.LSPEdit = exports.RuleTag = exports.Severity = exports.Visibility = exports.Info = exports.Diagnostics = exports.Rename = exports.PrettyPrinter = exports.Position = exports.CurrentScope = void 0;
+exports.Rename = exports.PrettyPrinter = exports.Position = exports.CurrentScope = exports.ABAPFile = exports.RulesRunner = exports.SpaghettiScope = exports.SyntaxLogic = exports.ABAPObject = exports.Tokens = exports.ExpressionsCDS = exports.CDSParser = exports.LanguageServerTypes = exports.DDLParser = exports.NativeSQL = exports.MacroContent = exports.MacroCall = exports.applyEditList = exports.applyEditSingle = exports.SpaghettiScopeNode = exports.AbstractFile = exports.Token = exports.ScopeType = exports.BasicTypes = exports.TypedIdentifier = exports.AbstractType = exports.VirtualPosition = exports.Comment = exports.Unknown = exports.Empty = exports.Identifier = exports.Nodes = exports.Types = exports.Expressions = exports.Statements = exports.Structures = exports.SkipLogic = exports.Objects = exports.ArtifactsRules = exports.ArtifactsObjects = exports.ArtifactsABAP = exports.BuiltIn = exports.MethodLengthStats = exports.LanguageServer = exports.Registry = exports.CyclomaticComplexityStats = exports.Version = exports.Config = exports.Issue = exports.MemoryFile = void 0;
+exports.LSPEdit = exports.RuleTag = exports.Severity = exports.Visibility = exports.Info = exports.Diagnostics = void 0;
 const issue_1 = require("./issue");
 Object.defineProperty(exports, "Issue", { enumerable: true, get: function () { return issue_1.Issue; } });
 const config_1 = require("./config");
@@ -75,8 +75,6 @@ Object.defineProperty(exports, "NativeSQL", { enumerable: true, get: function ()
 const edit_helper_1 = require("./edit_helper");
 Object.defineProperty(exports, "applyEditSingle", { enumerable: true, get: function () { return edit_helper_1.applyEditSingle; } });
 Object.defineProperty(exports, "applyEditList", { enumerable: true, get: function () { return edit_helper_1.applyEditList; } });
-const _reference_1 = require("./abap/5_syntax/_reference");
-Object.defineProperty(exports, "ReferenceType", { enumerable: true, get: function () { return _reference_1.ReferenceType; } });
 const _builtin_1 = require("./abap/5_syntax/_builtin");
 Object.defineProperty(exports, "BuiltIn", { enumerable: true, get: function () { return _builtin_1.BuiltIn; } });
 const abap_file_1 = require("./abap/abap_file");
@@ -109,9 +107,4 @@ const diagnostics_1 = require("./lsp/diagnostics");
 Object.defineProperty(exports, "Diagnostics", { enumerable: true, get: function () { return diagnostics_1.Diagnostics; } });
 const _edit_1 = require("./lsp/_edit");
 Object.defineProperty(exports, "LSPEdit", { enumerable: true, get: function () { return _edit_1.LSPEdit; } });
-const statement_flow_1 = require("./abap/flow/statement_flow");
-Object.defineProperty(exports, "StatementFlow", { enumerable: true, get: function () { return statement_flow_1.StatementFlow; } });
-const flow_graph_1 = require("./abap/flow/flow_graph");
-Object.defineProperty(exports, "FlowGraph", { enumerable: true, get: function () { return flow_graph_1.FlowGraph; } });
-Object.defineProperty(exports, "FLOW_EDGE_TYPE", { enumerable: true, get: function () { return flow_graph_1.FLOW_EDGE_TYPE; } });
 //# sourceMappingURL=index.js.map

package/build/src/objects/application_descriptors_fiori.js

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

package/build/src/objects/brf_plus_system_application.js

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

package/build/src/objects/index.js

@@ -26,11 +26,13 @@ __exportStar(require("./atc_check_category"), exports);
 __exportStar(require("./atc_check_object"), exports);
 __exportStar(require("./atc_check_variant"), exports);
 __exportStar(require("./authorization_check_field"), exports);
+__exportStar(require("./application_descriptors_fiori"), exports);
 __exportStar(require("./authorization_group"), exports);
 __exportStar(require("./authorization_object_class"), exports);
 __exportStar(require("./authorization_object_extension"), exports);
 __exportStar(require("./authorization_object"), exports);
 __exportStar(require("./behavior_definition"), exports);
+__exportStar(require("./brf_plus_system_application"), exports);
 __exportStar(require("./bsp_application"), exports);
 __exportStar(require("./business_add_in_implementation"), exports);
 __exportStar(require("./business_catalog_app_assignment"), exports);

package/build/src/objects/rename/renamer_helper.js

@@ -2,11 +2,11 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.RenamerHelper = void 0;
 const vscode_languageserver_types_1 = require("vscode-languageserver-types");
-const __1 = require("../..");
 const syntax_1 = require("../../abap/5_syntax/syntax");
 const _scope_type_1 = require("../../abap/5_syntax/_scope_type");
 const virtual_position_1 = require("../../virtual_position");
 const _abap_object_1 = require("../_abap_object");
+const _reference_1 = require("../../abap/5_syntax/_reference");
 class RenamerHelper {
     constructor(reg) {
         this.reg = reg;
@@ -147,11 +147,11 @@ class RenamerHelper {
         let ret = [];
         if (node.getIdentifier().stype !== _scope_type_1.ScopeType.BuiltIn) {
             for (const r of node.getData().references) {
-                if (r.referenceType === __1.ReferenceType.ConstructorReference) {
+                if (r.referenceType === _reference_1.ReferenceType.ConstructorReference) {
                     continue;
                 }
                 if (((_a = r.resolved) === null || _a === void 0 ? void 0 : _a.equals(identifier))
-                    && r.referenceType !== __1.ReferenceType.InferredType
+                    && r.referenceType !== _reference_1.ReferenceType.InferredType
                     && !(r.position.getStart() instanceof virtual_position_1.VirtualPosition)) {
                     ret.push(r.position);
                 }

package/build/src/registry.js

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

package/build/src/rules/cloud_types.js

@@ -38,8 +38,10 @@ class CloudTypes {
     static isCloud(obj) {
         return obj instanceof Objects.ApplicationJobCatalogEntry
             || obj instanceof Objects.ApplicationJobTemplate
+            || obj instanceof Objects.APIReleaseState
             || obj instanceof Objects.AssignmentServiceToAuthorizationGroup
             || obj instanceof Objects.ATCCheckCategory
+            || obj instanceof Objects.ApplicationDescriptorsFiori
             || obj instanceof Objects.ATCCheckObject
             || obj instanceof Objects.ATCCheckVariant
             || obj instanceof Objects.AuthorizationCheckField

package/build/src/rules/exit_or_check.js

@@ -59,7 +59,7 @@ https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#check-vs-re
             else if (this.conf.allowCheck === false && get instanceof Statements.Check && stack.length === 0) {
                 const message = "CHECK is not allowed outside of loops";
                 let tokensString = statement.concatTokens();
-                tokensString = tokensString.slice(statement.getFirstToken().getEnd().getCol());
+                tokensString = tokensString.split("CHECK")[1].trim();
                 const replacement = "IF NOT " + tokensString + "\n  RETURN.\nENDIF.";
                 const fix = edit_helper_1.EditHelper.replaceRange(file, statement.getFirstToken().getStart(), statement.getLastToken().getEnd(), replacement);
                 const issue = issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, fix);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.113.109",
+    "version": "2.113.111",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -50,20 +50,20 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@microsoft/api-extractor": "^7.52.3",
+        "@microsoft/api-extractor": "^7.52.7",
         "@types/chai": "^4.3.20",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^22.14.1",
+        "@types/node": "^22.15.17",
         "chai": "^4.5.0",
-        "eslint": "^9.24.0",
-        "mocha": "^11.1.0",
+        "eslint": "^9.26.0",
+        "mocha": "^11.2.2",
         "c8": "^10.1.3",
         "source-map-support": "^0.5.21",
         "ts-json-schema-generator": "^2.4.0",
         "typescript": "^5.8.3"
     },
     "dependencies": {
-        "fast-xml-parser": "^5.2.0",
+        "fast-xml-parser": "^5.2.3",
         "json5": "^2.2.3",
         "vscode-languageserver-types": "^3.17.5"
     }