@abaplint/core 2.85.31 → 2.85.32

package/build/abaplint.d.ts

@@ -1912,6 +1912,7 @@ declare namespace Expressions {
         SQLCompareOperator,
         SQLCompare,
         SQLCond,
+        SQLFieldListLoop,
         SQLFieldList,
         SQLFieldName,
         SQLForAllEntries,
@@ -4894,6 +4895,10 @@ declare class SQLFieldList extends Expression {
     getRunnable(): IStatementRunnable;
 }
 
+declare class SQLFieldListLoop extends Expression {
+    getRunnable(): IStatementRunnable;
+}
+
 declare class SQLFieldName extends Expression {
     getRunnable(): IStatementRunnable;
 }

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

@@ -158,6 +158,7 @@ __exportStar(require("./sql_client"), exports);
 __exportStar(require("./sql_compare_operator"), exports);
 __exportStar(require("./sql_compare"), exports);
 __exportStar(require("./sql_cond"), exports);
+__exportStar(require("./sql_field_list_loop"), exports);
 __exportStar(require("./sql_field_list"), exports);
 __exportStar(require("./sql_field_name"), exports);
 __exportStar(require("./sql_for_all_entries"), exports);

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

@@ -3,25 +3,15 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.SelectLoop = void 0;
 const combi_1 = require("../combi");
 const _1 = require(".");
-const version_1 = require("../../../version");
 const sql_order_by_1 = require("./sql_order_by");
 const sql_having_1 = require("./sql_having");
-const sql_path_1 = require("./sql_path");
-const sql_as_name_1 = require("./sql_as_name");
-const sql_case_1 = require("./sql_case");
 const sql_into_structure_1 = require("./sql_into_structure");
 const sql_field_list_1 = require("./sql_field_list");
 const sql_hints_1 = require("./sql_hints");
-const sql_function_1 = require("./sql_function");
+const sql_field_list_loop_1 = require("./sql_field_list_loop");
 class SelectLoop extends combi_1.Expression {
     getRunnable() {
         const where = (0, combi_1.seq)("WHERE", _1.SQLCond);
-        const comma = (0, combi_1.opt)((0, combi_1.ver)(version_1.Version.v740sp05, ","));
-        const as = (0, combi_1.seq)("AS", sql_as_name_1.SQLAsName);
-        const someField = (0, combi_1.seq)((0, combi_1.alt)(_1.SQLFieldName, sql_path_1.SQLPath, _1.SQLAggregation, sql_function_1.SQLFunction, sql_case_1.SQLCase), (0, combi_1.optPrio)(as), comma);
-        const fieldList = (0, combi_1.seq)((0, combi_1.star)(someField), (0, combi_1.alt)(_1.SQLFieldName, sql_path_1.SQLPath), (0, combi_1.optPrio)(as), comma, (0, combi_1.star)(someField));
-        // todo, use SQLFieldList instead?
-        const fields = (0, combi_1.alt)("*", _1.Dynamic, fieldList);
         const client = "CLIENT SPECIFIED";
         const bypass = "BYPASSING BUFFER";
         const up = (0, combi_1.seq)("UP TO", _1.SQLSource, "ROWS");
@@ -29,7 +19,7 @@ class SelectLoop extends combi_1.Expression {
         const tab = (0, combi_1.seq)(_1.SQLIntoTable, (0, combi_1.alt)(pack, (0, combi_1.seq)(_1.SQLFrom, pack), (0, combi_1.seq)(pack, _1.SQLFrom)));
         const perm = (0, combi_1.per)(_1.SQLFrom, where, up, sql_order_by_1.SQLOrderBy, sql_having_1.SQLHaving, client, bypass, _1.SQLGroupBy, _1.SQLForAllEntries, (0, combi_1.alt)(tab, sql_into_structure_1.SQLIntoStructure));
         const strict = (0, combi_1.seq)(_1.SQLFrom, "FIELDS", sql_field_list_1.SQLFieldList, where, sql_into_structure_1.SQLIntoStructure, up);
-        const ret = (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)((0, combi_1.seq)((0, combi_1.optPrio)("DISTINCT"), fields, perm), strict), (0, combi_1.optPrio)(sql_hints_1.SQLHints));
+        const ret = (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)((0, combi_1.seq)((0, combi_1.optPrio)("DISTINCT"), sql_field_list_loop_1.SQLFieldListLoop, perm), strict), (0, combi_1.optPrio)(sql_hints_1.SQLHints));
         return ret;
     }
 }

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

