@abaplint/core 2.90.1 → 2.90.2

package/build/src/registry.js

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

package/build/src/rules/downport.js

@@ -1224,12 +1224,14 @@ ${indentation}    output = ${topTarget}.`;
                 body += indentation + uniqueName + " = " + base.concatTokens() + ".\n";
             }
             let end = "";
-            for (const forLoop of (valueBody === null || valueBody === void 0 ? void 0 : valueBody.findDirectExpressions(Expressions.For)) || []) {
-                const outlineFor = this.outlineFor(forLoop, indentation, lowFile, highSyntax);
-                body += outlineFor.body;
-                end = outlineFor.end + `.\n` + end;
-                indentation += "  ";
-            }
+            /*
+                  for (const forLoop of valueBody?.findDirectExpressions(Expressions.For) || []) {
+                    const outlineFor = this.outlineFor(forLoop, indentation, lowFile, highSyntax);
+                    body += outlineFor.body;
+                    end = outlineFor.end + `.\n` + end;
+                    indentation += "  ";
+                  }
+            */
             let structureName = uniqueName;
             let added = false;
             let skip = false;
@@ -1247,6 +1249,12 @@ ${indentation}    output = ${topTarget}.`;
                     }
                     body += indentation + structureName + "-" + b.concatTokens() + ".\n";
                 }
