@abaplint/cli 2.120.33 → 2.120.34

package/build/cli.js

@@ -26709,6 +26709,11 @@ class ObjectOriented {
         if (found) {
             return found;
         }
+        if (name.includes("~")) {
+            // event of an implemented interface, ie. "if_name~event_name"
+            const [interfaceName, eventName] = name.split("~");
+            return this.searchEvent(this.scope.findInterfaceDefinition(interfaceName), eventName);
+        }
         for (const a of def.getAliases() || []) {
             if (a.getName().toUpperCase() === name.toUpperCase()) {
                 const comp = a.getComponent();
@@ -27777,6 +27782,11 @@ class TypeUtils {
                 }
                 return false;
             }
+            else if (target instanceof basic_1.TableType
+                || target instanceof basic_1.DataReference
+                || target instanceof basic_1.ObjectReferenceType) {
+                return false;
+            }
             return true;
         }
         else if (source instanceof basic_1.StructureType) {
@@ -27801,7 +27811,8 @@ class TypeUtils {
             }
             else if (target instanceof basic_1.VoidType
                 || target instanceof basic_1.AnyType
-                || target instanceof basic_1.DataType) {
+                || target instanceof basic_1.DataType
+                || target instanceof basic_1.UnknownType) {
                 return true;
             }
             return false;
@@ -70060,7 +70071,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.33";
+        return "2.120.34";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -73264,7 +73275,8 @@ class CDSFieldOrder {
     }
     getAssociationNames(tree) {
         const names = new Set();
-        for (const assoc of tree.findAllExpressions(expressions_1.CDSAssociation)) {
+        const found = [...tree.findAllExpressions(expressions_1.CDSAssociation), ...tree.findAllExpressions(expressions_1.CDSComposition)];
+        for (const assoc of found) {
             const relation = assoc.findDirectExpression(expressions_1.CDSRelation);
             if (relation === undefined) {
                 continue;
@@ -73382,7 +73394,6 @@ const _irule_1 = __webpack_require__(/*! ./_irule */ "../core/build/src/rules/_i
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "../core/build/src/rules/_basic_rule_config.js");
 const objects_1 = __webpack_require__(/*! ../objects */ "../core/build/src/objects/index.js");
 const expressions_1 = __webpack_require__(/*! ../cds/expressions */ "../core/build/src/cds/expressions/index.js");
-const cds_name_1 = __webpack_require__(/*! ../cds/expressions/cds_name */ "../core/build/src/cds/expressions/cds_name.js");
 class CDSNamingConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
@@ -73454,7 +73465,7 @@ https://help.sap.com/docs/SAP_S4HANA_ON-PREMISE/ee6ff9b281d8448f96b4fe6c89f2bdc8
         if (file === undefined) {
             return [];
         }
-        const name = this.getCDSName(tree);
+        const name = o.getDefinitionName();
         if (name === undefined) {
             return [];
         }
@@ -73471,13 +73482,6 @@ https://help.sap.com/docs/SAP_S4HANA_ON-PREMISE/ee6ff9b281d8448f96b4fe6c89f2bdc8
         }
         return [];
     }
-    getCDSName(tree) {
-        const nameNodes = tree.findDirectExpressions(cds_name_1.CDSName);
-        if (nameNodes.length > 0) {
-            return nameNodes[0].getFirstToken().getStr();
-        }
-        return undefined;
-    }
     getAllowedPrefixes(tree) {
         const root = tree.get();
         if (root instanceof expressions_1.CDSExtendView) {
@@ -88496,17 +88500,19 @@ class NoDynamicStuff extends _abap_rule_1.ABAPRule {
             extendedInformation: `Dynamic constructs cannot be checked statically, they are not found by
 where-used lists and refactorings, and they can introduce injection vulnerabilities.
 
-Dynamic tokens containing a literal, eg. CALL METHOD ('CL_FOO')=>('BAR'), are not reported,
-as these are equivalent to the static variant.
+Dynamic tokens are also reported when containing a literal, eg. CALL METHOD go_calendar->('RESET_DAY_INFO'),
+the syntax check does not resolve these either, so they behave like any other dynamic token.
 
 Dynamic SQL is reported by rule dangerous_statement.`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Security],
             badExample: `CALL METHOD (lv_class)=>(lv_method).
+CALL METHOD go_calendar->('RESET_DAY_INFO').
 CALL FUNCTION lv_function_module.
 CREATE OBJECT ref TYPE (lv_class).
 ASSIGN (lv_name) TO <fs>.
 SORT tab BY (lv_field).`,
             goodExample: `cl_class=>method( ).
+go_calendar->reset_day_info( ).
 CALL FUNCTION 'ZFOO'.
 CREATE OBJECT ref TYPE cl_class.
 ASSIGN foo TO <fs>.
@@ -88618,10 +88624,8 @@ SORT tab BY field.`,
         for (const source of node.findAllExpressions(Expressions.MethodSource)) {
             // Dynamic is always a direct child of MethodSource, dynamics further down
             // are parameters of the method call and not part of the method name
-            for (const dynamic of source.findDirectExpressions(Expressions.Dynamic)) {
-                if (this.isLiteral(dynamic) === false) {
-                    return true;
-                }
+            if (source.findDirectExpression(Expressions.Dynamic) !== undefined) {
+                return true;
             }
         }
         return false;
@@ -88632,12 +88636,7 @@ SORT tab BY field.`,
         return component !== undefined && this.isLiteral(component) === false;
     }
     hasDynamic(node) {
-        for (const dynamic of node.findAllExpressions(Expressions.Dynamic)) {
-            if (this.isLiteral(dynamic) === false) {
-                return true;
-            }
-        }
-        return false;
+        return node.findFirstExpression(Expressions.Dynamic) !== undefined;
     }
     notLiteral(node) {
         return node !== undefined && this.isLiteral(node) === false;
@@ -100634,6 +100633,10 @@ exports.DECLARATION_STUFF = [
     Statements.DataBegin,
     Statements.Constant,
     Statements.ConstantBegin,
+    Statements.Static,
+    Statements.StaticBegin,
+    Statements.FieldSymbol,
+    Statements.Ranges,
     Statements.Tables,
     Statements.Include, // this is not super correct, but anyhow
     Statements.Parameter,

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.120.33",
+    "version": "2.120.34",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -39,7 +39,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.120.33",
+        "@abaplint/core": "^2.120.34",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",