@abaplint/core 2.93.50 → 2.93.52

package/build/abaplint.d.ts

@@ -1835,6 +1835,7 @@ declare namespace Expressions {
         Cast,
         ClassFinal,
         ClassFriends,
+        SimpleTarget,
         ClassGlobal_2 as ClassGlobal,
         ClassName,
         Color,
@@ -4886,6 +4887,10 @@ declare class SimpleSource4 extends Expression {
     getRunnable(): IStatementRunnable;
 }
 
+declare class SimpleTarget extends Expression {
+    getRunnable(): IStatementRunnable;
+}
+
 declare class SimpleType extends AbstractType {
     toText(): string;
     toABAP(): string;

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

@@ -30,6 +30,7 @@ __exportStar(require("./call_transformation_parameters"), exports);
 __exportStar(require("./cast"), exports);
 __exportStar(require("./class_final"), exports);
 __exportStar(require("./class_friends"), exports);
+__exportStar(require("./simple_target"), exports);
 __exportStar(require("./class_global"), exports);
 __exportStar(require("./class_name"), exports);
 __exportStar(require("./color"), exports);

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

@@ -8,7 +8,7 @@ const version_1 = require("../../../version");
 class ReduceNext extends combi_1.Expression {
     getRunnable() {
         const calcAssign = (0, combi_1.ver)(version_1.Version.v754, (0, combi_1.alt)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.WPlus), "="), (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WDash), "="), "/=", "*=", "&&="));
-        const fields = (0, combi_1.seq)(_1.SimpleFieldChain, (0, combi_1.altPrio)("=", calcAssign), _1.Source);
+        const fields = (0, combi_1.seq)(_1.SimpleTarget, (0, combi_1.altPrio)("=", calcAssign), _1.Source);
         return (0, combi_1.seq)("NEXT", (0, combi_1.plus)(fields));
     }
 }

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

@@ -0,0 +1,22 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.SimpleTarget = void 0;
+const combi_1 = require("../combi");
+const _1 = require(".");
+const tokens_1 = require("../../1_lexer/tokens");
+const attribute_name_1 = require("./attribute_name");
+const dereference_1 = require("./dereference");
+class SimpleTarget extends combi_1.Expression {
+    getRunnable() {
+        const attr = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.InstanceArrow), attribute_name_1.AttributeName);
+        const comp = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.Dash), _1.ComponentName);
+        const something = (0, combi_1.starPrio)((0, combi_1.altPrio)(dereference_1.Dereference, attr, comp, _1.TableExpression));
+        const cast = (0, combi_1.seq)((0, combi_1.altPrio)(_1.Cast, _1.NewObject), _1.Arrow, _1.FieldAll);
+        const clas = (0, combi_1.seq)(_1.ClassName, (0, combi_1.tok)(tokens_1.StaticArrow), attribute_name_1.AttributeName);
+        const start = (0, combi_1.altPrio)(cast, clas, _1.TargetField, _1.TargetFieldSymbol);
+        const fields = (0, combi_1.seq)((0, combi_1.optPrio)(_1.FieldOffset), (0, combi_1.optPrio)(_1.FieldLength));
+        return (0, combi_1.seq)(start, something, fields);
+    }
+}
+exports.SimpleTarget = SimpleTarget;
+//# sourceMappingURL=simple_target.js.map

package/build/src/abap/2_statements/statements/class_definition.js

@@ -11,7 +11,7 @@ class ClassDefinition {
         const risk = (0, combi_1.seq)("RISK LEVEL", level);
         const time = (0, combi_1.alt)("LONG", "MEDIUM", "SHORT");
         const duration = (0, combi_1.seq)("DURATION", time);
-        const blah = (0, combi_1.per)(expressions_1.ClassGlobal, expressions_1.ClassFinal, "ABSTRACT", (0, combi_1.seq)("INHERITING FROM", expressions_1.SuperClassName), create, "FOR TESTING", risk, "SHARED MEMORY ENABLED", duration, (0, combi_1.ver)(version_1.Version.v754, (0, combi_1.seq)("FOR BEHAVIOR OF", expressions_1.SimpleName)), expressions_1.ClassFriends);
+        const blah = (0, combi_1.per)(expressions_1.ClassGlobal, expressions_1.ClassFinal, "ABSTRACT", (0, combi_1.seq)("INHERITING FROM", expressions_1.SuperClassName), create, "FOR TESTING", risk, "SHARED MEMORY ENABLED", duration, (0, combi_1.ver)(version_1.Version.v754, (0, combi_1.seq)("FOR BEHAVIOR OF", expressions_1.NamespaceSimpleName)), expressions_1.ClassFriends);
         const def = (0, combi_1.seq)("DEFINITION", (0, combi_1.opt)(blah));
         return (0, combi_1.seq)("CLASS", expressions_1.ClassName, def);
     }

