@abaplint/core 2.119.49 → 2.119.50

package/build/src/abap/2_statements/expressions/_derived_types.js

@@ -0,0 +1,53 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.commonDerivedTypes = commonDerivedTypes;
+exports.derivedTypesAlt = derivedTypesAlt;
+const combi_1 = require("../combi");
+const _1 = require(".");
+const version_1 = require("../../../version");
+function commonDerivedTypes() {
+    const entity = (0, combi_1.alt)(_1.TypeName, _1.EntityAssociation);
+    // more-specific keyword phrases (e.g. "FAILED LATE") must precede less-specific ones ("FAILED")
+    return [
+        (0, combi_1.ver)(version_1.Release.v774, (0, combi_1.seq)("FAILED LATE", entity)),
+        (0, combi_1.ver)(version_1.Release.v774, (0, combi_1.seq)("MAPPED LATE", entity)),
+        (0, combi_1.ver)(version_1.Release.v774, (0, combi_1.seq)("REPORTED LATE", entity)),
+        (0, combi_1.ver)(version_1.Release.v777, (0, combi_1.seq)("FAILED EARLY", entity)),
+        (0, combi_1.ver)(version_1.Release.v777, (0, combi_1.seq)("MAPPED EARLY", entity)),
+        (0, combi_1.ver)(version_1.Release.v777, (0, combi_1.seq)("REPORTED EARLY", entity)),
+        (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("FAILED", entity)),
+        (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("MAPPED", entity)),
+        (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("REPORTED", entity)),
+        (0, combi_1.ver)(version_1.Release.v773, (0, combi_1.seq)("FUNCTION IMPORT", entity)),
+        (0, combi_1.ver)(version_1.Release.v773, (0, combi_1.seq)("FUNCTION RESULT", entity)),
+        (0, combi_1.ver)(version_1.Release.v773, (0, combi_1.seq)("ACTION RESULT", entity)),
+        (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("INSTANCE AUTHORIZATION KEY", entity)),
+        (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("INSTANCE AUTHORIZATION REQUEST", entity)),
+        (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("INSTANCE AUTHORIZATION RESULT", entity)),
+        (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("INSTANCE FEATURES KEY", entity)),
+        (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("INSTANCE FEATURES REQUEST", entity)),
+        (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("INSTANCE FEATURES RESULT", entity)),
+        (0, combi_1.ver)(version_1.Release.v780, (0, combi_1.seq)("AUTHORIZATION RESULT", entity)),
+        (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("AUTHORIZATION KEY", entity)),
+        (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("FEATURES RESULT", entity)),
+        (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("FEATURES KEY", entity)),
+        (0, combi_1.ver)(version_1.Release.v780, (0, combi_1.seq)("PERMISSIONS KEY", entity)),
+        (0, combi_1.seq)("READ IMPORT", entity),
+        (0, combi_1.seq)("READ RESULT", entity),
+        (0, combi_1.ver)(version_1.Release.v915, (0, combi_1.seq)("READ CHANGES", entity)),
+        (0, combi_1.seq)("CREATE", entity),
+        (0, combi_1.seq)("DELETE", entity),
+        (0, combi_1.seq)("UPDATE", entity),
+        (0, combi_1.ver)(version_1.Release.v777, (0, combi_1.seq)("LOCK", entity)),
+        (0, combi_1.ver)(version_1.Release.v775, (0, combi_1.seq)("KEY OF", entity)),
+        (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("DETERMINATION", entity)),
+        (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("VALIDATION", entity)),
+        (0, combi_1.ver)(version_1.Release.v778, (0, combi_1.seq)("CHANGE", entity)),
+    ];
+}
+function derivedTypesAlt(...extra) {
+    const all = [...extra, ...commonDerivedTypes()];
+    const [first, second, ...rest] = all;
+    return (0, combi_1.altPrio)(first, second, ...rest);
+}
+//# sourceMappingURL=_derived_types.js.map

package/build/src/abap/2_statements/expressions/type_structure.js

@@ -3,24 +3,16 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.TypeStructure = void 0;
 const combi_1 = require("../combi");
 const _1 = require(".");
