@abaplint/cli 2.113.231 → 2.113.233

package/build/cli.js

@@ -4550,7 +4550,7 @@ const component_chain_simple_1 = __webpack_require__(/*! ./component_chain_simpl
 const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
 class ComponentCompareSimple extends combi_1.Expression {
     getRunnable() {
-        const source = (0, combi_1.alt)(_1.SimpleSource4, (0, combi_1.ver)(version_1.Version.v740sp02, _1.Source, version_1.Version.OpenABAP));
+        const source = (0, combi_1.altPrio)((0, combi_1.ver)(version_1.Version.v740sp02, _1.Source, version_1.Version.OpenABAP), _1.SimpleSource4);
         const ret = (0, combi_1.seq)((0, combi_1.altPrio)(component_chain_simple_1.ComponentChainSimple, _1.Dynamic), "=", source);
         return (0, combi_1.plus)(ret);
     }
@@ -16309,12 +16309,11 @@ class ReadTable {
         const comparing = (0, combi_1.seq)("COMPARING", (0, combi_1.alt)((0, combi_1.plus)(expressions_1.FieldSub), (0, combi_1.plus)(expressions_1.Dynamic)));
         const index = (0, combi_1.seq)("INDEX", expressions_1.Source);
         const components = (0, combi_1.seq)((0, combi_1.alt)(expressions_1.Field, expressions_1.Dynamic), "COMPONENTS", expressions_1.ComponentCompareSimple);
-        const source = (0, combi_1.alt)(expressions_1.SimpleSource2, (0, combi_1.ver)(version_1.Version.v740sp02, expressions_1.Source, version_1.Version.OpenABAP));
-        const key = (0, combi_1.seq)((0, combi_1.altPrio)("WITH KEY", "WITH TABLE KEY"), (0, combi_1.alt)(expressions_1.ComponentCompareSimple, components, (0, combi_1.seq)((0, combi_1.optPrio)("="), source)));
+        const key = (0, combi_1.seq)((0, combi_1.altPrio)("WITH KEY", "WITH TABLE KEY"), (0, combi_1.alt)(expressions_1.ComponentCompareSimple, components, (0, combi_1.seq)((0, combi_1.optPrio)("="), expressions_1.Source)));
         const using = (0, combi_1.seq)("USING KEY", (0, combi_1.alt)(expressions_1.Field, expressions_1.Dynamic));
-        const from = (0, combi_1.seq)("FROM", source);
+        const from = (0, combi_1.seq)("FROM", expressions_1.Source);
         const perm = (0, combi_1.per)((0, combi_1.alt)(index, key, from), expressions_1.ReadTableTarget, using, comparing, "CASTING", (0, combi_1.seq)("TRANSPORTING", (0, combi_1.altPrio)("ALL FIELDS", "NO FIELDS", transporting_fields_1.TransportingFields)), "BINARY SEARCH");
-        return (0, combi_1.seq)("READ TABLE", source, (0, combi_1.opt)(perm));
+        return (0, combi_1.seq)("READ TABLE", (0, combi_1.alt)(expressions_1.SimpleSource2, (0, combi_1.ver)(version_1.Version.v740sp02, expressions_1.Source, version_1.Version.OpenABAP)), (0, combi_1.opt)(perm));
     }
 }
 exports.ReadTable = ReadTable;
@@ -23980,9 +23979,30 @@ class TypeUtils {
                 return true;
             }
             else if (source instanceof basic_1.StructureType) {
-                if (this.structureContainsString(target) && !this.structureContainsString(source)) {
+                const targetDeep = this.structureContainsString(target);
+                const sourceDeep = this.structureContainsString(source);
+                if (targetDeep && !sourceDeep) {
                     return false;
                 }
+                const targetComponents = target.getComponents();
+                const sourceComponents = source.getComponents();
+                if (targetComponents.length !== sourceComponents.length) {
+                    if (targetDeep === true || sourceDeep === true) {
+                        return false;
+                    }
+                }
+                for (let i = 0; i < targetComponents.length; i++) {
+                    if (sourceComponents[i] === undefined) {
+                        continue;
+                    }
+                    // hmm
+                    if (sourceComponents[i].type instanceof basic_1.StringType && !(targetComponents[i].type instanceof basic_1.StringType)) {
+                        return false;
+                    }
+                    else if (!(sourceComponents[i].type instanceof basic_1.StringType) && targetComponents[i].type instanceof basic_1.StringType) {
+                        return false;
+                    }
+                }
                 return true;
             }
             else if (target.containsVoid() === true) {
@@ -25458,7 +25478,9 @@ class ComponentCompareSimple {
                 else if (c.get() instanceof Expressions.Dynamic) {
                     targetType = undefined;
                 }
-                else if (c.get() instanceof Expressions.Source || c.get() instanceof Expressions.SimpleSource4) {
+                else if (c.get() instanceof Expressions.Source
+                    || c.get() instanceof Expressions.SimpleSource4
+                    || c.get() instanceof Expressions.SimpleSource2) {
                     const sourceType = source_1.Source.runSyntax(c, input, targetType);
                     if (targetType && new _type_utils_1.TypeUtils(input.scope).isAssignable(sourceType, targetType) === false) {
                         const message = "ComponentCompareSimple, incompatible types";
@@ -25976,7 +25998,7 @@ class FieldAssignment {
                 const text = c.concatTokens();
                 if (text !== "-" && context instanceof basic_1.StructureType) {
                     context = context.getComponentByName(text);
-                    if (context === undefined && targetType.containsVoid() === false) {
+                    if (context === undefined) {
                         const message = `field ${text} does not exist in structure`;
                         input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                         return;
@@ -33130,6 +33152,7 @@ const _object_oriented_1 = __webpack_require__(/*! ../_object_oriented */ "./nod
 const _scope_type_1 = __webpack_require__(/*! ../_scope_type */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_scope_type.js");
 const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
+const visibility_1 = __webpack_require__(/*! ../../4_file_information/visibility */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/visibility.js");
 class MethodImplementation {
     runSyntax(node, input) {
         const helper = new _object_oriented_1.ObjectOriented(input.scope);
@@ -33143,6 +33166,13 @@ class MethodImplementation {
             return;
         }
         const { method: methodDefinition } = helper.searchMethodName(classDefinition, methodName);
+        if (classDefinition.isForTesting()
+            && (methodDefinition === null || methodDefinition === void 0 ? void 0 : methodDefinition.getVisibility()) !== visibility_1.Visibility.Private
+            && ["SETUP", "TEARDOWN", "CLASS_SETUP", "CLASS_TEARDOWN"].includes(methodName.toUpperCase())) {
+            const message = "Special test method \"" + methodName + "\" must be private";
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+            return;
+        }
         const start = node.getFirstToken().getStart();
         if ((methodDefinition === null || methodDefinition === void 0 ? void 0 : methodDefinition.isStatic()) === false) {
             input.scope.push(_scope_type_1.ScopeType.MethodInstance, methodName, start, input.filename);
@@ -33983,6 +34013,11 @@ class ReadTable {
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return;
             }
+            else if (sourceType instanceof basic_1.TableType && sourceType.getAccessType() === basic_1.TableAccessType.hashed) {
+                const message = "INDEX on hashed table not possible";
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+                return;
+            }
         }
         const fromSource = node.findExpressionAfterToken("FROM");
         if (fromSource) {
@@ -54736,7 +54771,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.231";
+        return "2.113.233";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.231",
+    "version": "2.113.233",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,16 +38,16 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.113.231",
+        "@abaplint/core": "^2.113.233",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.7.2",
+        "@types/node": "^24.9.1",
         "@types/progress": "^2.0.7",
         "chai": "^4.5.0",
         "p-limit": "^3.1.0",
         "chalk": "^5.6.2",
-        "eslint": "^9.37.0",
+        "eslint": "^9.38.0",
         "glob": "^11.0.3",
         "json5": "^2.2.3",
         "memfs": "^4.49.0",