@abaplint/transpiler-cli 2.5.76 → 2.5.78

package/build/bundle.js

@@ -8581,7 +8581,7 @@ const expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/
 const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
 class CallFunction {
     getMatcher() {
-        const starting = (0, combi_1.seq)("STARTING NEW TASK", expressions_1.SimpleSource2);
+        const starting = (0, combi_1.verNot)(version_1.Version.Cloud, (0, combi_1.seq)("STARTING NEW TASK", expressions_1.SimpleSource2));
         const update = (0, combi_1.verNot)(version_1.Version.Cloud, (0, combi_1.str)("IN UPDATE TASK"));
         const unit = (0, combi_1.seq)("UNIT", expressions_1.Source);
         const background = (0, combi_1.verNot)(version_1.Version.Cloud, (0, combi_1.seq)("IN BACKGROUND", (0, combi_1.altPrio)("TASK", unit)));
@@ -19434,6 +19434,18 @@ class CurrentScope {
         (_a = this.current) === null || _a === void 0 ? void 0 : _a.getData().references.push({ position, resolved: referencing, referenceType: type, extra });
     }
     ///////////////////////////
+    findFunctionModule(name) {
+        if (name === undefined) {
+            return undefined;
+        }
+        for (const fugr of this.reg.getObjectsByType("FUGR")) {
+            const func = fugr.getModule(name);
+            if (func !== undefined) {
+                return func;
+            }
+        }
+        return undefined;
+    }
     findObjectDefinition(name) {
         if (name === undefined) {
             return undefined;
@@ -23566,9 +23578,10 @@ const tokens_1 = __webpack_require__(/*! ../../1_lexer/tokens */ "./node_modules
 const attribute_name_1 = __webpack_require__(/*! ./attribute_name */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/attribute_name.js");
 const component_name_1 = __webpack_require__(/*! ./component_name */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/component_name.js");
 const types_1 = __webpack_require__(/*! ../../types */ "./node_modules/@abaplint/core/build/src/abap/types/index.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
 class MethodSource {
     runSyntax(node, scope, filename) {
-        var _a;
+        var _a, _b, _c;
         const helper = new _object_oriented_1.ObjectOriented(scope);
         const children = node.getChildren().slice();
         const first = children.shift();
@@ -23580,6 +23593,15 @@ class MethodSource {
             context = (_a = scope.findVariable("me")) === null || _a === void 0 ? void 0 : _a.getType();
             children.unshift(first);
         }
+        if (scope.getVersion() === version_1.Version.Cloud
+            && first.get() instanceof Expressions.Dynamic
+            && first instanceof nodes_1.ExpressionNode
+            && ((_b = children[0]) === null || _b === void 0 ? void 0 : _b.concatTokens()) === "=>") {
+            const name = (_c = first.findDirectExpression(Expressions.Constant)) === null || _c === void 0 ? void 0 : _c.concatTokens().replace(/'/g, "");
+            if (name !== undefined && scope.findClassDefinition(name) === undefined) {
+                throw new Error(`Class "${name}" not found/released`);
+            }
+        }
         if (context instanceof basic_1.VoidType) {
             // todo, if there are more dynamic with variables, the references for the variables are not added?
             return context;
@@ -23610,7 +23632,7 @@ class MethodSource {
                         continue;
                     }
                 }
-                catch (_b) {
+                catch (_d) {
                     // ignore
                 }
                 // try looking for method name
@@ -25598,6 +25620,7 @@ const source_1 = __webpack_require__(/*! ../expressions/source */ "./node_module
 const target_1 = __webpack_require__(/*! ../expressions/target */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/target.js");
 const field_chain_1 = __webpack_require__(/*! ../expressions/field_chain */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/field_chain.js");
 const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
 class CallFunction {
     runSyntax(node, scope, filename) {
         // todo, lots of work here, similar to receive.ts
@@ -25606,6 +25629,13 @@ class CallFunction {
         if (chain) {
             new field_chain_1.FieldChain().runSyntax(chain, scope, filename, _reference_1.ReferenceType.DataReadReference);
         }
+        else if (scope.getVersion() === version_1.Version.Cloud
+            && node.findDirectExpression(Expressions.Destination) === undefined) {
+            const functionName = name === null || name === void 0 ? void 0 : name.concatTokens().replace(/'/, "");
+            if (scope.findFunctionModule(functionName) === undefined) {
+                throw new Error(`Function module "${functionName}" not found/released`);
+            }
+        }
         // just recurse
         for (const s of node.findAllExpressions(Expressions.Source)) {
             new source_1.Source().runSyntax(s, scope, filename);
@@ -26416,7 +26446,10 @@ class CreateObject {
                 }
                 else if (found instanceof basic_1.ObjectReferenceType) {
                     const id = found.getIdentifier();
-                    if (id instanceof types_1.ClassDefinition && cdef === undefined) {
+                    if (id instanceof types_1.InterfaceDefinition && type === undefined) {
+                        throw new Error("Interface reference, cannot be instantiated");
+                    }
+                    else if (id instanceof types_1.ClassDefinition && cdef === undefined) {
                         cdef = id;
                     }
                     if (type === undefined && id instanceof types_1.ClassDefinition && id.isAbstract() === true) {
@@ -39166,10 +39199,15 @@ class LanguageServer {
         }
         const text = new pretty_printer_1.PrettyPrinter(file, this.reg.getConfig()).run();
         const rows = file.getRawRows();
-        return [{
-                range: LServer.Range.create(0, 0, rows.length, rows[rows.length - 1].length + 1),
-                newText: text,
-            }];
+        if (text === file.getRaw()) {
+            return [];
+        }
+        else {
+            return [{
+                    range: LServer.Range.create(0, 0, rows.length, rows[rows.length - 1].length + 1),
+                    newText: text,
+                }];
+        }
     }
     // https://microsoft.github.io/language-server-protocol/specifications/specification-3-14/#textDocument_publishDiagnostics
     diagnostics(textDocument) {
@@ -46079,7 +46117,6 @@ exports.PrettyPrinter = void 0;
 const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
 const fix_keyword_case_1 = __webpack_require__(/*! ./fix_keyword_case */ "./node_modules/@abaplint/core/build/src/pretty_printer/fix_keyword_case.js");
 const indent_1 = __webpack_require__(/*! ./indent */ "./node_modules/@abaplint/core/build/src/pretty_printer/indent.js");
-const remove_sequential_blanks_1 = __webpack_require__(/*! ./remove_sequential_blanks */ "./node_modules/@abaplint/core/build/src/pretty_printer/remove_sequential_blanks.js");
 const position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
 const indentation_1 = __webpack_require__(/*! ../rules/indentation */ "./node_modules/@abaplint/core/build/src/rules/indentation.js");
 class PrettyPrinter {
@@ -46109,8 +46146,6 @@ class PrettyPrinter {
         }
         const indentation = new indent_1.Indent(this.options);
         this.result = indentation.execute(this.file, this.result);
-        const removeBlanks = new remove_sequential_blanks_1.RemoveSequentialBlanks(this.config);
-        this.result = removeBlanks.execute(this.file, this.result);
         return this.result;
     }
 }
@@ -46119,66 +46154,6 @@ exports.PrettyPrinter = PrettyPrinter;
 
 /***/ }),
 
-/***/ "./node_modules/@abaplint/core/build/src/pretty_printer/remove_sequential_blanks.js":
-/*!******************************************************************************************!*\
-  !*** ./node_modules/@abaplint/core/build/src/pretty_printer/remove_sequential_blanks.js ***!
-  \******************************************************************************************/
-/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
-
-"use strict";
-
-Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.RemoveSequentialBlanks = void 0;
-const rules_1 = __webpack_require__(/*! ../rules */ "./node_modules/@abaplint/core/build/src/rules/index.js");
-class RemoveSequentialBlanks {
-    constructor(config) {
-        this.config = config;
-    }
-    execute(file, modified) {
-        const sequentialBlankConfig = this.getSequentialBlankConfig();
-        if (sequentialBlankConfig) {
-            return this.withoutSequentialBlanks(file, modified, sequentialBlankConfig.lines);
-        }
-        return modified;
-    }
-    withoutSequentialBlanks(file, modified, threshold) {
-        const rows = file.getRawRows();
-        let blanks = 0;
-        const rowsToRemove = [];
-        const newBlankCount = (current, row) => {
-            return rules_1.SequentialBlank.isBlankOrWhitespace(row) ? current + 1 : 0;
-        };
-        for (let i = 0; i < rows.length; i++) {
-            blanks = newBlankCount(blanks, rows[i]);
-            if (blanks === threshold) {
-                // count additional blanks
-                for (let j = i; j < rows.length; j++) {
-                    if (rules_1.SequentialBlank.isBlankOrWhitespace(rows[j])) {
-                        rowsToRemove.push(j);
-                    }
-                    else {
-                        break;
-                    }
-                }
-            }
-        }
-        return this.removeRows(modified.split("\n"), rowsToRemove);
-    }
-    removeRows(lines, rowsToRemove) {
-        const withoutRemoved = lines.filter((_, idx) => {
-            return rowsToRemove.indexOf(idx) === -1;
-        });
-        return withoutRemoved.join("\n").trim();
-    }
-    getSequentialBlankConfig() {
-        return this.config.readByRule(new rules_1.SequentialBlank().getMetadata().key);
-    }
-}
-exports.RemoveSequentialBlanks = RemoveSequentialBlanks;
-//# sourceMappingURL=remove_sequential_blanks.js.map
-
-/***/ }),
-
 /***/ "./node_modules/@abaplint/core/build/src/registry.js":
 /*!***********************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/registry.js ***!
@@ -46251,7 +46226,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.97.4";
+        return "2.97.7";
     }
     getDDICReferences() {
         return this.references;
@@ -71307,7 +71282,7 @@ const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/tr
 const expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/transpiler/build/src/expressions/index.js");
 class CallTranspiler {
     transpile(node, traversal) {
-        var _a, _b, _c, _d, _e;
+        var _a, _b, _c, _d;
         const chain = node.findDirectExpression(abaplint.Expressions.MethodCallChain);
         if (chain) {
             let pre = "";
@@ -71317,34 +71292,12 @@ class CallTranspiler {
                 pre = traversal.traverse(receiving).getCode() + ".set(";
                 post = ")";
             }
+            post += ";";
             const exceptions = node.findFirstExpression(abaplint.Expressions.ParameterListExceptions);
             if (exceptions) {
-                pre = "try {\n" + pre;
-            }
-            post += ";";
-            if (exceptions) {
-                post += `\nabap.builtin.sy.get().subrc.set(0);
-} catch (e) {
-if (e.classic) {
-  switch (e.classic.toUpperCase()) {\n`;
-                for (const e of exceptions.findAllExpressions(abaplint.Expressions.ParameterException)) {
-                    const name = e.getFirstToken().getStr().toUpperCase();
-                    const value = (_c = e.findFirstExpression(abaplint.Expressions.SimpleName)) === null || _c === void 0 ? void 0 : _c.getFirstToken().getStr().toUpperCase();
-                    if (value === undefined) {
-                        continue;
-                    }
-                    if (name === "OTHERS") {
-                        post += `default: abap.builtin.sy.get().subrc.set(${value}); break;\n`;
-                    }
-                    else {
-                        post += `case "${name}": abap.builtin.sy.get().subrc.set(${value}); break;\n`;
-                    }
-                }
-                post += `  }
-} else {
-  throw e;
-}
-}`;
+                const build = this.buildExceptions(exceptions);
+                pre = build.pre + pre;
+                post += build.post;
             }
             const chainChunk = traversal.traverse(chain);
             let chainCode = chainChunk.getCode();
@@ -71360,7 +71313,7 @@ if (e.classic) {
         const methodSource = node.findDirectExpression(abaplint.Expressions.MethodSource);
         if (methodSource) {
             let body = "";
-            const nameToken = (_d = methodSource.getLastChild()) === null || _d === void 0 ? void 0 : _d.getFirstToken();
+            const nameToken = (_c = methodSource.getLastChild()) === null || _c === void 0 ? void 0 : _c.getFirstToken();
             const m = nameToken ? traversal.findMethodReference(nameToken, traversal.findCurrentScopeByToken(nameToken)) : undefined;
             const methodCallBody = node.findDirectExpression(abaplint.Expressions.MethodCallBody);
             if (methodCallBody) {
@@ -71368,12 +71321,18 @@ if (e.classic) {
             }
             let pre = "";
             let post = "";
-            const receiving = (_e = node.findFirstExpression(abaplint.Expressions.MethodParameters)) === null || _e === void 0 ? void 0 : _e.findExpressionAfterToken("RECEIVING");
+            const receiving = (_d = node.findFirstExpression(abaplint.Expressions.MethodParameters)) === null || _d === void 0 ? void 0 : _d.findExpressionAfterToken("RECEIVING");
             if (receiving) {
                 const target = traversal.traverse(receiving.findDirectExpression(abaplint.Expressions.Target));
                 pre = target.getCode() + ".set(";
                 post = ")";
             }
+            const exceptions = node.findFirstExpression(abaplint.Expressions.ParameterListExceptions);
+            if (exceptions) {
+                const build = this.buildExceptions(exceptions);
+                pre = build.pre + pre;
+                post += build.post;
+            }
             let ms = new expressions_1.MethodSourceTranspiler(pre).transpile(methodSource, traversal).getCode();
             if (ms === "await super.get().constructor") {
                 // semantics of constructors in JS vs ABAP is different, so the "constructor_" has been introduced,
@@ -71386,6 +71345,35 @@ if (e.classic) {
         }
         throw new Error("CallTranspiler, todo");
     }
+    buildExceptions(node) {
+        var _a;
+        let pre = "";
+        let post = "";
+        pre = "try {\n" + pre;
+        post += `\nabap.builtin.sy.get().subrc.set(0);
+} catch (e) {
+if (e.classic) {
+  switch (e.classic.toUpperCase()) {\n`;
+        for (const e of node.findAllExpressions(abaplint.Expressions.ParameterException)) {
+            const name = e.getFirstToken().getStr().toUpperCase();
+            const value = (_a = e.findFirstExpression(abaplint.Expressions.SimpleName)) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStr().toUpperCase();
+            if (value === undefined) {
+                continue;
+            }
+            if (name === "OTHERS") {
+                post += `default: abap.builtin.sy.get().subrc.set(${value}); break;\n`;
+            }
+            else {
+                post += `case "${name}": abap.builtin.sy.get().subrc.set(${value}); break;\n`;
+            }
+        }
+        post += `  }
+} else {
+  throw e;
+}
+}`;
+        return { pre, post };
+    }
 }
 exports.CallTranspiler = CallTranspiler;
 //# sourceMappingURL=call.js.map
@@ -79278,6 +79266,7 @@ const defaultOptions = {
   stopNodes: [],
   // transformTagName: false,
   // transformAttributeName: false,
+  oneListGroup: false
 };
 
 function Builder(options) {
@@ -79348,6 +79337,7 @@ Builder.prototype.j2x = function(jObj, level) {
     } else if (Array.isArray(jObj[key])) {
       //repeated nodes
       const arrLen = jObj[key].length;
+      let listTagVal = "";
       for (let j = 0; j < arrLen; j++) {
         const item = jObj[key][j];
         if (typeof item === 'undefined') {
@@ -79357,11 +79347,19 @@ Builder.prototype.j2x = function(jObj, level) {
           else val += this.indentate(level) + '<' + key + '/' + this.tagEndChar;
           // val += this.indentate(level) + '<' + key + '/' + this.tagEndChar;
         } else if (typeof item === 'object') {
-          val += this.processTextOrObjNode(item, key, level)
+          if(this.options.oneListGroup ){
+            listTagVal += this.j2x(item, level + 1).val;
+          }else{
+            listTagVal += this.processTextOrObjNode(item, key, level)
+          }
         } else {
-          val += this.buildTextValNode(item, key, '', level);
+          listTagVal += this.buildTextValNode(item, key, '', level);
         }
       }
+      if(this.options.oneListGroup){
+        listTagVal = this.buildObjectNode(listTagVal, key, '', level);
+      }
+      val += listTagVal;
     } else {
       //nested node
       if (this.options.attributesGroupName && key === this.options.attributesGroupName) {
@@ -80588,8 +80586,20 @@ function assignAttributes(obj, attrMap, jpath, options){
 }
 
 function isLeafTag(obj, options){
+  const { textNodeName } = options;
   const propCount = Object.keys(obj).length;
-  if( propCount === 0 || (propCount === 1 && obj[options.textNodeName]) ) return true;
+  
+  if (propCount === 0) {
+    return true;
+  }
+
+  if (
+    propCount === 1 &&
+    (obj[textNodeName] || typeof obj[textNodeName] === "boolean" || obj[textNodeName] === 0)
+  ) {
+    return true;
+  }
+
   return false;
 }
 exports.prettify = prettify;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.5.76",
+  "version": "2.5.78",
   "description": "Transpiler - Command Line Interface",
   "bin": {
     "abap_transpile": "./abap_transpile"
@@ -25,15 +25,15 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/transpiler": "^2.5.76",
+    "@abaplint/transpiler": "^2.5.78",
     "@types/glob": "^7.2.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.5",
     "@types/node": "^18.15.11",
-    "@abaplint/core": "^2.97.4",
+    "@abaplint/core": "^2.97.7",
     "progress": "^2.0.3",
-    "webpack": "^5.77.0",
+    "webpack": "^5.78.0",
     "webpack-cli": "^5.0.1",
-    "typescript": "^5.0.3"
+    "typescript": "^5.0.4"
   }
 }