@abaplint/core 2.88.8 → 2.89.0

package/build/abaplint.d.ts

@@ -741,6 +741,7 @@ declare class Class extends ABAPObject {
     getNameFromXML(): string | undefined;
     getCategory(): string | undefined;
     getLocalsImpFile(): ABAPFile | undefined;
+    getTestclassFile(): ABAPFile | undefined;
     private parseXML;
 }
 
@@ -1952,6 +1953,7 @@ declare namespace Expressions {
         TypeTableKey,
         TypeTable,
         Type_2 as Type,
+        SQLUpTo,
         ValueBody,
         ValueBodyLines,
         Value,
@@ -2651,7 +2653,8 @@ export declare const enum IdentifierMeta {
     BuiltIn = "built-in",
     DDIC = "ddic",
     Static = "static",
-    Enum = "enum"
+    Enum = "enum",
+    SelectionScreenTab = "selection_screen_tab"
 }
 
 export declare interface IDependency {
@@ -4977,6 +4980,10 @@ declare class SQLTarget extends Expression {
     getRunnable(): IStatementRunnable;
 }
 
+declare class SQLUpTo extends Expression {
+    getRunnable(): IStatementRunnable;
+}
+
 declare class StartOfSelection implements IStatement {
     getMatcher(): IStatementRunnable;
 }

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

@@ -200,6 +200,7 @@ __exportStar(require("./type_param"), exports);
 __exportStar(require("./type_table_key"), exports);
 __exportStar(require("./type_table"), exports);
 __exportStar(require("./type"), exports);
+__exportStar(require("./sql_up_to"), exports);
 __exportStar(require("./value_body"), exports);
 __exportStar(require("./value_body_lines"), exports);
 __exportStar(require("./value"), exports);

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

@@ -8,16 +8,16 @@ const sql_group_by_1 = require("./sql_group_by");
 const sql_into_structure_1 = require("./sql_into_structure");
 const tokens_1 = require("../../1_lexer/tokens");
 const sql_field_name_1 = require("./sql_field_name");
+const sql_up_to_1 = require("./sql_up_to");
 class Select extends combi_1.Expression {
     getRunnable() {
         const into = (0, combi_1.altPrio)(_1.SQLIntoTable, sql_into_structure_1.SQLIntoStructure);
         const where = (0, combi_1.seq)("WHERE", _1.SQLCond);
-        const up = (0, combi_1.seq)("UP TO", _1.SQLSource, "ROWS");
         const offset = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)("OFFSET", _1.SQLSource));
         const client = (0, combi_1.str)("CLIENT SPECIFIED");
         const bypass = (0, combi_1.str)("BYPASSING BUFFER");
         const fields = (0, combi_1.seq)("FIELDS", _1.SQLFieldList);
-        const perm = (0, combi_1.per)(_1.SQLFrom, into, _1.SQLForAllEntries, where, _1.SQLOrderBy, up, offset, client, _1.SQLHaving, bypass, sql_group_by_1.SQLGroupBy, fields, _1.DatabaseConnection);
+        const perm = (0, combi_1.per)(_1.SQLFrom, into, _1.SQLForAllEntries, where, _1.SQLOrderBy, sql_up_to_1.SQLUpTo, offset, client, _1.SQLHaving, bypass, sql_group_by_1.SQLGroupBy, fields, _1.DatabaseConnection);
         const paren = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WParenLeftW), sql_field_name_1.SQLFieldName, (0, combi_1.tok)(tokens_1.WParenRightW));
         const ret = (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)("DISTINCT", (0, combi_1.optPrio)((0, combi_1.seq)("SINGLE", (0, combi_1.optPrio)("FOR UPDATE")))), (0, combi_1.optPrio)((0, combi_1.altPrio)(_1.SQLFieldList, paren)), perm, (0, combi_1.optPrio)(_1.SQLHints));
         return ret;

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

@@ -9,16 +9,16 @@ const sql_into_structure_1 = require("./sql_into_structure");
 const sql_field_list_1 = require("./sql_field_list");
 const sql_hints_1 = require("./sql_hints");
 const sql_field_list_loop_1 = require("./sql_field_list_loop");
