@abaplint/core 2.80.8 → 2.81.1

package/README.md

@@ -1,7 +1,7 @@
-# @abaplint/core
-
-[abaplint](https://abaplint.org/) core library
-
-Exposes functionallity like the parser and rules, which can be used in other projects.
-
+# @abaplint/core
+
+[abaplint](https://abaplint.org/) core library
+
+Exposes functionallity like the parser and rules, which can be used in other projects.
+
 For more information see https://github.com/abaplint/abaplint

package/build/abaplint.d.ts

@@ -999,6 +999,8 @@ export declare class CurrentScope {
     private constructor();
     addType(type: TypedIdentifier | undefined): void;
     addTypeNamed(name: string, type: TypedIdentifier | undefined): void;
+    addExtraLikeType(type: TypedIdentifier | undefined): void;
+    addExtraLikeTypeNamed(name: string, type: TypedIdentifier | undefined): void;
     addClassDefinition(c: IClassDefinition): void;
     addFormDefinitions(f: readonly IFormDefinition[]): void;
     addInterfaceDefinition(i: IInterfaceDefinition): void;
@@ -1024,6 +1026,7 @@ export declare class CurrentScope {
     findInterfaceDefinition(name: string): IInterfaceDefinition | undefined;
     findFormDefinition(name: string): IFormDefinition | undefined;
     findType(name: string | undefined): TypedIdentifier | undefined;
+    findExtraLikeType(name: string | undefined): TypedIdentifier | undefined;
     findVariable(name: string | undefined): TypedIdentifier | undefined;
     getDDIC(): DDIC;
     getDDICReferences(): IDDICReferences;
@@ -3067,6 +3070,9 @@ declare interface IScopeData {
     types: {
         [name: string]: TypedIdentifier;
     };
+    extraLikeTypes: {
+        [name: string]: TypedIdentifier;
+    };
     deferred: Token[];
     cdefs: IClassDefinition[];
     idefs: IInterfaceDefinition[];
@@ -4312,6 +4318,7 @@ export declare enum ScopeType {
     Form = "form",
     FunctionModule = "function",
     Method = "method",
+    MethodInstance = "method_instance",
     For = "for",
     OpenSQL = "open_sql"
 }
@@ -4595,6 +4602,7 @@ export declare class SpaghettiScopeNode extends ScopeData implements ISpaghettiS
     listFormDefinitions(): IFormDefinition[];
     findInterfaceDefinition(name: string): IInterfaceDefinition | undefined;
     findType(name: string): TypedIdentifier | undefined;
+    findExtraLikeType(name: string): TypedIdentifier | undefined;
     findVariable(name: string): TypedIdentifier | undefined;
     findWriteReference(pos: Position): TypedIdentifier | undefined;
     findTableReference(pos: Position): string | undefined;

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

@@ -7,7 +7,8 @@ const tokens_1 = require("../../1_lexer/tokens");
 class SQLArithmetics extends combi_1.Expression {
     getRunnable() {
         const operator = (0, combi_1.altPrio)((0, combi_1.tok)(tokens_1.WPlusW), (0, combi_1.tok)(tokens_1.WDashW), "*", "/");
-        return (0, combi_1.seq)(_1.SQLFieldName, (0, combi_1.starPrio)((0, combi_1.seq)(operator, _1.SQLFieldName)));
+        const field = (0, combi_1.alt)(_1.SQLFieldName, _1.SQLFunction);
+        return (0, combi_1.seq)(field, (0, combi_1.starPrio)((0, combi_1.seq)(operator, field)));
     }
 }
 exports.SQLArithmetics = SQLArithmetics;

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

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.SQLFunction = void 0;
+const constant_1 = require("./constant");
 const version_1 = require("../../../version");
 const tokens_1 = require("../../1_lexer/tokens");
 const combi_1 = require("../combi");
@@ -9,10 +10,20 @@ const sql_alias_field_1 = require("./sql_alias_field");
 const sql_field_name_1 = require("./sql_field_name");
 class SQLFunction extends combi_1.Expression {
     getRunnable() {
+        const param = (0, combi_1.alt)(sql_field_name_1.SQLFieldName, sql_alias_field_1.SQLAliasField, SQLFunction, constant_1.Constant);
+        const castTypes = (0, combi_1.altPrio)((0, combi_1.seq)("CHAR", (0, combi_1.tok)(tokens_1.ParenLeftW), integer_1.Integer, (0, combi_1.tok)(tokens_1.WParenRightW)), "FLTP");
+        const abs = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)("abs", (0, combi_1.tok)(tokens_1.ParenLeftW), param, (0, combi_1.tok)(tokens_1.WParenRightW)));
+        const cast = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)("cast", (0, combi_1.tok)(tokens_1.ParenLeftW), param, "AS", castTypes, (0, combi_1.tok)(tokens_1.WParenRightW)));
+        const ceil = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)("ceil", (0, combi_1.tok)(tokens_1.ParenLeftW), param, (0, combi_1.tok)(tokens_1.WParenRightW)));
+        const coalesce = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)("coalesce", (0, combi_1.tok)(tokens_1.ParenLeftW), param, ",", param, (0, combi_1.tok)(tokens_1.WParenRightW)));
+        const concat = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)("concat", (0, combi_1.tok)(tokens_1.ParenLeftW), param, ",", param, (0, combi_1.tok)(tokens_1.WParenRightW)));
+        const div = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)("div", (0, combi_1.tok)(tokens_1.ParenLeftW), param, ",", param, (0, combi_1.tok)(tokens_1.WParenRightW)));
+        const floor = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)("floor", (0, combi_1.tok)(tokens_1.ParenLeftW), param, (0, combi_1.tok)(tokens_1.WParenRightW)));
+        const length = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)("length", (0, combi_1.tok)(tokens_1.ParenLeftW), param, (0, combi_1.tok)(tokens_1.WParenRightW)));
+        const mod = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)("mod", (0, combi_1.tok)(tokens_1.ParenLeftW), param, ",", param, (0, combi_1.tok)(tokens_1.WParenRightW)));
+        const replace = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)("replace", (0, combi_1.tok)(tokens_1.ParenLeftW), param, ",", param, ",", param, (0, combi_1.tok)(tokens_1.WParenRightW)));
         const uuid = (0, combi_1.ver)(version_1.Version.v754, (0, combi_1.seq)("uuid", (0, combi_1.tok)(tokens_1.ParenLeftW), (0, combi_1.tok)(tokens_1.WParenRightW)));
