@abaplint/cli 2.105.15 → 2.105.17

package/build/cli.js

@@ -7196,7 +7196,7 @@ const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/co
 const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 class RaiseWith extends combi_1.Expression {
     getRunnable() {
-        const wit = (0, combi_1.seq)("WITH", _1.Source, (0, combi_1.opt)(_1.Source), (0, combi_1.opt)(_1.Source), (0, combi_1.opt)(_1.Source));
+        const wit = (0, combi_1.seq)("WITH", _1.SimpleSource3, (0, combi_1.opt)(_1.SimpleSource3), (0, combi_1.opt)(_1.SimpleSource3), (0, combi_1.opt)(_1.SimpleSource3));
         return wit;
     }
 }
@@ -26009,7 +26009,7 @@ class MethodSource {
             && first.get() instanceof Expressions.Dynamic
             && first instanceof nodes_1.ExpressionNode
             && ((_b = children[0]) === null || _b === void 0 ? void 0 : _b.concatTokens()) === "=>") {
-            const name = (_c = first.findDirectExpression(Expressions.Constant)) === null || _c === void 0 ? void 0 : _c.concatTokens().replace(/'/g, "");
+            const name = (_c = first.findDirectExpression(Expressions.Constant)) === null || _c === void 0 ? void 0 : _c.concatTokens().replace(/'/g, "").replace(/`/g, "");
             if (name !== undefined && scope.findClassDefinition(name) === undefined) {
                 throw new Error(`Class "${name}" not found/released`);
             }
@@ -26283,7 +26283,7 @@ const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ ".
 const source_1 = __webpack_require__(/*! ./source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
 class RaiseWith {
     runSyntax(node, scope, filename) {
-        for (const f of node.findDirectExpressions(Expressions.Source)) {
+        for (const f of node.findDirectExpressions(Expressions.SimpleSource3)) {
             new source_1.Source().runSyntax(f, scope, filename);
         }
     }
@@ -30959,13 +30959,33 @@ const source_1 = __webpack_require__(/*! ../expressions/source */ "./node_module
 const target_1 = __webpack_require__(/*! ../expressions/target */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/target.js");
 const fstarget_1 = __webpack_require__(/*! ../expressions/fstarget */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/fstarget.js");
 const component_cond_1 = __webpack_require__(/*! ../expressions/component_cond */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/component_cond.js");
+const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
 class ModifyInternal {
     runSyntax(node, scope, filename) {
         for (const s of node.findDirectExpressions(Expressions.Source)) {
             new source_1.Source().runSyntax(s, scope, filename);
         }
-        for (const t of node.findDirectExpressions(Expressions.Target)) {
-            new target_1.Target().runSyntax(t, scope, filename);
+        // there is only one
+        const targetExpression = node.findFirstExpression(Expressions.Target);
+        if (targetExpression) {
+            // it might be a dynamic target
+            const targetType = new target_1.Target().runSyntax(targetExpression, scope, filename);
+            if (targetType instanceof basic_1.VoidType
+                || targetType instanceof basic_1.AnyType
+                || targetType instanceof basic_1.UnknownType) {
+                // ok
+            }
+            else if (targetType instanceof basic_1.TableType) {
+                if (node.findDirectTokenByText("TABLE")
+                    && node.findDirectTokenByText("INDEX")
+                    && targetType.isWithHeader() === false) {
+                    // MODIFY TABLE INDEX
+                    throw new Error("Table does not have header line");
+                }
+            }
+            else {
+                throw new Error("Not an internal table");
+            }
         }
         const target = node.findDirectExpression(Expressions.FSTarget);
         if (target) {
@@ -31203,6 +31223,9 @@ class Parameter {
         if (nameToken && nameToken.getStr().length > 8) {
             throw new Error("Parameter name too long, " + nameToken.getStr());
         }
+        if (node.findDirectTokenByText("RADIOBUTTON") && node.findDirectTokenByText("LENGTH")) {
+            throw new Error("RADIOBUTTON and LENGTH not possible together");
+        }
         const bfound = new basic_types_1.BasicTypes(filename, scope).parseType(node);
         if (nameToken && bfound) {
             scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(nameToken, filename, bfound));
@@ -34146,13 +34169,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) {
@@ -34519,6 +34542,32 @@ class ExpressionNode extends _abstract_node_1.AbstractNode {
         }
         throw new Error("ExpressionNode, getFirstToken, no children");
     }
+    concatTokensWithLinebreaks() {
+        let str = "";
+        let prev;
+        for (const token of this.getTokens()) {
+            if (token instanceof tokens_1.Pragma) {
+                continue;
+            }
+            if (str === "") {
+                str = token.getStr();
+            }
+            else if (prev && prev.getStr().length + prev.getCol() === token.getCol()
+                && prev.getRow() === token.getRow()) {
+                str = str + token.getStr();
+            }
+            else {
+                if (prev && prev.getRow() !== token.getRow()) {
+                    str = str + "\n" + token.getStr();
+                }
+                else {
+                    str = str + " " + token.getStr();
+                }
+            }
+            prev = token;
+        }
+        return str;
+    }
     concatTokens() {
         let str = "";
         let prev;
@@ -39747,7 +39796,12 @@ class Config {
         return this.config;
     }
     readByKey(rule, key) {
-        return this.config["rules"][rule] ? this.config["rules"][rule][key] : undefined;
+        if (this.config["rules"]) {
+            return this.config["rules"][rule] ? this.config["rules"][rule][key] : undefined;
+        }
+        else {
+            return undefined;
+        }
     }
     readByRule(rule) {
         return this.config["rules"][rule];
@@ -42159,13 +42213,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) + ", " +
@@ -51162,7 +51216,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.105.15";
+        return "2.105.17";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -51478,10 +51532,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],
         };
@@ -51685,10 +51739,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],
         };
@@ -51826,49 +51880,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 ) ).`,
         };
     }
@@ -52344,15 +52398,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.`,
         };
     }
@@ -52422,6 +52476,7 @@ const expressions_1 = __webpack_require__(/*! ../abap/2_statements/expressions *
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
 const version_1 = __webpack_require__(/*! ../version */ "./node_modules/@abaplint/core/build/src/version.js");
 const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
+const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
 class AvoidUseConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
@@ -52431,11 +52486,11 @@ class AvoidUseConf extends _basic_rule_config_1.BasicRuleConfig {
         this.define = true;
         /** Detects statics */
         this.statics = true;
-        /** Detects DEFAULT KEY definitions, from version v740sp02 and up */
+        /** Detects DEFAULT KEY definitions, from version v740sp02 and up. Use pseudo comment DEFAULT_KEY to ignore */
         this.defaultKey = true;
         /** Detects BREAK and BREAK-POINTS */
         this.break = true;
-        /** Detects TEST SEAMS */
+        /** Detects TEST SEAMS. Use pseudo comment TEST_SEAM_USAGE to ignore */
         this.testSeams = true;
         /** Detects DESCRIBE TABLE LINES, use lines() instead */
         this.describeLines = true;
@@ -52456,16 +52511,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],
         };
@@ -52483,7 +52538,9 @@ BREAK points`,
         var _a;
         const issues = [];
         let isStaticsBlock = false;
-        for (const statementNode of file.getStatements()) {
+        const statements = file.getStatements();
+        for (let i = 0; i < statements.length; i++) {
+            const statementNode = statements[i];
             const statement = statementNode.get();
             let message = undefined;
             let fix = undefined;
@@ -52511,6 +52568,10 @@ BREAK points`,
                 message = "EXPORT TO DATABASE";
             }
             else if (this.conf.testSeams && statement instanceof Statements.TestSeam) {
+                const next = statements[i + 1];
+                if ((next === null || next === void 0 ? void 0 : next.get()) instanceof _statement_1.Comment && next.concatTokens().includes("EC TEST_SEAM_USAGE")) {
+                    continue;
+                }
                 message = "TEST-SEAM";
             }
             else if (this.conf.statics && statement instanceof Statements.Static && isStaticsBlock === false) {
@@ -52530,6 +52591,10 @@ BREAK points`,
                 const tt = (_a = statementNode.findFirstExpression(expressions_1.TypeTable)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(expressions_1.TypeTableKey);
                 const token = tt === null || tt === void 0 ? void 0 : tt.findDirectTokenByText("DEFAULT");
                 if (tt && token) {
+                    const next = statements[i + 1];
+                    if ((next === null || next === void 0 ? void 0 : next.get()) instanceof _statement_1.Comment && next.concatTokens().includes("EC DEFAULT_KEY")) {
+                        continue;
+                    }
                     message = "DEFAULT KEY";
                     issues.push(issue_1.Issue.atToken(file, token, this.getDescription(message), this.getMetadata().key, this.conf.severity));
                 }
@@ -52586,11 +52651,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.`,
         };
     }
@@ -52683,19 +52748,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.`,
         };
     }
@@ -52785,9 +52850,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.`,
         };
     }
@@ -52852,10 +52917,10 @@ class CDSCommentStyle {
             key: "cds_comment_style",
             title: "CDS Comment Style",
             shortDescription: `Check for obsolete comment style`,
-            extendedInformation: `Check for obsolete comment style
-
-Comments starting with "--" are considered obsolete
-
+            extendedInformation: `Check for obsolete comment style
+
+Comments starting with "--" are considered obsolete
+
 https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abencds_general_syntax_rules.htm`,
             tags: [_irule_1.RuleTag.SingleFile],
             badExample: "-- this is a comment",
@@ -52922,10 +52987,10 @@ class CDSLegacyView {
             title: "CDS Legacy View",
             shortDescription: `Identify CDS Legacy Views`,
             // eslint-disable-next-line max-len
-            extendedInformation: `Use DEFINE VIEW ENTITY instead of DEFINE VIEW
-
-https://blogs.sap.com/2021/10/16/a-new-generation-of-cds-views-how-to-migrate-your-cds-views-to-cds-view-entities/
-
+            extendedInformation: `Use DEFINE VIEW ENTITY instead of DEFINE VIEW
+
+https://blogs.sap.com/2021/10/16/a-new-generation-of-cds-views-how-to-migrate-your-cds-views-to-cds-view-entities/
+
 v755 and up`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Upport],
         };
@@ -53080,10 +53145,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.`,
@@ -53259,17 +53324,17 @@ class ChangeIfToCase extends _abap_rule_1.ABAPRule {
             title: "Change IF to CASE",
             shortDescription: `Finds IF constructs that can be changed to CASE`,
             // eslint-disable-next-line max-len
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-case-to-else-if-for-multiple-alternative-conditions
-
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-case-to-else-if-for-multiple-alternative-conditions
+
 If the first comparison is a boolean compare, no issue is reported.`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
-            badExample: `IF l_fcat-fieldname EQ 'FOO'.
-ELSEIF l_fcat-fieldname = 'BAR'
-    OR l_fcat-fieldname = 'MOO'.
+            badExample: `IF l_fcat-fieldname EQ 'FOO'.
+ELSEIF l_fcat-fieldname = 'BAR'
+    OR l_fcat-fieldname = 'MOO'.
 ENDIF.`,
-            goodExample: `CASE l_fcat-fieldname.
-  WHEN 'FOO'.
-  WHEN 'BAR' OR 'MOO'.
+            goodExample: `CASE l_fcat-fieldname.
+  WHEN 'FOO'.
+  WHEN 'BAR' OR 'MOO'.
 ENDCASE.`,
         };
     }
