@abaplint/core 2.113.107 → 2.113.109

package/build/abaplint.d.ts

@@ -4381,8 +4381,10 @@ declare class MethodDefinition extends Identifier implements IMethodDefinition {
     private readonly static;
     private readonly raising;
     private readonly exceptions;
+    private readonly className;
     constructor(node: StatementNode, visibility: Visibility, input: SyntaxInput);
     getVisibility(): Visibility;
+    getClassName(): string;
     isRedefinition(): boolean;
     isAbstract(): boolean;
     isStatic(): boolean;

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

@@ -11,7 +11,7 @@ class StringTemplateFormatting extends combi_1.Expression {
         const alphaOptions = (0, combi_1.altPrio)("OUT", "RAW", "IN", _1.Source);
         const alignOptions = (0, combi_1.altPrio)("LEFT", "RIGHT", "CENTER", _1.Source, dynamic_1.Dynamic);
         const dateTimeOptions = (0, combi_1.altPrio)("RAW", "ISO", "USER", "ENVIRONMENT", _1.Source, dynamic_1.Dynamic);
-        const timeStampOptions = (0, combi_1.altPrio)("SPACE", "ISO", "USER", "ENVIRONMENT", _1.Source);
+        const timeStampOptions = (0, combi_1.altPrio)("SPACE", "ISO", "USER", "ENVIRONMENT", _1.Source, dynamic_1.Dynamic);
         const numberOptions = (0, combi_1.altPrio)("RAW", "USER", "ENVIRONMENT", _1.Source, dynamic_1.Dynamic);
         const signOptions = (0, combi_1.altPrio)("LEFT", "LEFTPLUS", "LEFTSPACE", "RIGHT", "RIGHTPLUS", "RIGHTSPACE", _1.Source);
         const caseOptions = (0, combi_1.altPrio)("RAW", "UPPER", "LOWER", _1.Source, dynamic_1.Dynamic);

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

@@ -18,6 +18,7 @@ class MethodDefinition extends _identifier_1.Identifier {
             throw new Error("MethodDefinition, expected MethodDef as part of input node");
         }
         super(found.getFirstToken(), input.filename);
+        this.className = input.scope.getName();
         this.redefinition = false;
         if (node.findDirectExpression(Expressions.Redefinition)) {
             this.redefinition = true;
@@ -70,6 +71,9 @@ class MethodDefinition extends _identifier_1.Identifier {
     getVisibility() {
         return this.visibility;
     }
+    getClassName() {
+        return this.className;
+    }
     isRedefinition() {
         return this.redefinition;
     }

package/build/src/registry.js

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

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.107",
+    "version": "2.113.109",
     "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.3",
         "@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"
     }