@abaplint/core 2.120.25 → 2.120.26

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

@@ -0,0 +1,35 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.checkDatabaseFields = checkDatabaseFields;
+const basic_1 = require("../../types/basic");
+const _syntax_input_1 = require("../_syntax_input");
+function checkDatabaseFields(fields, dbSources, input, token) {
+    if (dbSources.length > 1) {
+        return;
+    }
+    const first = dbSources[0];
+    if (first === undefined) {
+        // then its voided
+        return;
+    }
+    const type = first.parseType(input.scope.getRegistry());
+    if (type instanceof basic_1.VoidType || type instanceof basic_1.UnknownType) {
+        return;
+    }
+    if (!(type instanceof basic_1.StructureType)) {
+        const message = "checkFields, expected structure, " + type.constructor.name;
+        input.issues.push((0, _syntax_input_1.syntaxIssue)(input, token, message));
+        return;
+    }
+    for (const field of fields) {
+        if (field === "*") {
+            continue;
+        }
+        if (/^\w+$/.test(field) && type.getComponentByName(field) === undefined) {
+            const message = `checkFields, field ${field} not found`;
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, token, message));
+            return;
+        }
+    }
+}
+//# sourceMappingURL=_check_database_fields.js.map

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

@@ -48,6 +48,7 @@ const dynamic_1 = require("./dynamic");
 const _reference_1 = require("../_reference");
 const _syntax_input_1 = require("../_syntax_input");
 const version_1 = require("../../../version");
+const _check_database_fields_1 = require("./_check_database_fields");
 const isSimple = /^\w+$/;
 class Select {
     static runSyntax(node, input, skipImplicitInto = false, selectLoop = false) {
@@ -72,7 +73,7 @@ class Select {
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
             return;
         }
-        this.checkFields(fields, dbSources, input, node);
+        (0, _check_database_fields_1.checkDatabaseFields)(fields.map(field => field.code), dbSources, input, node.getFirstToken());
         const intoExpression = this.handleInto(node, input, fields, dbSources);
         const fae = node.findDirectExpression(Expressions.SQLForAllEntries);
         if (fae) {
@@ -299,35 +300,6 @@ class Select {
         }
         return undefined;
     }
-    static checkFields(fields, dbSources, input, node) {
-        if (dbSources.length > 1) {
-            return;
-        }
-        const first = dbSources[0];
-        if (first === undefined) {
-            // then its voided
-            return;
-        }
-        const type = first.parseType(input.scope.getRegistry());
-        if (type instanceof basic_1.VoidType || type instanceof basic_1.UnknownType) {
-            return;
-        }
-        if (!(type instanceof basic_1.StructureType)) {
-            const message = "checkFields, expected structure, " + type.constructor.name;
-            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
-            return;
-        }
-        for (const field of fields) {
-            if (field.code === "*") {
-                continue;
-            }
-            if (isSimple.test(field.code) && type.getComponentByName(field.code) === undefined) {
-                const message = `checkFields, field ${field.code} not found`;
-                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
-                return;
-            }
-        }
-    }
     static countResultType(scope) {
         if ((0, version_1.releaseAtLeast)(scope.getRelease(), version_1.Release.v750)
             || scope.getOpenABAP()

package/build/src/abap/5_syntax/statements/delete_database.js

@@ -38,6 +38,7 @@ const Expressions = __importStar(require("../../2_statements/expressions"));
 const source_1 = require("../expressions/source");
 const dynamic_1 = require("../expressions/dynamic");
 const database_table_1 = require("../expressions/database_table");
+const _check_database_fields_1 = require("../expressions/_check_database_fields");
 class DeleteDatabase {
     runSyntax(node, input) {
         for (const s of node.findAllExpressions(Expressions.Source)) {
@@ -51,7 +52,14 @@ class DeleteDatabase {
         }
         const dbtab = node.findFirstExpression(Expressions.DatabaseTable);
         if (dbtab !== undefined) {
-            database_table_1.DatabaseTable.runSyntax(dbtab, input);
+            const dbSource = database_table_1.DatabaseTable.runSyntax(dbtab, input);
+            const fields = node.findAllExpressions(Expressions.SQLCompare)
+                .map(compare => { var _a; return (_a = compare.findDirectExpression(Expressions.SQLFieldName)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase(); })
+                .filter(field => field !== undefined);
+            for (const orderBy of node.findAllExpressions(Expressions.SQLOrderBy)) {
+                fields.push(...orderBy.findAllExpressions(Expressions.SQLFieldName).map(field => field.concatTokens().toUpperCase()));
+            }
+            (0, _check_database_fields_1.checkDatabaseFields)(fields, [dbSource], input, node.getFirstToken());
         }
     }
 }

package/build/src/abap/5_syntax/statements/update_database.js

@@ -42,11 +42,19 @@ const _typed_identifier_1 = require("../../types/_typed_identifier");
 const identifier_1 = require("../../1_lexer/tokens/identifier");
 const database_table_1 = require("../expressions/database_table");
 const dynamic_1 = require("../expressions/dynamic");
+const _check_database_fields_1 = require("../expressions/_check_database_fields");
 class UpdateDatabase {
     runSyntax(node, input) {
         const dbtab = node.findFirstExpression(Expressions.DatabaseTable);
         if (dbtab !== undefined) {
-            database_table_1.DatabaseTable.runSyntax(dbtab, input);
+            const dbSource = database_table_1.DatabaseTable.runSyntax(dbtab, input);
+            const fields = node.findAllExpressions(Expressions.SQLFieldAndValue)
+                .flatMap(fieldAndValue => fieldAndValue.findDirectExpressions(Expressions.SQLFieldName))
+                .map(field => field.concatTokens().toUpperCase());
+            fields.push(...node.findAllExpressions(Expressions.SQLCompare)
+                .map(compare => { var _a; return (_a = compare.findDirectExpression(Expressions.SQLFieldName)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase(); })
+                .filter(field => field !== undefined));
+            (0, _check_database_fields_1.checkDatabaseFields)(fields, [dbSource], input, node.getFirstToken());
         }
         const tableName = node.findDirectExpression(Expressions.DatabaseTable);
         const tokenName = tableName === null || tableName === void 0 ? void 0 : tableName.getFirstToken();

package/build/src/registry.js

@@ -75,7 +75,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.25";
+        return "2.120.26";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/package.json

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