@@ -53406,8 +53471,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],
@@ -53488,11 +53553,11 @@ class CheckComments extends _abap_rule_1.ABAPRule {
         return {
             key: "check_comments",
             title: "Check Comments",
-            shortDescription: `
+            shortDescription: `
 Various checks for comment usage.`,
-            extendedInformation: `
-Detects end of line comments. Comments starting with "#EC" or "##" are ignored
-
+            extendedInformation: `
+Detects end of line comments. Comments starting with "#EC" or "##" are ignored
+
 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],
             badExample: `WRITE 2. " descriptive comment`,
@@ -53654,9 +53719,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],
         };
@@ -53731,14 +53796,14 @@ class CheckSubrc extends _abap_rule_1.ABAPRule {
             key: "check_subrc",
             title: "Check sy-subrc",
             shortDescription: `Check sy-subrc`,
-            extendedInformation: `Pseudo comment "#EC CI_SUBRC can be added to suppress findings
-
-If sy-dbcnt is checked after database statements, it is considered okay.
-
-"SELECT SINGLE @abap_true FROM " is considered as an existence check, also "SELECT COUNT( * )" is considered okay
-
-If IS ASSIGNED is checked after assigning, it is considered okay.
-
+            extendedInformation: `Pseudo comment "#EC CI_SUBRC can be added to suppress findings
+
+If sy-dbcnt is checked after database statements, it is considered okay.
+
+"SELECT SINGLE @abap_true FROM " is considered as an existence check, also "SELECT COUNT( * )" is considered okay
+
+If IS ASSIGNED is checked after assigning, it is considered okay.
+
 FIND statement with MATCH COUNT is considered okay if subrc is not checked`,
             tags: [_irule_1.RuleTag.SingleFile],
             pseudoComment: "EC CI_SUBRC",
@@ -54291,17 +54356,17 @@ class ClassicExceptionsOverlap extends _abap_rule_1.ABAPRule {
             shortDescription: `Find overlapping classic exceptions`,
             extendedInformation: `When debugging its typically good to know exactly which exception is caught`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `CALL FUNCTION 'SOMETHING'
-  EXCEPTIONS
-    system_failure        = 1 MESSAGE lv_message
-    communication_failure = 1 MESSAGE lv_message
-    resource_failure      = 1
+            badExample: `CALL FUNCTION 'SOMETHING'
+  EXCEPTIONS
+    system_failure        = 1 MESSAGE lv_message
+    communication_failure = 1 MESSAGE lv_message
+    resource_failure      = 1
     OTHERS                = 1.`,
-            goodExample: `CALL FUNCTION 'SOMETHING'
-  EXCEPTIONS
-    system_failure        = 1 MESSAGE lv_message
-    communication_failure = 2 MESSAGE lv_message
-    resource_failure      = 3
+            goodExample: `CALL FUNCTION 'SOMETHING'
+  EXCEPTIONS
+    system_failure        = 1 MESSAGE lv_message
+    communication_failure = 2 MESSAGE lv_message
+    resource_failure      = 3
     OTHERS                = 4.`,
         };
     }
@@ -54547,7 +54612,7 @@ class CommentedCode extends _abap_rule_1.ABAPRule {
             key: "commented_code",
             title: "Find commented code",
             shortDescription: `Detects usage of commented out code.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#delete-code-instead-of-commenting-it
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#delete-code-instead-of-commenting-it
 https://docs.abapopenchecks.org/checks/14/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
         };
@@ -54779,10 +54844,10 @@ class ConstructorVisibilityPublic {
             key: "constructor_visibility_public",
             title: "Check constructor visibility is public",
             shortDescription: `Constructor must be placed in the public section, even if the class is not CREATE PUBLIC.`,
-            extendedInformation: `
-This only applies to global classes.
-
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#if-your-global-class-is-create-private-leave-the-constructor-public
+            extendedInformation: `
+This only applies to global classes.
+
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#if-your-global-class-is-create-private-leave-the-constructor-public
 https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abeninstance_constructor_guidl.htm`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };
@@ -54857,8 +54922,8 @@ class ContainsTab extends _abap_rule_1.ABAPRule {
             key: "contains_tab",
             title: "Code contains tab",
             shortDescription: `Checks for usage of tabs (enable to enforce spaces)`,
-            extendedInformation: `
-https://docs.abapopenchecks.org/checks/09/
+            extendedInformation: `
+https://docs.abapopenchecks.org/checks/09/
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#indent-and-snap-to-tab`,
             tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };
@@ -54943,10 +55008,10 @@ class CyclicOO {
             key: "cyclic_oo",
             title: "Cyclic OO",
             shortDescription: `Finds cyclic OO references`,
-            extendedInformation: `Runs for global INTF + CLAS objects
-
-Objects must be without syntax errors for this rule to take effect
-
+            extendedInformation: `Runs for global INTF + CLAS objects
+
+Objects must be without syntax errors for this rule to take effect
+
 References in testclass includes are ignored`,
         };
     }
@@ -55188,7 +55253,7 @@ class DangerousStatement extends _abap_rule_1.ABAPRule {
             key: "dangerous_statement",
             title: "Dangerous statement",
             shortDescription: `Detects potentially dangerous statements`,
-            extendedInformation: `Dynamic SQL: Typically ABAP logic does not need dynamic SQL,
+            extendedInformation: `Dynamic SQL: Typically ABAP logic does not need dynamic SQL,
 dynamic SQL can potentially create SQL injection problems`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Security],
         };
@@ -55392,13 +55457,13 @@ class DefinitionsTop extends _abap_rule_1.ABAPRule {
             shortDescription: `Checks that definitions are placed at the beginning of METHODs, FORMs and FUNCTIONs.`,
             extendedInformation: `https://docs.abapopenchecks.org/checks/17/`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
-            badExample: `FROM foo.
-  WRITE 'hello'.
-  DATA int TYPE i.
+            badExample: `FROM foo.
+  WRITE 'hello'.
+  DATA int TYPE i.
 ENDFORM.`,
-            goodExample: `FROM foo.
-  DATA int TYPE i.
-  WRITE 'hello'.
+            goodExample: `FROM foo.
+  DATA int TYPE i.
+  WRITE 'hello'.
 ENDFORM.`,
         };
     }
@@ -55934,39 +55999,39 @@ class Downport {
             key: "downport",
             title: "Downport statement",
             shortDescription: `Downport functionality`,
-            extendedInformation: `Much like the 'commented_code' rule this rule loops through unknown statements and tries parsing with
-a higher level language version. If successful, various rules are applied to downport the statement.
-Target downport version is always v702, thus rule is only enabled if target version is v702.
-
-Current rules:
-* NEW transformed to CREATE OBJECT, opposite of https://rules.abaplint.org/use_new/
-* DATA() definitions are outlined, opposite of https://rules.abaplint.org/prefer_inline/
-* FIELD-SYMBOL() definitions are outlined
-* CONV is outlined
-* COND is outlined
-* REDUCE is outlined
-* SWITCH is outlined
-* FILTER is outlined
-* APPEND expression is outlined
-* INSERT expression is outlined
-* EMPTY KEY is changed to DEFAULT KEY, opposite of DEFAULT KEY in https://rules.abaplint.org/avoid_use/
-* CAST changed to ?=
-* LOOP AT method_call( ) is outlined
-* VALUE # with structure fields
-* VALUE # with internal table lines
-* Table Expressions are outlined
-* SELECT INTO @DATA definitions are outlined
-* Some occurrences of string template formatting option ALPHA changed to function module call
-* SELECT/INSERT/MODIFY/DELETE/UPDATE "," in field list removed, "@" in source/targets removed
-* PARTIALLY IMPLEMENTED removed, it can be quick fixed via rule implement_methods
-* RAISE EXCEPTION ... MESSAGE
-* Moving with +=, -=, /=, *=, &&= is expanded
-* line_exists and line_index is downported to READ TABLE
-* ENUMs, but does not nessesarily give the correct type and value
-* MESSAGE with non simple source
-
-Only one transformation is applied to a statement at a time, so multiple steps might be required to do the full downport.
-
+            extendedInformation: `Much like the 'commented_code' rule this rule loops through unknown statements and tries parsing with
+a higher level language version. If successful, various rules are applied to downport the statement.
+Target downport version is always v702, thus rule is only enabled if target version is v702.
+
+Current rules:
+* NEW transformed to CREATE OBJECT, opposite of https://rules.abaplint.org/use_new/
+* DATA() definitions are outlined, opposite of https://rules.abaplint.org/prefer_inline/
+* FIELD-SYMBOL() definitions are outlined
+* CONV is outlined
+* COND is outlined
+* REDUCE is outlined
+* SWITCH is outlined
+* FILTER is outlined
+* APPEND expression is outlined
+* INSERT expression is outlined
+* EMPTY KEY is changed to DEFAULT KEY, opposite of DEFAULT KEY in https://rules.abaplint.org/avoid_use/
+* CAST changed to ?=
+* LOOP AT method_call( ) is outlined
+* VALUE # with structure fields
+* VALUE # with internal table lines
+* Table Expressions are outlined
+* SELECT INTO @DATA definitions are outlined
+* Some occurrences of string template formatting option ALPHA changed to function module call
+* SELECT/INSERT/MODIFY/DELETE/UPDATE "," in field list removed, "@" in source/targets removed
+* PARTIALLY IMPLEMENTED removed, it can be quick fixed via rule implement_methods
+* RAISE EXCEPTION ... MESSAGE
+* Moving with +=, -=, /=, *=, &&= is expanded
+* line_exists and line_index is downported to READ TABLE
+* ENUMs, but does not nessesarily give the correct type and value
+* MESSAGE with non simple source
+
+Only one transformation is applied to a statement at a time, so multiple steps might be required to do the full downport.
+
 Make sure to test the downported code, it might not always be completely correct.`,
             tags: [_irule_1.RuleTag.Downport, _irule_1.RuleTag.Quickfix],
         };
@@ -56540,10 +56605,10 @@ Make sure to test the downported code, it might not always be completely correct
                 const fieldName = f.concatTokens();
                 fieldDefinition += indentation + "        " + fieldName + " TYPE " + tableName + "-" + fieldName + ",\n";
             }
-            fieldDefinition = `DATA: BEGIN OF ${name},
+            fieldDefinition = `DATA: BEGIN OF ${name},
 ${fieldDefinition}${indentation}      END OF ${name}.`;
         }
-        const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, high.getStart(), `${fieldDefinition}
+        const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, high.getStart(), `${fieldDefinition}
 ${indentation}`);
         const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, inlineData.getFirstToken().getStart(), inlineData.getLastToken().getEnd(), name);
         const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
@@ -56587,12 +56652,12 @@ ${indentation}`);
         }
         const uniqueName = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
         const name = ((_c = inlineData.findFirstExpression(Expressions.TargetField)) === null || _c === void 0 ? void 0 : _c.concatTokens()) || "error";
-        let fix1 = edit_helper_1.EditHelper.insertAt(lowFile, high.getStart(), `TYPES: BEGIN OF ${uniqueName},
-${fieldDefinitions}${indentation}      END OF ${uniqueName}.
-${indentation}DATA ${name} TYPE STANDARD TABLE OF ${uniqueName} WITH DEFAULT KEY.
+        let fix1 = edit_helper_1.EditHelper.insertAt(lowFile, high.getStart(), `TYPES: BEGIN OF ${uniqueName},
+${fieldDefinitions}${indentation}      END OF ${uniqueName}.
+${indentation}DATA ${name} TYPE STANDARD TABLE OF ${uniqueName} WITH DEFAULT KEY.
 ${indentation}`);
         if (fieldDefinitions === "") {
-            fix1 = edit_helper_1.EditHelper.insertAt(lowFile, high.getStart(), `DATA ${name} TYPE STANDARD TABLE OF ${tableName} WITH DEFAULT KEY.
+            fix1 = edit_helper_1.EditHelper.insertAt(lowFile, high.getStart(), `DATA ${name} TYPE STANDARD TABLE OF ${tableName} WITH DEFAULT KEY.
 ${indentation}`);
         }
         const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, inlineData.getFirstToken().getStart(), inlineData.getLastToken().getEnd(), name);
