@abaplint/core 2.87.0 → 2.88.2

package/build/src/rules/{pragma_placement.js → pragma_style.js}

@@ -1,23 +1,28 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.PragmaPlacement = exports.PragmaPlacementConf = void 0;
+exports.PragmaStyle = exports.PragmaStyleConf = 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 PragmaPlacementConf extends _basic_rule_config_1.BasicRuleConfig {
+const keyword_case_1 = require("./keyword_case");
+class PragmaStyleConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        this.style = keyword_case_1.KeywordCaseStyle.Upper;
+    }
 }
-exports.PragmaPlacementConf = PragmaPlacementConf;
-class PragmaPlacement extends _abap_rule_1.ABAPRule {
+exports.PragmaStyleConf = PragmaStyleConf;
+class PragmaStyle extends _abap_rule_1.ABAPRule {
     constructor() {
         super(...arguments);
-        this.conf = new PragmaPlacementConf();
+        this.conf = new PragmaStyleConf();
     }
     getMetadata() {
         return {
-            key: "pragma_placement",
-            title: "Pragma Placement",
-            shortDescription: `Place pragmas at end of statements`,
+            key: "pragma_style",
+            title: "Pragma Style",
+            shortDescription: `Check pragmas placment and case`,
             tags: [_irule_1.RuleTag.SingleFile],
             extendedInformation: `https://help.sap.com/doc/abapdocu_cp_index_htm/CLOUD/en-US/abenpragma.htm`,
             badExample: `DATA field ##NO_TEXT TYPE i.`,
@@ -29,6 +34,9 @@ class PragmaPlacement extends _abap_rule_1.ABAPRule {
     }
     setConfig(conf) {
         this.conf = conf;
+        if (this.conf.style === undefined) {
+            this.conf.style = keyword_case_1.KeywordCaseStyle.Upper;
+        }
     }
     runParsed(file) {
         const issues = [];
@@ -47,10 +55,20 @@ class PragmaPlacement extends _abap_rule_1.ABAPRule {
                     issues.push(issue);
                     continue; // max one finding per statement
                 }
+                if (this.conf.style === keyword_case_1.KeywordCaseStyle.Upper && p.getStr() !== p.getStr().toUpperCase()) {
+                    const message = "Upper case pragmas";
+                    const issue = issue_1.Issue.atToken(file, p, message, this.getMetadata().key, this.conf.severity);
+                    issues.push(issue);
+                }
+                else if (this.conf.style === keyword_case_1.KeywordCaseStyle.Lower && p.getStr() !== p.getStr().toLowerCase()) {
+                    const message = "Lower case pragmas";
+                    const issue = issue_1.Issue.atToken(file, p, message, this.getMetadata().key, this.conf.severity);
+                    issues.push(issue);
+                }
             }
         }
         return issues;
     }
 }
-exports.PragmaPlacement = PragmaPlacement;
-//# sourceMappingURL=pragma_placement.js.map
+exports.PragmaStyle = PragmaStyle;
+//# sourceMappingURL=pragma_style.js.map

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

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/slow_parameter_passing.js

@@ -49,7 +49,7 @@ class SlowParameterPassing {
                 }
                 const writes = this.listWritePositions(m, id);
                 if (writes.length === 0) {
-                    const message = "Parameter passed by VALUE but not changed";
+                    const message = "Parameter " + id.getName() + " passed by VALUE but not changed";
                     issues.push(issue_1.Issue.atIdentifier(id, message, this.getMetadata().key, this.getConfig().severity));
                 }
             }

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_pragma.js