@@ -0,0 +1,21 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.SQLFieldListLoop = void 0;
+const combi_1 = require("../combi");
+const _1 = require(".");
+const version_1 = require("../../../version");
+const sql_function_1 = require("./sql_function");
+const sql_path_1 = require("./sql_path");
+// loop must include one field from the database table
+class SQLFieldListLoop extends combi_1.Expression {
+    getRunnable() {
+        const comma = (0, combi_1.opt)((0, combi_1.ver)(version_1.Version.v740sp05, ","));
+        const as = (0, combi_1.seq)("AS", _1.SQLAsName);
+        const someField = (0, combi_1.seq)((0, combi_1.alt)(_1.SQLFieldName, sql_path_1.SQLPath, _1.SQLAggregation, sql_function_1.SQLFunction, _1.SQLCase), (0, combi_1.optPrio)(as), comma);
+        const fieldList = (0, combi_1.seq)((0, combi_1.star)(someField), (0, combi_1.alt)(_1.SQLFieldName, sql_path_1.SQLPath), (0, combi_1.optPrio)(as), comma, (0, combi_1.star)(someField));
+        const fields = (0, combi_1.alt)("*", _1.Dynamic, fieldList);
+        return fields;
+    }
+}
+exports.SQLFieldListLoop = SQLFieldListLoop;
+//# sourceMappingURL=sql_field_list_loop.js.map

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

@@ -9,7 +9,11 @@ class Constant {
             return new basic_1.IntegerType("I");
         }
         else if (node.getFirstToken().getStr().startsWith("'")) {
-            return new basic_1.CharacterType(10);
+            let len = node.getFirstToken().getStr().length - 2;
+            if (len <= 0) {
+                len = 1;
+            }
+            return new basic_1.CharacterType(len);
         }
         else {
             return new basic_1.StringType("STRING");

package/build/src/registry.js

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

package/build/src/rules/downport.js

@@ -291,8 +291,10 @@ Only one transformation is applied to a statement at a time, so multiple steps m
         }
     }
     downportSelectInline(low, high, lowFile, highSyntax) {
-        if (!(low.get() instanceof _statement_1.Unknown)
-            || !(high.get() instanceof Statements.Select)) {
+        if (!(low.get() instanceof _statement_1.Unknown)) {
+            return undefined;
+        }
+        else if (!(high.get() instanceof Statements.Select) && !(high.get() instanceof Statements.SelectLoop)) {
             return undefined;
         }
         // as first step outline the @DATA, note that void types are okay, as long the field names are specified
@@ -325,7 +327,10 @@ Only one transformation is applied to a statement at a time, so multiple steps m
             return undefined;
         }
         const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
-        const fieldList = high.findFirstExpression(Expressions.SQLFieldList);
+        let fieldList = high.findFirstExpression(Expressions.SQLFieldList);
+        if (fieldList === undefined) {
+            fieldList = high.findFirstExpression(Expressions.SQLFieldListLoop);
+        }
         if (fieldList === undefined) {
             return undefined;
         }
@@ -368,26 +373,31 @@ ${indentation}`);
         if (targets.length !== 1) {
             return undefined;
         }
+        const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
         const inlineData = targets[0].findFirstExpression(Expressions.InlineData);
         if (inlineData === undefined) {
             return undefined;
         }
         const sqlFrom = high.findAllExpressions(Expressions.SQLFromSource);
-        if (sqlFrom.length !== 1) {
-            return undefined;
+        if (sqlFrom.length === 0) {
+            return issue_1.Issue.atToken(lowFile, high.getFirstToken(), "Error outlining, sqlFrom not found", this.getMetadata().key, this.conf.severity);
         }
-        const tableName = (_b = sqlFrom[0].findDirectExpression(Expressions.DatabaseTable)) === null || _b === void 0 ? void 0 : _b.concatTokens();
+        let tableName = (_b = sqlFrom[0].findDirectExpression(Expressions.DatabaseTable)) === null || _b === void 0 ? void 0 : _b.concatTokens();
         if (tableName === undefined) {
             return undefined;
         }
-        const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
         const fieldList = high.findFirstExpression(Expressions.SQLFieldList);
         if (fieldList === undefined) {
             return undefined;
         }
         let fieldDefinitions = "";
         for (const f of fieldList.findDirectExpressions(Expressions.SQLFieldName)) {
-            const fieldName = f.concatTokens();
+            let fieldName = f.concatTokens();
+            if (fieldName.includes("~")) {
+                const split = fieldName.split("~");
+                fieldName = split[0];
+                tableName = split[1];
+            }
             fieldDefinitions += indentation + "        " + fieldName + " TYPE " + tableName + "-" + fieldName + ",\n";
         }
         const uniqueName = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
@@ -1103,11 +1113,18 @@ ${indentation}    output = ${topTarget}.`;
     }
     buildCondBody(body, uniqueName, indent, lowFile, highSyntax) {
         let code = "";
+        let first = true;
         for (const c of body.getChildren()) {
             if (c instanceof nodes_1.TokenNode) {
                 switch (c.getFirstToken().getStr().toUpperCase()) {
                     case "WHEN":
-                        code += indent + "IF ";
+                        if (first === true) {
+                            code += indent + "IF ";
+                            first = false;
+                        }
+                        else {
+                            code += indent + "ELSEIF ";
+                        }
                         break;
                     case "THEN":
                         code += ".\n";

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.85.31",
+    "version": "2.85.32",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",