@@ -56660,7 +56725,7 @@ ${indentation}`);
             const uniqueName = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
             const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
             const firstToken = high.getFirstToken();
-            const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, firstToken.getStart(), `DATA ${uniqueName} LIKE LINE OF ${target === null || target === void 0 ? void 0 : target.concatTokens()}.
+            const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, firstToken.getStart(), `DATA ${uniqueName} LIKE LINE OF ${target === null || target === void 0 ? void 0 : target.concatTokens()}.
 ${indentation}${uniqueName} = ${source.concatTokens()}.\n${indentation}`);
             const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, source.getFirstToken().getStart(), source.getLastToken().getEnd(), uniqueName);
             const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
@@ -56698,7 +56763,7 @@ ${indentation}${uniqueName} = ${source.concatTokens()}.\n${indentation}`);
             const uniqueName = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
             const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
             const firstToken = high.getFirstToken();
-            const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, firstToken.getStart(), `DATA ${uniqueName} LIKE LINE OF ${target === null || target === void 0 ? void 0 : target.concatTokens()}.
+            const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, firstToken.getStart(), `DATA ${uniqueName} LIKE LINE OF ${target === null || target === void 0 ? void 0 : target.concatTokens()}.
 ${indentation}${uniqueName} = ${source.concatTokens()}.\n${indentation}`);
             const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, source.getFirstToken().getStart(), source.getLastToken().getEnd(), uniqueName);
             const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
@@ -56740,14 +56805,14 @@ ${indentation}${uniqueName} = ${source.concatTokens()}.\n${indentation}`);
             const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
             const firstToken = high.getFirstToken();
             // note that the tabix restore should be done before throwing the exception
-            const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, firstToken.getStart(), `DATA ${uniqueName} LIKE LINE OF ${pre}.
-${indentation}DATA ${tabixBackup} LIKE sy-tabix.
-${indentation}${tabixBackup} = sy-tabix.
-${indentation}READ TABLE ${pre} ${condition}INTO ${uniqueName}.
-${indentation}sy-tabix = ${tabixBackup}.
-${indentation}IF sy-subrc <> 0.
-${indentation}  RAISE EXCEPTION TYPE cx_sy_itab_line_not_found.
-${indentation}ENDIF.
+            const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, firstToken.getStart(), `DATA ${uniqueName} LIKE LINE OF ${pre}.
+${indentation}DATA ${tabixBackup} LIKE sy-tabix.
+${indentation}${tabixBackup} = sy-tabix.
+${indentation}READ TABLE ${pre} ${condition}INTO ${uniqueName}.
+${indentation}sy-tabix = ${tabixBackup}.
+${indentation}IF sy-subrc <> 0.
+${indentation}  RAISE EXCEPTION TYPE cx_sy_itab_line_not_found.
+${indentation}ENDIF.
 ${indentation}`);
             const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, startToken.getStart(), tableExpression.getLastToken().getEnd(), uniqueName);
             const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
@@ -56804,7 +56869,7 @@ ${indentation}`);
         const className = classNames[0].concatTokens();
         const targetName = (_b = target.findFirstExpression(Expressions.TargetField)) === null || _b === void 0 ? void 0 : _b.concatTokens();
         const indentation = " ".repeat(node.getFirstToken().getStart().getCol() - 1);
-        const code = `  DATA ${targetName} TYPE REF TO ${className}.
+        const code = `  DATA ${targetName} TYPE REF TO ${className}.
 ${indentation}CATCH ${className} INTO ${targetName}.`;
         const fix = edit_helper_1.EditHelper.replaceRange(lowFile, node.getStart(), node.getEnd(), code);
         return issue_1.Issue.atToken(lowFile, node.getFirstToken(), "Outline DATA", this.getMetadata().key, this.conf.severity, fix);