-        const abs = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)("abs", (0, combi_1.tok)(tokens_1.ParenLeftW), (0, combi_1.altPrio)(sql_field_name_1.SQLFieldName, sql_alias_field_1.SQLAliasField), (0, combi_1.tok)(tokens_1.WParenRightW)));
-        const cast = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)("cast", (0, combi_1.tok)(tokens_1.ParenLeftW), (0, combi_1.altPrio)(sql_field_name_1.SQLFieldName, sql_alias_field_1.SQLAliasField), "AS CHAR", (0, combi_1.tok)(tokens_1.ParenLeftW), integer_1.Integer, (0, combi_1.tok)(tokens_1.WParenRightW), (0, combi_1.tok)(tokens_1.WParenRightW)));
-        return (0, combi_1.altPrio)(uuid, abs, cast);
+        return (0, combi_1.altPrio)(uuid, abs, ceil, floor, cast, div, mod, coalesce, concat, replace, length);
     }
 }
 exports.SQLFunction = SQLFunction;

package/build/src/abap/2_statements/statements/move.js

@@ -10,9 +10,10 @@ class Move {
         const mov = (0, combi_1.verNot)(version_1.Version.Cloud, "MOVE");
         const move = (0, combi_1.seq)(mov, (0, combi_1.altPrio)((0, combi_1.seq)("EXACT", expressions_1.Source, "TO", expressions_1.Target), (0, combi_1.seq)(expressions_1.Source, (0, combi_1.altPrio)("?TO", "TO"), expressions_1.Target)));
         const calcAssign = (0, combi_1.ver)(version_1.Version.v754, (0, combi_1.alt)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.WPlus), "="), (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WDash), "="), "/=", "*=", "&&="));