+const version_1 = require("../../../version");
+const _derived_types_1 = require("./_derived_types");
 class TypeStructure extends combi_1.Expression {
     getRunnable() {
-        // todo, add version,
-        const hier = (0, combi_1.seq)("HIERARCHY", _1.NamespaceSimpleName);
-        const create = (0, combi_1.seq)("CREATE", (0, combi_1.alt)(_1.NamespaceSimpleName, _1.EntityAssociation));
-        const update = (0, combi_1.seq)("UPDATE", (0, combi_1.alt)(_1.NamespaceSimpleName, _1.EntityAssociation));
-        const readResult = (0, combi_1.seq)("READ RESULT", (0, combi_1.alt)(_1.NamespaceSimpleName, _1.EntityAssociation));
-        const readLink = (0, combi_1.seq)("READ LINK", _1.EntityAssociation);
-        const action = (0, combi_1.seq)("ACTION IMPORT", _1.Source);
-        const permissionsRequest = (0, combi_1.seq)("PERMISSIONS REQUEST", _1.NamespaceSimpleName);
-        const evt = (0, combi_1.seq)("EVENT", _1.EventName);
-        const failedEarly = (0, combi_1.seq)("FAILED EARLY", _1.NamespaceSimpleName);
-        const mappedEarly = (0, combi_1.seq)("MAPPED EARLY", _1.NamespaceSimpleName);
-        const reportedEarly = (0, combi_1.seq)("REPORTED EARLY", _1.NamespaceSimpleName);
-        const structure = (0, combi_1.seq)("STRUCTURE FOR", (0, combi_1.altPrio)(hier, evt, create, update, action, permissionsRequest, readLink, readResult));
-        const response = (0, combi_1.seq)("RESPONSE FOR", (0, combi_1.altPrio)(failedEarly, mappedEarly, reportedEarly));
-        const request = (0, combi_1.seq)("REQUEST FOR CHANGE", _1.NamespaceSimpleName);
-        return (0, combi_1.seq)("TYPE", (0, combi_1.altPrio)(structure, response, request));
+        const entity = (0, combi_1.alt)(_1.TypeName, _1.EntityAssociation);
+        const derivedTypes = (0, _derived_types_1.derivedTypesAlt)((0, combi_1.ver)(version_1.Release.v779, (0, combi_1.seq)("FUNCTION REQUEST", entity)), (0, combi_1.ver)(version_1.Release.v779, (0, combi_1.seq)("ACTION REQUEST", entity)), (0, combi_1.seq)("ACTION IMPORT", _1.Source), (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("GLOBAL AUTHORIZATION REQUEST", entity)), (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("GLOBAL AUTHORIZATION RESULT", entity)), (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("GLOBAL FEATURES REQUEST", entity)), (0, combi_1.ver)(version_1.Release.v781, (0, combi_1.seq)("GLOBAL FEATURES RESULT", entity)), (0, combi_1.ver)(version_1.Release.v780, (0, combi_1.seq)("AUTHORIZATION REQUEST", entity)), (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("FEATURES REQUEST", entity)), (0, combi_1.seq)("PERMISSIONS REQUEST", _1.NamespaceSimpleName), (0, combi_1.ver)(version_1.Release.v780, (0, combi_1.seq)("PERMISSIONS RESULT", entity)), (0, combi_1.seq)("READ LINK", _1.EntityAssociation), (0, combi_1.seq)("HIERARCHY", _1.NamespaceSimpleName), (0, combi_1.seq)("EVENT", _1.EventName));
+        const structure = (0, combi_1.ver)(version_1.Release.v774, (0, combi_1.seq)("STRUCTURE FOR", derivedTypes));
+        const response = (0, combi_1.ver)(version_1.Release.v776, (0, combi_1.seq)("RESPONSE FOR", (0, combi_1.altPrio)((0, combi_1.seq)("FAILED EARLY", _1.NamespaceSimpleName), (0, combi_1.seq)("FAILED LATE", _1.NamespaceSimpleName), (0, combi_1.seq)("FAILED", _1.NamespaceSimpleName), (0, combi_1.seq)("MAPPED EARLY", _1.NamespaceSimpleName), (0, combi_1.seq)("MAPPED LATE", _1.NamespaceSimpleName), (0, combi_1.seq)("MAPPED", _1.NamespaceSimpleName), (0, combi_1.seq)("REPORTED EARLY", _1.NamespaceSimpleName), (0, combi_1.seq)("REPORTED LATE", _1.NamespaceSimpleName), (0, combi_1.seq)("REPORTED", _1.NamespaceSimpleName))));
+        const request = (0, combi_1.ver)(version_1.Release.v778, (0, combi_1.seq)("REQUEST FOR", (0, combi_1.alt)("CHANGE", "DELETE"), _1.NamespaceSimpleName));
+        return (0, combi_1.seq)("TYPE", (0, combi_1.altPrio)(structure, response, request), (0, combi_1.optPrio)("VALUE IS INITIAL"));
     }
 }
 exports.TypeStructure = TypeStructure;

package/build/src/abap/2_statements/expressions/type_table.js