+const sql_up_to_1 = require("./sql_up_to");
 class SelectLoop extends combi_1.Expression {
     getRunnable() {
         const where = (0, combi_1.seq)("WHERE", _1.SQLCond);
         const client = "CLIENT SPECIFIED";
         const bypass = "BYPASSING BUFFER";
-        const up = (0, combi_1.seq)("UP TO", _1.SQLSource, "ROWS");
         const pack = (0, combi_1.seq)("PACKAGE SIZE", _1.SQLSource);
         const tab = (0, combi_1.seq)(_1.SQLIntoTable, (0, combi_1.alt)(pack, (0, combi_1.seq)(_1.SQLFrom, pack), (0, combi_1.seq)(pack, _1.SQLFrom)));
-        const perm = (0, combi_1.per)(_1.SQLFrom, where, up, sql_order_by_1.SQLOrderBy, sql_having_1.SQLHaving, client, bypass, _1.SQLGroupBy, _1.SQLForAllEntries, (0, combi_1.alt)(tab, sql_into_structure_1.SQLIntoStructure));
-        const strict = (0, combi_1.seq)(_1.SQLFrom, "FIELDS", sql_field_list_1.SQLFieldList, where, sql_into_structure_1.SQLIntoStructure, up);
+        const perm = (0, combi_1.per)(_1.SQLFrom, where, sql_up_to_1.SQLUpTo, sql_order_by_1.SQLOrderBy, sql_having_1.SQLHaving, client, bypass, _1.SQLGroupBy, _1.SQLForAllEntries, (0, combi_1.alt)(tab, sql_into_structure_1.SQLIntoStructure));
+        const strict = (0, combi_1.seq)(_1.SQLFrom, "FIELDS", sql_field_list_1.SQLFieldList, where, sql_into_structure_1.SQLIntoStructure, sql_up_to_1.SQLUpTo);
         const ret = (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)((0, combi_1.seq)((0, combi_1.optPrio)("DISTINCT"), sql_field_list_loop_1.SQLFieldListLoop, perm), strict), (0, combi_1.optPrio)(sql_hints_1.SQLHints));
         return ret;
     }

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

@@ -0,0 +1,13 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.SQLUpTo = void 0;
+const combi_1 = require("../combi");
+const sql_source_1 = require("./sql_source");
+class SQLUpTo extends combi_1.Expression {
+    getRunnable() {
+        const up = (0, combi_1.seq)("UP TO", sql_source_1.SQLSource, "ROWS");
+        return up;
+    }
+}
+exports.SQLUpTo = SQLUpTo;
+//# sourceMappingURL=sql_up_to.js.map

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

@@ -5,8 +5,13 @@ const Expressions = require("../../2_statements/expressions");
 const _typed_identifier_1 = require("../../types/_typed_identifier");
 const data_definition_1 = require("../expressions/data_definition");
 const unknown_type_1 = require("../../types/basic/unknown_type");
+const basic_types_1 = require("../basic_types");
 class Data {
     runSyntax(node, scope, filename) {
+        const val = node.findFirstExpression(Expressions.Value);
+        if (val) {
+            new basic_types_1.BasicTypes(filename, scope).findValue(node);
+        }
         const dd = node.findFirstExpression(Expressions.DataDefinition);
         if (dd) {
             return new data_definition_1.DataDefinition().runSyntax(dd, scope, filename);

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

@@ -18,7 +18,10 @@ class SelectionScreen {
                 { name: "DYNNR", type: new basic_1.CharacterType(4) },
                 { name: "ACTIVETAB", type: new basic_1.CharacterType(132) },
             ]);
-            scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(name, filename, type));
+            scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(name, filename, type, ["selection_screen_tab" /* SelectionScreenTab */]));
+        }
+        else if (concat.startsWith("SELECTION-SCREEN TAB")) {
+            scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(name, filename, new basic_1.CharacterType(83), ["selection_screen_tab" /* SelectionScreenTab */]));
         }
         else {
             scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(name, filename, new basic_1.CharacterType(83)));

package/build/src/objects/class.js