+                else if (b instanceof nodes_1.ExpressionNode && b.get() instanceof Expressions.For) {
+                    const outlineFor = this.outlineFor(b, indentation, lowFile, highSyntax);
+                    body += outlineFor.body;
+                    end = outlineFor.end + `.\n` + end;
+                    indentation += "  ";
+                }
                 else if (b.get() instanceof Expressions.Source) {
                     structureName = b.concatTokens();
                     if (base && (valueBody === null || valueBody === void 0 ? void 0 : valueBody.findDirectTokenByText("(")) === undefined) {
@@ -1302,6 +1310,10 @@ ${indentation}    output = ${topTarget}.`;
             }
             const found = spag.findVariable(name);
             if (found === undefined) {
+                const source = f.findFirstExpression(Expressions.Source);
+                if (source) {
+                    ret += indentation + "DATA(" + name + `) = ${source.concatTokens()}.\n`;
+                }
                 continue;
             }
             const type = found.getType().getQualifiedName() ? (_a = found.getType().getQualifiedName()) === null || _a === void 0 ? void 0 : _a.toLowerCase() : found.getType().toABAP();

package/build/src/rules/function_module_recommendations.js

@@ -12,28 +12,29 @@ class FunctionModuleRecommendationsConf extends _basic_rule_config_1.BasicRuleCo
         /** Tuple of Function Module Name to be replaced, the recommended alternative and
          * the version from which the recommendation is valid.*/
         this.recommendations = [
-            { name: "SUBST_GET_FILE_LIST", replace: "see note 1686357" },
-            { name: "ROUND", replace: "use built in function: round()" },
-            { name: "SO_NEW_DOCUMENT_ATT_SEND_API1", replace: "use CL_BCS" },
+            { name: "CALCULATE_HASH_FOR_RAW", replace: "use CL_ABAP_HMAC" },
             { name: "ECATT_CONV_XSTRING_TO_STRING", replace: "use CL_BINARY_CONVERT" },
-            { name: "SCMS_STRING_TO_XSTRING", replace: "use CL_BINARY_CONVERT" },
-            { name: "JOB_CREATE", replace: "use CL_BP_ABAP_JOB" },
-            { name: "JOB_SUBMIT", replace: "use CL_BP_ABAP_JOB" },
+            { name: "F4_FILENAME", replace: "use CL_GUI_FRONTEND_SERVICES" },
+            { name: "FUNCTION_EXISTS", replace: "surround with try-catch CX_SY_DYN_CALL_ILLEGAL_METHOD instead" },
             { name: "GUI_DOWNLOAD", replace: "use CL_GUI_FRONTEND_SERVICES" },
             { name: "GUI_UPLOAD", replace: "use CL_GUI_FRONTEND_SERVICES" },
-            { name: "WS_FILENAME_GET", replace: "use CL_GUI_FRONTEND_SERVICES" },
-            { name: "F4_FILENAME", replace: "use CL_GUI_FRONTEND_SERVICES" },
-            { name: "SAPGUI_PROGRESS_INDICATOR", replace: "use CL_PROGRESS_INDICATOR" },
             { name: "GUID_CREATE", replace: "use CL_SYSTEM_UUID" },
-            { name: "SSFC_BASE64_DECODE", replace: "use class CL_HTTP_UTILITY methods" },
-            { name: "SSFC_BASE64_ENCODE", replace: "use class CL_HTTP_UTILITY methods" },
-            { name: "SCMS_BASE64_DECODE_STR", replace: "use class CL_HTTP_UTILITY methods" },
-            { name: "POPUP_TO_DECIDE", replace: "use POPUP_TO_CONFIRM" },
-            { name: "REUSE_ALV_GRID_DISPLAY", replace: "use CL_SALV_TABLE=>FACTORY or CL_GUI_ALV_GRID" },
-            { name: "CALCULATE_HASH_FOR_RAW", replace: "use CL_ABAP_HMAC" },
-            { name: "FUNCTION_EXISTS", replace: "surround with try-catch CX_SY_DYN_CALL_ILLEGAL_METHOD instead" },
             { name: "IGN_TIMESTAMP_DIFFERENCE", replace: "use CL_ABAP_TSTMP" },
             { name: "IGN_TIMESTAMP_PLUSMINUS", replace: "use CL_ABAP_TSTMP" },
+            { name: "JOB_CREATE", replace: "use CL_BP_ABAP_JOB" },
+            { name: "JOB_SUBMIT", replace: "use CL_BP_ABAP_JOB" },
+            { name: "POPUP_TO_DECIDE", replace: "use POPUP_TO_CONFIRM" },
+            { name: "POPUP_TO_GET_VALUE", replace: "use POPUP_GET_VALUES" },
+            { name: "REUSE_ALV_GRID_DISPLAY", replace: "use CL_SALV_TABLE=>FACTORY or CL_GUI_ALV_GRID" },
+            { name: "ROUND", replace: "use built in function: round()" },
+            { name: "SAPGUI_PROGRESS_INDICATOR", replace: "use CL_PROGRESS_INDICATOR" },
+            { name: "SCMS_BASE64_DECODE_STR", replace: "use class CL_HTTP_UTILITY methods" },
+            { name: "SCMS_STRING_TO_XSTRING", replace: "use CL_BINARY_CONVERT" },
+            { name: "SO_NEW_DOCUMENT_ATT_SEND_API1", replace: "use CL_BCS" },
+            { name: "SSFC_BASE64_DECODE", replace: "use class CL_HTTP_UTILITY methods" },
+            { name: "SSFC_BASE64_ENCODE", replace: "use class CL_HTTP_UTILITY methods" },
+            { name: "SUBST_GET_FILE_LIST", replace: "see note 1686357" },
+            { name: "WS_FILENAME_GET", replace: "use CL_GUI_FRONTEND_SERVICES" },
         ];
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.90.1",
+    "version": "2.90.2",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -45,10 +45,10 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@microsoft/api-extractor": "^7.23.1",
+        "@microsoft/api-extractor": "^7.23.2",
         "@types/chai": "^4.3.1",
         "@types/mocha": "^9.1.1",
-        "@types/node": "^17.0.31",
+        "@types/node": "^17.0.32",
         "chai": "^4.3.6",
         "eslint": "^8.15.0",
         "mocha": "^10.0.0",
@@ -60,7 +60,7 @@
     "dependencies": {
         "fast-xml-parser": "^4.0.7",
         "json5": "^2.2.1",
-        "vscode-languageserver-protocol": "^3.16.0",
-        "vscode-languageserver-types": "^3.16.0"
+        "vscode-languageserver-protocol": "=3.16.0",
+        "vscode-languageserver-types": "=3.16.0"
     }
 }