@@ -6,6 +6,7 @@ const _1 = require(".");
 const field_chain_1 = require("./field_chain");
 const type_table_key_1 = require("./type_table_key");
 const version_1 = require("../../../version");
+const _derived_types_1 = require("./_derived_types");
 class TypeTable extends combi_1.Expression {
     getRunnable() {
         const header = "WITH HEADER LINE";
@@ -19,7 +20,9 @@ class TypeTable extends combi_1.Expression {
         // "WITH" is not allowed as a field name in keys
         const typetable = (0, combi_1.alt)(generic, (0, combi_1.seq)(normal1, (0, combi_1.alt)((0, combi_1.opt)((0, combi_1.per)(header, initial, (0, combi_1.plusPrio)(type_table_key_1.TypeTableKey))), (0, combi_1.seq)((0, combi_1.plus)(type_table_key_1.TypeTableKey), (0, combi_1.optPrio)(initial))), (0, combi_1.optPrio)("VALUE IS INITIAL")));
         const occurs = (0, combi_1.seq)("OCCURS", (0, combi_1.altPrio)(_1.Integer, field_chain_1.FieldChain));
-        const derived = (0, combi_1.ver)(version_1.Release.v754, (0, combi_1.seq)("TABLE FOR", (0, combi_1.altPrio)("ACTION IMPORT", "ACTION RESULT", "CREATE", "EVENT", "REPORTED EARLY", "READ IMPORT", "FAILED EARLY", "FAILED", "LOCK", "DETERMINATION", "READ RESULT", "UPDATE", "DELETE"), (0, combi_1.alt)(_1.TypeName, _1.EntityAssociation)), { also: combi_1.AlsoIn.OpenABAP });
+        const entity = (0, combi_1.alt)(_1.TypeName, _1.EntityAssociation);
+        const derivedTypes = (0, _derived_types_1.derivedTypesAlt)((0, combi_1.ver)(version_1.Release.v773, (0, combi_1.seq)("ACTION IMPORT", entity)), (0, combi_1.seq)("READ LINK", entity), (0, combi_1.seq)("EVENT", entity));
+        const derived = (0, combi_1.ver)(version_1.Release.v770, (0, combi_1.seq)("TABLE FOR", derivedTypes, (0, combi_1.optPrio)("VALUE IS INITIAL")), { also: combi_1.AlsoIn.OpenABAP });
         const oldType = (0, combi_1.seq)((0, combi_1.opt)("REF TO"), _1.TypeName, (0, combi_1.alt)((0, combi_1.seq)(occurs, (0, combi_1.opt)(header)), header));
         const oldLike = (0, combi_1.seq)((0, combi_1.opt)("REF TO"), field_chain_1.FieldChain, (0, combi_1.alt)((0, combi_1.seq)(occurs, (0, combi_1.opt)(header)), header));
         const typeLine = (0, combi_1.seq)("LINE OF", _1.TypeName, occurs, header);

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

@@ -684,25 +684,12 @@ class BasicTypes {
         return name.includes("\\") || name.includes("~");
     }
     isRAPTableFor(text) {
-        return text.startsWith("TYPE TABLE FOR ACTION IMPORT ")
-            || text.startsWith("TYPE TABLE FOR ACTION RESULT ")
-            || text.startsWith("TYPE TABLE FOR CREATE ")
-            || text.startsWith("TYPE TABLE FOR DELETE ")
-            || text.startsWith("TYPE TABLE FOR DETERMINATION ")
-            || text.startsWith("TYPE TABLE FOR EVENT ")
-            || text.startsWith("TYPE TABLE FOR FAILED ")
-            || text.startsWith("TYPE TABLE FOR FAILED EARLY ")
-            || text.startsWith("TYPE TABLE FOR LOCK ")
-            || text.startsWith("TYPE TABLE FOR READ ")
-            || text.startsWith("TYPE TABLE FOR READ IMPORT ")
-            || text.startsWith("TYPE TABLE FOR READ RESULT ")
-            || text.startsWith("TYPE TABLE FOR REPORTED EARLY ")
-            || text.startsWith("TYPE TABLE FOR UPDATE ");
+        return text.startsWith("TYPE TABLE FOR ");
     }
     isRAPTypeStructure(text) {
         return text.startsWith("TYPE STRUCTURE FOR ")
             || text.startsWith("TYPE RESPONSE FOR ")
-            || text.startsWith("TYPE REQUEST FOR CHANGE ");
+            || text.startsWith("TYPE REQUEST FOR ");
     }
     // todo, rewrite this method
     resolveTypeChain(expr) {

package/build/src/registry.js

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

package/build/src/rules/avoid_use.js

@@ -89,6 +89,8 @@ TEST-SEAMS: https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md
 
 BREAK points`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+            badExample: `DESCRIBE TABLE items LINES count.`,
+            goodExample: `count = lines( items ).`,
         };
     }
     getDescription(statement) {

package/build/src/rules/class_attribute_names.js

@@ -37,6 +37,14 @@ class ClassAttributeNames extends _abap_rule_1.ABAPRule {
             title: "Class attributes naming",
             shortDescription: `Allows you to enforce a pattern, such as a prefix, for class variable names.`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],
+            badExample: `CLASS zcl_foo DEFINITION PUBLIC.
+  PUBLIC SECTION.
+    DATA counter TYPE i.
+ENDCLASS.`,
+            goodExample: `CLASS zcl_foo DEFINITION PUBLIC.
+  PUBLIC SECTION.
+    DATA mv_counter TYPE i.
+ENDCLASS.`,
         };
     }
     getDescription(actual, expected) {

package/build/src/rules/constructor_visibility_public.js

@@ -24,6 +24,14 @@ 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],
+            badExample: `CLASS zcl_foo DEFINITION PUBLIC CREATE PRIVATE.
+  PRIVATE SECTION.
+    METHODS constructor.
+ENDCLASS.`,
+            goodExample: `CLASS zcl_foo DEFINITION PUBLIC CREATE PRIVATE.
+  PUBLIC SECTION.
+    METHODS constructor.
+ENDCLASS.`,
         };
     }
     getMessage() {

package/build/src/rules/dangerous_statement.js

@@ -79,6 +79,8 @@ class DangerousStatement extends _abap_rule_1.ABAPRule {
             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],
+            badExample: `SELECT * FROM (lv_table_name) INTO TABLE @DATA(lt_rows).`,
+            goodExample: `SELECT * FROM zknown_table INTO TABLE @DATA(lt_rows).`,
         };
     }
     getDescription(statement) {

package/build/src/rules/db_operation_in_loop.js

@@ -54,6 +54,17 @@ class DbOperationInLoop extends _abap_rule_1.ABAPRule {
             title: "Database operation in loop",
             shortDescription: `Database operation in LOOP/DO/WHILE`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Performance],
+            badExample: `LOOP AT lt_items INTO DATA(ls_item).
+  SELECT SINGLE name FROM zcustomer INTO @DATA(lv_name) WHERE id = @ls_item-customer_id.
+ENDLOOP.`,
+            goodExample: `ASSERT lines( lt_items ) > 0.
+SELECT id, name FROM zcustomer INTO TABLE @DATA(lt_customers)
+  FOR ALL ENTRIES IN @lt_items
+  WHERE id = @lt_items-customer_id.
+
+LOOP AT lt_items INTO DATA(ls_item).
+  READ TABLE lt_customers INTO DATA(ls_customer) WITH KEY id = ls_item-customer_id.
+ENDLOOP.`,
         };
     }
     getConfig() {

package/build/src/rules/exit_or_check.js

@@ -63,6 +63,12 @@ Use RETURN to leave procesing blocks instead.`,
 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],
+            badExample: `CHECK is_valid = abap_true.
+do_something( ).`,
+            goodExample: `IF is_valid = abap_false.
+  RETURN.
+ENDIF.
+do_something( ).`,
         };
     }
     getConfig() {

package/build/src/rules/form_tables_obsolete.js

@@ -55,6 +55,10 @@ class FormTablesObsolete extends _abap_rule_1.ABAPRule {
             shortDescription: `Checks for TABLES parameters in forms.`,
             extendedInformation: `https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abapform_tables.htm`,
             tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `FORM update_items TABLES items.
+ENDFORM.`,
+            goodExample: `FORM update_items CHANGING items TYPE ty_items.
+ENDFORM.`,
         };
     }
     getMessage() {

package/build/src/rules/function_module_recommendations.js

@@ -110,6 +110,10 @@ class FunctionModuleRecommendations extends _abap_rule_1.ABAPRule {
             shortDescription: `Suggests replacements for various function modules`,
             extendedInformation: `https://docs.abapopenchecks.org/checks/53/`,
             tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `CALL FUNCTION 'GUID_CREATE'
+  IMPORTING
+    ev_guid_32 = guid.`,
+            goodExample: `guid = cl_system_uuid=>create_uuid_c32_static( ).`,
         };
     }
     getConfig() {

package/build/src/rules/line_length.js

@@ -27,6 +27,9 @@ class LineLength extends _abap_rule_1.ABAPRule {
             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],
+            badExample: "DATA long_text TYPE string VALUE 'this line keeps going until it is longer than the configured maximum'.",
+            goodExample: `DATA long_text TYPE string.
+long_text = 'split long expressions across multiple lines'.`,
         };
     }
     getConfig() {

package/build/src/rules/local_class_naming.js

@@ -30,6 +30,10 @@ class LocalClassNaming extends _abap_rule_1.ABAPRule {
             title: "Local class naming conventions",
             shortDescription: `Allows you to enforce a pattern, such as a prefix, for local class names.`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],
+            badExample: `CLASS helper DEFINITION.
+ENDCLASS.`,
+            goodExample: `CLASS lcl_helper DEFINITION.
+ENDCLASS.`,
         };
     }
     getDescription(expected, actual) {

package/build/src/rules/method_parameter_names.js

@@ -34,6 +34,12 @@ class MethodParameterNames {
             title: "Method parameter naming conventions",
             shortDescription: `Allows you to enforce a pattern, such as a prefix, for method parameter names`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],
+            badExample: `METHODS get_name
+  IMPORTING customer TYPE string
+  RETURNING VALUE(name) TYPE string.`,
+            goodExample: `METHODS get_name
+  IMPORTING iv_customer TYPE string
+  RETURNING VALUE(rv_name) TYPE string.`,
         };
     }
     getDescription(expected, actual) {

package/build/src/rules/no_aliases.js

@@ -25,6 +25,13 @@ class NoAliases extends _abap_rule_1.ABAPRule {
             title: "No ALIASES",
             shortDescription: `Detects use of the ALIAS statement`,
             tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `INTERFACE lif_foo.
+  METHODS long_method_name.
+  ALIASES short FOR long_method_name.
+ENDINTERFACE.`,
+            goodExample: `INTERFACE lif_foo.
+  METHODS long_method_name.
+ENDINTERFACE.`,
         };
     }
     getConfig() {

package/build/src/rules/no_comments_between_methods.js

@@ -53,8 +53,22 @@ class NoCommentsBetweenMethods extends _abap_rule_1.ABAPRule {
         return {
             key: "no_comments_between_methods",
             title: "No comments between methods in global classes",
-            shortDescription: `Its not possible to have comments between methods in global classes.`,
+            shortDescription: `It's not possible to have comments between methods in global classes.`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Syntax],
+            badExample: `CLASS zcl_foo IMPLEMENTATION.
+  METHOD first.
+  ENDMETHOD.
+  " comment
+  METHOD second.
+  ENDMETHOD.
+ENDCLASS.`,
+            goodExample: `CLASS zcl_foo IMPLEMENTATION.
+  METHOD first.
+  ENDMETHOD.
+
+  METHOD second.
+  ENDMETHOD.
+ENDCLASS.`,
         };
     }
     getConfig() {

package/build/src/rules/no_public_attributes.js

@@ -31,6 +31,14 @@ class NoPublicAttributes extends _abap_rule_1.ABAPRule {
 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],
+            badExample: `CLASS zcl_foo DEFINITION PUBLIC.
+  PUBLIC SECTION.
+    DATA counter TYPE i.
+ENDCLASS.`,
+            goodExample: `CLASS zcl_foo DEFINITION PUBLIC.
+  PRIVATE SECTION.
+    DATA counter TYPE i.
+ENDCLASS.`,
         };
     }
     getDescription(name) {

package/build/src/rules/prefix_is_current_class.js

@@ -67,6 +67,14 @@ class PrefixIsCurrentClass extends _abap_rule_1.ABAPRule {
             // eslint-disable-next-line max-len
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#omit-the-self-reference-me-when-calling-an-instance-attribute-or-method`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
+            badExample: `METHOD run.
+  zcl_current=>helper( ).
+  me->do_work( ).
+ENDMETHOD.`,
+            goodExample: `METHOD run.
+  helper( ).
+  do_work( ).
+ENDMETHOD.`,
         };
     }
     getConfig() {

package/build/src/rules/static_call_via_instance.js

@@ -29,6 +29,12 @@ class StaticCallViaInstance extends _abap_rule_1.ABAPRule {
             shortDescription: `Static method call via instance variable`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#dont-call-static-methods-through-instance-variables`,
             tags: [_irule_1.RuleTag.Styleguide],
+            badExample: `METHOD run.
+  NEW lcl_helper( )->static_method( ).
+ENDMETHOD.`,
+            goodExample: `METHOD run.
+  lcl_helper=>static_method( ).
+ENDMETHOD.`,
         };
     }
     getConfig() {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.119.49",
+    "version": "2.119.50",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",