package/build/src/abap/2_statements/statements/commit_entities.js

@@ -6,7 +6,7 @@ const expressions_1 = require("../expressions");
 const version_1 = require("../../../version");
 class CommitEntities {
     getMatcher() {
-        const s = (0, combi_1.seq)("COMMIT ENTITIES", (0, combi_1.optPrio)("IN SIMULATION MODE"), (0, combi_1.star)((0, combi_1.seq)("RESPONSE OF", expressions_1.SimpleName, "FAILED", expressions_1.Target, "REPORTED", expressions_1.Target)));
+        const s = (0, combi_1.seq)("COMMIT ENTITIES", (0, combi_1.optPrio)("IN SIMULATION MODE"), (0, combi_1.star)((0, combi_1.seq)("RESPONSE OF", expressions_1.NamespaceSimpleName, "FAILED", expressions_1.Target, "REPORTED", expressions_1.Target)));
         return (0, combi_1.ver)(version_1.Version.v754, s);
     }
 }

package/build/src/abap/2_statements/statements/method_def.js

@@ -13,7 +13,7 @@ class MethodDef {
         const tableFunction = (0, combi_1.seq)("TABLE FUNCTION", (0, combi_1.regex)(/^\w+?$/));
         // todo, this is only from version something
         const ddl = "DDL OBJECT OPTIONS CDS SESSION CLIENT REQUIRED";
-        const behavior = (0, combi_1.altPrio)((0, combi_1.seq)("VALIDATE ON SAVE IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("MODIFY IMPORTING", expressions_1.MethodParamName, "FOR ACTION", expressions_1.TypeName, "RESULT", expressions_1.MethodParamName), (0, combi_1.seq)("FEATURES IMPORTING", expressions_1.MethodParamName, "REQUEST", expressions_1.NamespaceSimpleName, "FOR", expressions_1.NamespaceSimpleName, "RESULT", expressions_1.MethodParamName), (0, combi_1.seq)("DETERMINE ON MODIFY IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName));
+        const behavior = (0, combi_1.altPrio)((0, combi_1.seq)("VALIDATE ON SAVE IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("MODIFY IMPORTING", expressions_1.MethodParamName, "FOR ACTION", expressions_1.TypeName, "RESULT", expressions_1.MethodParamName), (0, combi_1.seq)("READ IMPORTING", expressions_1.MethodParamName, "FOR READ", expressions_1.TypeName, "RESULT", expressions_1.MethodParamName), (0, combi_1.seq)("FEATURES IMPORTING", expressions_1.MethodParamName, "REQUEST", expressions_1.NamespaceSimpleName, "FOR", expressions_1.NamespaceSimpleName, "RESULT", expressions_1.MethodParamName), (0, combi_1.seq)("DETERMINE ON MODIFY IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName), (0, combi_1.seq)("DETERMINE ON SAVE IMPORTING", expressions_1.MethodParamName, "FOR", expressions_1.TypeName));
         // todo, this is only from version something
         const amdp = (0, combi_1.seq)("AMDP OPTIONS CDS SESSION CLIENT CURRENT", (0, combi_1.optPrio)(expressions_1.MethodDefImporting), (0, combi_1.optPrio)(expressions_1.MethodDefExporting), (0, combi_1.optPrio)(expressions_1.MethodDefRaising));
         const ret = (0, combi_1.seq)((0, combi_1.altPrio)("CLASS-METHODS", "METHODS"), expressions_1.MethodName, (0, combi_1.alt)((0, combi_1.seq)((0, combi_1.optPrio)(expressions_1.Abstract), expressions_1.EventHandler), parameters, testing, (0, combi_1.seq)("FOR", (0, combi_1.alt)(tableFunction, ddl, behavior)), amdp, "NOT AT END OF MODE", (0, combi_1.optPrio)(expressions_1.Redefinition)));

package/build/src/abap/2_statements/statements/modify_entities.js

@@ -8,7 +8,7 @@ class ModifyEntities {
     getMatcher() {
         const fieldsWith = (0, combi_1.seq)("FIELDS (", (0, combi_1.plus)(expressions_1.SimpleName), ") WITH", expressions_1.Source);
         const operation = (0, combi_1.alt)((0, combi_1.seq)("UPDATE SET FIELDS WITH", expressions_1.Source), (0, combi_1.seq)("UPDATE", fieldsWith), (0, combi_1.seq)("DELETE FROM", expressions_1.Source), (0, combi_1.seq)("EXECUTE", expressions_1.SimpleName, "FROM", expressions_1.Source), (0, combi_1.seq)("CREATE", (0, combi_1.optPrio)("AUTO FILL CID"), fieldsWith));
-        const s = (0, combi_1.seq)("MODIFY ENTITIES OF", expressions_1.SimpleName, (0, combi_1.opt)("IN LOCAL MODE"), "ENTITY", expressions_1.SimpleName, operation, (0, combi_1.optPrio)((0, combi_1.seq)("MAPPED", expressions_1.Target)), (0, combi_1.per)((0, combi_1.seq)("FAILED", expressions_1.Target), (0, combi_1.seq)("RESULT", expressions_1.Target), (0, combi_1.seq)("REPORTED", expressions_1.Target)));
+        const s = (0, combi_1.seq)("MODIFY ENTITIES OF", expressions_1.NamespaceSimpleName, (0, combi_1.opt)("IN LOCAL MODE"), "ENTITY", expressions_1.SimpleName, operation, (0, combi_1.optPrio)((0, combi_1.seq)("MAPPED", expressions_1.Target)), (0, combi_1.per)((0, combi_1.seq)("FAILED", expressions_1.Target), (0, combi_1.seq)("RESULT", expressions_1.Target), (0, combi_1.seq)("REPORTED", expressions_1.Target)));
         return (0, combi_1.ver)(version_1.Version.v754, s);
     }
 }

package/build/src/abap/2_statements/statements/raise.js

@@ -10,7 +10,8 @@ class Raise {
         const messid = (0, combi_1.seq)("MESSAGE ID", expressions_1.Source, "NUMBER", (0, combi_1.altPrio)(expressions_1.MessageNumber, expressions_1.Source), (0, combi_1.optPrio)(expressions_1.RaiseWith));
         const exporting = (0, combi_1.seq)("EXPORTING", expressions_1.ParameterListS);
         const from = (0, combi_1.seq)("TYPE", expressions_1.ClassName, (0, combi_1.opt)((0, combi_1.alt)((0, combi_1.ver)(version_1.Version.v750, (0, combi_1.alt)(mess, messid)), (0, combi_1.ver)(version_1.Version.v752, "USING MESSAGE"))), (0, combi_1.optPrio)(exporting));
-        const clas = (0, combi_1.seq)((0, combi_1.optPrio)("RESUMABLE"), "EXCEPTION", (0, combi_1.altPrio)(from, (0, combi_1.ver)(version_1.Version.v752, expressions_1.Source), expressions_1.SimpleSource2));
+        const pre = (0, combi_1.alt)((0, combi_1.seq)((0, combi_1.optPrio)("RESUMABLE"), "EXCEPTION"), "SHORTDUMP");
+        const clas = (0, combi_1.seq)(pre, (0, combi_1.altPrio)(from, (0, combi_1.ver)(version_1.Version.v752, expressions_1.Source), expressions_1.SimpleSource2));
         const ret = (0, combi_1.seq)("RAISE", (0, combi_1.altPrio)(clas, expressions_1.ExceptionName));
         return ret;
     }

package/build/src/abap/2_statements/statements/read_entities.js

@@ -7,7 +7,7 @@ const version_1 = require("../../../version");
 const tokens_1 = require("../../1_lexer/tokens");
 class ReadEntities {
     getMatcher() {
-        const s = (0, combi_1.seq)("READ ENTITIES OF", expressions_1.SimpleName, "IN LOCAL MODE", "ENTITY", expressions_1.SimpleName, (0, combi_1.opt)((0, combi_1.seq)("BY", expressions_1.AssociationName)), "FIELDS", (0, combi_1.tok)(tokens_1.WParenLeftW), (0, combi_1.plus)(expressions_1.SimpleName), (0, combi_1.tok)(tokens_1.WParenRightW), "WITH", expressions_1.Source, "RESULT", expressions_1.Target, (0, combi_1.optPrio)((0, combi_1.seq)("LINK", expressions_1.Target)), (0, combi_1.optPrio)((0, combi_1.seq)("FAILED", expressions_1.Target)), (0, combi_1.optPrio)((0, combi_1.seq)("REPORTED", expressions_1.Target)));
+        const s = (0, combi_1.seq)("READ ENTITIES OF", expressions_1.NamespaceSimpleName, "IN LOCAL MODE", "ENTITY", expressions_1.SimpleName, (0, combi_1.opt)((0, combi_1.seq)("BY", expressions_1.AssociationName)), "FIELDS", (0, combi_1.tok)(tokens_1.WParenLeftW), (0, combi_1.plus)(expressions_1.SimpleName), (0, combi_1.tok)(tokens_1.WParenRightW), "WITH", expressions_1.Source, "RESULT", expressions_1.Target, (0, combi_1.optPrio)((0, combi_1.seq)("LINK", expressions_1.Target)), (0, combi_1.optPrio)((0, combi_1.seq)("FAILED", expressions_1.Target)), (0, combi_1.optPrio)((0, combi_1.seq)("REPORTED", expressions_1.Target)));
         return (0, combi_1.ver)(version_1.Version.v754, s);
     }
 }

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

@@ -3,6 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.ReduceNext = void 0;
 const Expressions = require("../../2_statements/expressions");
 const source_1 = require("./source");
+const target_1 = require("./target");
 class ReduceNext {
     runSyntax(node, scope, filename) {
         if (node === undefined) {
@@ -11,6 +12,9 @@ class ReduceNext {
         for (const s of node.findDirectExpressions(Expressions.Source)) {
             new source_1.Source().runSyntax(s, scope, filename);
         }
+        for (const s of node.findDirectExpressions(Expressions.SimpleTarget)) {
+            new target_1.Target().runSyntax(s, scope, filename);
+        }
     }
 }
 exports.ReduceNext = ReduceNext;

package/build/src/registry.js

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

package/build/src/rules/downport.js

@@ -1713,7 +1713,7 @@ ${indentation}    output = ${topTarget}.`;
                 if (concat.toUpperCase() === "NEXT") {
                     continue;
                 }
-                else if (n.get() instanceof Expressions.SimpleFieldChain) {
+                else if (n.get() instanceof Expressions.SimpleTarget) {
                     body += indentation + "  " + concat + " ";
                 }
                 else if (n.get() instanceof Expressions.Source) {

package/build/src/rules/slow_parameter_passing.js

@@ -50,6 +50,9 @@ class SlowParameterPassing {
                 const writes = this.listWritePositions(m, id);
                 if (writes.length === 0) {
                     const message = "Parameter " + id.getName() + " passed by VALUE but not changed";
+                    if (this.reg.isFileDependency(id.getFilename()) === true) {
+                        continue;
+                    }
                     issues.push(issue_1.Issue.atIdentifier(id, message, this.getMetadata().key, this.getConfig().severity));
                 }
             }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.93.50",
+    "version": "2.93.52",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -49,9 +49,9 @@
         "@microsoft/api-extractor": "^7.33.4",
         "@types/chai": "^4.3.3",
         "@types/mocha": "^10.0.0",
-        "@types/node": "^18.11.3",
+        "@types/node": "^18.11.5",
         "chai": "^4.3.6",
-        "eslint": "^8.25.0",
+        "eslint": "^8.26.0",
         "mocha": "^10.1.0",
         "c8": "^7.12.0",
         "source-map-support": "^0.5.21",