@@ -0,0 +1,126 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.UnnecessaryPragma = exports.UnnecessaryPragmaConf = void 0;
+const issue_1 = require("../issue");
+const Statements = require("../abap/2_statements/statements");
+const Expressions = require("../abap/2_statements/expressions");
+const _abap_rule_1 = require("./_abap_rule");
+const _basic_rule_config_1 = require("./_basic_rule_config");
+const _statement_1 = require("../abap/2_statements/statements/_statement");
+const _irule_1 = require("./_irule");
+class UnnecessaryPragmaConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.UnnecessaryPragmaConf = UnnecessaryPragmaConf;
+class UnnecessaryPragma extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new UnnecessaryPragmaConf();
+    }
+    getMetadata() {
+        return {
+            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`,
+            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.`,
+            goodExample: `TRY.
+    ...
+  CATCH zcx_abapgit_exception.
+    RETURN.
+ENDTRY.
+MESSAGE w125(zbar) WITH c_foo INTO message.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const issues = [];
+        let noHandler = false;
+        const statements = file.getStatements();
+        for (let i = 0; i < statements.length; i++) {
+            const statement = statements[i];
+            if (statement.get() instanceof Statements.EndTry) {
+                noHandler = false;
+            }
+            else if (statement.get() instanceof _statement_1.Comment) {
+                continue;
+            }
+            else if (noHandler === true && !(statement.get() instanceof Statements.Catch)) {
+                const message = "NO_HANDLER pragma or pseudo comment can be removed";
+                const issue = issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity);
+                issues.push(issue);
+                noHandler = false;
+            }
+            else {
+                noHandler = this.containsNoHandler(statement, statements[i + 1]);
+            }
+            issues.push(...this.checkText(statement, file));
+            issues.push(...this.checkNeeded(statement, file));
+        }
+        return issues;
+    }
+    checkText(statement, file) {
+        const p = statement.getPragmas().find(t => t.getStr().toUpperCase() === "##NO_TEXT");
+        if (p === undefined) {
+            return [];
+        }
+        if (statement.findFirstExpression(Expressions.ConstantString) === undefined
+            && statement.findFirstExpression(Expressions.StringTemplate) === undefined) {
+            const message = "There is no text, NO_TEXT can be removed";
+            return [issue_1.Issue.atToken(file, p, message, this.getMetadata().key, this.getConfig().severity)];
+        }
+        return [];
+    }
+    checkNeeded(statement, file) {
+        const p = statement.getPragmas().find(t => t.getStr().toUpperCase() === "##NEEDED");
+        if (p === undefined) {
+            return [];
+        }
+        if (statement.findFirstExpression(Expressions.InlineData) === undefined
+            && !(statement.get() instanceof Statements.Parameter)
+            && !(statement.get() instanceof Statements.Data)
+            && !(statement.get() instanceof Statements.DataEnd)
+            && !(statement.get() instanceof Statements.Type)
+            && !(statement.get() instanceof Statements.TypeEnd)
+            && !(statement.get() instanceof Statements.Constant)
+            && !(statement.get() instanceof Statements.ConstantEnd)
+            && !(statement.get() instanceof Statements.TypeEnum)
+            && !(statement.get() instanceof Statements.TypeEnumEnd)
+            && !(statement.get() instanceof Statements.MethodImplementation)
+            && !(statement.get() instanceof Statements.MethodDef)
+            && statement.findFirstExpression(Expressions.InlineFS) === undefined) {
+            const message = "There is no data definition, NEEDED can be removed";
+            return [issue_1.Issue.atToken(file, p, message, this.getMetadata().key, this.getConfig().severity)];
+        }
+        return [];
+    }
+    containsNoHandler(statement, next) {
+        for (const t of statement.getPragmas()) {
+            if (t.getStr().toUpperCase() === "##NO_HANDLER") {
+                return true;
+            }
+        }
+        if (next
+            && next.get() instanceof _statement_1.Comment
+            && next.concatTokens().toUpperCase().includes("#EC NO_HANDLER")) {
+            return true;
+        }
+        return false;
+    }
+}
+exports.UnnecessaryPragma = UnnecessaryPragma;
+//# sourceMappingURL=unnecessary_pragma.js.map

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
-
-Skips event parameters.
-
-Note that this currently does not work if the source code uses macros.
-
+            extendedInformation: `WARNING: slow
+
+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",

package/build/src/rules/use_bool_expression.js

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

package/build/src/rules/use_line_exists.js

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

package/build/src/rules/use_new.js

@@ -23,10 +23,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 #( ).`,

package/build/src/rules/when_others_last.js

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