@@ -56961,21 +57026,21 @@ ${indentation}CATCH ${className} INTO ${targetName}.`;
             }
             startToken = node.getFirstToken();
         }
-        const withs = ((_f = node.findDirectExpression(Expressions.RaiseWith)) === null || _f === void 0 ? void 0 : _f.findDirectExpressions(Expressions.Source)) || [];
+        const withs = ((_f = node.findDirectExpression(Expressions.RaiseWith)) === null || _f === void 0 ? void 0 : _f.findDirectExpressions(Expressions.SimpleSource3)) || [];
         const className = ((_g = node.findDirectExpression(Expressions.ClassName)) === null || _g === void 0 ? void 0 : _g.concatTokens()) || "ERROR";
         const uniqueName1 = this.uniqueName(node.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
         const uniqueName2 = this.uniqueName(node.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
         const indentation = " ".repeat(node.getFirstToken().getStart().getCol() - 1);
-        let abap = `DATA ${uniqueName1} LIKE if_t100_message=>t100key.
-${indentation}${uniqueName1}-msgid = ${id}.
+        let abap = `DATA ${uniqueName1} LIKE if_t100_message=>t100key.
+${indentation}${uniqueName1}-msgid = ${id}.
 ${indentation}${uniqueName1}-msgno = ${number}.\n`;
         if (withs.length > 0) {
-            abap += `${indentation}${uniqueName1}-attr1 = 'IF_T100_DYN_MSG~MSGV1'.
-${indentation}${uniqueName1}-attr2 = 'IF_T100_DYN_MSG~MSGV2'.
-${indentation}${uniqueName1}-attr3 = 'IF_T100_DYN_MSG~MSGV3'.
+            abap += `${indentation}${uniqueName1}-attr1 = 'IF_T100_DYN_MSG~MSGV1'.
+${indentation}${uniqueName1}-attr2 = 'IF_T100_DYN_MSG~MSGV2'.
+${indentation}${uniqueName1}-attr3 = 'IF_T100_DYN_MSG~MSGV3'.
 ${indentation}${uniqueName1}-attr4 = 'IF_T100_DYN_MSG~MSGV4'.\n`;
         }
-        abap += `${indentation}DATA ${uniqueName2} TYPE REF TO ${className}.
+        abap += `${indentation}DATA ${uniqueName2} TYPE REF TO ${className}.
 ${indentation}CREATE OBJECT ${uniqueName2} EXPORTING textid = ${uniqueName1}.\n`;
         if (withs.length > 0) {
             abap += `${indentation}${uniqueName2}->if_t100_dyn_msg~msgty = 'E'.\n`;
@@ -57087,10 +57152,10 @@ ${indentation}CREATE OBJECT ${uniqueName2} EXPORTING textid = ${uniqueName1}.\n`
         let code = "";
         if (sourceRef.findFirstExpression(Expressions.TableExpression)) {
             const uniqueName = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
-            code = `ASSIGN ${sourceRef.concatTokens()} TO FIELD-SYMBOL(<${uniqueName}>).
-IF sy-subrc <> 0.
-  RAISE EXCEPTION TYPE cx_sy_itab_line_not_found.
-ENDIF.
+            code = `ASSIGN ${sourceRef.concatTokens()} TO FIELD-SYMBOL(<${uniqueName}>).
+IF sy-subrc <> 0.
+  RAISE EXCEPTION TYPE cx_sy_itab_line_not_found.
+ENDIF.
 GET REFERENCE OF <${uniqueName}> INTO ${target.concatTokens()}`;
         }
         else {
@@ -57166,20 +57231,20 @@ GET REFERENCE OF <${uniqueName}> INTO ${target.concatTokens()}`;
         const uniqueName = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
         const uniqueFS = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
         const uniqueNameIndex = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
-        code += `         items LIKE ${loopSourceName},
-       END OF ${groupTargetName}type.
-DATA ${groupTargetName}tab TYPE STANDARD TABLE OF ${groupTargetName}type WITH DEFAULT KEY.
-DATA ${uniqueName} LIKE LINE OF ${groupTargetName}tab.
+        code += `         items LIKE ${loopSourceName},
+       END OF ${groupTargetName}type.
+DATA ${groupTargetName}tab TYPE STANDARD TABLE OF ${groupTargetName}type WITH DEFAULT KEY.
+DATA ${uniqueName} LIKE LINE OF ${groupTargetName}tab.
 LOOP AT ${loopSourceName} ${(_l = high.findFirstExpression(Expressions.LoopTarget)) === null || _l === void 0 ? void 0 : _l.concatTokens()}.\n`;
         if (groupIndexName !== undefined) {
             code += `DATA(${uniqueNameIndex}) = sy-tabix.\n`;
         }
-        code += `READ TABLE ${groupTargetName}tab ASSIGNING FIELD-SYMBOL(<${uniqueFS}>) WITH KEY ${condition}.
+        code += `READ TABLE ${groupTargetName}tab ASSIGNING FIELD-SYMBOL(<${uniqueFS}>) WITH KEY ${condition}.
 IF sy-subrc = 0.\n`;
         if (groupCountName !== undefined) {
             code += `  <${uniqueFS}>-${groupCountName} = <${uniqueFS}>-${groupCountName} + 1.\n`;
         }
-        code += `  INSERT ${loopTargetName}${isReference ? "->*" : ""} INTO TABLE <${uniqueFS}>-items.
+        code += `  INSERT ${loopTargetName}${isReference ? "->*" : ""} INTO TABLE <${uniqueFS}>-items.
 ELSE.\n`;
         code += `  CLEAR ${uniqueName}.\n`;
         for (const c of group.findAllExpressions(Expressions.LoopGroupByComponent)) {
@@ -57200,8 +57265,8 @@ ELSE.\n`;
         }
         code += `  INSERT ${loopTargetName}${isReference ? "->*" : ""} INTO TABLE ${uniqueName}-items.\n`;
         code += `  INSERT ${uniqueName} INTO TABLE ${groupTargetName}tab.\n`;
-        code += `ENDIF.
-ENDLOOP.
+        code += `ENDIF.
+ENDLOOP.
 LOOP AT ${groupTargetName}tab ${groupTarget}.`;
         let fix = edit_helper_1.EditHelper.replaceRange(lowFile, high.getFirstToken().getStart(), high.getLastToken().getEnd(), code);
         for (const l of ((_m = highFile.getStructure()) === null || _m === void 0 ? void 0 : _m.findAllStructures(Structures.Loop)) || []) {
@@ -57349,7 +57414,7 @@ LOOP AT ${groupTargetName}tab ${groupTarget}.`;
         const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
         let code = `CLEAR ${target.concatTokens()}.\n`;
         for (const fieldAssignment of fieldAssignments) {
-            code += indentation + target.concatTokens() + "-" + fieldAssignment.concatTokens() + `.\n`;
+            code += indentation + target.concatTokens() + "-" + fieldAssignment.concatTokensWithLinebreaks() + `.\n`;
         }
         code = code.trimEnd();
         const start = high.getFirstToken().getStart();
@@ -57373,7 +57438,7 @@ LOOP AT ${groupTargetName}tab ${groupTarget}.`;
         const enumName = (_b = high.findExpressionAfterToken("ENUM")) === null || _b === void 0 ? void 0 : _b.concatTokens();
         const structureName = (_c = high.findExpressionAfterToken("STRUCTURE")) === null || _c === void 0 ? void 0 : _c.concatTokens();
         // all ENUMS are char like?
-        let code = `TYPES ${enumName} TYPE string.
+        let code = `TYPES ${enumName} TYPE string.
 CONSTANTS: BEGIN OF ${structureName},\n`;
         let count = 1;
         for (const e of enumStructure.findDirectStatements(Statements.TypeEnum).concat(enumStructure.findDirectStatements(Statements.Type))) {
@@ -57417,14 +57482,14 @@ CONSTANTS: BEGIN OF ${structureName},\n`;
         const tabixBackup = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
         const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
         // restore tabix before exeption
-        const code = `FIELD-SYMBOLS ${uniqueName} LIKE LINE OF ${tName}.
-${indentation}DATA ${tabixBackup} LIKE sy-tabix.
-${indentation}${tabixBackup} = sy-tabix.
-${indentation}READ TABLE ${tName} ${condition}ASSIGNING ${uniqueName}.
-${indentation}sy-tabix = ${tabixBackup}.
-${indentation}IF sy-subrc <> 0.
-${indentation}  RAISE EXCEPTION TYPE cx_sy_itab_line_not_found.
-${indentation}ENDIF.
+        const code = `FIELD-SYMBOLS ${uniqueName} LIKE LINE OF ${tName}.
+${indentation}DATA ${tabixBackup} LIKE sy-tabix.
+${indentation}${tabixBackup} = sy-tabix.
+${indentation}READ TABLE ${tName} ${condition}ASSIGNING ${uniqueName}.
+${indentation}sy-tabix = ${tabixBackup}.
+${indentation}IF sy-subrc <> 0.
+${indentation}  RAISE EXCEPTION TYPE cx_sy_itab_line_not_found.
+${indentation}ENDIF.
 ${indentation}${uniqueName}`;
         const start = target.getFirstToken().getStart();
         const end = (_a = tableExpression.findDirectTokenByText("]")) === null || _a === void 0 ? void 0 : _a.getEnd();
@@ -57508,11 +57573,11 @@ ${indentation}${uniqueName}`;
             const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
             const source = (_b = templateSource === null || templateSource === void 0 ? void 0 : templateSource.findDirectExpression(Expressions.Source)) === null || _b === void 0 ? void 0 : _b.concatTokens();
             const uniqueName = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
-            const code = `DATA ${uniqueName} TYPE string.
-${indentation}CALL FUNCTION '${functionName}'
-${indentation}  EXPORTING
-${indentation}    input  = ${source}
-${indentation}  IMPORTING
+            const code = `DATA ${uniqueName} TYPE string.
+${indentation}CALL FUNCTION '${functionName}'
+${indentation}  EXPORTING
+${indentation}    input  = ${source}
+${indentation}  IMPORTING
 ${indentation}    output = ${uniqueName}.\n`;
             const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, high.getFirstToken().getStart(), code);
             const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, child.getFirstToken().getStart(), child.getLastToken().getEnd(), uniqueName);
@@ -58824,12 +58889,12 @@ class EasyToFindMessages {
             key: "easy_to_find_messages",
             title: "Easy to find messages",
             shortDescription: `Make messages easy to find`,
-            extendedInformation: `All messages must be statically referenced exactly once
-
-Only MESSAGE and RAISE statments are counted as static references
-
-Also see rule "message_exists"
-
+            extendedInformation: `All messages must be statically referenced exactly once
+
+Only MESSAGE and RAISE statments are counted as static references
+
+Also see rule "message_exists"
+
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#make-messages-easy-to-find`,
             tags: [_irule_1.RuleTag.Styleguide],
         };
@@ -58914,8 +58979,8 @@ class EmptyLineinStatement extends _abap_rule_1.ABAPRule {
             key: "empty_line_in_statement",
             title: "Find empty lines in statements",
             shortDescription: `Checks that statements do not contain empty lines.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#dont-obsess-with-separating-blank-lines
-
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#dont-obsess-with-separating-blank-lines
+
 https://docs.abapopenchecks.org/checks/41/`,
             tags: [_irule_1.RuleTag.Quickfix, _irule_1.RuleTag.Whitespace, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
             badExample: `WRITE\n\nhello.`,
@@ -59207,10 +59272,10 @@ class ExitOrCheck extends _abap_rule_1.ABAPRule {
         return {
             key: "exit_or_check",
             title: "Find EXIT or CHECK outside loops",
-            shortDescription: `Detects usages of EXIT or CHECK statements outside of loops.
+            shortDescription: `Detects usages of EXIT or CHECK statements outside of loops.
 Use RETURN to leave procesing blocks instead.`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abenleave_processing_blocks.htm
-https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abapcheck_processing_blocks.htm
+            extendedInformation: `https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abenleave_processing_blocks.htm
+https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abapcheck_processing_blocks.htm
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#check-vs-return`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
         };
@@ -59293,12 +59358,12 @@ class ExpandMacros extends _abap_rule_1.ABAPRule {
             key: "expand_macros",
             title: "Expand Macros",
             shortDescription: `Allows expanding macro calls with quick fixes`,
-            extendedInformation: `Macros: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abenmacros_guidl.htm
-
+            extendedInformation: `Macros: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abenmacros_guidl.htm
+
 Note that macros/DEFINE cannot be used in the ABAP Cloud programming model`,
-            badExample: `DEFINE _hello.
-  WRITE 'hello'.
-END-OF-DEFINITION.
+            badExample: `DEFINE _hello.
+  WRITE 'hello'.
+END-OF-DEFINITION.
 _hello.`,
             goodExample: `WRITE 'hello'.`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.Upport],
@@ -59385,7 +59450,7 @@ class Exporting extends _abap_rule_1.ABAPRule {
             shortDescription: `Detects EXPORTING statements which can be omitted.`,
             badExample: `call_method( EXPORTING foo = bar ).`,
             goodExample: `call_method( foo = bar ).`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#omit-the-optional-keyword-exporting
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#omit-the-optional-keyword-exporting
 https://docs.abapopenchecks.org/checks/30/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
         };
@@ -59483,7 +59548,7 @@ class ForbiddenIdentifier extends _abap_rule_1.ABAPRule {
             key: "forbidden_identifier",
             title: "Forbidden Identifier",
             shortDescription: `Forbid use of specified identifiers, list of regex.`,
-            extendedInformation: `Used in the transpiler to find javascript keywords in ABAP identifiers,
+            extendedInformation: `Used in the transpiler to find javascript keywords in ABAP identifiers,
 https://github.com/abaplint/transpiler/blob/bda94b8b56e2b7f2f87be2168f12361aa530220e/packages/transpiler/src/validation.ts#L44`,
             tags: [_irule_1.RuleTag.SingleFile],
         };
@@ -59725,8 +59790,8 @@ class ForbiddenVoidType {
             key: "forbidden_void_type",
             title: "Forbidden Void Types",
             shortDescription: `Avoid usage of specified void types.`,
-            extendedInformation: `Inspiration:
-BOOLEAN, BOOLE_D, CHAR01, CHAR1, CHAR10, CHAR12, CHAR128, CHAR2, CHAR20, CHAR4, CHAR70,
+            extendedInformation: `Inspiration:
+BOOLEAN, BOOLE_D, CHAR01, CHAR1, CHAR10, CHAR12, CHAR128, CHAR2, CHAR20, CHAR4, CHAR70,
 DATS, TIMS, DATUM, FLAG, INT4, NUMC3, NUMC4, SAP_BOOL, TEXT25, TEXT80, X255, XFELD`,
         };
     }
@@ -59969,7 +60034,7 @@ class FullyTypeITabs extends _abap_rule_1.ABAPRule {
             key: "fully_type_itabs",
             title: "Fully type internal tables",
             shortDescription: `No implict table types or table keys`,
-            badExample: `DATA lt_foo TYPE TABLE OF ty.
+            badExample: `DATA lt_foo TYPE TABLE OF ty.
 DATA lt_bar TYPE STANDARD TABLE OF ty.`,
             goodExample: `DATA lt_foo TYPE STANDARD TABLE OF ty WITH EMPTY KEY.`,
             tags: [_irule_1.RuleTag.SingleFile],
@@ -60154,26 +60219,26 @@ class FunctionalWriting extends _abap_rule_1.ABAPRule {
             key: "functional_writing",
             title: "Use functional writing",
             shortDescription: `Detects usage of call method when functional style calls can be used.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-functional-to-procedural-calls
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-functional-to-procedural-calls
 https://docs.abapopenchecks.org/checks/07/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
-            badExample: `CALL METHOD zcl_class=>method( ).
-CALL METHOD cl_abap_typedescr=>describe_by_name
-  EXPORTING
-    p_name         = 'NAME'
-  RECEIVING
-    p_descr_ref    = lr_typedescr
-  EXCEPTIONS
-    type_not_found = 1
+            badExample: `CALL METHOD zcl_class=>method( ).
+CALL METHOD cl_abap_typedescr=>describe_by_name
+  EXPORTING
+    p_name         = 'NAME'
+  RECEIVING
+    p_descr_ref    = lr_typedescr
+  EXCEPTIONS
+    type_not_found = 1
     OTHERS         = 2.`,
-            goodExample: `zcl_class=>method( ).
-cl_abap_typedescr=>describe_by_name(
-  EXPORTING
-    p_name         = 'NAME'
-  RECEIVING
-    p_descr_ref    = lr_typedescr
-  EXCEPTIONS
-    type_not_found = 1
+            goodExample: `zcl_class=>method( ).
+cl_abap_typedescr=>describe_by_name(
+  EXPORTING
+    p_name         = 'NAME'
+  RECEIVING
+    p_descr_ref    = lr_typedescr
+  EXCEPTIONS
+    type_not_found = 1
     OTHERS         = 2 ).`,
         };
     }
@@ -60284,14 +60349,14 @@ class GlobalClass extends _abap_rule_1.ABAPRule {
             key: "global_class",
             title: "Global class checks",
             shortDescription: `Checks related to global classes`,
-            extendedInformation: `* global classes must be in own files
-
-* file names must match class name
-
-* file names must match interface name
-
-* global classes must be global definitions
-
+            extendedInformation: `* global classes must be in own files
+
+* file names must match class name
+
+* file names must match interface name
+
+* global classes must be global definitions
+
 * global interfaces must be global definitions`,
             tags: [_irule_1.RuleTag.Syntax],
         };
@@ -60390,8 +60455,8 @@ class IdenticalConditions extends _abap_rule_1.ABAPRule {
         return {
             key: "identical_conditions",
             title: "Identical conditions",
-            shortDescription: `Find identical conditions in IF + CASE + WHILE etc
-
+            shortDescription: `Find identical conditions in IF + CASE + WHILE etc
+
 Prerequsites: code is pretty printed with identical cAsE`,
             tags: [_irule_1.RuleTag.SingleFile],
         };
@@ -60525,23 +60590,23 @@ class IdenticalContents extends _abap_rule_1.ABAPRule {
             key: "identical_contents",
             title: "Identical contents",
             shortDescription: `Find identical contents in blocks inside IFs, both in the beginning and in the end.`,
-            extendedInformation: `
-Prerequsites: code is pretty printed with identical cAsE
-
+            extendedInformation: `
+Prerequsites: code is pretty printed with identical cAsE
+
 Chained statments are ignored`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `IF foo = bar.
-  WRITE 'bar'.
-  WRITE 'world'.
-ELSE.
-  WRITE 'foo'.
-  WRITE 'world'.
+            badExample: `IF foo = bar.
+  WRITE 'bar'.
+  WRITE 'world'.
+ELSE.
+  WRITE 'foo'.
+  WRITE 'world'.
 ENDIF.`,
-            goodExample: `IF foo = bar.
-  WRITE 'bar'.
-ELSE.
-  WRITE 'foo'.
-ENDIF.
+            goodExample: `IF foo = bar.
+  WRITE 'bar'.
+ELSE.
+  WRITE 'foo'.
+ENDIF.
 WRITE 'world'.`,
         };
     }
@@ -60644,12 +60709,12 @@ class IdenticalDescriptions {
             key: "identical_descriptions",
             title: "Identical descriptions",
             shortDescription: `Searches for objects with the same type and same description`,
-            extendedInformation: `Case insensitive
-
-Only checks the master language descriptions
-
-Dependencies are skipped
-
+            extendedInformation: `Case insensitive
+
+Only checks the master language descriptions
+
+Dependencies are skipped
+
 Works for: INTF, CLAS, DOMA, DTEL, FUNC in same FUGR`,
             tags: [],
         };
@@ -60823,43 +60888,43 @@ class IfInIf extends _abap_rule_1.ABAPRule {
             key: "if_in_if",
             title: "IF in IF",
             shortDescription: `Detects nested ifs which can be refactored.`,
-            extendedInformation: `
-Directly nested IFs without ELSE can be refactored to a single condition using AND.
-
-ELSE condtions with directly nested IF refactored to ELSEIF, quickfixes are suggested for this case.
-
-https://docs.abapopenchecks.org/checks/01/
+            extendedInformation: `
+Directly nested IFs without ELSE can be refactored to a single condition using AND.
+
+ELSE condtions with directly nested IF refactored to ELSEIF, quickfixes are suggested for this case.
+
+https://docs.abapopenchecks.org/checks/01/
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#keep-the-nesting-depth-low`,
-            badExample: `IF condition1.
-  IF condition2.
-    ...
-  ENDIF.
-ENDIF.
-
-IF condition1.
-  ...
-ELSE.
-  IF condition2.
-    ...
-  ENDIF.
+            badExample: `IF condition1.
+  IF condition2.
+    ...
+  ENDIF.
+ENDIF.
+
+IF condition1.
+  ...
+ELSE.
+  IF condition2.
+    ...
+  ENDIF.
 ENDIF.`,
-            goodExample: `IF ( condition1 ) AND ( condition2 ).
-  ...
-ENDIF.
-
-IF condition1.
-  ...
-ELSEIF condition2.
-  ...
-ENDIF.
-
-CASE variable.
-  WHEN value1.
-  ...
-  WHEN value2.
-    IF condition2.
-      ...
-    ENDIF.
+            goodExample: `IF ( condition1 ) AND ( condition2 ).
+  ...
+ENDIF.
+
+IF condition1.
+  ...
+ELSEIF condition2.
+  ...
+ENDIF.
+
+CASE variable.
+  WHEN value1.
+  ...
+  WHEN value2.
+    IF condition2.
+      ...
+    ENDIF.
 ENDCASE.`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
         };
@@ -61044,9 +61109,9 @@ class ImplementMethods extends _abap_rule_1.ABAPRule {
         for (const i of ((_a = file.getStructure()) === null || _a === void 0 ? void 0 : _a.findAllStatements(Statements.ClassImplementation)) || []) {
             const name = (_b = i.findFirstExpression(Expressions.ClassName)) === null || _b === void 0 ? void 0 : _b.getFirstToken().getStr().toUpperCase();
             if (name === impl.identifier.getName().toUpperCase()) {
-                return edit_helper_1.EditHelper.insertAt(file, i.getLastToken().getEnd(), `
-  METHOD ${methodName.toLowerCase()}.
-    RETURN. " todo, implement method
+                return edit_helper_1.EditHelper.insertAt(file, i.getLastToken().getEnd(), `
+  METHOD ${methodName.toLowerCase()}.
+    RETURN. " todo, implement method
   ENDMETHOD.`);
             }
         }
@@ -61234,19 +61299,19 @@ class InStatementIndentation extends _abap_rule_1.ABAPRule {
             key: "in_statement_indentation",
             title: "In-statement indentation",
             shortDescription: "Checks alignment within statements which span multiple lines.",
-            extendedInformation: `Lines following the first line should be indented once (2 spaces).
-      
-For block declaration statements, lines after the first should be indented an additional time (default: +2 spaces)
+            extendedInformation: `Lines following the first line should be indented once (2 spaces).
+      
+For block declaration statements, lines after the first should be indented an additional time (default: +2 spaces)
 to distinguish them better from code within the block.`,
-            badExample: `IF 1 = 1
-  AND 2 = 2.
-  WRITE 'hello' &&
-  'world'.
+            badExample: `IF 1 = 1
+  AND 2 = 2.
+  WRITE 'hello' &&
+  'world'.
 ENDIF.`,
-            goodExample: `IF 1 = 1
-    AND 2 = 2.
-  WRITE 'hello' &&
-    'world'.
+            goodExample: `IF 1 = 1
+    AND 2 = 2.
+  WRITE 'hello' &&
+    'world'.
 ENDIF.`,
             tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
         };
@@ -61751,9 +61816,9 @@ class IntfReferencingClas {
             key: "intf_referencing_clas",
             title: "INTF referencing CLAS",
             shortDescription: `Interface contains references to class`,
-            extendedInformation: `Only global interfaces are checked.
-      Only first level references are checked.
-      Exception class references are ignored.
+            extendedInformation: `Only global interfaces are checked.
+      Only first level references are checked.
+      Exception class references are ignored.
       Void references are ignored.`,
         };
     }
@@ -62348,8 +62413,8 @@ class LineBreakStyle {
         return {
             key: "line_break_style",
             title: "Makes sure line breaks are consistent in the ABAP code",
-            shortDescription: `Enforces LF as newlines in ABAP files
-
+            shortDescription: `Enforces LF as newlines in ABAP files
+
 abapGit does not work with CRLF`,
             tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.SingleFile],
         };
@@ -62418,7 +62483,7 @@ class LineLength extends _abap_rule_1.ABAPRule {
             key: "line_length",
             title: "Line length",
             shortDescription: `Detects lines exceeding the provided maximum length.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#stick-to-a-reasonable-line-length
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#stick-to-a-reasonable-line-length
 https://docs.abapopenchecks.org/checks/04/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };
@@ -62489,7 +62554,7 @@ class LineOnlyPunc extends _abap_rule_1.ABAPRule {
             key: "line_only_punc",
             title: "Line containing only punctuation",
             shortDescription: `Detects lines containing only punctuation.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#close-brackets-at-line-end
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#close-brackets-at-line-end
 https://docs.abapopenchecks.org/checks/16/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
             badExample: "zcl_class=>method(\n).",
@@ -62749,8 +62814,8 @@ class LocalVariableNames extends _abap_rule_1.ABAPRule {
         return {
             key: "local_variable_names",
             title: "Local variable naming conventions",
-            shortDescription: `
-Allows you to enforce a pattern, such as a prefix, for local variables, constants and field symbols.
+            shortDescription: `
+Allows you to enforce a pattern, such as a prefix, for local variables, constants and field symbols.
 Regexes are case-insensitive.`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],
         };
@@ -62897,10 +62962,10 @@ class MainFileContents {
             key: "main_file_contents",
             title: "Main file contents",
             shortDescription: `Checks related to report declarations.`,
-            extendedInformation: `Does not run if the target version is Cloud
-
-* PROGs must begin with "REPORT <name>." or "PROGRAM <name>.
-* TYPEs must begin with "TYPE-POOL <name>."
+            extendedInformation: `Does not run if the target version is Cloud
+
+* PROGs must begin with "REPORT <name>." or "PROGRAM <name>.
+* TYPEs must begin with "TYPE-POOL <name>."
 `,
         };
     }
@@ -63016,17 +63081,17 @@ class ManyParentheses extends _abap_rule_1.ABAPRule {
             title: "Too many parentheses",
             shortDescription: `Searches for expressions where extra parentheses can safely be removed`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
-            badExample: `
-IF ( destination IS INITIAL ).
-ENDIF.
-IF foo = boo AND ( bar = lar AND moo = loo ).
-ENDIF.
+            badExample: `
+IF ( destination IS INITIAL ).
+ENDIF.
+IF foo = boo AND ( bar = lar AND moo = loo ).
+ENDIF.
 `,
-            goodExample: `
-IF destination IS INITIAL.
-ENDIF.
-IF foo = boo AND bar = lar AND moo = loo.
-ENDIF.
+            goodExample: `
+IF destination IS INITIAL.
+ENDIF.
+IF foo = boo AND bar = lar AND moo = loo.
+ENDIF.
 `,
         };
     }
@@ -63200,14 +63265,14 @@ class MaxOneMethodParameterPerLine extends _abap_rule_1.ABAPRule {
             title: "Max one method parameter definition per line",
             shortDescription: `Keep max one method parameter description per line`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Whitespace],
-            badExample: `
-METHODS apps_scope_token
-  IMPORTING
+            badExample: `
+METHODS apps_scope_token
+  IMPORTING
     body TYPE bodyapps_scope_token client_id TYPE str.`,
-            goodExample: `
-METHODS apps_scope_token
-  IMPORTING
-    body      TYPE bodyapps_scope_token
+            goodExample: `
+METHODS apps_scope_token
+  IMPORTING
+    body      TYPE bodyapps_scope_token
     client_id TYPE str.`,
         };
     }
@@ -63272,11 +63337,11 @@ class MaxOneStatement extends _abap_rule_1.ABAPRule {
             key: "max_one_statement",
             title: "Max one statement per line",
             shortDescription: `Checks that each line contains only a single statement.`,
-            extendedInformation: `Does not report empty statements, use rule empty_statement for detecting empty statements.
-
-Does not report anything for chained statements.
-
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#no-more-than-one-statement-per-line
+            extendedInformation: `Does not report empty statements, use rule empty_statement for detecting empty statements.
+
+Does not report anything for chained statements.
+
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#no-more-than-one-statement-per-line
 https://docs.abapopenchecks.org/checks/11/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
             badExample: `WRITE foo. WRITE bar.`,
@@ -63581,6 +63646,7 @@ exports.MethodImplementedTwice = MethodImplementedTwice;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.MethodLength = exports.MethodLengthConf = void 0;
 const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const Objects = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
 const method_length_stats_1 = __webpack_require__(/*! ../utils/method_length_stats */ "./node_modules/@abaplint/core/build/src/utils/method_length_stats.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
@@ -63613,8 +63679,8 @@ class MethodLength {
             key: "method_length",
             title: "Method/Form Length",
             shortDescription: `Checks relating to method/form length.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#keep-methods-small
-
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#keep-methods-small
+
 Abstract methods without statements are considered okay.`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };
@@ -63642,6 +63708,12 @@ Abstract methods without statements are considered okay.`,
         return this;
     }
     run(obj) {
+        var _a;
+        if (this.conf.ignoreTestClasses === true
+            && obj instanceof Objects.Class
+            && ((_a = obj.getClassDefinition()) === null || _a === void 0 ? void 0 : _a.isForTesting) === true) {
+            return [];
+        }
         const methodStats = method_length_stats_1.MethodLengthStats.run(obj);
         const methodIssues = this.check(methodStats, "METHOD");
         let formIssues = [];
@@ -63713,10 +63785,10 @@ class MethodOverwritesBuiltIn extends _abap_rule_1.ABAPRule {
             key: "method_overwrites_builtin",
             title: "Method name overwrites builtin function",
             shortDescription: `Checks Method names that overwrite builtin SAP functions`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abenbuilt_in_functions_overview.htm
-
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-obscuring-built-in-functions
-
+            extendedInformation: `https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abenbuilt_in_functions_overview.htm
+
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-obscuring-built-in-functions
+
 Interface method names are ignored`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
         };
@@ -64286,7 +64358,7 @@ class Nesting extends _abap_rule_1.ABAPRule {
             key: "nesting",
             title: "Check nesting depth",
             shortDescription: `Checks for methods exceeding a maximum nesting depth`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#keep-the-nesting-depth-low
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#keep-the-nesting-depth-low
 https://docs.abapopenchecks.org/checks/74/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };
@@ -64529,7 +64601,7 @@ class NoChainedAssignment extends _abap_rule_1.ABAPRule {
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#dont-chain-assignments`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
             badExample: `var1 = var2 = var3.`,
-            goodExample: `var2 = var3.
+            goodExample: `var2 = var3.
 var1 = var2.`,
         };
     }
@@ -64588,8 +64660,8 @@ class NoExternalFormCalls extends _abap_rule_1.ABAPRule {
             key: "no_external_form_calls",
             title: "No external FORM calls",
             shortDescription: `Detect external form calls`,
-            badExample: `PERFORM foo IN PROGRAM bar.
-
+            badExample: `PERFORM foo IN PROGRAM bar.
+
 PERFORM foo(bar).`,
             tags: [_irule_1.RuleTag.SingleFile],
         };
@@ -64650,17 +64722,17 @@ class NoInlineInOptionalBranches extends _abap_rule_1.ABAPRule {
             key: "no_inline_in_optional_branches",
             title: "Don't declare inline in optional branches",
             shortDescription: `Don't declare inline in optional branches`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#dont-declare-inline-in-optional-branches
-
-Considered optional branches:
-* inside IF/ELSEIF/ELSE
-* inside LOOP
-* inside WHILE
-* inside CASE/WHEN, CASE TYPE OF
-* inside DO
-* inside SELECT loops
-
-Not considered optional branches:
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#dont-declare-inline-in-optional-branches
+
+Considered optional branches:
+* inside IF/ELSEIF/ELSE
+* inside LOOP
+* inside WHILE
+* inside CASE/WHEN, CASE TYPE OF
+* inside DO
+* inside SELECT loops
+
+Not considered optional branches:
 * TRY/CATCH/CLEANUP`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };
@@ -64759,12 +64831,12 @@ class NoPrefixes extends _abap_rule_1.ABAPRule {
             key: "no_prefixes",
             title: "No Prefixes",
             shortDescription: `Dont use hungarian notation`,
-            extendedInformation: `
-Note: not prefixing TYPES will require changing the errorNamespace in the abaplint configuration,
-allowing all types to become voided, abaplint will then provide less precise syntax errors.
-
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-encodings-esp-hungarian-notation-and-prefixes
-
+            extendedInformation: `
+Note: not prefixing TYPES will require changing the errorNamespace in the abaplint configuration,
+allowing all types to become voided, abaplint will then provide less precise syntax errors.
+
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-encodings-esp-hungarian-notation-and-prefixes
+
 https://github.com/SAP/styleguides/blob/main/clean-abap/sub-sections/AvoidEncodings.md`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
             badExample: `DATA lv_foo TYPE i.`,
@@ -64937,7 +65009,7 @@ class NoPublicAttributes extends _abap_rule_1.ABAPRule {
         return {
             key: "no_public_attributes",
             title: "No public attributes",
-            shortDescription: `Checks that classes and interfaces don't contain any public attributes.
+            shortDescription: `Checks that classes and interfaces don't contain any public attributes.
 Exceptions are excluded from this rule.`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#members-private-by-default-protected-only-if-needed`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
@@ -65038,13 +65110,13 @@ class NoYodaConditions extends _abap_rule_1.ABAPRule {
             key: "no_yoda_conditions",
             title: "No Yoda conditions",
             shortDescription: `Finds Yoda conditions and reports issues`,
-            extendedInformation: `https://en.wikipedia.org/wiki/Yoda_conditions
-
+            extendedInformation: `https://en.wikipedia.org/wiki/Yoda_conditions
+
 Conditions with operators CP, NP, CS, NS, CA, NA, CO, CN are ignored`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `IF 0 <> sy-subrc.
+            badExample: `IF 0 <> sy-subrc.
 ENDIF.`,
-            goodExample: `IF sy-subrc <> 0.
+            goodExample: `IF sy-subrc <> 0.
 ENDIF.`,
         };
     }
@@ -65145,8 +65217,8 @@ class NROBConsistency {
             key: "nrob_consistency",
             title: "Number range consistency",
             shortDescription: `Consistency checks for number ranges`,
-            extendedInformation: `Issue reported if percentage warning is over 50%
-
+            extendedInformation: `Issue reported if percentage warning is over 50%
+
 Issue reported if the referenced domain is not found(taking error namespace into account)`,
             tags: [_irule_1.RuleTag.SingleFile],
         };
@@ -65423,58 +65495,58 @@ class ObsoleteStatement extends _abap_rule_1.ABAPRule {
             title: "Obsolete statements",
             shortDescription: `Checks for usages of certain obsolete statements`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix],
-            extendedInformation: `
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-functional-to-procedural-language-constructs
-
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-obsolete-language-elements
-
-SET EXTENDED CHECK: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapset_extended_check.htm
-
-IS REQUESTED: https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abenlogexp_requested.htm
-
-WITH HEADER LINE: https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abapdata_header_line.htm
-
-FIELD-SYMBOLS STRUCTURE: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapfield-symbols_obsolete_typing.htm
-
-TYPE-POOLS: from 702, https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abennews-71-program_load.htm
-
-LOAD addition: from 702, https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abennews-71-program_load.htm
-
-COMMUICATION: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapcommunication.htm
-
-OCCURS: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapdata_occurs.htm
-
-PARAMETER: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abapparameter.htm
-
-RANGES: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abapranges.htm
-
-PACK: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abappack.htm
-
-MOVE: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapmove_obs.htm
-
-SELECT without INTO: https://help.sap.com/doc/abapdocu_731_index_htm/7.31/en-US/abapselect_obsolete.htm
-SELECT COUNT(*) is considered okay
-
-FREE MEMORY: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abapfree_mem_id_obsolete.htm
-
-SORT BY FS: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abapsort_itab_obsolete.htm
-
-CALL TRANSFORMATION OBJECTS: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abapcall_transformation_objects.htm
-
-POSIX REGEX: https://help.sap.com/doc/abapdocu_755_index_htm/7.55/en-US/index.htm
-
-OCCURENCES: check for OCCURENCES vs OCCURRENCES
-
+            extendedInformation: `
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-functional-to-procedural-language-constructs
+
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-obsolete-language-elements
+
+SET EXTENDED CHECK: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapset_extended_check.htm
+
+IS REQUESTED: https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abenlogexp_requested.htm
+
+WITH HEADER LINE: https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abapdata_header_line.htm
+
+FIELD-SYMBOLS STRUCTURE: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapfield-symbols_obsolete_typing.htm
+
+TYPE-POOLS: from 702, https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abennews-71-program_load.htm
+
+LOAD addition: from 702, https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abennews-71-program_load.htm
+
+COMMUICATION: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapcommunication.htm
+
+OCCURS: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapdata_occurs.htm
+
+PARAMETER: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abapparameter.htm
+
+RANGES: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abapranges.htm
+
+PACK: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abappack.htm
+
+MOVE: https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapmove_obs.htm
+
+SELECT without INTO: https://help.sap.com/doc/abapdocu_731_index_htm/7.31/en-US/abapselect_obsolete.htm
+SELECT COUNT(*) is considered okay
+
+FREE MEMORY: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abapfree_mem_id_obsolete.htm
+
+SORT BY FS: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abapsort_itab_obsolete.htm
+
+CALL TRANSFORMATION OBJECTS: https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abapcall_transformation_objects.htm
+
+POSIX REGEX: https://help.sap.com/doc/abapdocu_755_index_htm/7.55/en-US/index.htm
+
+OCCURENCES: check for OCCURENCES vs OCCURRENCES
+
 CLIENT SPECIFIED, from 754: https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/index.htm?file=abapselect_client_obsolete.htm`,
-            badExample: `REFRESH itab.
-
-COMPUTE foo = 2 + 2.
-
-MULTIPLY lv_foo BY 2.
-
-INTERFACE intf LOAD.
-
-IF foo IS SUPPLIED.
+            badExample: `REFRESH itab.
+
+COMPUTE foo = 2 + 2.
+
+MULTIPLY lv_foo BY 2.
+
+INTERFACE intf LOAD.
+
+IF foo IS SUPPLIED.
 ENDIF.`,
         };
     }
@@ -65814,9 +65886,9 @@ class OmitParameterName {
             key: "omit_parameter_name",
             title: "Omit parameter name",
             shortDescription: `Omit the parameter name in single parameter calls`,
-            extendedInformation: `
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#omit-the-parameter-name-in-single-parameter-calls
-
+            extendedInformation: `
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#omit-the-parameter-name-in-single-parameter-calls
+
 EXPORTING must already be omitted for this rule to take effect, https://rules.abaplint.org/exporting/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix],
             badExample: `method( param = 2 ).`,
@@ -66022,20 +66094,20 @@ class OmitReceiving extends _abap_rule_1.ABAPRule {
             shortDescription: `Omit RECEIVING`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#omit-receiving`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
-            badExample: `
-      upload_pack(
-        EXPORTING
-          io_client       = lo_client
-          iv_url          = iv_url
-          iv_deepen_level = iv_deepen_level
-          it_hashes       = lt_hashes
-        RECEIVING
+            badExample: `
+      upload_pack(
+        EXPORTING
+          io_client       = lo_client
+          iv_url          = iv_url
+          iv_deepen_level = iv_deepen_level
+          it_hashes       = lt_hashes
+        RECEIVING
           rt_objects      = et_objects ).`,
-            goodExample: `
-      et_objects = upload_pack(
-        io_client       = lo_client
-        iv_url          = iv_url
-        iv_deepen_level = iv_deepen_level
+            goodExample: `
+      et_objects = upload_pack(
+        io_client       = lo_client
+        iv_url          = iv_url
+        iv_deepen_level = iv_deepen_level
         it_hashes       = lt_hashes ).`,
         };
     }
@@ -66099,8 +66171,8 @@ class Parser702Chaining extends _abap_rule_1.ABAPRule {
         return {
             key: "parser_702_chaining",
             title: "Parser Error, bad chanining on 702",
-            shortDescription: `ABAP on 702 does not allow for method chaining with IMPORTING/EXPORTING/CHANGING keywords,
-this rule finds these and reports errors.
+            shortDescription: `ABAP on 702 does not allow for method chaining with IMPORTING/EXPORTING/CHANGING keywords,
+this rule finds these and reports errors.
 Only active on target version 702 and below.`,
             tags: [_irule_1.RuleTag.Syntax, _irule_1.RuleTag.SingleFile],
         };
@@ -66180,8 +66252,8 @@ class ParserError {
         return {
             key: "parser_error",
             title: "Parser error",
-            shortDescription: `Checks for syntax not recognized by abaplint.
-
+            shortDescription: `Checks for syntax not recognized by abaplint.
+
 See recognized syntax at https://syntax.abaplint.org`,
             tags: [_irule_1.RuleTag.Syntax, _irule_1.RuleTag.SingleFile],
         };
@@ -66266,7 +66338,7 @@ class ParserMissingSpace extends _abap_rule_1.ABAPRule {
         return {
             key: "parser_missing_space",
             title: "Parser Error, missing space",
-            shortDescription: `In special cases the ABAP language allows for not having spaces before or after string literals.
+            shortDescription: `In special cases the ABAP language allows for not having spaces before or after string literals.
 This rule makes sure the spaces are consistently required across the language.`,
             tags: [_irule_1.RuleTag.Syntax, _irule_1.RuleTag.Whitespace, _irule_1.RuleTag.SingleFile],
             badExample: `IF ( foo = 'bar').`,
@@ -66678,25 +66750,25 @@ class PreferInline {
             key: "prefer_inline",
             title: "Prefer Inline Declarations",
             shortDescription: `Prefer inline to up-front declarations.`,
-            extendedInformation: `EXPERIMENTAL
-
-Activates if language version is v740sp02 or above.
-
-Variables must be local(METHOD or FORM).
-
-No generic or void typed variables. No syntax errors.
-
-First position used must be a full/pure write.
-
-Move statment is not a cast(?=)
-
+            extendedInformation: `EXPERIMENTAL
+
+Activates if language version is v740sp02 or above.
+
+Variables must be local(METHOD or FORM).
+
+No generic or void typed variables. No syntax errors.
+
+First position used must be a full/pure write.
+
+Move statment is not a cast(?=)
+
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-inline-to-up-front-declarations`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Upport, _irule_1.RuleTag.Experimental, _irule_1.RuleTag.Quickfix],
-            badExample: `DATA foo TYPE i.
-foo = 2.
-DATA percentage TYPE decfloat34.
+            badExample: `DATA foo TYPE i.
+foo = 2.
+DATA percentage TYPE decfloat34.
 percentage = ( comment_number / abs_statement_number ) * 100.`,
-            goodExample: `DATA(foo) = 2.
+            goodExample: `DATA(foo) = 2.
 DATA(percentage) = CONV decfloat34( comment_number / abs_statement_number ) * 100.`,
         };
     }
@@ -66910,18 +66982,18 @@ class PreferIsNot extends _abap_rule_1.ABAPRule {
             key: "prefer_is_not",
             title: "Prefer IS NOT to NOT IS",
             shortDescription: `Prefer IS NOT to NOT IS`,
-            extendedInformation: `
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-is-not-to-not-is
-
+            extendedInformation: `
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-is-not-to-not-is
+
 "if not is_valid( )." examples are skipped`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
-            goodExample: `IF variable IS NOT INITIAL.
-IF variable NP 'TODO*'.
-IF variable <> 42.
+            goodExample: `IF variable IS NOT INITIAL.
+IF variable NP 'TODO*'.
+IF variable <> 42.
 IF variable CO 'hello'.`,
-            badExample: `IF NOT variable IS INITIAL.
-IF NOT variable CP 'TODO*'.
-IF NOT variable = 42.
+            badExample: `IF NOT variable IS INITIAL.
+IF NOT variable CP 'TODO*'.
+IF NOT variable = 42.
 IF NOT variable CA 'hello'.`,
         };
     }
@@ -67109,14 +67181,14 @@ class PreferRaiseExceptionNew extends _abap_rule_1.ABAPRule {
             key: "prefer_raise_exception_new",
             title: "Prefer RAISE EXCEPTION NEW to RAISE EXCEPTION TYPE",
             shortDescription: `Prefer RAISE EXCEPTION NEW to RAISE EXCEPTION TYPE`,
-            extendedInformation: `
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-raise-exception-new-to-raise-exception-type
-
+            extendedInformation: `
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-raise-exception-new-to-raise-exception-type
+
 From 752 and up`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.Upport],
             goodExample: `RAISE EXCEPTION NEW cx_generation_error( previous = exception ).`,
-            badExample: `RAISE EXCEPTION TYPE cx_generation_error
-  EXPORTING
+            badExample: `RAISE EXCEPTION TYPE cx_generation_error
+  EXPORTING
     previous = exception.`,
         };
     }
@@ -67194,7 +67266,7 @@ class PreferReturningToExporting extends _abap_rule_1.ABAPRule {
             key: "prefer_returning_to_exporting",
             title: "Prefer RETURNING to EXPORTING",
             shortDescription: `Prefer RETURNING to EXPORTING. Generic types cannot be RETURNING.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-returning-to-exporting
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-returning-to-exporting
 https://docs.abapopenchecks.org/checks/44/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };
@@ -67291,8 +67363,8 @@ class PreferXsdbool extends _abap_rule_1.ABAPRule {
             key: "prefer_xsdbool",
             title: "Prefer xsdbool over boolc",
             shortDescription: `Prefer xsdbool over boolc`,
-            extendedInformation: `Activates if language version is v740sp08 or above.
-
+            extendedInformation: `Activates if language version is v740sp08 or above.
+
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-xsdbool-to-set-boolean-variables`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Upport, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
             badExample: `DATA(sdf) = boolc( 1 = 2 ).`,
@@ -67586,26 +67658,26 @@ class ReduceProceduralCode extends _abap_rule_1.ABAPRule {
             key: "reduce_procedural_code",
             title: "Reduce procedural code",
             shortDescription: `Checks FORM and FUNCTION-MODULE have few statements`,
-            extendedInformation: `Delegate logic to a class method instead of using FORM or FUNCTION-MODULE.
-
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-object-orientation-to-procedural-programming
-
+            extendedInformation: `Delegate logic to a class method instead of using FORM or FUNCTION-MODULE.
+
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-object-orientation-to-procedural-programming
+
 Comments are not counted as statements.`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
-            badExample: `FORM foo.
-  DATA lv_bar TYPE i.
-  lv_bar = 2 + 2.
-  IF lv_bar = 4.
-    WRITE 'hello world'.
-  ENDIF.
-  DATA lv_bar TYPE i.
-  lv_bar = 2 + 2.
-  IF lv_bar = 4.
-    WRITE 'hello world'.
-  ENDIF.
+            badExample: `FORM foo.
+  DATA lv_bar TYPE i.
+  lv_bar = 2 + 2.
+  IF lv_bar = 4.
+    WRITE 'hello world'.
+  ENDIF.
+  DATA lv_bar TYPE i.
+  lv_bar = 2 + 2.
+  IF lv_bar = 4.
+    WRITE 'hello world'.
+  ENDIF.
 ENDFORM.`,
-            goodExample: `FORM foo.
-  NEW zcl_global_class( )->run_logic( ).
+            goodExample: `FORM foo.
+  NEW zcl_global_class( )->run_logic( ).
 ENDFORM.`,
         };
     }
@@ -67849,10 +67921,10 @@ class RemoveDescriptions {
         return {
             key: "remove_descriptions",
             title: "Remove descriptions",
-            shortDescription: `Ensures you have no descriptions in metadata of methods, parameters, etc.
-
-Class descriptions are required, see rule description_empty.
-
+            shortDescription: `Ensures you have no descriptions in metadata of methods, parameters, etc.
+
+Class descriptions are required, see rule description_empty.
+
 Consider using ABAP Doc for documentation.`,
             tags: [],
         };
@@ -67977,16 +68049,16 @@ class RFCErrorHandling extends _abap_rule_1.ABAPRule {
             tags: [_irule_1.RuleTag.SingleFile],
             shortDescription: `Checks that exceptions 'system_failure' and 'communication_failure' are handled in RFC calls`,
             extendedInformation: `https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abenrfc_exception.htm`,
-            badExample: `
-CALL FUNCTION 'ZRFC'
+            badExample: `
+CALL FUNCTION 'ZRFC'
   DESTINATION lv_rfc.`,
-            goodExample: `
-CALL FUNCTION 'ZRFC'
-  DESTINATION lv_rfc
-  EXCEPTIONS
-    system_failure        = 1 MESSAGE msg
-    communication_failure = 2 MESSAGE msg
-    resource_failure      = 3
+            goodExample: `
+CALL FUNCTION 'ZRFC'
+  DESTINATION lv_rfc
+  EXCEPTIONS
+    system_failure        = 1 MESSAGE msg
+    communication_failure = 2 MESSAGE msg
+    resource_failure      = 3
     OTHERS                = 4.`,
         };
     }
@@ -68070,11 +68142,11 @@ class SelectAddOrderBy {
             key: "select_add_order_by",
             title: "SELECT add ORDER BY",
             shortDescription: `SELECTs add ORDER BY clause`,
-            extendedInformation: `
-This will make sure that the SELECT statement returns results in the same sequence on different databases
-
-add ORDER BY PRIMARY KEY if in doubt
-
+            extendedInformation: `
+This will make sure that the SELECT statement returns results in the same sequence on different databases
+
+add ORDER BY PRIMARY KEY if in doubt
+
 If the target is a sorted/hashed table, no issue is reported`,
             tags: [_irule_1.RuleTag.SingleFile],
         };
@@ -68203,14 +68275,14 @@ class SelectPerformance {
             key: "select_performance",
             title: "SELECT performance",
             shortDescription: `Various checks regarding SELECT performance.`,
-            extendedInformation: `ENDSELECT: not reported when the corresponding SELECT has PACKAGE SIZE
-
+            extendedInformation: `ENDSELECT: not reported when the corresponding SELECT has PACKAGE SIZE
+
 SELECT *: not reported if using INTO/APPENDING CORRESPONDING FIELDS OF`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Performance],
-            badExample: `SELECT field1, field2 FROM table
-    INTO @DATA(structure) UP TO 1 ROWS ORDER BY field3 DESCENDING.
+            badExample: `SELECT field1, field2 FROM table
+    INTO @DATA(structure) UP TO 1 ROWS ORDER BY field3 DESCENDING.
 ENDSELECT.`,
-            goodExample: `SELECT field1, field2 FROM table UP TO 1 ROWS
+            goodExample: `SELECT field1, field2 FROM table UP TO 1 ROWS
   INTO TABLE @DATA(table) ORDER BY field3 DESCENDING`,
         };
     }
@@ -68322,8 +68394,8 @@ class SelectSingleFullKey {
             key: "select_single_full_key",
             title: "Detect SELECT SINGLE which are possibily not unique",
             shortDescription: `Detect SELECT SINGLE which are possibily not unique`,
-            extendedInformation: `Table definitions must be known, ie. inside the errorNamespace
-
+            extendedInformation: `Table definitions must be known, ie. inside the errorNamespace
+
 If the statement contains a JOIN it is not checked`,
             pseudoComment: "EC CI_NOORDER",
             tags: [],
@@ -68740,8 +68812,8 @@ class SICFConsistency {
             key: "sicf_consistency",
             title: "SICF consistency",
             shortDescription: `Checks the validity of ICF services`,
-            extendedInformation: `* Class defined in handler must exist
-* Class must not have any syntax errors
+            extendedInformation: `* Class defined in handler must exist
+* Class must not have any syntax errors
 * Class must implement interface IF_HTTP_EXTENSION`,
         };
     }
@@ -69108,8 +69180,8 @@ class SpaceBeforeDot extends _abap_rule_1.ABAPRule {
             key: "space_before_dot",
             title: "Space before dot",
             shortDescription: `Checks for extra spaces before dots at the ends of statements`,
-            extendedInformation: `
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#be-consistent
+            extendedInformation: `
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#be-consistent
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#condense-your-code`,
             tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
             badExample: `WRITE bar .`,
@@ -69295,12 +69367,12 @@ class SQLValueConversion {
             key: "sql_value_conversion",
             title: "Implicit SQL Value Conversion",
             shortDescription: `Ensure types match when selecting from database`,
-            extendedInformation: `
-* Integer to CHAR conversion
-* Integer to NUMC conversion
-* NUMC to Integer conversion
-* CHAR to Integer conversion
-* Source field longer than database field, CHAR -> CHAR
+            extendedInformation: `
+* Integer to CHAR conversion
+* Integer to NUMC conversion
+* NUMC to Integer conversion
+* CHAR to Integer conversion
+* Source field longer than database field, CHAR -> CHAR
 * Source field longer than database field, NUMC -> NUMC`,
             tags: [],
         };
@@ -69372,7 +69444,7 @@ class StartAtTab extends _abap_rule_1.ABAPRule {
             key: "start_at_tab",
             title: "Start at tab",
             shortDescription: `Checks that statements start at tabstops.`,
-            extendedInformation: `Reports max 100 issues per file
+            extendedInformation: `Reports max 100 issues per file
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#indent-and-snap-to-tab`,
             tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
             badExample: ` WRITE a.`,
@@ -69549,12 +69621,12 @@ class StrictSQL extends _abap_rule_1.ABAPRule {
             key: "strict_sql",
             title: "Strict SQL",
             shortDescription: `Strict SQL`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abapinto_clause.htm
-
-https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abenopensql_strict_mode_750.htm
-
-Also see separate rule sql_escape_host_variables
-
+            extendedInformation: `https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abapinto_clause.htm
+
+https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abenopensql_strict_mode_750.htm
+
+Also see separate rule sql_escape_host_variables
+
 Activates from v750 and up`,
             tags: [_irule_1.RuleTag.Upport, _irule_1.RuleTag.Quickfix],
         };
@@ -69806,11 +69878,11 @@ class SyModification extends _abap_rule_1.ABAPRule {
             key: "sy_modification",
             title: "Modification of SY fields",
             shortDescription: `Finds modification of sy fields`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abensystem_fields.htm
-
+            extendedInformation: `https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abensystem_fields.htm
+
 Changes to SY-TVAR* fields are not reported`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `sy-uname = 2.
+            badExample: `sy-uname = 2.
 sy = sy.`,
         };
     }
@@ -69872,8 +69944,8 @@ class TABLEnhancementCategory {
             key: "tabl_enhancement_category",
             title: "TABL enhancement category must be set",
             shortDescription: `Checks that tables do not have the enhancement category 'not classified'.`,
-            extendedInformation: `SAP note 3063227 changes the default to 'Cannot be enhanced'.
-
+            extendedInformation: `SAP note 3063227 changes the default to 'Cannot be enhanced'.
+
 You may use standard report RS_DDIC_CLASSIFICATION_FINAL for adjustment.`,
             tags: [],
         };
@@ -70001,9 +70073,9 @@ class TypeFormParameters extends _abap_rule_1.ABAPRule {
             title: "Type FORM parameters",
             shortDescription: `Checks for untyped FORM parameters`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `FORM foo USING bar.
+            badExample: `FORM foo USING bar.
 ENDFORM.`,
-            goodExample: `FORM foo USING bar TYPE string.
+            goodExample: `FORM foo USING bar TYPE string.
 ENDFORM.`,
         };
     }
@@ -70673,38 +70745,38 @@ class UnnecessaryPragma extends _abap_rule_1.ABAPRule {
             key: "unnecessary_pragma",
             title: "Unnecessary Pragma",
             shortDescription: `Finds pragmas which can be removed`,
-            extendedInformation: `* NO_HANDLER with handler
-
-* NEEDED without definition
-
-* NO_TEXT without texts
-
-* SUBRC_OK where sy-subrc is checked
-
+            extendedInformation: `* NO_HANDLER with handler
+
+* NEEDED without definition
+
+* NO_TEXT without texts
+
+* SUBRC_OK where sy-subrc is checked
+
 NO_HANDLER inside macros are not checked`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `TRY.
-    ...
-  CATCH zcx_abapgit_exception ##NO_HANDLER.
-    RETURN. " it has a handler
-ENDTRY.
-MESSAGE w125(zbar) WITH c_foo INTO message ##NEEDED ##NO_TEXT.
-SELECT SINGLE * FROM tadir INTO @DATA(sdfs) ##SUBRC_OK.
-IF sy-subrc <> 0.
+            badExample: `TRY.
+    ...
+  CATCH zcx_abapgit_exception ##NO_HANDLER.
+    RETURN. " it has a handler
+ENDTRY.
+MESSAGE w125(zbar) WITH c_foo INTO message ##NEEDED ##NO_TEXT.
+SELECT SINGLE * FROM tadir INTO @DATA(sdfs) ##SUBRC_OK.
+IF sy-subrc <> 0.
 ENDIF.`,
-            goodExample: `TRY.
-    ...
-  CATCH zcx_abapgit_exception.
-    RETURN.
-ENDTRY.
-MESSAGE w125(zbar) WITH c_foo INTO message.
-SELECT SINGLE * FROM tadir INTO @DATA(sdfs).
-IF sy-subrc <> 0.
-ENDIF.
-
-DATA: BEGIN OF blah ##NEEDED,
-        test1 TYPE string,
-        test2 TYPE string,
+            goodExample: `TRY.
+    ...
+  CATCH zcx_abapgit_exception.
+    RETURN.
+ENDTRY.
+MESSAGE w125(zbar) WITH c_foo INTO message.
+SELECT SINGLE * FROM tadir INTO @DATA(sdfs).
+IF sy-subrc <> 0.
+ENDIF.
+
+DATA: BEGIN OF blah ##NEEDED,
+        test1 TYPE string,
+        test2 TYPE string,
       END OF blah.`,
         };
     }
@@ -70862,18 +70934,18 @@ class UnnecessaryReturn extends _abap_rule_1.ABAPRule {
             shortDescription: `Finds unnecessary RETURN statements`,
             extendedInformation: `Finds unnecessary RETURN statements`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
-            badExample: `FORM hello1.
-  WRITE 'world'.
-  RETURN.
-ENDFORM.
-
-FORM foo.
-  IF 1 = 2.
-    RETURN.
-  ENDIF.
+            badExample: `FORM hello1.
+  WRITE 'world'.
+  RETURN.
+ENDFORM.
+
+FORM foo.
+  IF 1 = 2.
+    RETURN.
+  ENDIF.
 ENDFORM.`,
-            goodExample: `FORM hello2.
-  WRITE 'world'.
+            goodExample: `FORM hello2.
+  WRITE 'world'.
 ENDFORM.`,
         };
     }
@@ -71240,17 +71312,17 @@ class UnusedMethods {
             key: "unused_methods",
             title: "Unused methods",
             shortDescription: `Checks for unused methods`,
-            extendedInformation: `Checks private and protected methods.
-
-Unused methods are not reported if the object contains parser or syntax errors.
-
-Skips:
-* methods FOR TESTING
-* methods SETUP + TEARDOWN + CLASS_SETUP + CLASS_TEARDOWN in testclasses
-* class_constructor + constructor methods
-* event handlers
-* methods that are redefined
-* INCLUDEs
+            extendedInformation: `Checks private and protected methods.
+
+Unused methods are not reported if the object contains parser or syntax errors.
+
+Skips:
+* methods FOR TESTING
+* methods SETUP + TEARDOWN + CLASS_SETUP + CLASS_TEARDOWN in testclasses
+* class_constructor + constructor methods
+* event handlers
+* methods that are redefined
+* INCLUDEs
 `,
             tags: [],
             pragma: "##CALLED",
@@ -71640,23 +71712,23 @@ class UnusedVariables {
             key: "unused_variables",
             title: "Unused variables",
             shortDescription: `Checks for unused variables and constants`,
-            extendedInformation: `Skips event parameters.
-
-Note that this currently does not work if the source code uses macros.
-
-Unused variables are not reported if the object contains parser or syntax errors.
-
+            extendedInformation: `Skips event parameters.
+
+Note that this currently does not work if the source code uses macros.
+
+Unused variables are not reported if the object contains parser or syntax errors.
+
 Errors found in INCLUDES are reported for the main program.`,
             tags: [_irule_1.RuleTag.Quickfix],
             pragma: "##NEEDED",
             pseudoComment: "EC NEEDED",
-            badExample: `DATA: BEGIN OF blah1,
-      test  TYPE string,
-      test2 TYPE string,
+            badExample: `DATA: BEGIN OF blah1,
+      test  TYPE string,
+      test2 TYPE string,
     END OF blah1.`,
-            goodExample: `DATA: BEGIN OF blah2 ##NEEDED,
-      test  TYPE string,
-      test2 TYPE string,
+            goodExample: `DATA: BEGIN OF blah2 ##NEEDED,
+      test  TYPE string,
+      test2 TYPE string,
     END OF blah2.`,
         };
     }
@@ -71875,15 +71947,15 @@ class UseBoolExpression extends _abap_rule_1.ABAPRule {
             shortDescription: `Use boolean expression, xsdbool from 740sp08 and up, boolc from 702 and up`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-xsdbool-to-set-boolean-variables`,
             tags: [_irule_1.RuleTag.Upport, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
-            badExample: `IF line IS INITIAL.
-  has_entries = abap_false.
-ELSE.
-  has_entries = abap_true.
-ENDIF.
-
+            badExample: `IF line IS INITIAL.
+  has_entries = abap_false.
+ELSE.
+  has_entries = abap_true.
+ENDIF.
+
 DATA(fsdf) = COND #( WHEN foo <> bar THEN abap_true ELSE abap_false ).`,
-            goodExample: `DATA(has_entries) = xsdbool( line IS NOT INITIAL ).
-
+            goodExample: `DATA(has_entries) = xsdbool( line IS NOT INITIAL ).
+
 DATA(fsdf) = xsdbool( foo <> bar ).`,
         };
     }
@@ -72059,15 +72131,15 @@ class UseLineExists extends _abap_rule_1.ABAPRule {
             key: "use_line_exists",
             title: "Use line_exists",
             shortDescription: `Use line_exists, from 740sp02 and up`,
-            extendedInformation: `
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-line_exists-to-read-table-or-loop-at
-
+            extendedInformation: `
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-line_exists-to-read-table-or-loop-at
+
 Not reported if the READ TABLE statement contains BINARY SEARCH.`,
             tags: [_irule_1.RuleTag.Upport, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
-            badExample: `READ TABLE my_table TRANSPORTING NO FIELDS WITH KEY key = 'A'.
-IF sy-subrc = 0.
+            badExample: `READ TABLE my_table TRANSPORTING NO FIELDS WITH KEY key = 'A'.
+IF sy-subrc = 0.
 ENDIF.`,
-            goodExample: `IF line_exists( my_table[ key = 'A' ] ).
+            goodExample: `IF line_exists( my_table[ key = 'A' ] ).
 ENDIF.`,
         };
     }
@@ -72177,10 +72249,10 @@ class UseNew extends _abap_rule_1.ABAPRule {
             key: "use_new",
             title: "Use NEW",
             shortDescription: `Checks for deprecated CREATE OBJECT statements.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-new-to-create-object
-
-If the target variable is referenced in the CREATE OBJECT statement, no errors are issued
-
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-new-to-create-object
+
+If the target variable is referenced in the CREATE OBJECT statement, no errors are issued
+
 Applicable from v740sp02 and up`,
             badExample: `CREATE OBJECT ref.`,
             goodExample: `ref = NEW #( ).`,
@@ -72278,13 +72350,13 @@ class WhenOthersLast extends _abap_rule_1.ABAPRule {
             title: "WHEN OTHERS last",
             shortDescription: `Checks that WHEN OTHERS is placed the last within a CASE statement.`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `CASE bar.
-  WHEN OTHERS.
-  WHEN 2.
+            badExample: `CASE bar.
+  WHEN OTHERS.
+  WHEN 2.
 ENDCASE.`,
-            goodExample: `CASE bar.
-  WHEN 2.
-  WHEN OTHERS.
+            goodExample: `CASE bar.
+  WHEN 2.
+  WHEN OTHERS.
 ENDCASE.`,
         };
     }
@@ -75117,14 +75189,13 @@ const parseXml = function(xmlData) {
 
         textData = this.saveTextToParentTag(textData, currentNode, jPath);
 
+        let val = this.parseTextData(tagExp, currentNode.tagname, jPath, true, false, true, true);
+        if(val == undefined) val = "";
+
         //cdata should be set even if it is 0 length string
         if(this.options.cdataPropName){
-          // let val = this.parseTextData(tagExp, this.options.cdataPropName, jPath + "." + this.options.cdataPropName, true, false, true);
-          // if(!val) val = "";
           currentNode.add(this.options.cdataPropName, [ { [this.options.textNodeName] : tagExp } ]);
         }else{
-          let val = this.parseTextData(tagExp, currentNode.tagname, jPath, true, false, true);
-          if(val == undefined) val = "";
           currentNode.add(this.options.textNodeName, val);
         }