@abaplint/cli 2.120.24 → 2.120.26

package/build/cli.js

@@ -27579,6 +27579,12 @@ class TypeUtils {
         if (source2 instanceof basic_1.HexType && this.isCalculated(node2) && source1 instanceof basic_1.IntegerType) {
             return false;
         }
+        if (source1 instanceof basic_1.DateType && this.isCalculated(node2) && source2 instanceof basic_1.IntegerType) {
+            return false;
+        }
+        if (source2 instanceof basic_1.DateType && this.isCalculated(node1) && source1 instanceof basic_1.IntegerType) {
+            return false;
+        }
         return true;
     }
     structureContainsString(structure) {
@@ -29097,6 +29103,51 @@ exports.BasicTypes = BasicTypes;
 
 /***/ },
 
+/***/ "../core/build/src/abap/5_syntax/expressions/_check_database_fields.js"
+/*!*****************************************************************************!*\
+  !*** ../core/build/src/abap/5_syntax/expressions/_check_database_fields.js ***!
+  \*****************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.checkDatabaseFields = checkDatabaseFields;
+const basic_1 = __webpack_require__(/*! ../../types/basic */ "../core/build/src/abap/types/basic/index.js");
+const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
+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
+
+/***/ },
+
 /***/ "../core/build/src/abap/5_syntax/expressions/_check_offset_length.js"
 /*!***************************************************************************!*\
   !*** ../core/build/src/abap/5_syntax/expressions/_check_offset_length.js ***!
@@ -29576,6 +29627,7 @@ const source_1 = __webpack_require__(/*! ./source */ "../core/build/src/abap/5_s
 const method_call_chain_1 = __webpack_require__(/*! ./method_call_chain */ "../core/build/src/abap/5_syntax/expressions/method_call_chain.js");
 const source_field_symbol_1 = __webpack_require__(/*! ./source_field_symbol */ "../core/build/src/abap/5_syntax/expressions/source_field_symbol.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
+const basic_1 = __webpack_require__(/*! ../../types/basic */ "../core/build/src/abap/types/basic/index.js");
 const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "../core/build/src/abap/5_syntax/_type_utils.js");
 class Compare {
     static runSyntax(node, input) {
@@ -29593,7 +29645,15 @@ class Compare {
         if (node.findDirectExpression(Expressions.CompareOperator)
             && new _type_utils_1.TypeUtils(input.scope).isCompareable(sourceTypes[0], sourceTypes[1], sources[0], sources[1]) === false
             && sourceTypes.length === 2) {
-            const message = "Incompatible types for comparison";
+            let message = "Incompatible types for comparison";
+            if ((sourceTypes[0] instanceof basic_1.DateType
+                && sourceTypes[1] instanceof basic_1.IntegerType
+                && sources[1].findFirstExpression(Expressions.ArithOperator))
+                || (sourceTypes[1] instanceof basic_1.DateType
+                    && sourceTypes[0] instanceof basic_1.IntegerType
+                    && sources[0].findFirstExpression(Expressions.ArithOperator))) {
+                message = "Date cannot be compared with arithmetic result of type Integer";
+            }
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
         }
     }
@@ -33737,6 +33797,7 @@ const dynamic_1 = __webpack_require__(/*! ./dynamic */ "../core/build/src/abap/5
 const _reference_1 = __webpack_require__(/*! ../_reference */ "../core/build/src/abap/5_syntax/_reference.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
 const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
+const _check_database_fields_1 = __webpack_require__(/*! ./_check_database_fields */ "../core/build/src/abap/5_syntax/expressions/_check_database_fields.js");
 const isSimple = /^\w+$/;
 class Select {
     static runSyntax(node, input, skipImplicitInto = false, selectLoop = false) {
@@ -33761,7 +33822,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) {
@@ -33988,35 +34049,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()
@@ -37625,7 +37657,12 @@ const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/bui
 const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "../core/build/src/abap/5_syntax/_type_utils.js");
 class Catch {
     runSyntax(node, input) {
-        var _a;
+        var _a, _b, _c;
+        const target = node.findDirectExpression(Expressions.Target);
+        const inlineTarget = (_b = (_a = target === null || target === void 0 ? void 0 : target.findDirectExpression(Expressions.InlineData)) === null || _a === void 0 ? void 0 : _a.findFirstExpression(Expressions.TargetField)) === null || _b === void 0 ? void 0 : _b.getFirstToken();
+        if (inlineTarget && inlineTarget.getStr().length > 30) {
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, inlineTarget, "Inline exception name longer than 30 characters"));
+        }
         const names = new Set();
         for (const c of node.findDirectExpressions(Expressions.ClassName)) {
             const token = c.getFirstToken();
@@ -37649,9 +37686,8 @@ class Catch {
             }
             names.add(className);
         }
-        const target = node.findDirectExpression(Expressions.Target);
         if (target === null || target === void 0 ? void 0 : target.findDirectExpression(Expressions.InlineData)) {
-            const token = (_a = target.findFirstExpression(Expressions.TargetField)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
+            const token = (_c = target.findFirstExpression(Expressions.TargetField)) === null || _c === void 0 ? void 0 : _c.getFirstToken();
             if (token) {
                 const classNames = Array.from(names);
                 const unknownClass = classNames.find(name => input.scope.findClassDefinition(name) === undefined);
@@ -39365,6 +39401,7 @@ const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expr
 const source_1 = __webpack_require__(/*! ../expressions/source */ "../core/build/src/abap/5_syntax/expressions/source.js");
 const dynamic_1 = __webpack_require__(/*! ../expressions/dynamic */ "../core/build/src/abap/5_syntax/expressions/dynamic.js");
 const database_table_1 = __webpack_require__(/*! ../expressions/database_table */ "../core/build/src/abap/5_syntax/expressions/database_table.js");
+const _check_database_fields_1 = __webpack_require__(/*! ../expressions/_check_database_fields */ "../core/build/src/abap/5_syntax/expressions/_check_database_fields.js");
 class DeleteDatabase {
     runSyntax(node, input) {
         for (const s of node.findAllExpressions(Expressions.Source)) {
@@ -39378,7 +39415,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());
         }
     }
 }
@@ -46323,11 +46367,19 @@ const _typed_identifier_1 = __webpack_require__(/*! ../../types/_typed_identifie
 const identifier_1 = __webpack_require__(/*! ../../1_lexer/tokens/identifier */ "../core/build/src/abap/1_lexer/tokens/identifier.js");
 const database_table_1 = __webpack_require__(/*! ../expressions/database_table */ "../core/build/src/abap/5_syntax/expressions/database_table.js");
 const dynamic_1 = __webpack_require__(/*! ../expressions/dynamic */ "../core/build/src/abap/5_syntax/expressions/dynamic.js");
+const _check_database_fields_1 = __webpack_require__(/*! ../expressions/_check_database_fields */ "../core/build/src/abap/5_syntax/expressions/_check_database_fields.js");
 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();
@@ -47601,6 +47653,7 @@ const include_type_1 = __webpack_require__(/*! ../statements/include_type */ "..
 const type_1 = __webpack_require__(/*! ../statements/type */ "../core/build/src/abap/5_syntax/statements/type.js");
 const Basic = __importStar(__webpack_require__(/*! ../../types/basic */ "../core/build/src/abap/types/basic/index.js"));
 const _scope_type_1 = __webpack_require__(/*! ../_scope_type */ "../core/build/src/abap/5_syntax/_scope_type.js");
+const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
 class Types {
     runSyntax(node, input, qualifiedNamePrefix) {
         const name = node.findFirstExpression(Expressions.NamespaceSimpleName).getFirstToken();
@@ -47615,6 +47668,9 @@ class Types {
                 if (ctyp instanceof Statements.Type) {
                     const found = new type_1.Type().runSyntax(c, input, qualifiedNamePrefix + name.getStr() + "-");
                     if (found) {
+                        if (found.getName().length > 30) {
+                            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, found.getToken(), "Structure field name longer than 30 characters"));
+                        }
                         components.push({ name: found.getName(), type: found.getType() });
                     }
                 }
@@ -47631,6 +47687,9 @@ class Types {
             else if (c instanceof nodes_1.StructureNode && ctyp instanceof Structures.Types) {
                 const found = new Types().runSyntax(c, input, qualifiedNamePrefix + name.getStr() + "-");
                 if (found) {
+                    if (found.getName().length > 30) {
+                        input.issues.push((0, _syntax_input_1.syntaxIssue)(input, found.getToken(), "Structure field name longer than 30 characters"));
+                    }
                     components.push({ name: found.getName(), type: found.getType() });
                 }
             }
@@ -51731,7 +51790,7 @@ class ClassDefinition extends _identifier_1.Identifier {
         this.checkEventNameConflicts(input);
         this.checkClassNameLength(input);
         this.checkMethodNameLength(input);
-        this.checkClassConstructorStatic(input);
+        this.checkClassConstructor(input);
     }
     getFriends() {
         return this.friends;
@@ -51801,11 +51860,17 @@ class ClassDefinition extends _identifier_1.Identifier {
             }
         }
     }
-    checkClassConstructorStatic(input) {
+    checkClassConstructor(input) {
         for (const m of this.methodDefs.getAll()) {
-            if (m.getName().toUpperCase() === "CLASS_CONSTRUCTOR" && m.isStatic() === false) {
+            if (m.getName().toUpperCase() !== "CLASS_CONSTRUCTOR") {
+                continue;
+            }
+            if (m.isStatic() === false) {
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, m.getToken(), "CLASS_CONSTRUCTOR must be static"));
             }
+            else if (m.getVisibility() !== visibility_1.Visibility.Public) {
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, m.getToken(), "CLASS_CONSTRUCTOR must be declared in the public section"));
+            }
         }
     }
     checkInterfaceVisibility(input, node) {
@@ -52970,6 +53035,7 @@ const _object_oriented_1 = __webpack_require__(/*! ../5_syntax/_object_oriented
 const _reference_1 = __webpack_require__(/*! ../5_syntax/_reference */ "../core/build/src/abap/5_syntax/_reference.js");
 const identifier_1 = __webpack_require__(/*! ../1_lexer/tokens/identifier */ "../core/build/src/abap/1_lexer/tokens/identifier.js");
 const _scope_type_1 = __webpack_require__(/*! ../5_syntax/_scope_type */ "../core/build/src/abap/5_syntax/_scope_type.js");
+const _syntax_input_1 = __webpack_require__(/*! ../5_syntax/_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
 // todo:
 // this.exceptions = [];
 // also consider RAISING vs EXCEPTIONS
@@ -52992,6 +53058,7 @@ class MethodParameters {
         input.scope.push(_scope_type_1.ScopeType.MethodDefinition, "method definition", node.getStart(), input.filename);
         try {
             this.parse(node, input, parentName, abstractMethod);
+            this.checkNameLengths(input);
             this.checkDuplicateNames();
         }
         finally {
@@ -53143,6 +53210,13 @@ class MethodParameters {
             names.add(name);
         }
     }
+    checkNameLengths(input) {
+        for (const parameter of this.getAll()) {
+            if (parameter.getName().length > 30) {
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, parameter.getToken(), "Method parameter name longer than 30 characters"));
+            }
+        }
+    }
     workaroundRAP(node, input, parentName) {
         const resultName = node.findExpressionAfterToken("RESULT");
         const concat = node.concatTokens().toUpperCase();
@@ -69677,7 +69751,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.24";
+        return "2.120.26";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.120.24",
+    "version": "2.120.26",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -39,7 +39,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.120.24",
+        "@abaplint/core": "^2.120.26",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",