@abaplint/core 2.113.108 → 2.113.110

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(): {
@@ -4875,11 +4893,13 @@ declare namespace Objects {
         ATCCheckObject,
         ATCCheckVariant,
         AuthorizationCheckField,
+        ApplicationDescriptorsFiori,
         AuthorizationGroup,
         AuthorizationObjectClass,
         AuthorizationObjectExtension,
         AuthorizationObject,
         BehaviorDefinition,
+        BRFPlusSystemApplication,
         BSPApplication,
         BusinessAddInImplementation,
         BusinessCatalogAppAssignment,

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/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/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/registry.js

@@ -67,7 +67,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.108";
+        return "2.113.110";
     }
     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/no_chained_assignment.js

@@ -7,6 +7,7 @@ const issue_1 = require("../issue");
 const _abap_rule_1 = require("./_abap_rule");
 const _basic_rule_config_1 = require("./_basic_rule_config");
 const _irule_1 = require("./_irule");
+const edit_helper_1 = require("../edit_helper");
 class NoChainedAssignmentConf extends _basic_rule_config_1.BasicRuleConfig {
 }
 exports.NoChainedAssignmentConf = NoChainedAssignmentConf;
@@ -21,7 +22,7 @@ class NoChainedAssignment extends _abap_rule_1.ABAPRule {
             title: "No chained assignment",
             shortDescription: `Find chained assingments and reports issues`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#dont-chain-assignments`,
-            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix],
             badExample: `var1 = var2 = var3.`,
             goodExample: `var2 = var3.
 var1 = var2.`,
@@ -41,12 +42,23 @@ var1 = var2.`,
             }
             if (s.findDirectExpressions(Expressions.Target).length >= 2) {
                 const message = "No chained assignment";
-                const issue = issue_1.Issue.atStatement(file, s, message, this.getMetadata().key);
+                const fix = this.buildFix(file, s);
+                const issue = issue_1.Issue.atStatement(file, s, message, this.getMetadata().key, this.getConfig().severity, fix);
                 issues.push(issue);
             }
         }
         return issues;
     }
+    buildFix(file, node) {
+        // window of 3 expressions
+        const children = node.getChildren();
+        let res = "";
+        for (let i = children.length - 4; i >= 0; i = i - 2) {
+            const concat = children[i].concatTokens() + " " + children[i + 1].concatTokens() + " " + children[i + 2].concatTokens();
+            res += concat + ".\n";
+        }
+        return edit_helper_1.EditHelper.replaceRange(file, node.getStart(), node.getEnd(), res.trimEnd());
+    }
 }
 exports.NoChainedAssignment = NoChainedAssignment;
 //# sourceMappingURL=no_chained_assignment.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.113.108",
+    "version": "2.113.110",
     "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.1",
+        "@microsoft/api-extractor": "^7.52.4",
         "@types/chai": "^4.3.20",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^22.13.10",
+        "@types/node": "^22.14.1",
         "chai": "^4.5.0",
-        "eslint": "^9.22.0",
+        "eslint": "^9.24.0",
         "mocha": "^11.1.0",
         "c8": "^10.1.3",
         "source-map-support": "^0.5.21",
-        "ts-json-schema-generator": "^2.3.0",
-        "typescript": "^5.8.2"
+        "ts-json-schema-generator": "^2.4.0",
+        "typescript": "^5.8.3"
     },
     "dependencies": {
-        "fast-xml-parser": "^5.0.9",
+        "fast-xml-parser": "^5.2.0",
         "json5": "^2.2.3",
         "vscode-languageserver-types": "^3.17.5"
     }