@abaplint/core 2.105.3 → 2.105.5

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

@@ -7,7 +7,6 @@ const position_1 = require("../../position");
 const spaghetti_scope_1 = require("./spaghetti_scope");
 const _identifier_1 = require("../4_file_information/_identifier");
 const _scope_type_1 = require("./_scope_type");
-const _reference_1 = require("./_reference");
 const syntax_1 = require("./syntax");
 class CurrentScope {
     static buildDefault(reg, obj) {
@@ -99,8 +98,14 @@ class CurrentScope {
         this.current.getData().forms.push(...f);
     }
     addInterfaceDefinition(i) {
-        var _a;
-        (_a = this.current) === null || _a === void 0 ? void 0 : _a.getData().idefs.push(i);
+        if (this.current === undefined) {
+            return;
+        }
+        const name = i.getName().toUpperCase();
+        if (this.current.getData().cdefs[name] !== undefined) {
+            throw new Error(`Interface "${name}" already defined`);
+        }
+        this.current.getData().idefs[name] = i;
     }
     addNamedIdentifier(name, identifier) {
         if (this.current === undefined) {
@@ -195,11 +200,10 @@ class CurrentScope {
         }
         return false;
     }
-    // todo, found + type can be removed from method output?
     existsObject(name) {
         var _a, _b, _c;
         if (name === undefined) {
-            return { found: false };
+            return undefined;
         }
         let prefixRTTI = "";
         if (this.parentObj.getType() === "PROG") {
@@ -213,28 +217,28 @@ class CurrentScope {
             if (findLocalClass.isGlobal() === true) {
                 prefixRTTI = "";
             }
-            return { found: true, id: findLocalClass, type: _reference_1.ReferenceType.ObjectOrientedReference, ooType: "CLAS", RTTIName: prefixRTTI + "\\CLASS=" + findLocalClass.getName() };
+            return { id: findLocalClass, ooType: "CLAS", RTTIName: prefixRTTI + "\\CLASS=" + findLocalClass.getName() };
         }
         const globalClas = this.reg.getObject("CLAS", name);
         if (globalClas) {
-            return { found: true, id: globalClas.getIdentifier(), type: _reference_1.ReferenceType.ObjectOrientedReference, ooType: "CLAS", RTTIName: "\\CLASS=" + globalClas.getName() };
+            return { id: globalClas.getIdentifier(), ooType: "CLAS", RTTIName: "\\CLASS=" + globalClas.getName() };
         }
         const findLocalInterface = (_b = this.current) === null || _b === void 0 ? void 0 : _b.findInterfaceDefinition(name);
         if (findLocalInterface) {
             if (findLocalInterface.isGlobal() === true) {
                 prefixRTTI = "";
             }
-            return { found: true, id: findLocalInterface, type: _reference_1.ReferenceType.ObjectOrientedReference, ooType: "INTF", RTTIName: prefixRTTI + "\\INTERFACE=" + findLocalInterface.getName() };
+            return { id: findLocalInterface, ooType: "INTF", RTTIName: prefixRTTI + "\\INTERFACE=" + findLocalInterface.getName() };
         }
         const globalIntf = this.reg.getObject("INTF", name);
         if (globalIntf) {
-            return { found: true, id: globalIntf.getIdentifier(), type: _reference_1.ReferenceType.ObjectOrientedReference, ooType: "INTF", RTTIName: "\\INTERFACE=" + globalIntf.getName() };
+            return { id: globalIntf.getIdentifier(), ooType: "INTF", RTTIName: "\\INTERFACE=" + globalIntf.getName() };
         }
         const def = (_c = this.current) === null || _c === void 0 ? void 0 : _c.findDeferred(name);
         if (def !== undefined) {
-            return { found: true, id: def };
+            return { id: def };
         }
-        return { found: false };
+        return undefined;
     }
     ///////////////////////////
     /** Lookup class in local and global scope */

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

@@ -332,6 +332,8 @@ class TypeUtils {
         else if (source instanceof basic_1.ObjectReferenceType) {
             if (target instanceof basic_1.XSequenceType
                 || target instanceof basic_1.IntegerType
+                || target instanceof basic_1.StructureType
+                || target instanceof basic_1.TableType
                 || target instanceof basic_1.XStringType) {
                 return false;
             }

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

@@ -771,7 +771,7 @@ class BasicTypes {
         else if (firstNode.get() instanceof Expressions.ClassName) {
             const obj = this.scope.findObjectDefinition(firstName);
             if (obj === undefined) {
-                if (this.scope.existsObject(firstName).found === true) {
+                if (this.scope.existsObject(firstName) !== undefined) {
                     return undefined;
                 }
                 else if (this.scope.getDDIC().inErrorNamespace(firstName) === true) {
@@ -818,7 +818,7 @@ class BasicTypes {
                 return new Types.GenericObjectReferenceType();
             }
             const search = this.scope.existsObject(name);
-            if (search.found === true && search.id) {
+            if (search === null || search === void 0 ? void 0 : search.id) {
                 this.scope.addReference(chain.getFirstToken(), search.id, _reference_1.ReferenceType.ObjectOrientedReference, this.filename, { ooType: search.ooType, ooName: name });
                 return new Types.ObjectReferenceType(search.id, { qualifiedName: name, RTTIName: search.RTTIName });
             }

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

@@ -171,8 +171,8 @@ class FieldChain {
                 return new basic_1.GenericObjectReferenceType();
             }
             const found = scope.existsObject(classNam);
-            if (found.found === true && found.id) {
-                scope.addReference(classTok, found.id, found.type, filename);
+            if (found === null || found === void 0 ? void 0 : found.id) {
+                scope.addReference(classTok, found.id, _reference_1.ReferenceType.ObjectOrientedReference, filename);
                 return new basic_1.ObjectReferenceType(found.id);
             }
             else if (scope.getDDIC().inErrorNamespace(classNam) === false) {

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

@@ -29,8 +29,13 @@ class MethodCallParam {
         else if (child instanceof nodes_1.ExpressionNode
             && (child.get() instanceof Expressions.Source
                 || child.get() instanceof Expressions.ConstantString)) {
-            if (!(method instanceof basic_1.VoidType) && method.getParameters().getImporting().length === 0) {
-                throw new Error("Method \"" + method.getName() + "\" has no importing parameters");
+            if (!(method instanceof basic_1.VoidType)) {
+                if (method.getParameters().getImporting().length === 0) {
+                    throw new Error("Method \"" + method.getName() + "\" has no importing parameters");
+                }
+                else if (method.getParameters().getRequiredParameters().length > 1) {
+                    throw new Error("Method \"" + method.getName() + "\" has more than one importing or changing parameter");
+                }
             }
             let targetType = undefined;
             if (!(method instanceof basic_1.VoidType)) {

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

@@ -12,6 +12,7 @@ const field_offset_1 = require("./field_offset");
 const _reference_1 = require("../_reference");
 const table_expression_1 = require("./table_expression");
 const expressions_1 = require("../../2_statements/expressions");
+const field_length_1 = require("./field_length");
 class Target {
     runSyntax(node, scope, filename) {
         const concat = node.concatTokens();
@@ -94,8 +95,18 @@ class Target {
         }
         const offset = node.findDirectExpression(Expressions.FieldOffset);
         if (offset) {
+            if (context instanceof basic_1.XStringType || context instanceof basic_1.StringType) {
+                throw new Error("xstring/string offset/length in writer position not possible");
+            }
             new field_offset_1.FieldOffset().runSyntax(offset, scope, filename);
         }
+        const length = node.findDirectExpression(Expressions.FieldLength);
+        if (length) {
+            if (context instanceof basic_1.XStringType || context instanceof basic_1.StringType) {
+                throw new Error("xstring/string offset/length in writer position not possible");
+            }
+            new field_length_1.FieldLength().runSyntax(length, scope, filename);
+        }
         return context;
     }
     /////////////////////////////////

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

@@ -9,7 +9,7 @@ class ScopeData {
         this.data = {
             vars: {},
             cdefs: {},
-            idefs: [], // todo, refactor to object
+            idefs: {},
             forms: [], // todo, refactor to object
             types: {},
             extraLikeTypes: {},
@@ -111,26 +111,13 @@ class SpaghettiScopeNode extends ScopeData {
         }
         return undefined;
     }
-    // todo, can be deleted, not called from anywhere?
-    listFormDefinitions() {
-        let search = this;
-        const ret = [];
-        while (search !== undefined) {
-            for (const form of search.getData().forms) {
-                ret.push(form);
-            }
-            search = search.getParent();
-        }
-        return ret;
-    }
-    // todo, optimize
     findInterfaceDefinition(name) {
         let search = this;
+        const upper = name.toUpperCase();
         while (search !== undefined) {
-            for (const idef of search.getData().idefs) {
-                if (idef.getName().toUpperCase() === name.toUpperCase()) {
-                    return idef;
-                }
+            const idef = search.getData().idefs[upper];
+            if (idef) {
+                return idef;
             }
             search = search.getParent();
         }
@@ -140,9 +127,9 @@ class SpaghettiScopeNode extends ScopeData {
         let search = this;
         const upper = name.toUpperCase();
         while (search !== undefined) {
-            const data = search.getData();
-            if (data.types[upper]) {
-                return data.types[upper];
+            const found = search.getData().types[upper];
+            if (found) {
+                return found;
             }
             search = search.getParent();
         }

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

@@ -15,8 +15,8 @@ class Catch {
             const token = c.getFirstToken();
             const className = token.getStr().toUpperCase();
             const found = scope.existsObject(className);
-            if (found.found === true && found.id) {
-                scope.addReference(token, found.id, found.type, filename);
+            if (found === null || found === void 0 ? void 0 : found.id) {
+                scope.addReference(token, found.id, _reference_1.ReferenceType.ObjectOrientedReference, filename);
             }
             else if (scope.getDDIC().inErrorNamespace(className) === false) {
                 const extra = { ooName: className, ooType: "Void" };
@@ -35,7 +35,7 @@ class Catch {
         if (target === null || target === void 0 ? void 0 : target.findDirectExpression(Expressions.InlineData)) {
             const token = (_b = target.findFirstExpression(Expressions.TargetField)) === null || _b === void 0 ? void 0 : _b.getFirstToken();
             const found = scope.existsObject(firstClassName);
-            if (token && found.found === true && firstClassName && found.id) {
+            if (token && firstClassName && (found === null || found === void 0 ? void 0 : found.id)) {
                 const identifier = new _typed_identifier_1.TypedIdentifier(token, filename, new basic_1.ObjectReferenceType(found.id), ["inline" /* IdentifierMeta.InlineDefinition */]);
                 scope.addIdentifier(identifier);
                 scope.addReference(token, identifier, _reference_1.ReferenceType.DataWriteReference, filename);

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

@@ -26,7 +26,7 @@ class ClassLocalFriends {
             const className = classNames[i].concatTokens();
             // make sure to check also DEFINITION DEFERRED
             const found = scope.existsObject(className);
-            if (found.found === false) {
+            if (found === undefined) {
                 throw new Error(`Class ${className.toUpperCase()} not found`);
             }
         }

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

@@ -20,8 +20,8 @@ class Raise {
         const className = classTok === null || classTok === void 0 ? void 0 : classTok.getStr();
         if (className) {
             const found = scope.existsObject(className);
-            if (found.found === true && found.id) {
-                scope.addReference(classTok, found.id, found.type, filename);
+            if (found === null || found === void 0 ? void 0 : found.id) {
+                scope.addReference(classTok, found.id, _reference_1.ReferenceType.ObjectOrientedReference, filename);
                 const def = scope.findObjectDefinition(className);
                 method = (_b = helper.searchMethodName(def, "CONSTRUCTOR")) === null || _b === void 0 ? void 0 : _b.method;
             }

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

@@ -84,13 +84,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/abap/types/basic/void_type.js

@@ -3,8 +3,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.VoidType = void 0;
 const _abstract_type_1 = require("./_abstract_type");
 class VoidType extends _abstract_type_1.AbstractType {
-    constructor(voided, name) {
-        super({ qualifiedName: name });
+    constructor(voided, qualifiedName) {
+        super({ qualifiedName: qualifiedName });
         this.voided = voided;
     }
     getVoided() {

package/build/src/lsp/help.js

@@ -19,13 +19,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) + ", " +

package/build/src/objects/index.js

@@ -99,6 +99,29 @@ __exportStar(require("./message_class"), exports);
 __exportStar(require("./messaging_channel"), exports);
 __exportStar(require("./mime_object"), exports);
 __exportStar(require("./namespace"), exports);
+__exportStar(require("./neptune_api"), exports);
+__exportStar(require("./neptune_app"), exports);
+__exportStar(require("./neptune_custom_color"), exports);
+__exportStar(require("./neptune_custom_js_helper"), exports);
+__exportStar(require("./neptune_documentation"), exports);
+__exportStar(require("./neptune_enhancement"), exports);
+__exportStar(require("./neptune_launchpad_layout"), exports);
+__exportStar(require("./neptune_launchpad"), exports);
+__exportStar(require("./neptune_login_page"), exports);
+__exportStar(require("./neptune_media_library"), exports);
+__exportStar(require("./neptune_media_pack"), exports);
+__exportStar(require("./neptune_metadata"), exports);
+__exportStar(require("./neptune_mobile_client"), exports);
+__exportStar(require("./neptune_odata"), exports);
+__exportStar(require("./neptune_policy"), exports);
+__exportStar(require("./neptune_rfc_mapping"), exports);
+__exportStar(require("./neptune_rich_text_template"), exports);
+__exportStar(require("./neptune_splash_screen"), exports);
+__exportStar(require("./neptune_sticky_banner"), exports);
+__exportStar(require("./neptune_tile_group"), exports);
+__exportStar(require("./neptune_tile_layout"), exports);
+__exportStar(require("./neptune_tile"), exports);
+__exportStar(require("./neptune_url_alias"), exports);
 __exportStar(require("./number_range"), exports);
 __exportStar(require("./oauth2_profile"), exports);
 __exportStar(require("./object_characteristic"), exports);
@@ -123,6 +146,7 @@ __exportStar(require("./service_definition"), exports);
 __exportStar(require("./shared_memory"), exports);
 __exportStar(require("./smart_form"), exports);
 __exportStar(require("./smart_style"), exports);
+__exportStar(require("./standard_task"), exports);
 __exportStar(require("./switch_assignment_hierarchy"), exports);
 __exportStar(require("./switch_assignments"), exports);
 __exportStar(require("./table_type"), exports);
@@ -140,27 +164,4 @@ __exportStar(require("./web_dynpro_application"), exports);
 __exportStar(require("./web_dynpro_component_configuration"), exports);
 __exportStar(require("./web_dynpro_component"), exports);
 __exportStar(require("./web_mime"), exports);
-__exportStar(require("./neptune_metadata"), exports);
-__exportStar(require("./neptune_app"), exports);
-__exportStar(require("./neptune_api"), exports);
-__exportStar(require("./neptune_launchpad"), exports);
-__exportStar(require("./neptune_tile_group"), exports);
-__exportStar(require("./neptune_tile"), exports);
-__exportStar(require("./neptune_policy"), exports);
-__exportStar(require("./neptune_launchpad_layout"), exports);
-__exportStar(require("./neptune_tile_layout"), exports);
-__exportStar(require("./neptune_enhancement"), exports);
-__exportStar(require("./neptune_login_page"), exports);
-__exportStar(require("./neptune_sticky_banner"), exports);
-__exportStar(require("./neptune_splash_screen"), exports);
-__exportStar(require("./neptune_custom_color"), exports);
-__exportStar(require("./neptune_rich_text_template"), exports);
-__exportStar(require("./neptune_custom_js_helper"), exports);
-__exportStar(require("./neptune_documentation"), exports);
-__exportStar(require("./neptune_rfc_mapping"), exports);
-__exportStar(require("./neptune_media_library"), exports);
-__exportStar(require("./neptune_media_pack"), exports);
-__exportStar(require("./neptune_url_alias"), exports);
-__exportStar(require("./neptune_odata"), exports);
-__exportStar(require("./neptune_mobile_client"), exports);
 //# sourceMappingURL=index.js.map

package/build/src/objects/neptune_api.js

@@ -8,7 +8,7 @@ class NeptuneAPI extends _abstract_object_1.AbstractObject {
     }
     getAllowedNaming() {
         return {
-            maxLength: 30,
+            maxLength: 100,
             allowNamespace: true,
         };
     }

package/build/src/objects/standard_task.js

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

package/build/src/registry.js

@@ -65,7 +65,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.105.3";
+        return "2.105.5";
     }
     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],
         };

package/build/src/rules/abapdoc.js

@@ -26,10 +26,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/align_parameters.js

@@ -22,49 +22,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/ambiguous_statement.js

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

package/build/src/rules/avoid_use.js

@@ -43,16 +43,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],
         };

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.`,
         };
     }