@abaplint/core 2.119.66 → 2.120.0

package/README.md

@@ -1,7 +1,7 @@
-# @abaplint/core
-
-[abaplint](https://abaplint.org) core library
-
-Exposes functionality like the parser and rules, which can be used in other projects.
-
+# @abaplint/core
+
+[abaplint](https://abaplint.org) core library
+
+Exposes functionality 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

@@ -1491,7 +1491,7 @@ export declare class Config implements IConfiguration {
     getGlobal(): IGlobalConfig;
     getSyntaxSetttings(): ISyntaxSettings;
     getRelease(): ABAPRelease;
-    getOpenABAP(): boolean;
+    isOpenABAP(): boolean;
     getVersion(): Version;
     getLanguageVersion(): LanguageVersion;
     private checkVersion;
@@ -3475,7 +3475,7 @@ export declare interface IConfiguration {
     getVersion(): Version;
     getRelease(): ABAPRelease;
     getLanguageVersion(): LanguageVersion;
-    getOpenABAP(): boolean;
+    isOpenABAP(): boolean;
     getSyntaxSetttings(): ISyntaxSettings;
     readByRule(rule: string): any;
     readByKey(rule: string, key: string): any;

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

@@ -45,7 +45,7 @@ class CurrentScope {
         return this.reg.getConfig().getLanguageVersion();
     }
     getOpenABAP() {
-        return this.reg.getConfig().getOpenABAP();
+        return this.reg.getConfig().isOpenABAP();
     }
     getRegistry() {
         return this.reg;

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

@@ -0,0 +1,59 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.checkOffsetLength = checkOffsetLength;
+const Expressions = __importStar(require("../../2_statements/expressions"));
+const basic_1 = require("../../types/basic");
+const _syntax_input_1 = require("../_syntax_input");
+const _builtin_1 = require("../_builtin");
+// Offsets or lengths cannot be specified for STRING or XSTRING fields
+// passed as a method parameter source. Returns true if an issue was reported.
+function checkOffsetLength(source, sourceType, method, input) {
+    const chain = source.findDirectExpression(Expressions.FieldChain);
+    const isCalculated = source.findDirectExpression(Expressions.Source) !== undefined;
+    const hasOffsetOrLength = chain !== undefined
+        && isCalculated === false
+        && (chain.findDirectExpression(Expressions.FieldOffset) !== undefined
+            || chain.findDirectExpression(Expressions.FieldLength) !== undefined);
+    if (hasOffsetOrLength
+        && !(method instanceof _builtin_1.BuiltInMethod)
+        && (sourceType instanceof basic_1.StringType || sourceType instanceof basic_1.XStringType)) {
+        const message = `Offsets or lengths cannot be specified for fields of type "STRING" or "XSTRING" in the current statement`;
+        input.issues.push((0, _syntax_input_1.syntaxIssue)(input, source.getFirstToken(), message));
+        return true;
+    }
+    return false;
+}
+//# sourceMappingURL=_check_offset_length.js.map

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

@@ -43,6 +43,7 @@ const source_1 = require("./source");
 const _type_utils_1 = require("../_type_utils");
 const _syntax_input_1 = require("../_syntax_input");
 const constant_1 = require("./constant");
+const _check_offset_length_1 = require("./_check_offset_length");
 class MethodCallParam {
     static runSyntax(node, input, method) {
         if (!(node.get() instanceof Expressions.MethodCallParam)) {
@@ -103,6 +104,9 @@ class MethodCallParam {
             let sourceType = undefined;
             if (child.get() instanceof Expressions.Source) {
                 sourceType = source_1.Source.runSyntax(child, input, targetType);
+                if ((0, _check_offset_length_1.checkOffsetLength)(child, sourceType, method, input)) {
+                    return;
+                }
             }
             else if (child.get() instanceof Expressions.ConstantString) {
                 sourceType = constant_1.Constant.runSyntax(child);

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

@@ -45,6 +45,7 @@ const _syntax_input_1 = require("../_syntax_input");
 const assert_error_1 = require("../assert_error");
 const field_chain_1 = require("./field_chain");
 const _reference_1 = require("../_reference");
+const _check_offset_length_1 = require("./_check_offset_length");
 class MethodParameters {
     constructor() {
         this.requiredParameters = undefined;
@@ -215,7 +216,6 @@ class MethodParameters {
         }
     }
     parameterListS(node, input, method) {
-        var _a;
         if (node === undefined) {
             return [];
         }
@@ -223,14 +223,21 @@ class MethodParameters {
             throw new assert_error_1.AssertError("parameterListS, unexpected node");
         }
         const ret = [];
+        const supplied = new Set();
         for (const c of node.getChildren()) {
             if (!(c.get() instanceof Expressions.ParameterS) || !(c instanceof nodes_1.ExpressionNode)) {
                 throw new assert_error_1.AssertError("parameterListS, unexpected node, child");
             }
-            const name = (_a = c.findDirectExpression(Expressions.ParameterName)) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStr().toUpperCase();
-            if (name === undefined) {
+            const nameNode = c.findDirectExpression(Expressions.ParameterName);
+            const name = nameNode === null || nameNode === void 0 ? void 0 : nameNode.getFirstToken().getStr().toUpperCase();
+            if (name === undefined || nameNode === undefined) {
                 throw new assert_error_1.AssertError("parameterListS, no name determined");
             }
+            if (supplied.has(name)) {
+                const message = `Method parameter "${name}" is supplied more than once`;
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, nameNode.getFirstToken(), message));
+            }
+            supplied.add(name);
             const source = c.findDirectExpression(Expressions.Source);
             if (source === undefined) {
                 throw new assert_error_1.AssertError("parameterListS, no source found");
@@ -248,6 +255,7 @@ class MethodParameters {
                 targetType = method;
             }
             let sourceType = source_1.Source.runSyntax(source, input, targetType);
+            (0, _check_offset_length_1.checkOffsetLength)(source, sourceType, method, input);
             if (sourceType === undefined) {
                 if (method instanceof basic_1.VoidType) {
                     sourceType = method;
@@ -263,7 +271,6 @@ class MethodParameters {
         return ret;
     }
     parameterListT(node, input) {
-        var _a;
         if (node === undefined) {
             return [];
         }
@@ -271,14 +278,21 @@ class MethodParameters {
             throw new assert_error_1.AssertError("parameterListT, unexpected node");
         }
         const ret = [];
+        const supplied = new Set();
         for (const c of node.getChildren()) {
             if (!(c.get() instanceof Expressions.ParameterT) || !(c instanceof nodes_1.ExpressionNode)) {
                 throw new assert_error_1.AssertError("parameterListT, unexpected node, child");
             }
-            const name = (_a = c.findDirectExpression(Expressions.ParameterName)) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStr().toUpperCase();
-            if (name === undefined) {
+            const nameNode = c.findDirectExpression(Expressions.ParameterName);
+            const name = nameNode === null || nameNode === void 0 ? void 0 : nameNode.getFirstToken().getStr().toUpperCase();
+            if (name === undefined || nameNode === undefined) {
                 throw new assert_error_1.AssertError("parameterListT, no name determined");
             }
+            if (supplied.has(name)) {
+                const message = `Method parameter "${name}" is supplied more than once`;
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, nameNode.getFirstToken(), message));
+            }
+            supplied.add(name);
             const target = c.findDirectExpression(Expressions.Target);
             if (target === undefined) {
                 throw new assert_error_1.AssertError("parameterListT, no target found");

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

@@ -41,6 +41,7 @@ const basic_1 = require("../../types/basic");
 const target_1 = require("../expressions/target");
 const _reference_1 = require("../_reference");
 const _syntax_input_1 = require("../_syntax_input");
+const _type_utils_1 = require("../_type_utils");
 class Catch {
     runSyntax(node, input) {
         var _a, _b;
@@ -91,7 +92,22 @@ class Catch {
             }
         }
         else if (target) {
-            target_1.Target.runSyntax(target, input);
+            const targetType = target_1.Target.runSyntax(target, input);
+            if (targetType instanceof basic_1.ObjectReferenceType) {
+                for (const c of node.findDirectExpressions(Expressions.ClassName)) {
+                    const token = c.getFirstToken();
+                    const className = token.getStr().toUpperCase();
+                    const found = input.scope.existsObject(className);
+                    if (found === null || found === void 0 ? void 0 : found.id) {
+                        const catchType = new basic_1.ObjectReferenceType(found.id);
+                        if (new _type_utils_1.TypeUtils(input.scope).isAssignableStrict(catchType, targetType) === false) {
+                            const message = "CATCH target not compatible with " + className;
+                            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, token, message));
+                            return;
+                        }
+                    }
+                }
+            }
         }
     }
 }

package/build/src/config.js

@@ -181,7 +181,7 @@ class Config {
         }
         return (0, version_1.versionToABAPRelease)(v);
     }
-    getOpenABAP() {
+    isOpenABAP() {
         return this.getRelease() === version_1.Release["open-abap"];
     }
     getVersion() {

package/build/src/lsp/help.js

@@ -51,13 +51,13 @@ class Help {
     }
     static dumpABAP(file, 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) + ", " +
@@ -212,9 +212,9 @@ class Help {
         return ret + "</ul>";
     }
     static tokens(file) {
-        let inner = `<table><tr><td><b>String</b></td><td><b>Type</b></td>
-    <td><b>Row</b></td><td><b>Column</b></td>
-    <td><b>vRow</b></td><td><b>vColumn</b></td>
+        let inner = `<table><tr><td><b>String</b></td><td><b>Type</b></td>
+    <td><b>Row</b></td><td><b>Column</b></td>
+    <td><b>vRow</b></td><td><b>vColumn</b></td>
     </tr>`;
         for (const token of file.getTokens()) {
             const tStart = token.getStart();

package/build/src/registry.js

@@ -75,7 +75,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.119.66";
+        return "2.120.0";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/build/src/rules/7bit_ascii.js

@@ -17,10 +17,10 @@ class SevenBitAscii {
             key: "7bit_ascii",
             title: "Check for 7bit ascii",
             shortDescription: `Only allow characters from the 7bit ASCII set.`,
-            extendedInformation: `https://docs.abapopenchecks.org/checks/05/
-
-https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abencharacter_set_guidl.htm
-
+            extendedInformation: `https://docs.abapopenchecks.org/checks/05/
+
+https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abencharacter_set_guidl.htm
+
 Checkes files with extensions ".abap" and ".asddls"`,
             tags: [_irule_1.RuleTag.SingleFile],
             badExample: `WRITE '뽑'.`,

package/build/src/rules/abapdoc.js

@@ -28,10 +28,10 @@ class Abapdoc extends _abap_rule_1.ABAPRule {
             key: "abapdoc",
             title: "Check abapdoc",
             shortDescription: `Various checks regarding abapdoc.`,
-            extendedInformation: `Base rule checks for existence of abapdoc for public class methods and all interface methods.
-
-Plus class and interface definitions.
-
+            extendedInformation: `Base rule checks for existence of abapdoc for public class methods and all interface methods.
+
+Plus class and interface definitions.
+
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#abap-doc-only-for-public-apis`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
         };

package/build/src/rules/add_test_attributes.js

@@ -54,27 +54,27 @@ class AddTestAttributes extends _abap_rule_1.ABAPRule {
             title: "Add test attributes for tests classes with test methods",
             shortDescription: `Add test attributes DURATION and RISK LEVEL for tests classes with test methods`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `CLASS ltcl_test1 DEFINITION FINAL FOR TESTING.
-  PUBLIC SECTION.
-  PROTECTED SECTION.
-  PRIVATE SECTION.
-    METHODS test FOR TESTING RAISING cx_static_check.
-ENDCLASS.
-
-CLASS ltcl_test1 IMPLEMENTATION.
-  METHOD test.
-  ENDMETHOD.
+            badExample: `CLASS ltcl_test1 DEFINITION FINAL FOR TESTING.
+  PUBLIC SECTION.
+  PROTECTED SECTION.
+  PRIVATE SECTION.
+    METHODS test FOR TESTING RAISING cx_static_check.
+ENDCLASS.
+
+CLASS ltcl_test1 IMPLEMENTATION.
+  METHOD test.
+  ENDMETHOD.
 ENDCLASS.`,
-            goodExample: `CLASS ltcl_test2 DEFINITION FINAL FOR TESTING DURATION SHORT RISK LEVEL HARMLESS.
-  PUBLIC SECTION.
-  PROTECTED SECTION.
-  PRIVATE SECTION.
-    METHODS test FOR TESTING RAISING cx_static_check.
-ENDCLASS.
-
-CLASS ltcl_test2 IMPLEMENTATION.
-  METHOD test.
-  ENDMETHOD.
+            goodExample: `CLASS ltcl_test2 DEFINITION FINAL FOR TESTING DURATION SHORT RISK LEVEL HARMLESS.
+  PUBLIC SECTION.
+  PROTECTED SECTION.
+  PRIVATE SECTION.
+    METHODS test FOR TESTING RAISING cx_static_check.
+ENDCLASS.
+
+CLASS ltcl_test2 IMPLEMENTATION.
+  METHOD test.
+  ENDMETHOD.
 ENDCLASS.`,
         };
     }

package/build/src/rules/align_parameters.js

@@ -55,49 +55,49 @@ 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/
-
-If parameters are on the same row, no issues are reported, see
+            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/
+
+If parameters are on the same row, no issues are reported, see
 https://rules.abaplint.org/max_one_method_parameter_per_line/ for splitting parameters to lines`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix],
-            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
-    moo = 2 ).
-
-DATA(sdf) = VALUE type(
-  common_val     = 2
-  another_common = 5
-  ( row_value = 4
+            goodExample: `CALL FUNCTION 'FOOBAR'
+  EXPORTING
+    foo       = 2
+    parameter = 3.
+
+foobar( moo   = 1
+        param = 1 ).
+
+foo = VALUE #(
+    foo = bar
+    moo = 2 ).
+
+DATA(sdf) = VALUE type(
+  common_val     = 2
+  another_common = 5
+  ( row_value = 4
     value_foo = 5 ) ).`,
         };
     }

package/build/src/rules/align_type_expressions.js

@@ -63,37 +63,37 @@ class AlignTypeExpressions extends _abap_rule_1.ABAPRule {
             key: "align_type_expressions",
             title: "Align TYPE expressions",
             shortDescription: `Align TYPE expressions in statements`,
-            extendedInformation: `
-Currently works for METHODS + BEGIN OF
-
-If BEGIN OF has an INCLUDE TYPE its ignored
-
-Also note that clean ABAP does not recommend aligning TYPE clauses:
+            extendedInformation: `
+Currently works for METHODS + BEGIN OF
+
+If BEGIN OF has an INCLUDE TYPE its ignored
+
+Also note that clean ABAP does not recommend aligning TYPE clauses:
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#dont-align-type-clauses`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Quickfix],
-            badExample: `
-TYPES: BEGIN OF foo,
-         bar TYPE i,
-         foobar TYPE i,
-       END OF foo.
-
-INTERFACE lif.
-  METHODS bar
-    IMPORTING
-      foo TYPE i
-      foobar TYPE i.
+            badExample: `
+TYPES: BEGIN OF foo,
+         bar TYPE i,
+         foobar TYPE i,
+       END OF foo.
+
+INTERFACE lif.
+  METHODS bar
+    IMPORTING
+      foo TYPE i
+      foobar TYPE i.
 ENDINTERFACE.`,
-            goodExample: `
-TYPES: BEGIN OF foo,
-         bar    TYPE i,
-         foobar TYPE i,
-       END OF foo.
-
-INTERFACE lif.
-  METHODS bar
-    IMPORTING
-      foo    TYPE i
-      foobar TYPE i.
+            goodExample: `
+TYPES: BEGIN OF foo,
+         bar    TYPE i,
+         foobar TYPE i,
+       END OF foo.
+
+INTERFACE lif.
+  METHODS bar
+    IMPORTING
+      foo    TYPE i
+      foobar TYPE i.
 ENDINTERFACE.`,
         };
     }

package/build/src/rules/ambiguous_statement.js

@@ -53,16 +53,16 @@ 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.
-DELETE TABLE itab FROM 2.
-MODIFY zfoo FROM @wa.
+            goodExample: `DELETE foo FROM @bar.
+DELETE TABLE itab FROM 2.
+MODIFY zfoo FROM @wa.
 MODIFY TABLE foo FROM bar.`,
         };
     }

package/build/src/rules/avoid_use.js

@@ -77,16 +77,16 @@ 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
-
-DESCRIBE TABLE LINES: use lines() instead (quickfix exists)
-
-TEST-SEAMS: https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-test-seams-as-temporary-workaround
-
+            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)
+
+TEST-SEAMS: https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-test-seams-as-temporary-workaround
+
 BREAK points`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
             badExample: `DESCRIBE TABLE items LINES count.`,

package/build/src/rules/begin_end_names.js

@@ -57,11 +57,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

@@ -54,20 +54,20 @@ 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.
-  DATA field TYPE i.
-  INCLUDE STRUCTURE dselc.
+            goodExample: `DATA BEGIN OF foo.
+  DATA field TYPE i.
+  INCLUDE STRUCTURE dselc.
 DATA END OF foo.`,
         };
     }

package/build/src/rules/call_transaction_authority_check.js

@@ -57,9 +57,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.`,
         };
     }