@@ -78,6 +78,14 @@ class Class extends _abap_object_1.ABAPObject {
         }
         return undefined;
     }
+    getTestclassFile() {
+        for (const file of this.getABAPFiles()) {
+            if (file.getFilename().endsWith(".clas.testclasses.abap")) {
+                return file;
+            }
+        }
+        return undefined;
+    }
     /////////////////////////
     parseXML() {
         if (this.parsedXML !== undefined) {

package/build/src/registry.js

@@ -68,7 +68,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.88.8";
+        return "2.89.0";
     }
     getDDICReferences() {
         return this.references;

package/build/src/rules/check_transformation_exists.js

@@ -40,13 +40,15 @@ class CheckTransformationExists extends _abap_rule_1.ABAPRule {
         }
         for (const s of file.getStatements()) {
             if (s.get() instanceof statements_1.CallTransformation) {
-                const name = s.findFirstExpression(expressions_1.NamespaceSimpleName);
-                if (name === undefined) {
+                const nameExpression = s.findFirstExpression(expressions_1.NamespaceSimpleName);
+                if (nameExpression === undefined) {
                     continue;
                 }
-                const tok = name.getFirstToken();
-                if (this.reg.getObject("XSLT", tok.getStr()) === undefined) {
-                    const issue = issue_1.Issue.atToken(file, tok, this.getDescription(tok.getStr()), this.getMetadata().key);
+                const tok = nameExpression.getFirstToken();
+                const name = tok.getStr();
+                if (this.reg.inErrorNamespace(name) === true
+                    && this.reg.getObject("XSLT", name) === undefined) {
+                    const issue = issue_1.Issue.atToken(file, tok, this.getDescription(name), this.getMetadata().key);
                     output.push(issue);
                 }
             }

package/build/src/rules/classic_exceptions_overlap.js

@@ -0,0 +1,69 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ClassicExceptionsOverlap = exports.ClassicExceptionsOverlapConf = void 0;
+const issue_1 = require("../issue");
+const _abap_rule_1 = require("./_abap_rule");
+const _basic_rule_config_1 = require("./_basic_rule_config");
+const Expressions = require("../abap/2_statements/expressions");
+const _irule_1 = require("./_irule");
+class ClassicExceptionsOverlapConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.ClassicExceptionsOverlapConf = ClassicExceptionsOverlapConf;
+class ClassicExceptionsOverlap extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new ClassicExceptionsOverlapConf();
+    }
+    getMetadata() {
+        return {
+            key: "classic_exceptions_overlap",
+            title: "Classic exceptions overlap when catching",
+            shortDescription: `Find overlapping classic exceptions`,
+            extendedInformation: `When debugging its typically good to know exactly which exception is caught`,
+            tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `  EXCEPTIONS
+    system_failure        = 1 MESSAGE lv_message
+    communication_failure = 1 MESSAGE lv_message
+    resource_failure      = 1
+    OTHERS                = 1.`,
+            goodExample: `  EXCEPTIONS
+    system_failure        = 1 MESSAGE lv_message
+    communication_failure = 2 MESSAGE lv_message
+    resource_failure      = 3
+    OTHERS                = 4.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        var _a;
+        const output = [];
+        const struc = file.getStructure();
+        if (struc === undefined) {
+            return []; // syntax error
+        }
+        for (const p of struc.findAllExpressions(Expressions.ParameterListExceptions)) {
+            const set = new Set();
+            for (const e of p.findAllExpressions(Expressions.ParameterException)) {
+                const text = (_a = e.findDirectExpression(Expressions.SimpleName)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase();
+                if (text === undefined) {
+                    continue;
+                }
+                if (set.has(text)) {
+                    const message = "Exception overlap, " + text;
+                    const issue = issue_1.Issue.atToken(file, e.getFirstToken(), message, this.getMetadata().key, this.getConfig().severity);
+                    output.push(issue);
+                    break;
+                }
+                set.add(text);
+            }
+        }
+        return output;
+    }
+}
+exports.ClassicExceptionsOverlap = ClassicExceptionsOverlap;
+//# sourceMappingURL=classic_exceptions_overlap.js.map

package/build/src/rules/definitions_top.js

@@ -122,7 +122,8 @@ class DefinitionsTop extends _abap_rule_1.ABAPRule {
         return undefined;
     }
     buildFix(file, statement, start) {
-        const concat = statement.concatTokens();
+        let concat = statement.concatTokens();
+        concat = concat.replace(/,$/, ".");
         const fix1 = edit_helper_1.EditHelper.deleteStatement(file, statement);
         const indentation = " ".repeat(statement.getFirstToken().getCol() - 1);
         const fix2 = edit_helper_1.EditHelper.insertAt(file, start.getEnd(), "\n" + indentation + concat);

package/build/src/rules/double_space.js

@@ -145,6 +145,7 @@ class DoubleSpace extends _abap_rule_1.ABAPRule {
             const upper = prev.get().getStr().toUpperCase();
             if (prev instanceof nodes_1.TokenNodeRegex
                 || upper === "("
+                || upper === ")"
                 || upper === "CHANGING"
                 || upper === "EXPORTING"
                 || upper === "OTHERS") {

package/build/src/rules/downport.js

@@ -502,14 +502,15 @@ ${indentation}${uniqueName} = ${source.concatTokens()}.\n${indentation}`);
             const tabixBackup = this.uniqueName(node.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
             const indentation = " ".repeat(node.getFirstToken().getStart().getCol() - 1);
             const firstToken = node.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.
-${indentation}sy-tabix = ${tabixBackup}.
 ${indentation}`);
             const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, startToken.getStart(), tableExpression.getLastToken().getEnd(), uniqueName);
             const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
@@ -760,14 +761,15 @@ ${indentation}RAISE EXCEPTION ${uniqueName2}.`;
         const condition = this.tableCondition(tableExpression);
         const tabixBackup = this.uniqueName(node.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.
-${indentation}sy-tabix = ${tabixBackup}.
 ${indentation}${uniqueName}`;
         const start = target.getFirstToken().getStart();
         const end = (_a = tableExpression.findDirectTokenByText("]")) === null || _a === void 0 ? void 0 : _a.getEnd();

package/build/src/rules/identical_form_names.js

@@ -8,6 +8,7 @@ const _abap_object_1 = require("../objects/_abap_object");
 class IdenticalFormNamesConf extends _basic_rule_config_1.BasicRuleConfig {
 }
 exports.IdenticalFormNamesConf = IdenticalFormNamesConf;
+// todo: deprecation candidate? this is/should be handled by the syntax check?
 class IdenticalFormNames {
     constructor() {
         this.conf = new IdenticalFormNamesConf();

package/build/src/rules/index.js

@@ -30,9 +30,9 @@ __exportStar(require("./check_abstract"), exports);
 __exportStar(require("./check_comments"), exports);
 __exportStar(require("./check_ddic"), exports);
 __exportStar(require("./check_include"), exports);
-__exportStar(require("./unnecessary_pragma"), exports);
 __exportStar(require("./check_subrc"), exports);
 __exportStar(require("./check_syntax"), exports);
+__exportStar(require("./classic_exceptions_overlap"), exports);
 __exportStar(require("./check_text_elements"), exports);
 __exportStar(require("./check_transformation_exists"), exports);
 __exportStar(require("./class_attribute_names"), exports);
@@ -80,7 +80,7 @@ __exportStar(require("./line_break_style"), exports);
 __exportStar(require("./line_length"), exports);
 __exportStar(require("./line_only_punc"), exports);
 __exportStar(require("./local_class_naming"), exports);
-__exportStar(require("./local_testclass_location"), exports);
+__exportStar(require("./local_testclass_consistency"), exports);
 __exportStar(require("./local_variable_names"), exports);
 __exportStar(require("./main_file_contents"), exports);
 __exportStar(require("./many_parentheses"), exports);
@@ -99,6 +99,7 @@ __exportStar(require("./nesting"), exports);
 __exportStar(require("./newline_between_methods"), exports);
 __exportStar(require("./no_aliases"), exports);
 __exportStar(require("./no_chained_assignment"), exports);
+__exportStar(require("./no_inline_in_optional_branches"), exports);
 __exportStar(require("./no_public_attributes"), exports);
 __exportStar(require("./no_yoda_conditions"), exports);
 __exportStar(require("./nrob_consistency"), exports);
@@ -144,6 +145,7 @@ __exportStar(require("./types_naming"), exports);
 __exportStar(require("./uncaught_exception"), exports);
 __exportStar(require("./unknown_types"), exports);
 __exportStar(require("./unnecessary_chaining"), exports);
+__exportStar(require("./unnecessary_pragma"), exports);
 __exportStar(require("./unreachable_code"), exports);
 __exportStar(require("./unsecure_fae"), exports);
 __exportStar(require("./unused_ddic"), exports);

package/build/src/rules/local_testclass_consistency.js

@@ -0,0 +1,63 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.LocalTestclassConsistency = exports.LocalTestclassConsistencyConf = void 0;
+const issue_1 = require("../issue");
+const _abap_rule_1 = require("./_abap_rule");
+const _basic_rule_config_1 = require("./_basic_rule_config");
+const objects_1 = require("../objects");
+const _irule_1 = require("./_irule");
+const version_1 = require("../version");
+class LocalTestclassConsistencyConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.LocalTestclassConsistencyConf = LocalTestclassConsistencyConf;
+class LocalTestclassConsistency extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new LocalTestclassConsistencyConf();
+    }
+    getMetadata() {
+        return {
+            key: "local_testclass_consistency",
+            title: "Local testclass consistency",
+            shortDescription: `Checks that local test classes are placed in the test include, and class unit test flag is set`,
+            tags: [_irule_1.RuleTag.Syntax],
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file, obj) {
+        var _a;
+        const issues = [];
+        if (this.reg.getConfig().getVersion() === version_1.Version.v700) {
+            // 700 does not have testclass includes
+            return [];
+        }
+        if (!(obj instanceof objects_1.Class)) {
+            return [];
+        }
+        for (const c of file.getInfo().listClassDefinitions()) {
+            if (c.isLocal && c.isForTesting && !file.getFilename().includes(".testclasses.abap")) {
+                const message = "Place local testclass \"" + c.name + "\" in the testclass include";
+                const issue = issue_1.Issue.atIdentifier(c.identifier, message, this.getMetadata().key, this.conf.severity);
+                issues.push(issue);
+            }
+        }
+        if (file.getFilename().includes(".testclasses.") === true
+            && obj.getTestclassFile() !== undefined
+            && ((_a = obj.getXML()) === null || _a === void 0 ? void 0 : _a.includes("<WITH_UNIT_TESTS>X</WITH_UNIT_TESTS>")) === false) {
+            const id = obj.getIdentifier();
+            if (id) {
+                const message = "Has testclass, but XML does not set <WITH_UNIT_TESTS>";
+                const issue = issue_1.Issue.atIdentifier(id, message, this.getMetadata().key, this.conf.severity);
+                issues.push(issue);
+            }
+        }
+        return issues;
+    }
+}
+exports.LocalTestclassConsistency = LocalTestclassConsistency;
+//# sourceMappingURL=local_testclass_consistency.js.map

package/build/src/rules/no_inline_in_optional_branches.js

@@ -0,0 +1,71 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.NoInlineInOptionalBranches = exports.NoInlineInOptionalBranchesConf = void 0;
+const issue_1 = require("../issue");
+const _abap_rule_1 = require("./_abap_rule");
+const _basic_rule_config_1 = require("./_basic_rule_config");
+const Structures = require("../abap/3_structures/structures");
+const Expressions = require("../abap/2_statements/expressions");
+const _irule_1 = require("./_irule");
+class NoInlineInOptionalBranchesConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.NoInlineInOptionalBranchesConf = NoInlineInOptionalBranchesConf;
+class NoInlineInOptionalBranches extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new NoInlineInOptionalBranchesConf();
+    }
+    getMetadata() {
+        return {
+            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:
+* TRY/CATCH/CLEANUP`,
+            tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const output = [];
+        const struc = file.getStructure();
+        if (struc === undefined) {
+            return []; // syntax error
+        }
+        const candidates = [
+            ...struc.findAllStructures(Structures.If),
+            ...struc.findAllStructures(Structures.Loop),
+            ...struc.findAllStructures(Structures.While),
+            ...struc.findAllStructures(Structures.Case),
+            ...struc.findAllStructures(Structures.CaseType),
+            ...struc.findAllStructures(Structures.Do),
+            ...struc.findAllStructures(Structures.Select)
+        ];
+        for (const c of candidates) {
+            const inline = c.findFirstExpression(Expressions.InlineData);
+            if (inline) {
+                const message = "Don't declare inline in optional branches";
+                const issue = issue_1.Issue.atToken(file, c.getFirstToken(), message, this.getMetadata().key, this.getConfig().severity);
+                output.push(issue);
+            }
+        }
+        return output;
+    }
+}
+exports.NoInlineInOptionalBranches = NoInlineInOptionalBranches;
+//# sourceMappingURL=no_inline_in_optional_branches.js.map

package/build/src/rules/unused_variables.js

@@ -156,6 +156,7 @@ Unused variables are not reported if the object contains syntax errors. Errors f
                 }
                 else if (name === "ME"
                     || name === "SUPER"
+                    || meta.includes("selection_screen_tab" /* SelectionScreenTab */)
                     || meta.includes("event_parameter" /* EventParameter */)) {
                     // todo, workaround for "me" and "super", these should somehow be typed to built-in
                     continue;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.88.8",
+    "version": "2.89.0",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -45,10 +45,10 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@microsoft/api-extractor": "^7.22.0",
-        "@types/chai": "^4.3.0",
+        "@microsoft/api-extractor": "^7.22.2",
+        "@types/chai": "^4.3.1",
         "@types/mocha": "^9.1.0",
-        "@types/node": "^17.0.23",
+        "@types/node": "^17.0.24",
         "chai": "^4.3.6",
         "eslint": "^8.13.0",
         "mocha": "^9.2.2",

package/build/src/rules/local_testclass_location.js

@@ -1,54 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.LocalTestclassLocation = exports.LocalTestclassLocationConf = void 0;
-const issue_1 = require("../issue");
-const _abap_rule_1 = require("./_abap_rule");
-const _basic_rule_config_1 = require("./_basic_rule_config");
-const objects_1 = require("../objects");
-const _irule_1 = require("./_irule");
-const version_1 = require("../version");
-class LocalTestclassLocationConf extends _basic_rule_config_1.BasicRuleConfig {
-}
-exports.LocalTestclassLocationConf = LocalTestclassLocationConf;
-class LocalTestclassLocation extends _abap_rule_1.ABAPRule {
-    constructor() {
-        super(...arguments);
-        this.conf = new LocalTestclassLocationConf();
-    }
-    getMetadata() {
-        return {
-            key: "local_testclass_location",
-            title: "Local testclass location",
-            shortDescription: `Checks that local test classes are placed in the test include.`,
-            tags: [_irule_1.RuleTag.SingleFile],
-        };
-    }
-    getDescription(className) {
-        return "Place local testclass \"" + className + "\" in the testclass include";
-    }
-    getConfig() {
-        return this.conf;
-    }
-    setConfig(conf) {
-        this.conf = conf;
-    }
-    runParsed(file, obj) {
-        const issues = [];
-        if (this.reg.getConfig().getVersion() === version_1.Version.v700) {
-            // 700 does not have testclass includes
-            return [];
-        }
-        if (!(obj instanceof objects_1.Class)) {
-            return [];
-        }
-        for (const c of file.getInfo().listClassDefinitions()) {
-            if (c.isLocal && c.isForTesting && !file.getFilename().includes(".testclasses.abap")) {
-                const issue = issue_1.Issue.atIdentifier(c.identifier, this.getDescription(c.name), this.getMetadata().key, this.conf.severity);
-                issues.push(issue);
-            }
-        }
-        return issues;
-    }
-}
-exports.LocalTestclassLocation = LocalTestclassLocation;
-//# sourceMappingURL=local_testclass_location.js.map