@abaplint/core 2.80.8 → 2.81.1

package/build/src/rules/no_chained_assignment.js

@@ -0,0 +1,52 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.NoChainedAssignment = exports.NoChainedAssignmentConf = void 0;
+const Expressions = require("../abap/2_statements/expressions");
+const Statements = require("../abap/2_statements/statements");
+const issue_1 = require("../issue");
+const _abap_rule_1 = require("./_abap_rule");
+const _basic_rule_config_1 = require("./_basic_rule_config");
+const _irule_1 = require("./_irule");
+class NoChainedAssignmentConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.NoChainedAssignmentConf = NoChainedAssignmentConf;
+class NoChainedAssignment extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new NoChainedAssignmentConf();
+    }
+    getMetadata() {
+        return {
+            key: "no_chained_assignment",
+            title: "No chained assignment",
+            shortDescription: `Find chained assingments and reports issues`,
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#dont-chain-assignments`,
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
+            badExample: `var1 = var2 = var3.`,
+            goodExample: `var2 = var3.
+var1 = var2.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const issues = [];
+        for (const s of file.getStatements()) {
+            if (!(s.get() instanceof Statements.Move)) {
+                continue;
+            }
+            if (s.findDirectExpressions(Expressions.Target).length >= 2) {
+                const message = "No chained assignment";
+                const issue = issue_1.Issue.atStatement(file, s, message, this.getMetadata().key);
+                issues.push(issue);
+            }
+        }
+        return issues;
+    }
+}
+exports.NoChainedAssignment = NoChainedAssignment;
+//# sourceMappingURL=no_chained_assignment.js.map

package/build/src/rules/no_public_attributes.js

@@ -25,7 +25,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],

package/build/src/rules/no_yoda_conditions.js

@@ -24,13 +24,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.`,
         };
     }

package/build/src/rules/obsolete_statement.js

@@ -77,42 +77,42 @@ 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
-
-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
-
+            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
+
+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`,
         };
     }

package/build/src/rules/omit_parameter_name.js

@@ -22,9 +22,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 ).`,

package/build/src/rules/omit_receiving.js

@@ -21,20 +21,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 ).`,
         };
     }

package/build/src/rules/parser_702_chaining.js

@@ -19,8 +19,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],
         };

package/build/src/rules/parser_error.js

@@ -20,8 +20,8 @@ class ParserError extends _abap_rule_1.ABAPRule {
         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],
         };

package/build/src/rules/parser_missing_space.js

@@ -21,7 +21,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').`,

package/build/src/rules/prefer_inline.js

@@ -23,25 +23,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.
-
-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.
+
+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.`,
         };
     }
@@ -74,7 +74,7 @@ DATA(percentage) = CONV decfloat34( comment_number / abs_statement_number ) * 10
     }
     ///////////////////////////
     analyzeScope(node, obj) {
-        var _a;
+        var _a, _b;
         const ret = [];
         const vars = node.getData().vars;
         for (const name in vars) {
@@ -113,7 +113,8 @@ DATA(percentage) = CONV decfloat34( comment_number / abs_statement_number ) * 10
                 || statementType instanceof Statements.Catch
                 || statementType instanceof Statements.ReadTable
                 || statementType instanceof Statements.Loop)
-                || ((_a = writeStatement === null || writeStatement === void 0 ? void 0 : writeStatement.concatTokens()) === null || _a === void 0 ? void 0 : _a.includes("?="))) {
+                || ((_a = writeStatement === null || writeStatement === void 0 ? void 0 : writeStatement.concatTokens()) === null || _a === void 0 ? void 0 : _a.includes("?="))
+                || ((_b = writeStatement === null || writeStatement === void 0 ? void 0 : writeStatement.concatTokens()) === null || _b === void 0 ? void 0 : _b.includes(" #("))) {
                 continue;
             }
             const statement = edit_helper_1.EditHelper.findStatement(identifier.getToken(), file);

package/build/src/rules/prefer_is_not.js

@@ -21,16 +21,16 @@ 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*'.
+            goodExample: `IF variable IS NOT INITIAL.
+IF variable NP 'TODO*'.
 IF variable <> 42.`,
-            badExample: `IF NOT variable IS INITIAL.
-IF NOT variable CP 'TODO*'.
+            badExample: `IF NOT variable IS INITIAL.
+IF NOT variable CP 'TODO*'.
 IF NOT variable = 42.`,
         };
     }

package/build/src/rules/prefer_raise_exception_new.js

@@ -20,12 +20,12 @@ 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: `
+            extendedInformation: `
       https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-raise-exception-new-to-raise-exception-type`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
             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.`,
         };
     }

package/build/src/rules/prefer_returning_to_exporting.js

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

package/build/src/rules/prefer_xsdbool.js

