@abaplint/core 2.92.1 → 2.93.1

package/build/abaplint.d.ts

@@ -572,6 +572,10 @@ declare class CDSAggregate extends Expression {
     getRunnable(): IStatementRunnable;
 }
 
+declare class CDSAnnotate extends Expression {
+    getRunnable(): IStatementRunnable;
+}
+
 declare class CDSAnnotation extends Expression {
     getRunnable(): IStatementRunnable;
 }
@@ -649,12 +653,17 @@ declare class CDSJoin extends Expression {
 }
 
 declare class CDSMetadataExtension extends AbstractObject {
+    private parserError;
+    private parsedData;
     getType(): string;
     getAllowedNaming(): {
         maxLength: number;
         allowNamespace: boolean;
     };
+    hasParserError(): boolean | undefined;
+    parse(): IParseResult;
     getDescription(): string | undefined;
+    findSourceFile(): IFile | undefined;
 }
 
 declare class CDSName extends Expression {
@@ -2002,6 +2011,7 @@ declare namespace Expressions {
         TypeTableKey,
         TypeTable,
         Type_2 as Type,
+        ValueBodyLine,
         ValueBodyLines,
         ValueBody,
         Value,
@@ -2014,6 +2024,7 @@ export { Expressions }
 declare namespace ExpressionsCDS {
     export {
         CDSAggregate,
+        CDSAnnotate,
         CDSAnnotationArray,
         CDSAnnotationObject,
         CDSAnnotationSimple,
@@ -3984,6 +3995,7 @@ declare namespace Objects {
         BusinessFunctionAssignment,
         BusinessFunctionSetAssignment,
         BusinessObjectModel,
+        ParsedMetadataExtension,
         CDSMetadataExtension,
         ChangeDocument,
         ChapterOfBookStructure,
@@ -4228,6 +4240,10 @@ declare type ParsedDataDefinition = {
     tree: ExpressionNode | undefined;
 };
 
+declare type ParsedMetadataExtension = {
+    tree: ExpressionNode | undefined;
+};
+
 declare class PassByValue extends Expression {
     getRunnable(): IStatementRunnable;
 }
@@ -6106,6 +6122,10 @@ declare class ValueBody extends Expression {
     getRunnable(): IStatementRunnable;
 }
 
+declare class ValueBodyLine extends Expression {
+    getRunnable(): IStatementRunnable;
+}
+
 declare class ValueBodyLines extends Expression {
     getRunnable(): IStatementRunnable;
 }

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

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

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

@@ -2,16 +2,12 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ValueBody = void 0;
 const combi_1 = require("../combi");
-const tokens_1 = require("../../1_lexer/tokens");
 const _1 = require(".");
 const version_1 = require("../../../version");
-const value_body_lines_1 = require("./value_body_lines");
 class ValueBody extends combi_1.Expression {
     getRunnable() {
         const base = (0, combi_1.seq)("BASE", _1.Source);
-        // missing spaces caught by rule "parser_missing_space"
-        const foo = (0, combi_1.seq)((0, combi_1.altPrio)((0, combi_1.tok)(tokens_1.WParenLeftW), (0, combi_1.tok)(tokens_1.WParenLeft)), (0, combi_1.optPrio)((0, combi_1.altPrio)((0, combi_1.plusPrio)(_1.FieldAssignment), value_body_lines_1.ValueBodyLines, _1.Source)), (0, combi_1.altPrio)((0, combi_1.tok)(tokens_1.WParenRightW), (0, combi_1.tok)(tokens_1.ParenRightW)));
-        const strucOrTab = (0, combi_1.seq)((0, combi_1.optPrio)(_1.Let), (0, combi_1.optPrio)(base), (0, combi_1.star)(_1.For), (0, combi_1.plusPrio)((0, combi_1.altPrio)(_1.FieldAssignment, foo)));
+        const strucOrTab = (0, combi_1.seq)((0, combi_1.optPrio)(_1.Let), (0, combi_1.optPrio)(base), (0, combi_1.star)(_1.For), (0, combi_1.plusPrio)((0, combi_1.altPrio)(_1.FieldAssignment, _1.ValueBodyLine)));
         const tabdef = (0, combi_1.ver)(version_1.Version.v740sp08, (0, combi_1.altPrio)("OPTIONAL", (0, combi_1.seq)("DEFAULT", _1.Source)));
         return (0, combi_1.optPrio)((0, combi_1.altPrio)(strucOrTab, (0, combi_1.seq)(_1.Source, (0, combi_1.optPrio)(tabdef))));
     }

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

@@ -0,0 +1,16 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ValueBodyLine = void 0;
+const combi_1 = require("../combi");
+const tokens_1 = require("../../1_lexer/tokens");
+const _1 = require(".");
+const value_body_lines_1 = require("./value_body_lines");
+class ValueBodyLine extends combi_1.Expression {
+    getRunnable() {
+        // missing spaces caught by rule "parser_missing_space"
+        const ret = (0, combi_1.seq)((0, combi_1.altPrio)((0, combi_1.tok)(tokens_1.WParenLeftW), (0, combi_1.tok)(tokens_1.WParenLeft)), (0, combi_1.optPrio)((0, combi_1.altPrio)((0, combi_1.plusPrio)(_1.FieldAssignment), value_body_lines_1.ValueBodyLines, _1.Source)), (0, combi_1.altPrio)((0, combi_1.tok)(tokens_1.WParenRightW), (0, combi_1.tok)(tokens_1.ParenRightW)));
+        return ret;
+    }
+}
+exports.ValueBodyLine = ValueBodyLine;
+//# sourceMappingURL=value_body_line.js.map

package/build/src/abap/5_syntax/expressions/field_chain.js

@@ -36,7 +36,10 @@ class FieldChain {
             if (current === undefined) {
                 break;
             }
-            if (current.get() instanceof tokens_1.Dash) {
+            if (current.get() instanceof tokens_1.DashW) {
+                throw new Error("Ending with dash");
+            }
+            else if (current.get() instanceof tokens_1.Dash) {
                 if (context instanceof basic_1.UnknownType) {
                     throw new Error("Not a structure, type unknown, FieldChain");
                 }

package/build/src/abap/5_syntax/expressions/value_body.js

@@ -27,8 +27,16 @@ class ValueBody {
         for (const s of node.findDirectExpressions(Expressions.Source)) {
             type = new source_1.Source().runSyntax(s, scope, filename);
         }
-        for (const l of node.findDirectExpressions(Expressions.ValueBodyLines)) {
-            for (const s of l.findDirectExpressions(Expressions.Source)) {
+        for (const foo of node.findDirectExpressions(Expressions.ValueBodyLine)) {
+            for (const l of foo.findDirectExpressions(Expressions.ValueBodyLines)) {
+                for (const s of l.findDirectExpressions(Expressions.Source)) {
+                    new source_1.Source().runSyntax(s, scope, filename);
+                }
+            }
+            for (const s of foo.findDirectExpressions(Expressions.FieldAssignment)) {
+                new field_assignment_1.FieldAssignment().runSyntax(s, scope, filename, targetType);
+            }
+            for (const s of foo.findDirectExpressions(Expressions.Source)) {
                 new source_1.Source().runSyntax(s, scope, filename);
             }
         }

package/build/src/cds/cds_lexer.js

@@ -22,13 +22,25 @@ class Stream {
         return this.buffer.length;
     }
 }
+var Mode;
+(function (Mode) {
+    Mode[Mode["Default"] = 0] = "Default";
+    Mode[Mode["String"] = 1] = "String";
+    Mode[Mode["SingleLineComment"] = 2] = "SingleLineComment";
+    Mode[Mode["MultiLineComment"] = 3] = "MultiLineComment";
+})(Mode || (Mode = {}));
 class Result {
     constructor() {
         this.result = [];
     }
-    add(text, row, col) {
+    add(text, row, col, mode) {
         if (text.length > 0) {
-            this.result.push(new tokens_1.Identifier(new position_1.Position(row, col), text));
+            if (mode === Mode.SingleLineComment) {
+                this.result.push(new tokens_1.Comment(new position_1.Position(row, col), text));
+            }
+            else {
+                this.result.push(new tokens_1.Identifier(new position_1.Position(row, col), text));
+            }
         }
         return "";
     }
@@ -36,13 +48,6 @@ class Result {
         return this.result;
     }
 }
-var Mode;
-(function (Mode) {
-    Mode[Mode["Default"] = 0] = "Default";
-    Mode[Mode["String"] = 1] = "String";
-    Mode[Mode["SingleLineComment"] = 2] = "SingleLineComment";
-    Mode[Mode["MultiLineComment"] = 3] = "MultiLineComment";
-})(Mode || (Mode = {}));
 class CDSLexer {
     static run(file) {
         const result = new Result();
@@ -61,7 +66,7 @@ class CDSLexer {
             if (mode === Mode.String) {
                 build += next;
                 if (next === "'") {
-                    build = result.add(build, row, col);
+                    build = result.add(build, row, col, mode);
                     mode = Mode.Default;
                 }
                 continue;
@@ -69,20 +74,24 @@ class CDSLexer {
             // single line comment handling
             if (mode === Mode.SingleLineComment) {
                 if (next === "\n") {
+                    build = result.add(build, row, col, mode);
                     mode = Mode.Default;
                 }
                 else {
+                    build += next;
                     continue;
                 }
             }
             else if (mode === Mode.Default && next === "/" && nextNext === "/") {
                 mode = Mode.SingleLineComment;
-                build = result.add(build, row, col);
+                build = result.add(build, row, col, mode);
+                build += next;
                 continue;
             }
             else if (mode === Mode.Default && next === "-" && nextNext === "-") {
                 mode = Mode.SingleLineComment;
-                build = result.add(build, row, col);
+                build = result.add(build, row, col, mode);
+                build += next;
                 continue;
             }
             // multi line comment handling
@@ -97,7 +106,7 @@ class CDSLexer {
             }
             else if (mode === Mode.Default && next === "/" && nextNext === "*") {
                 mode = Mode.MultiLineComment;
-                build = result.add(build, row, col);
+                build = result.add(build, row, col, mode);
                 continue;
             }
             switch (next) {
@@ -106,10 +115,10 @@ class CDSLexer {
                     build += next;
                     break;
                 case " ":
-                    build = result.add(build, row, col);
+                    build = result.add(build, row, col, mode);
                     break;
                 case "\n":
-                    build = result.add(build, row, col);
+                    build = result.add(build, row, col, mode);
                     row++;
                     col = 0;
                     break;
@@ -130,15 +139,15 @@ class CDSLexer {
                 case "-":
                 case "*":
                 case "/":
-                    build = result.add(build, row, col);
-                    result.add(next, row, col);
+                    build = result.add(build, row, col, mode);
+                    result.add(next, row, col, mode);
                     break;
                 default:
                     build += next;
                     break;
             }
         }
-        result.add(build, row, col);
+        result.add(build, row, col, mode);
         return result.get();
     }
 }

package/build/src/cds/cds_parser.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.CDSParser = void 0;
+const tokens_1 = require("../abap/1_lexer/tokens");
 const combi_1 = require("../abap/2_statements/combi");
 const nodes_1 = require("../abap/nodes");
 const version_1 = require("../version");
@@ -12,7 +13,8 @@ class CDSParser {
         if (file === undefined) {
             return undefined;
         }
-        const tokens = cds_lexer_1.CDSLexer.run(file);
+        let tokens = cds_lexer_1.CDSLexer.run(file);
+        tokens = tokens.filter(t => !(t instanceof tokens_1.Comment));
         // console.dir(tokens);
         let res = combi_1.Combi.run(new Expressions.CDSDefineView(), tokens, version_1.defaultVersion);
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
@@ -21,6 +23,9 @@ class CDSParser {
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
             res = combi_1.Combi.run(new Expressions.CDSDefineProjection(), tokens, version_1.defaultVersion);
         }
+        if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
+            res = combi_1.Combi.run(new Expressions.CDSAnnotate(), tokens, version_1.defaultVersion);
+        }
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
             return undefined;
         }

package/build/src/cds/expressions/cds_annotate.js

@@ -0,0 +1,12 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.CDSAnnotate = void 0;
+const _1 = require(".");
+const combi_1 = require("../../abap/2_statements/combi");
+class CDSAnnotate extends combi_1.Expression {
+    getRunnable() {
+        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), "ANNOTATE", (0, combi_1.alt)("ENTITY", "VIEW"), _1.CDSName, "WITH", (0, combi_1.str)("{"), (0, combi_1.plus)((0, combi_1.seq)(_1.CDSElement, ";")), (0, combi_1.str)("}"), (0, combi_1.opt)(";"));
+    }
+}
+exports.CDSAnnotate = CDSAnnotate;
+//# sourceMappingURL=cds_annotate.js.map

package/build/src/cds/expressions/index.js

@@ -16,6 +16,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
 Object.defineProperty(exports, "__esModule", { value: true });
 __exportStar(require("./cds_aggregate"), exports);
 __exportStar(require("./cds_aggregate"), exports);
+__exportStar(require("./cds_annotate"), exports);
 __exportStar(require("./cds_annotation_array"), exports);
 __exportStar(require("./cds_annotation_object"), exports);
 __exportStar(require("./cds_annotation_simple"), exports);

package/build/src/objects/cds_metadata_extension.js

@@ -1,8 +1,14 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.CDSMetadataExtension = void 0;
+const cds_parser_1 = require("../cds/cds_parser");
 const _abstract_object_1 = require("./_abstract_object");
 class CDSMetadataExtension extends _abstract_object_1.AbstractObject {
+    constructor() {
+        super(...arguments);
+        this.parserError = undefined;
+        this.parsedData = undefined;
+    }
     getType() {
         return "DDLX";
     }
@@ -12,10 +18,31 @@ class CDSMetadataExtension extends _abstract_object_1.AbstractObject {
             allowNamespace: true,
         };
     }
+    hasParserError() {
+        return this.parserError;
+    }
+    parse() {
+        if (this.isDirty() === false) {
+            return { updated: false, runtime: 0 };
+        }
+        const start = Date.now();
+        this.parsedData = {
+            tree: undefined,
+        };
+        this.parsedData.tree = new cds_parser_1.CDSParser().parse(this.findSourceFile());
+        if (this.parsedData.tree === undefined) {
+            this.parserError = true;
+        }
+        this.dirty = false;
+        return { updated: true, runtime: Date.now() - start };
+    }
     getDescription() {
         // todo
         return undefined;
     }
+    findSourceFile() {
+        return this.getFiles().find(f => f.getFilename().endsWith(".asddlxs") || f.getFilename().endsWith(".acds"));
+    }
 }
 exports.CDSMetadataExtension = CDSMetadataExtension;
 //# sourceMappingURL=cds_metadata_extension.js.map

package/build/src/objects/data_definition.js

@@ -52,7 +52,7 @@ class DataDefinition extends _abstract_object_1.AbstractObject {
         super.setDirty();
     }
     findSourceFile() {
-        return this.getFiles().find(f => f.getFilename().endsWith(".asddls"));
+        return this.getFiles().find(f => f.getFilename().endsWith(".asddls") || f.getFilename().endsWith(".acds"));
     }
     hasParserError() {
         return this.parserError;

package/build/src/registry.js

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

package/build/src/rules/align_parameters.js

@@ -56,7 +56,13 @@ foobar( moo   = 1
 
 foo = VALUE #(
     foo = bar
-    moo = 2 ).`,
+    moo = 2 ).
+
+DATA(sdf) = VALUE type(
+  common_val     = 2
+  another_common = 5
+  ( row_value = 4
+    value_foo = 5 ) ).`,
         };
     }
     getConfig() {
@@ -169,6 +175,27 @@ foo = VALUE #(
                 candidates.push({ parameters });
             }
         }
+        for (const vb of stru.findAllExpressionsRecursive(Expressions.ValueBodyLine)) {
+            const parameters = [];
+            const fieldAssignments = vb.findDirectExpressions(Expressions.FieldAssignment);
+            if (fieldAssignments.length <= 1) {
+                continue;
+            }
+            for (const fs of fieldAssignments) {
+                const children = fs.getChildren();
+                if (children.length < 3) {
+                    continue; // unexpected
+                }
+                parameters.push({
+                    left: children[0],
+                    eq: children[1].getFirstToken().getStart(),
+                    right: children[2],
+                });
+            }
+            if (parameters.length > 0) {
+                candidates.push({ parameters });
+            }
+        }
         return candidates;
     }
     raiseAndCreateCandidates(stru) {

package/build/src/rules/cds_comment_style.js

@@ -0,0 +1,60 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.CDSCommentStyle = exports.CDSCommentStyleConf = void 0;
+const issue_1 = require("../issue");
+const _irule_1 = require("./_irule");
+const _basic_rule_config_1 = require("./_basic_rule_config");
+const objects_1 = require("../objects");
+const cds_lexer_1 = require("../cds/cds_lexer");
+const tokens_1 = require("../abap/1_lexer/tokens");
+class CDSCommentStyleConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.CDSCommentStyleConf = CDSCommentStyleConf;
+class CDSCommentStyle {
+    constructor() {
+        this.conf = new CDSCommentStyleConf();
+    }
+    getMetadata() {
+        return {
+            key: "cds_comment_style",
+            title: "CDS Comment Style",
+            shortDescription: `Check for obsolete comment style`,
+            extendedInformation: `Check for obsolete comment style
+
+Comments starting with "--" are considered obsolete
+
+https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abencds_general_syntax_rules.htm`,
+            tags: [_irule_1.RuleTag.SingleFile],
+            badExample: "-- this is a comment",
+            goodExample: "// this is a comment",
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    initialize(_reg) {
+        return this;
+    }
+    run(object) {
+        const issues = [];
+        if ((object.getType() === "DDLS" && object instanceof objects_1.DataDefinition) ||
+            (object.getType() === "DDLX" && object instanceof objects_1.CDSMetadataExtension)) {
+            const file = object.findSourceFile();
+            if (file === undefined) {
+                return issues;
+            }
+            const tokens = cds_lexer_1.CDSLexer.run(file);
+            for (const t of tokens) {
+                if (t instanceof tokens_1.Comment && t.getStr().startsWith("--")) {
+                    issues.push(issue_1.Issue.atToken(file, t, `Use "//" for comments instead of "--"`, this.getMetadata().key, this.getConfig().severity));
+                }
+            }
+        }
+        return issues;
+    }
+}
+exports.CDSCommentStyle = CDSCommentStyle;
+//# sourceMappingURL=cds_comment_style.js.map

package/build/src/rules/cds_legacy_view.js

@@ -24,7 +24,7 @@ class CDSLegacyView {
 https://blogs.sap.com/2021/10/16/a-new-generation-of-cds-views-how-to-migrate-your-cds-views-to-cds-view-entities/
 
 v755 and up`,
-            tags: [_irule_1.RuleTag.SingleFile],
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Upport],
         };
     }
     getConfig() {

package/build/src/rules/cds_parser_error.js

@@ -16,8 +16,8 @@ class CDSParserError {
         return {
             key: "cds_parser_error",
             title: "CDS Parser Error",
-            shortDescription: `CDS parsing, experimental`,
-            extendedInformation: ``,
+            shortDescription: `CDS parsing`,
+            extendedInformation: `Parses CDS and issues parser errors`,
             tags: [_irule_1.RuleTag.Syntax],
         };
     }
@@ -30,11 +30,12 @@ class CDSParserError {
     initialize(_reg) {
         return this;
     }
-    run(o) {
+    run(object) {
         const issues = [];
-        if (o.getType() === "DDLS" && o instanceof objects_1.DataDefinition) {
-            const hasError = o.hasParserError();
-            const file = o.findSourceFile();
+        if ((object.getType() === "DDLS" && object instanceof objects_1.DataDefinition) ||
+            (object.getType() === "DDLX" && object instanceof objects_1.CDSMetadataExtension)) {
+            const hasError = object.hasParserError();
+            const file = object.findSourceFile();
             if (hasError === true && file) {
                 issues.push(issue_1.Issue.atRow(file, 1, "CDS Parser error", this.getMetadata().key, this.getConfig().severity));
             }

package/build/src/rules/change_if_to_case.js

@@ -0,0 +1,125 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ChangeIfToCase = exports.ChangeIfToCaseConf = void 0;
+const issue_1 = require("../issue");
+const Expressions = require("../abap/2_statements/expressions");
+const Statements = require("../abap/2_statements/statements");
+const Structures = require("../abap/3_structures/structures");
+const _abap_rule_1 = require("./_abap_rule");
+const _basic_rule_config_1 = require("./_basic_rule_config");
+const _irule_1 = require("./_irule");
+class ChangeIfToCaseConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.ChangeIfToCaseConf = ChangeIfToCaseConf;
+class ChangeIfToCase extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new ChangeIfToCaseConf();
+    }
+    getMetadata() {
+        return {
+            key: "change_if_to_case",
+            title: "Change IF to CASE",
+            shortDescription: `Finds IF constructs that can be changed to CASE`,
+            // eslint-disable-next-line max-len
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-case-to-else-if-for-multiple-alternative-conditions`,
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
+            badExample: `IF l_fcat-fieldname EQ 'FOO'.
+ELSEIF l_fcat-fieldname = 'BAR'
+    OR l_fcat-fieldname = 'MOO'.
+ENDIF.`,
+            goodExample: `CASE l_fcat-fieldname.
+  WHEN 'FOO'.
+  WHEN 'BAR' OR 'MOO'.
+ENDCASE.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        var _a;
+        const issues = [];
+        const stru = file.getStructure();
+        if (stru === undefined) {
+            return issues;
+        }
+        for (const i of stru.findAllStructuresRecursive(Structures.If)) {
+            const conds = [];
+            const ifStatement = i.findDirectStatement(Statements.If);
+            if (ifStatement === undefined) {
+                continue;
+            }
+            conds.push(ifStatement === null || ifStatement === void 0 ? void 0 : ifStatement.findDirectExpression(Expressions.Cond));
+            for (const ei of i.findDirectStructures(Structures.ElseIf)) {
+                conds.push((_a = ei.findDirectStatement(Statements.ElseIf)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(Expressions.Cond));
+            }
+            if (conds.length === 1) {
+                continue;
+            }
+            const issue = this.analyze(conds);
+            if (issue === true) {
+                const message = "Change IF to CASE";
+                issues.push(issue_1.Issue.atStatement(file, ifStatement, message, this.getMetadata().key, this.getConfig().severity));
+            }
+        }
+        return issues;
+    }
+    analyze(conds) {
+        var _a, _b, _c, _d, _e;
+        const tuples = [];
+        for (const c of conds) {
+            if (c === undefined) {
+                continue;
+            }
+            if (c.findFirstExpression(Expressions.CondSub)) {
+                return false;
+            }
+            else if (c.findDirectTokenByText("AND") || c.findDirectTokenByText("EQUIV")) {
+                return false;
+            }
+            for (const compare of c.findAllExpressions(Expressions.Compare)) {
+                const op = (_a = compare.findDirectExpression(Expressions.CompareOperator)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase();
+                if (compare.getChildren().length !== 3) {
+                    return false;
+                }
+                else if (op !== "=" && op !== "EQ") {
+                    return false;
+                }
+                const left = (_c = (_b = compare.getChildren()[0]) === null || _b === void 0 ? void 0 : _b.concatTokens()) === null || _c === void 0 ? void 0 : _c.toUpperCase();
+                const right = (_e = (_d = compare.getChildren()[2]) === null || _d === void 0 ? void 0 : _d.concatTokens()) === null || _e === void 0 ? void 0 : _e.toUpperCase();
+                tuples.push({ left, right });
+            }
+        }
+        if (tuples.length === 1) {
+            return false;
+        }
+        let chain = "";
+        if (tuples[0].left === tuples[1].left) {
+            chain = tuples[0].left;
+        }
+        else if (tuples[0].left === tuples[1].right) {
+            chain = tuples[0].left;
+        }
+        else if (tuples[0].right === tuples[1].right) {
+            chain = tuples[0].right;
+        }
+        else if (tuples[0].right === tuples[1].left) {
+            chain = tuples[0].right;
+        }
+        else {
+            return false;
+        }
+        for (const t of tuples) {
+            if (t.left !== chain && t.right !== chain) {
+                return false;
+            }
+        }
+        return true;
+    }
+}
+exports.ChangeIfToCase = ChangeIfToCase;
+//# sourceMappingURL=change_if_to_case.js.map

package/build/src/rules/constant_classes.js

@@ -24,7 +24,7 @@ class ConstantClasses {
             title: "Validate constant classes",
             shortDescription: `Checks that a class contains exactly the constants corresponding to a domain's fixed values.`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-enumeration-classes-to-constants-interfaces`,
-            tags: [_irule_1.RuleTag.Syntax, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Experimental],
+            tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Experimental],
         };
     }
     initialize(reg) {

package/build/src/rules/downport.js

@@ -83,10 +83,19 @@ Only one transformation is applied to a statement at a time, so multiple steps m
         const version = this.lowReg.getConfig().getVersion();
         if (version === version_1.Version.v702 || version === version_1.Version.OpenABAP) {
             this.initHighReg();
-            this.graph = new include_graph_1.IncludeGraph(reg);
         }
         return this;
     }
+    listMainForInclude(filename) {
+        if (filename === undefined) {
+            return [];
+        }
+        // only initialize this.graph if needed
+        if (this.graph === undefined) {
+            this.graph = new include_graph_1.IncludeGraph(this.lowReg);
+        }
+        return this.graph.listMainForInclude(filename);
+    }
     run(lowObj) {
         var _a;
         const ret = [];
@@ -105,7 +114,7 @@ Only one transformation is applied to a statement at a time, so multiple steps m
         let highSyntaxObj = highObj;
         // for includes do the syntax check via a main program
         if (lowObj instanceof objects_1.Program && lowObj.isInclude()) {
-            const mains = this.graph.listMainForInclude((_a = lowObj.getMainABAPFile()) === null || _a === void 0 ? void 0 : _a.getFilename());
+            const mains = this.listMainForInclude((_a = lowObj.getMainABAPFile()) === null || _a === void 0 ? void 0 : _a.getFilename());
             if (mains.length <= 0) {
                 return [];
             }
@@ -115,8 +124,8 @@ Only one transformation is applied to a statement at a time, so multiple steps m
             }
             highSyntaxObj = this.highReg.findObjectForFile(f);
         }
-        const highSyntax = new syntax_1.SyntaxLogic(this.highReg, highSyntaxObj).run();
         for (const lowFile of lowObj.getABAPFiles()) {
+            let highSyntax = undefined;
             const highFile = highObj.getABAPFileByName(lowFile.getFilename());
             if (highFile === undefined) {
                 continue;
@@ -137,6 +146,9 @@ Only one transformation is applied to a statement at a time, so multiple steps m
                 const high = highStatements[i];
                 if ((low.get() instanceof _statement_1.Unknown && !(high.get() instanceof _statement_1.Unknown))
                     || high.findFirstExpression(Expressions.InlineData)) {
+                    if (highSyntax === undefined) {
+                        highSyntax = new syntax_1.SyntaxLogic(this.highReg, highSyntaxObj).run();
+                    }
                     const issue = this.checkStatement(low, high, lowFile, highSyntax, highFile);
                     if (issue) {
                         ret.push(issue);
@@ -1500,54 +1512,65 @@ ${indentation}    output = ${topTarget}.`;
             let end = "";
             let structureName = uniqueName;
             let added = false;
-            let skip = false;
             let data = "";
             let previous = undefined;
-            for (const b of (valueBody === null || valueBody === void 0 ? void 0 : valueBody.getChildren()) || []) {
-                if (b.concatTokens() === "(" && added === false) {
-                    structureName = this.uniqueName(firstToken.getStart(), lowFile.getFilename(), highSyntax);
-                    data = indentation + `DATA ${structureName} LIKE LINE OF ${uniqueName}.\n`;
-                }
-                if (b.get() instanceof Expressions.FieldAssignment) {
+            for (const a of (valueBody === null || valueBody === void 0 ? void 0 : valueBody.getChildren()) || []) {
+                if (a.get() instanceof Expressions.FieldAssignment) {
                     if (added === false) {
                         body += data;
                         added = true;
                     }
-                    body += indentation + structureName + "-" + b.concatTokens() + ".\n";
+                    body += indentation + structureName + "-" + a.concatTokens() + ".\n";
                 }
-                else if (b instanceof nodes_1.ExpressionNode && b.get() instanceof Expressions.For) {
-                    const outlineFor = this.outlineFor(b, indentation, lowFile, highSyntax);
+                else if (a instanceof nodes_1.ExpressionNode && a.get() instanceof Expressions.For) {
+                    const outlineFor = this.outlineFor(a, indentation, lowFile, highSyntax);
                     body += outlineFor.body;
                     end = outlineFor.end + `.\n` + end;
                     indentation += "  ";
                 }
-                else if (b.get() instanceof Expressions.Source) {
+                else if (a instanceof nodes_1.ExpressionNode && a.get() instanceof Expressions.Source) {
+                    // special handling for superflous value expression
                     if ((valueBody === null || valueBody === void 0 ? void 0 : valueBody.getChildren().length) === 1) {
-                        body += indentation + uniqueName + " = " + b.concatTokens() + `.\n`;
-                    }
-                    structureName = b.concatTokens();
-                    if (base && (valueBody === null || valueBody === void 0 ? void 0 : valueBody.findDirectTokenByText("(")) === undefined) {
-                        structureName = uniqueName;
+                        body += indentation + uniqueName + " = " + a.concatTokens() + `.\n`;
                     }
                 }
-                else if (b.get() instanceof Expressions.ValueBodyLines) {
-                    body += indentation + "APPEND " + b.concatTokens() + ` TO ${uniqueName}.\n`;
-                    skip = true;
+                else if (a instanceof nodes_1.ExpressionNode && a.get() instanceof Expressions.Let) {
+                    body += this.outlineLet(a, indentation, highSyntax, lowFile);
                 }
-                else if (b instanceof nodes_1.ExpressionNode && b.get() instanceof Expressions.Let) {
-                    body += this.outlineLet(b, indentation, highSyntax, lowFile);
-                }
-                else if (b.concatTokens() === ")") {
-                    if (added === false && (previous === null || previous === void 0 ? void 0 : previous.concatTokens()) === "(") {
-                        body += data;
-                        added = true;
-                    }
-                    if (skip === false) {
-                        body += indentation + `APPEND ${structureName} TO ${uniqueName}.\n`;
+                if (a instanceof nodes_1.ExpressionNode && a.get() instanceof Expressions.ValueBodyLine) {
+                    let skip = false;
+                    for (const b of (a === null || a === void 0 ? void 0 : a.getChildren()) || []) {
+                        if (b.concatTokens() === "(" && added === false) {
+                            structureName = this.uniqueName(firstToken.getStart(), lowFile.getFilename(), highSyntax);
+                            data = indentation + `DATA ${structureName} LIKE LINE OF ${uniqueName}.\n`;
+                        }
+                        if (b.get() instanceof Expressions.FieldAssignment) {
+                            if (added === false) {
+                                body += data;
+                                added = true;
+                            }
+                            body += indentation + structureName + "-" + b.concatTokens() + ".\n";
+                        }
+                        else if (b.get() instanceof Expressions.Source) {
+                            body += indentation + "APPEND " + b.concatTokens() + ` TO ${uniqueName}.\n`;
+                            skip = true;
+                        }
+                        else if (b.get() instanceof Expressions.ValueBodyLines) {
+                            body += indentation + "APPEND " + b.concatTokens() + ` TO ${uniqueName}.\n`;
+                            skip = true;
+                        }
+                        else if (b.concatTokens() === ")") {
+                            if (added === false && (previous === null || previous === void 0 ? void 0 : previous.concatTokens()) === "(") {
+                                body += data;
+                                added = true;
+                            }
+                            if (skip === false) {
+                                body += indentation + `APPEND ${structureName} TO ${uniqueName}.\n`;
+                            }
+                        }
+                        previous = b;
                     }
-                    skip = false;
                 }
-                previous = b;
             }
             if (body === "" && ((_b = valueBody === null || valueBody === void 0 ? void 0 : valueBody.getLastChild()) === null || _b === void 0 ? void 0 : _b.getFirstToken().getStr().toUpperCase()) === "OPTIONAL") {
                 const fieldChain = valueBody.findFirstExpression(Expressions.FieldChain);

package/build/src/rules/index.js

@@ -24,21 +24,21 @@ __exportStar(require("./avoid_use"), exports);
 __exportStar(require("./begin_end_names"), exports);
 __exportStar(require("./begin_single_include"), exports);
 __exportStar(require("./call_transaction_authority_check"), exports);
-__exportStar(require("./cds_parser_error"), exports);
+__exportStar(require("./cds_comment_style"), exports);
 __exportStar(require("./cds_legacy_view"), exports);
+__exportStar(require("./cds_parser_error"), exports);
 __exportStar(require("./chain_mainly_declarations"), exports);
+__exportStar(require("./change_if_to_case"), exports);
 __exportStar(require("./check_abstract"), exports);
 __exportStar(require("./check_comments"), exports);
 __exportStar(require("./check_ddic"), exports);
 __exportStar(require("./check_include"), exports);
 __exportStar(require("./check_subrc"), exports);
-__exportStar(require("./no_external_form_calls"), 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("./superfluous_value"), exports);
 __exportStar(require("./class_attribute_names"), exports);
+__exportStar(require("./classic_exceptions_overlap"), exports);
 __exportStar(require("./cloud_types"), exports);
 __exportStar(require("./colon_missing_space"), exports);
 __exportStar(require("./commented_code"), exports);
@@ -102,6 +102,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_external_form_calls"), exports);
 __exportStar(require("./no_inline_in_optional_branches"), exports);
 __exportStar(require("./no_public_attributes"), exports);
 __exportStar(require("./no_yoda_conditions"), exports);
@@ -140,6 +141,7 @@ __exportStar(require("./sql_escape_host_variables"), exports);
 __exportStar(require("./start_at_tab"), exports);
 __exportStar(require("./static_call_via_instance"), exports);
 __exportStar(require("./superclass_final"), exports);
+__exportStar(require("./superfluous_value"), exports);
 __exportStar(require("./sy_modification"), exports);
 __exportStar(require("./tabl_enhancement_category"), exports);
 __exportStar(require("./try_without_catch"), exports);

package/build/src/rules/parser_missing_space.js

@@ -50,7 +50,7 @@ This rule makes sure the spaces are consistently required across the language.`,
     }
     missingSpace(statement) {
         const found = statement.findAllExpressionsMulti([Expressions.CondSub, Expressions.SQLCond,
-            Expressions.ValueBody, Expressions.NewObject, Expressions.Cond,
+            Expressions.ValueBodyLine, Expressions.NewObject, Expressions.Cond,
             Expressions.ComponentCond, Expressions.ComponentCondSub, Expressions.MethodCallParam], true);
         let pos = undefined;
         for (const f of found) {
@@ -70,8 +70,8 @@ This rule makes sure the spaces are consistently required across the language.`,
             else if (type instanceof Expressions.SQLCond) {
                 pos = this.checkSQLCond(f);
             }
-            else if (type instanceof Expressions.ValueBody) {
-                pos = this.checkValueBody(f);
+            else if (type instanceof Expressions.ValueBodyLine) {
+                pos = this.checkValueBodyLine(f);
             }
             else if (type instanceof Expressions.NewObject) {
                 pos = this.checkNewObject(f);
@@ -173,7 +173,7 @@ This rule makes sure the spaces are consistently required across the language.`,
         }
         return undefined;
     }
-    checkValueBody(vb) {
+    checkValueBodyLine(vb) {
         var _a, _b;
         const children = vb.getChildren();
         for (let i = 0; i < children.length; i++) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.92.1",
+    "version": "2.93.1",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -50,7 +50,7 @@
         "@types/mocha": "^9.1.1",
         "@types/node": "^18.7.13",
         "chai": "^4.3.6",
-        "eslint": "^8.22.0",
+        "eslint": "^8.23.0",
         "mocha": "^10.0.0",
         "c8": "^7.12.0",
         "source-map-support": "^0.5.21",