-        const equals = (0, combi_1.altPrio)((0, combi_1.altPrio)("=", "?="), calcAssign);
+        const chained = (0, combi_1.seq)("=", (0, combi_1.star)((0, combi_1.seq)(expressions_1.Target, "=")));
+        const equals = (0, combi_1.altPrio)((0, combi_1.altPrio)(chained, "?="), calcAssign);
         // todo, move "?=" to CAST?
-        const eq = (0, combi_1.seq)((0, combi_1.plus)((0, combi_1.seq)(expressions_1.Target, equals)), expressions_1.Source);
+        const eq = (0, combi_1.seq)(expressions_1.Target, equals, expressions_1.Source);
         return (0, combi_1.altPrio)(move, eq);
     }
 }

package/build/src/abap/5_syntax/_current_scope.js

@@ -54,6 +54,25 @@ class CurrentScope {
         }
         this.current.getData().types[upper] = type;
     }
+    addExtraLikeType(type) {
+        if (type === undefined) {
+            return;
+        }
+        this.addExtraLikeTypeNamed(type.getName(), type);
+    }
+    addExtraLikeTypeNamed(name, type) {
+        if (type === undefined) {
+            return;
+        }
+        if (this.current === undefined) {
+            return;
+        }
+        const upper = name.toUpperCase();
+        if (this.current.getData().extraLikeTypes[upper] !== undefined) {
+            throw new Error(`Type name "${name}" already defined`);
+        }
+        this.current.getData().extraLikeTypes[upper] = type;
+    }
     addClassDefinition(c) {
         var _a;
         (_a = this.current) === null || _a === void 0 ? void 0 : _a.getData().cdefs.push(c);
@@ -231,6 +250,13 @@ class CurrentScope {
         }
         return (_a = this.current) === null || _a === void 0 ? void 0 : _a.findType(name);
     }
+    findExtraLikeType(name) {
+        var _a;
+        if (name === undefined) {
+            return undefined;
+        }
+        return (_a = this.current) === null || _a === void 0 ? void 0 : _a.findExtraLikeType(name);
+    }
     findVariable(name) {
         var _a;
         if (name === undefined) {

package/build/src/abap/5_syntax/_scope_type.js

@@ -14,6 +14,7 @@ var ScopeType;
     ScopeType["Form"] = "form";
     ScopeType["FunctionModule"] = "function";
     ScopeType["Method"] = "method";
+    ScopeType["MethodInstance"] = "method_instance";
     ScopeType["For"] = "for";
     ScopeType["OpenSQL"] = "open_sql";
 })(ScopeType = exports.ScopeType || (exports.ScopeType = {}));

package/build/src/abap/5_syntax/basic_types.js

