@abaplint/core 2.93.0 → 2.93.1

package/build/src/registry.js

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

package/build/src/rules/cds_comment_style.js

@@ -21,6 +21,8 @@ class CDSCommentStyle {
             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",

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

@@ -29,8 +29,8 @@ ELSEIF l_fcat-fieldname = 'BAR'
     OR l_fcat-fieldname = 'MOO'.
 ENDIF.`,
             goodExample: `CASE l_fcat-fieldname.
-  WHEN 'FOO.
-  WHEN 'BAR' OR MOO.
+  WHEN 'FOO'.
+  WHEN 'BAR' OR 'MOO'.
 ENDCASE.`,
         };
     }

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);

package/package.json

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