@abaplint/transpiler-cli 2.11.38 → 2.11.40

package/build/bundle.js

@@ -4379,7 +4379,7 @@ class For extends combi_1.Expression {
         const t = (0, combi_1.alt)(_1.TargetField, _1.TargetFieldSymbol);
         const groups = (0, combi_1.ver)(version_1.Version.v740sp08, (0, combi_1.seq)("GROUPS", t, "OF", t, "IN", _1.Source, (0, combi_1.optPrio)(groupBy)));
         const f = (0, combi_1.seq)("FOR", (0, combi_1.alt)(itera, inn, groups), (0, combi_1.optPrio)(_1.Let));
-        return (0, combi_1.ver)(version_1.Version.v740sp05, f);
+        return (0, combi_1.ver)(version_1.Version.v740sp05, f, version_1.Version.OpenABAP);
     }
 }
 exports.For = For;
@@ -53791,7 +53791,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.180";
+        return "2.113.181";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -79195,9 +79195,14 @@ class FieldChainTranspiler {
                 let name = c.getFirstToken().getStr().toLowerCase();
                 if (context instanceof abaplint.BasicTypes.ObjectReferenceType) {
                     const cdef = traversal.findClassDefinition(context.getIdentifierName(), scope);
-                    const attr = cdef?.getAttributes().findByName(c.getFirstToken().getStr());
+                    const tokenName = c.getFirstToken().getStr();
+                    const attr = cdef?.getAttributes().findByName(tokenName);
+                    // Do not mark constants as private JS fields. Constants are exposed on instances via constructor copying.
+                    const constants = cdef?.getAttributes().getConstants() || [];
+                    const isConstant = constants.some(cons => cons.getName().toUpperCase() === tokenName.toUpperCase());
                     if (feature_flags_1.FEATURE_FLAGS.PRIVATE_ATTRIBUTES === true
-                        && attr?.getVisibility() === abaplint.Visibility.Private) {
+                        && attr?.getVisibility() === abaplint.Visibility.Private
+                        && isConstant === false) {
                         const id = scope?.getParent()?.getParent()?.getIdentifier();
                         if (id?.stype === abaplint.ScopeType.ClassImplementation
                             && cdef?.getName().toUpperCase() === id.sname.toUpperCase()) {
@@ -81621,6 +81626,8 @@ const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/tr
 const type_name_or_infer_1 = __webpack_require__(/*! ./type_name_or_infer */ "./node_modules/@abaplint/transpiler/build/src/expressions/type_name_or_infer.js");
 const value_body_line_1 = __webpack_require__(/*! ./value_body_line */ "./node_modules/@abaplint/transpiler/build/src/expressions/value_body_line.js");
 const field_assignment_1 = __webpack_require__(/*! ./field_assignment */ "./node_modules/@abaplint/transpiler/build/src/expressions/field_assignment.js");
+const statements_1 = __webpack_require__(/*! ../statements */ "./node_modules/@abaplint/transpiler/build/src/statements/index.js");
+const source_field_symbol_1 = __webpack_require__(/*! ./source_field_symbol */ "./node_modules/@abaplint/transpiler/build/src/expressions/source_field_symbol.js");
 class ValueBodyTranspiler {
     transpile(typ, body, traversal) {
         if (!(typ.get() instanceof core_1.Expressions.TypeNameOrInfer)) {
@@ -81628,8 +81635,9 @@ class ValueBodyTranspiler {
         }
         let ret = new chunk_1.Chunk().appendString(new type_name_or_infer_1.TypeNameOrInfer().transpile(typ, traversal).getCode());
         const context = new type_name_or_infer_1.TypeNameOrInfer().findType(typ, traversal);
-        const hasLines = body.findDirectExpression(core_1.Expressions.ValueBodyLine) !== undefined;
+        let post = "";
         let extraFields = "";
+        const hasLines = body.findDirectExpression(core_1.Expressions.ValueBodyLine) !== undefined;
         for (const child of body.getChildren()) {
             if (child.get() instanceof core_1.Expressions.FieldAssignment && child instanceof core_1.Nodes.ExpressionNode) {
                 const transpiled = new field_assignment_1.FieldAssignmentTranspiler().transpile(child, traversal).getCode();
@@ -81655,11 +81663,47 @@ class ValueBodyTranspiler {
                 const source = traversal.traverse(child);
                 ret.appendString(".set(" + source.getCode() + ".clone())");
             }
+            else if (child.get() instanceof core_1.Expressions.For && child instanceof core_1.Nodes.ExpressionNode) {
+                if (child.getChildren().length !== 2) {
+                    throw new Error("ValueBody FOR todo, num");
+                }
+                const loop = child.findDirectExpression(core_1.Expressions.InlineLoopDefinition);
+                if (loop === undefined) {
+                    throw new Error("ValueBody FOR todo");
+                }
+                else if (loop.getChildren().length !== 3) {
+                    throw new Error("ValueBody FOR todo, num loop");
+                }
+                let targetDeclare = "";
+                let targetAction = "";
+                const fs = loop.findDirectExpression(core_1.Expressions.TargetFieldSymbol);
+                if (fs) {
+                    targetDeclare = new statements_1.FieldSymbolTranspiler().transpile(fs, traversal).getCode();
+                    const targetName = new source_field_symbol_1.SourceFieldSymbolTranspiler().transpile(fs, traversal).getCode();
+                    targetAction = `${targetName}.assign(unique1);`;
+                }
+                else {
+                    const field = traversal.traverse(loop.findDirectExpression(core_1.Expressions.TargetField));
+                    if (field === undefined) {
+                        throw new Error("ValueBody FOR empty field todo");
+                    }
+                    targetAction = `const ${field.getCode()} = unique1.clone();`;
+                }
+                const source = traversal.traverse(loop.findDirectExpression(core_1.Expressions.Source)).getCode();
+                const val = new type_name_or_infer_1.TypeNameOrInfer().transpile(typ, traversal).getCode();
+                ret = new chunk_1.Chunk().appendString(`await (async () => {
+${targetDeclare}
+const VAL = ${val};
+for await (const unique1 of abap.statements.loop(${source})) {
+  ${targetAction}
+  VAL`);
+                post = ";\n}\nreturn VAL;\n})()";
+            }
             else {
                 throw new Error("ValueBodyTranspiler, unknown " + child.get().constructor.name + " \"" + child.concatTokens()) + "\"";
             }
         }
-        return ret;
+        return ret.appendString(post);
     }
 }
 exports.ValueBodyTranspiler = ValueBodyTranspiler;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.11.38",
+  "version": "2.11.40",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -27,8 +27,8 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/core": "^2.113.180",
-    "@abaplint/transpiler": "^2.11.38",
+    "@abaplint/core": "^2.113.181",
+    "@abaplint/transpiler": "^2.11.40",
     "@types/glob": "^8.1.0",
     "@types/node": "^24.3.0",
     "@types/progress": "^2.0.7",