@@ -21,8 +21,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 ).`,

package/build/src/rules/remove_descriptions.js

@@ -26,10 +26,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: [],
         };

package/build/src/rules/rfc_error_handling.js

@@ -22,16 +22,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.`,
         };
     }

package/build/src/rules/select_add_order_by.js

@@ -21,11 +21,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],
         };

package/build/src/rules/select_performance.js

@@ -32,8 +32,8 @@ 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],
         };

package/build/src/rules/sicf_consistency.js

@@ -16,10 +16,10 @@ class SICFConsistency {
         return {
             key: "sicf_consistency",
             title: "SICF consistency",
-            shortDescription: `Checks the validity of ICF services:
-
-* Class defined in handler must exist
-* Class must not have any syntax errors
+            shortDescription: `Checks the validity of ICF services:
+
+* Class defined in handler must exist
+* Class must not have any syntax errors
 * Class must implement interface IF_HTTP_EXTENSION`,
         };
     }

package/build/src/rules/space_before_dot.js

@@ -29,8 +29,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 .`,

package/build/src/rules/start_at_tab.js

@@ -20,7 +20,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.`,

package/build/src/rules/sy_modification.js

@@ -21,8 +21,8 @@ class SyModification extends _abap_rule_1.ABAPRule {
             shortDescription: `Finds modification of sy fields`,
             extendedInformation: `https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abensystem_fields.htm`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `
-sy-uname = 2.
+            badExample: `
+sy-uname = 2.
 sy = sy.`,
         };
     }

package/build/src/rules/tabl_enhancement_category.js

@@ -17,8 +17,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: [],
         };

package/build/src/rules/unnecessary_chaining.js

@@ -0,0 +1,58 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.UnnecessaryChaining = exports.UnnecessaryChainingConf = void 0;
+const issue_1 = require("../issue");
+const _abap_rule_1 = require("./_abap_rule");
+const _basic_rule_config_1 = require("./_basic_rule_config");
+const _irule_1 = require("./_irule");
+class UnnecessaryChainingConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.UnnecessaryChainingConf = UnnecessaryChainingConf;
+class UnnecessaryChaining extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new UnnecessaryChainingConf();
+    }
+    getMetadata() {
+        return {
+            key: "unnecessary_chaining",
+            title: "Unnecessary Chaining",
+            shortDescription: `Find unnecessary chaining, all statements are checked`,
+            extendedInformation: ``,
+            tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `WRITE: bar.`,
+            goodExample: `WRITE bar.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        var _a, _b;
+        const issues = [];
+        const statements = file.getStatements();
+        for (let i = 0; i < statements.length; i++) {
+            const colon = statements[i].getColon();
+            if (colon === undefined) {
+                continue;
+            }
+            const next = (_a = statements[i + 1]) === null || _a === void 0 ? void 0 : _a.getColon();
+            const prev = (_b = statements[i - 1]) === null || _b === void 0 ? void 0 : _b.getColon();
+            if (next !== undefined && colon.getStart().equals(next.getStart())) {
+                continue;
+            }
+            else if (prev !== undefined && colon.getStart().equals(prev.getStart())) {
+                continue;
+            }
+            const message = "Unnecessary chaining";
+            const issue = issue_1.Issue.atStatement(file, statements[i], message, this.getMetadata().key);
+            issues.push(issue);
+        }
+        return issues;
+    }
+}
+exports.UnnecessaryChaining = UnnecessaryChaining;
+//# sourceMappingURL=unnecessary_chaining.js.map

package/build/src/rules/unreachable_code.js

@@ -67,6 +67,10 @@ class UnreachableCode extends _abap_rule_1.ABAPRule {
             return true;
         }
         else if (s instanceof Statements.Leave && n.findFirstExpression(Expressions.AndReturn) === undefined) {
+            const concat = n.concatTokens();
+            if (concat.includes(" TO LIST-PROCESSING")) {
+                return false;
+            }
             return true;
         }
         else if (s instanceof Statements.Return

package/build/src/rules/unused_methods.js

@@ -55,15 +55,15 @@ class UnusedMethods {
             key: "unused_methods",
             title: "Unused methods",
             shortDescription: `Checks for unused methods`,
-            extendedInformation: `Checks private and protected methods.
-
-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.
+
+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",

package/build/src/rules/unused_variables.js

@@ -64,12 +64,12 @@ class UnusedVariables {
             key: "unused_variables",
             title: "Unused variables",
             shortDescription: `Checks for unused variables and constants`,
-            extendedInformation: `WARNING: slow
-
-Experimental, might give false positives. Skips event parameters.
-
-Note that this currently does not work if the source code uses macros.
-
+            extendedInformation: `WARNING: slow
+
+Experimental, might give false positives. 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 syntax errors. Errors found in INCLUDES are reported for the main program.`,
             tags: [_irule_1.RuleTag.Quickfix],
             pragma: "##NEEDED",