@@ -79,8 +79,12 @@ class BasicTypes {
         }
         else {
             const name = children.shift().getFirstToken().getStr();
-            const found = this.scope.findVariable(name);
+            let found = this.scope.findVariable(name);
             type = found === null || found === void 0 ? void 0 : found.getType();
+            if (found === undefined) {
+                found = this.scope.findExtraLikeType(name);
+                type = found === null || found === void 0 ? void 0 : found.getType();
+            }
             if (found) {
                 this.scope.addReference(chain === null || chain === void 0 ? void 0 : chain.getFirstToken(), found, _reference_1.ReferenceType.TypeReference, this.filename);
             }

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

@@ -6,9 +6,8 @@ const basic_1 = require("../../types/basic");
 const source_1 = require("./source");
 class FieldAssignment {
     runSyntax(node, scope, filename, targetType) {
-        var _a;
-        const name = (_a = node.findDirectExpression(Expressions.FieldSub)) === null || _a === void 0 ? void 0 : _a.concatTokens();
-        if (name === undefined) {
+        const fieldSub = node.findDirectExpression(Expressions.FieldSub);
+        if (fieldSub === undefined) {
             throw new Error("FieldAssignment, FieldSub node not found");
         }
         const s = node.findDirectExpression(Expressions.Source);
@@ -17,10 +16,17 @@ class FieldAssignment {
         }
         let type = undefined;
         if (targetType instanceof basic_1.StructureType) {
-            type = targetType.getComponentByName(name);
-            if (type === undefined && targetType.containsVoid() === false) {
-                throw new Error(`field ${name} does not exist in structure`);
+            let context = targetType;
+            for (const c of fieldSub.getChildren()) {
+                const text = c.concatTokens();
+                if (text !== "-" && context instanceof basic_1.StructureType) {
+                    context = context.getComponentByName(text);
+                    if (context === undefined && targetType.containsVoid() === false) {
+                        throw new Error(`field ${text} does not exist in structure`);
+                    }
+                }
             }
+            type = context;
         }
         else if (targetType instanceof basic_1.VoidType) {
             type = targetType;

package/build/src/abap/5_syntax/spaghetti_scope.js

@@ -12,6 +12,7 @@ class ScopeData {
             idefs: [],
             forms: [],
             types: {},
+            extraLikeTypes: {},
             deferred: [],
             references: [],
         };
@@ -126,6 +127,18 @@ class SpaghettiScopeNode extends ScopeData {
         }
         return undefined;
     }
+    findExtraLikeType(name) {
+        let search = this;
+        const upper = name.toUpperCase();
+        while (search !== undefined) {
+            const data = search.getData();
+            if (data.extraLikeTypes[upper]) {
+                return data.extraLikeTypes[upper];
+            }
+            search = search.getParent();
+        }
+        return undefined;
+    }
     findVariable(name) {
         let search = this;
         const upper = name.toUpperCase();

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

@@ -28,8 +28,10 @@ class ClassImplementation {
         scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(new tokens_1.Identifier(new position_1.Position(1, 1), "me"), _builtin_1.BuiltIn.filename, new basic_1.ObjectReferenceType(classDefinition)));
         helper.addAliasedAttributes(classDefinition); // todo, this is not correct, take care of instance vs static
         scope.addList(classAttributes.getConstants());
-        scope.addList(classAttributes.getInstance()); // todo, this is not correct, take care of instance vs static
-        scope.addList(classAttributes.getStatic()); // todo, this is not correct, take care of instance vs static
+        scope.addList(classAttributes.getStatic());
+        for (const i of classAttributes.getInstance()) {
+            scope.addExtraLikeType(i);
+        }
         const implemented = helper.fromSuperClass(classDefinition);
         helper.fromInterfaces(classDefinition, implemented);
     }

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

@@ -11,17 +11,21 @@ class MethodImplementation {
         const className = scope.getName();
         const methodToken = node.findFirstExpression(Expressions.MethodName).getFirstToken();
         const methodName = methodToken === null || methodToken === void 0 ? void 0 : methodToken.getStr();
-        scope.push(_scope_type_1.ScopeType.Method, methodName, node.getFirstToken().getStart(), filename);
         const classDefinition = scope.findClassDefinition(className);
         if (classDefinition === undefined) {
-            scope.pop(node.getLastToken().getEnd());
+            //      scope.pop(node.getLastToken().getEnd());
             throw new Error("Class definition for \"" + className + "\" not found");
         }
         const { method: methodDefinition } = helper.searchMethodName(classDefinition, methodName);
         if (methodDefinition === undefined) {
-            scope.pop(node.getLastToken().getEnd());
+            //      scope.pop(node.getLastToken().getEnd());
             throw new Error("Method definition \"" + methodName + "\" not found");
         }
+        if (methodDefinition.isStatic() === false) {
+            scope.push(_scope_type_1.ScopeType.MethodInstance, methodName, node.getFirstToken().getStart(), filename);
+            scope.addList(classDefinition.getAttributes().getInstance());
+        }
+        scope.push(_scope_type_1.ScopeType.Method, methodName, node.getFirstToken().getStart(), filename);
         scope.addReference(methodToken, methodDefinition, _reference_1.ReferenceType.MethodImplementationReference, filename);
         scope.addList(methodDefinition.getParameters().getAll());
         for (const i of helper.findInterfaces(classDefinition)) {

package/build/src/abap/5_syntax/syntax.js

@@ -412,11 +412,16 @@ class SyntaxLogic {
         }
         else if (s instanceof Statements.EndForm
             || s instanceof Statements.EndFunction
-            || s instanceof Statements.EndMethod
             || s instanceof Statements.EndClass
             || s instanceof Statements.EndInterface) {
             this.scope.pop(node.getLastToken().getEnd());
         }
+        else if (s instanceof Statements.EndMethod) {
+            this.scope.pop(node.getLastToken().getEnd());
+            if (this.scope.getType() === _scope_type_1.ScopeType.MethodInstance) {
+                this.scope.pop(node.getLastToken().getEnd());
+            }
+        }
     }
 }
 exports.SyntaxLogic = SyntaxLogic;

package/build/src/abap/flow/flow_graph.js

@@ -72,13 +72,13 @@ class FlowGraph {
         this.label = label;
     }
     toDigraph() {
-        return `digraph G {
-labelloc="t";
-label="${this.label}";
-graph [fontname = "helvetica"];
-node [fontname = "helvetica", shape="box"];
-edge [fontname = "helvetica"];
-${this.toTextEdges()}
+        return `digraph G {
+labelloc="t";
+label="${this.label}";
+graph [fontname = "helvetica"];
+node [fontname = "helvetica", shape="box"];
+edge [fontname = "helvetica"];
+${this.toTextEdges()}
 }`;
     }
     listSources(node) {

package/build/src/lsp/help.js

@@ -9,13 +9,13 @@ const dump_scope_1 = require("./dump_scope");
 class Help {
     static find(reg, textDocument, position) {
         let content = "";
-        content = `
-    <a href="#_tokens" rel="no-refresh">Tokens</a> |
-    <a href="#_statements" rel="no-refresh">Statements</a> |
-    <a href="#_structure" rel="no-refresh">Structure</a> |
-    <a href="#_files" rel="no-refresh">Files</a> |
-    <a href="#_info" rel="no-refresh">Info Dump</a>
-    <hr>
+        content = `
+    <a href="#_tokens" rel="no-refresh">Tokens</a> |
+    <a href="#_statements" rel="no-refresh">Statements</a> |
+    <a href="#_structure" rel="no-refresh">Structure</a> |
+    <a href="#_files" rel="no-refresh">Files</a> |
+    <a href="#_info" rel="no-refresh">Info Dump</a>
+    <hr>
     ` +
             "<tt>" + textDocument.uri + " (" +
             (position.line + 1) + ", " +

package/build/src/registry.js

@@ -68,7 +68,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.80.8";
+        return "2.81.1";
     }
     getDDICReferences() {
         return this.references;

package/build/src/rules/abapdoc.js

@@ -23,7 +23,7 @@ class Abapdoc extends _abap_rule_1.ABAPRule {
         return {
             key: "abapdoc",
             title: "Check abapdoc",
-            shortDescription: `Various checks regarding abapdoc.
+            shortDescription: `Various checks regarding abapdoc.
 Base rule checks for existence of abapdoc for public class methods and all interface methods.`,
             tags: [_irule_1.RuleTag.SingleFile],
         };

package/build/src/rules/align_parameters.js

@@ -20,42 +20,42 @@ class AlignParameters extends _abap_rule_1.ABAPRule {
             key: "align_parameters",
             title: "Align Parameters",
             shortDescription: `Checks for vertially aligned parameters`,
-            extendedInformation: `Checks:
-* function module calls
-* method calls
-* VALUE constructors
-* NEW constructors
-* RAISE EXCEPTION statements
-* CREATE OBJECT statements
-* RAISE EVENT statements
-
-https://github.com/SAP/styleguides/blob/master/clean-abap/CleanABAP.md#align-parameters
-
-Does not take effect on non functional method calls, use https://rules.abaplint.org/functional_writing/
-
+            extendedInformation: `Checks:
+* function module calls
+* method calls
+* VALUE constructors
+* NEW constructors
+* RAISE EXCEPTION statements
+* CREATE OBJECT statements
+* RAISE EVENT statements
+
+https://github.com/SAP/styleguides/blob/master/clean-abap/CleanABAP.md#align-parameters
+
+Does not take effect on non functional method calls, use https://rules.abaplint.org/functional_writing/
+
 Also https://rules.abaplint.org/max_one_method_parameter_per_line/ can help aligning parameter syntax`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Styleguide],
-            badExample: `CALL FUNCTION 'FOOBAR'
-  EXPORTING
-    foo = 2
-    parameter = 3.
-
-foobar( moo = 1
-  param = 1 ).
-
-foo = VALUE #(
-    foo = bar
+            badExample: `CALL FUNCTION 'FOOBAR'
+  EXPORTING
+    foo = 2
+    parameter = 3.
+
+foobar( moo = 1
+  param = 1 ).
+
+foo = VALUE #(
+    foo = bar
         moo = 2 ).`,
-            goodExample: `CALL FUNCTION 'FOOBAR'
-  EXPORTING
-    foo       = 2
-    parameter = 3.
-
-foobar( moo   = 1
-        param = 1 ).
-
-foo = VALUE #(
-    foo = bar
+            goodExample: `CALL FUNCTION 'FOOBAR'
+  EXPORTING
+    foo       = 2
+    parameter = 3.
+
+foobar( moo   = 1
+        param = 1 ).
+
+foo = VALUE #(
+    foo = bar
     moo = 2 ).`,
         };
     }

package/build/src/rules/ambiguous_statement.js

@@ -20,14 +20,14 @@ class AmbiguousStatement extends _abap_rule_1.ABAPRule {
         return {
             key: "ambiguous_statement",
             title: "Check for ambigious statements",
-            shortDescription: `Checks for ambiguity between deleting or modifying from internal and database table
-Add "TABLE" keyword or "@" for escaping SQL variables
-
+            shortDescription: `Checks for ambiguity between deleting or modifying from internal and database table
+Add "TABLE" keyword or "@" for escaping SQL variables
+
 Only works if the target version is 740sp05 or above`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `DELETE foo FROM bar.
+            badExample: `DELETE foo FROM bar.
 MODIFY foo FROM bar.`,
-            goodExample: `DELETE foo FROM @bar.
+            goodExample: `DELETE foo FROM @bar.
 MODIFY TABLE foo FROM bar.`,
         };
     }

package/build/src/rules/avoid_use.js

@@ -35,12 +35,12 @@ class AvoidUse extends _abap_rule_1.ABAPRule {
             key: "avoid_use",
             title: "Avoid use of certain statements",
             shortDescription: `Detects usage of certain statements.`,
-            extendedInformation: `DEFAULT KEY: https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-default-key
-
-Macros: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abenmacros_guidl.htm
-
-STATICS: use CLASS-DATA instead
-
+            extendedInformation: `DEFAULT KEY: https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-default-key
+
+Macros: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abenmacros_guidl.htm
+
+STATICS: use CLASS-DATA instead
+
 DESCRIBE TABLE LINES: use lines() instead (quickfix exists)`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };

package/build/src/rules/begin_end_names.js

@@ -23,11 +23,11 @@ class BeginEndNames extends _abap_rule_1.ABAPRule {
             title: "Check BEGIN END names",
             shortDescription: `Check BEGIN OF and END OF names match, plus there must be statements between BEGIN and END`,
             tags: [_irule_1.RuleTag.Syntax, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
-            badExample: `DATA: BEGIN OF stru,
-      field TYPE i,
+            badExample: `DATA: BEGIN OF stru,
+      field TYPE i,
     END OF structure_not_the_same.`,
-            goodExample: `DATA: BEGIN OF stru,
-      field TYPE i,
+            goodExample: `DATA: BEGIN OF stru,
+      field TYPE i,
     END OF stru.`,
         };
     }

package/build/src/rules/begin_single_include.js

@@ -21,19 +21,19 @@ class BeginSingleInclude extends _abap_rule_1.ABAPRule {
             title: "BEGIN contains single INCLUDE",
             shortDescription: `Finds TYPE BEGIN with just one INCLUDE TYPE, and DATA with single INCLUDE STRUCTURE`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `TYPES: BEGIN OF dummy1.
-  INCLUDE TYPE dselc.
-TYPES: END OF dummy1.
-
-DATA BEGIN OF foo.
-INCLUDE STRUCTURE syst.
-DATA END OF foo.
-
-STATICS BEGIN OF bar.
-INCLUDE STRUCTURE syst.
+            badExample: `TYPES: BEGIN OF dummy1.
+  INCLUDE TYPE dselc.
+TYPES: END OF dummy1.
+
+DATA BEGIN OF foo.
+INCLUDE STRUCTURE syst.
+DATA END OF foo.
+
+STATICS BEGIN OF bar.
+INCLUDE STRUCTURE syst.
 STATICS END OF bar.`,
-            goodExample: `DATA BEGIN OF foo.
-INCLUDE STRUCTURE dselc.
+            goodExample: `DATA BEGIN OF foo.
+INCLUDE STRUCTURE dselc.
 DATA END OF foo.`,
         };
     }

package/build/src/rules/call_transaction_authority_check.js

@@ -24,9 +24,9 @@ class CallTransactionAuthorityCheck extends _abap_rule_1.ABAPRule {
             extendedInformation: `https://docs.abapopenchecks.org/checks/54/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Security],
             badExample: `CALL TRANSACTION 'FOO'.`,
-            goodExample: `TRY.
-    CALL TRANSACTION 'FOO' WITH AUTHORITY-CHECK.
-  CATCH cx_sy_authorization_error.
+            goodExample: `TRY.
+    CALL TRANSACTION 'FOO' WITH AUTHORITY-CHECK.
+  CATCH cx_sy_authorization_error.
 ENDTRY.`,
         };
     }

package/build/src/rules/chain_mainly_declarations.js

@@ -45,10 +45,10 @@ class ChainMainlyDeclarations extends _abap_rule_1.ABAPRule {
             key: "chain_mainly_declarations",
             title: "Chain mainly declarations",
             shortDescription: `Chain mainly declarations, allows chaining for the configured statements, reports errors for other statements.`,
-            extendedInformation: `
-https://docs.abapopenchecks.org/checks/23/
-
-https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abenchained_statements_guidl.htm
+            extendedInformation: `
+https://docs.abapopenchecks.org/checks/23/
+
+https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abenchained_statements_guidl.htm
 `,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
             badExample: `CALL METHOD: bar.`,

package/build/src/rules/check_abstract.js

@@ -23,8 +23,8 @@ class CheckAbstract extends _abap_rule_1.ABAPRule {
         return {
             key: "check_abstract",
             title: "Check abstract methods and classes",
-            shortDescription: `Checks abstract methods and classes:
-- class defined as abstract and final,
+            shortDescription: `Checks abstract methods and classes:
+- class defined as abstract and final,
 - non-abstract class contains abstract methods`,
             extendedInformation: `If a class defines only constants, use an interface instead`,
             tags: [_irule_1.RuleTag.SingleFile],

package/build/src/rules/check_comments.js

@@ -27,9 +27,9 @@ class CheckComments extends _abap_rule_1.ABAPRule {
         return {
             key: "check_comments",
             title: "Check Comments",
-            shortDescription: `
-Various checks for comment usage.
-
+            shortDescription: `
+Various checks for comment usage.
+
 * End of line comments. Comments starting with "#EC" or "##" are ignored`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#put-comments-before-the-statement-they-relate-to`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],

package/build/src/rules/check_include.js

@@ -17,9 +17,9 @@ class CheckInclude {
             key: "check_include",
             title: "Check INCLUDEs",
             shortDescription: `Checks INCLUDE statements`,
-            extendedInformation: `
-* Reports unused includes
-* Errors if the includes are not found
+            extendedInformation: `
+* Reports unused includes
+* Errors if the includes are not found
 * Error if including a main program`,
             tags: [_irule_1.RuleTag.Syntax],
         };

package/build/src/rules/check_no_handler_pragma.js

@@ -21,15 +21,15 @@ class CheckNoHandlerPragma extends _abap_rule_1.ABAPRule {
             title: "Check if NO_HANDLER can be removed",
             shortDescription: `Checks NO_HANDLER pragmas that can be removed`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `TRY.
-    ...
-  CATCH zcx_abapgit_exception ##NO_HANDLER.
-    RETURN. " it has a handler
+            badExample: `TRY.
+    ...
+  CATCH zcx_abapgit_exception ##NO_HANDLER.
+    RETURN. " it has a handler
 ENDTRY.`,
-            goodExample: `TRY.
-    ...
-  CATCH zcx_abapgit_exception.
-    RETURN.
+            goodExample: `TRY.
+    ...
+  CATCH zcx_abapgit_exception.
+    RETURN.
 ENDTRY.`,
         };
     }