@abaplint/core 2.118.5 → 2.118.7

package/build/src/abap/2_statements/statements/submit.js

@@ -17,6 +17,8 @@ class Submit {
         const exporting = (0, combi_1.str)("EXPORTING LIST TO MEMORY");
         const spool = (0, combi_1.seq)("SPOOL PARAMETERS", expressions_1.Source);
         const archive = (0, combi_1.seq)("ARCHIVE PARAMETERS", expressions_1.Source);
+        const archiveMode = (0, combi_1.seq)("ARCHIVE MODE", expressions_1.Source);
+        const receiver = (0, combi_1.seq)("RECEIVER", expressions_1.Source);
         const lineSize = (0, combi_1.seq)("LINE-SIZE", expressions_1.Source);
         const lineCount = (0, combi_1.seq)("LINE-COUNT", expressions_1.Source);
         const user = (0, combi_1.seq)("USER", expressions_1.Source);
@@ -35,7 +37,7 @@ class Submit {
         const imm = (0, combi_1.seq)("IMMEDIATELY", expressions_1.Source);
         const dest = (0, combi_1.seq)("DESTINATION", expressions_1.Source);
         const language = (0, combi_1.seq)("LANGUAGE", expressions_1.Source);
-        const perm = (0, combi_1.per)((0, combi_1.plus)(awith), selectionTable, (0, combi_1.plus)(awith), spool, lineSize, lineCount, archive, user, sset, ssetp, keep, cover, imm, layout, dest, coverText, listName, language, free, newList, uss, copies, datasetExpiration, "TO SAP-SPOOL", "WITHOUT SPOOL DYNPRO", "VIA SELECTION-SCREEN", exporting, expressions_1.AndReturn, job);
+        const perm = (0, combi_1.per)((0, combi_1.plus)(awith), selectionTable, (0, combi_1.plus)(awith), (0, combi_1.plus)(awith), spool, lineSize, lineCount, archive, archiveMode, receiver, user, sset, ssetp, keep, cover, imm, layout, dest, coverText, listName, language, free, newList, uss, copies, datasetExpiration, "TO SAP-SPOOL", "WITHOUT SPOOL DYNPRO", "VIA SELECTION-SCREEN", exporting, expressions_1.AndReturn, job);
         const ret = (0, combi_1.seq)("SUBMIT", prog, (0, combi_1.opt)(perm));
         return (0, combi_1.verNot)(version_1.Version.Cloud, ret);
     }

package/build/src/abap/5_syntax/_type_utils.js

@@ -439,6 +439,10 @@ class TypeUtils {
                     && targetKeyType !== table_type_1.TableKeyType.user) {
                     return false;
                 }
+                if (sourceKeyType === table_type_1.TableKeyType.empty && targetKeyType !== table_type_1.TableKeyType.empty
+                    && !(target.getRowType() instanceof basic_1.AnyType)) {
+                    return false;
+                }
                 return this.isAssignableStrict(source.getRowType(), target.getRowType());
             }
             else if (target instanceof basic_1.UnknownType
@@ -496,13 +500,6 @@ class TypeUtils {
                         return false;
                     }
                 }
-                const sourceKeyType = source.getOptions().keyType;
-                const targetKeyType = target.getOptions().keyType;
-                if (sourceKeyType !== targetKeyType
-                    && sourceKeyType !== table_type_1.TableKeyType.user
-                    && targetKeyType !== table_type_1.TableKeyType.user) {
-                    return false;
-                }
                 return true;
             }
             return false;

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

@@ -111,7 +111,7 @@ class MethodParameters {
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return;
             }
-            else if (item.targetType && new _type_utils_1.TypeUtils(input.scope).isAssignable(parameterType, item.targetType) === false) {
+            else if (item.targetType && new _type_utils_1.TypeUtils(input.scope).isAssignableStrict(parameterType, item.targetType) === false) {
                 const message = "Method parameter type not compatible, " + item.name;
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return;
@@ -139,7 +139,7 @@ class MethodParameters {
                 }
                 parameterType = parameter.getType();
             }
-            if (item.targetType && new _type_utils_1.TypeUtils(input.scope).isAssignable(parameterType, item.targetType) === false) {
+            if (item.targetType && new _type_utils_1.TypeUtils(input.scope).isAssignableStrict(parameterType, item.targetType) === false) {
                 const message = "Method parameter type not compatible, " + item.name;
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return;

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

@@ -322,16 +322,17 @@ class Select {
         if (dbSources.length !== 1) {
             return basic_1.VoidType.get("SELECT_todo3");
         }
+        const tableOptions = { withHeader: false, keyType: basic_1.TableKeyType.empty };
         if (dbSources[0] === undefined) {
             // then its a voided table
-            return basic_1.VoidType.get("SELECT_todo4");
+            return new basic_1.TableType(basic_1.VoidType.get("SELECT_todo4"), tableOptions, undefined);
         }
         const dbType = dbSources[0].parseType(scope.getRegistry());
         if (!(dbType instanceof basic_1.StructureType)) {
             return basic_1.VoidType.get("SELECT_todo5");
         }
         if (fields.length === 1 && fields[0].code === "*") {
-            return new basic_1.TableType(dbType, { withHeader: false, keyType: basic_1.TableKeyType.default }, undefined);
+            return new basic_1.TableType(dbType, tableOptions, undefined);
         }
         const allFieldsSimple = fields.every(f => isSimple.test(f.code));
         if (allFieldsSimple === true) {
@@ -343,7 +344,7 @@ class Select {
                 }
                 components.push({ name: field.code, type });
             }
-            return new basic_1.TableType(new basic_1.StructureType(components), { withHeader: false, keyType: basic_1.TableKeyType.default }, undefined);
+            return new basic_1.TableType(new basic_1.StructureType(components), tableOptions, undefined);
         }
         return basic_1.VoidType.get("SELECT_todo7");
     }

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

@@ -11,7 +11,7 @@ const _syntax_input_1 = require("../_syntax_input");
 class Split {
     runSyntax(node, input) {
         const intoTable = node.findTokenSequencePosition("INTO", "TABLE") !== undefined;
-        const type = intoTable ? new basic_1.TableType(basic_1.StringType.get(), { withHeader: false, keyType: basic_1.TableKeyType.user }) : basic_1.StringType.get();
+        const type = intoTable ? new basic_1.TableType(basic_1.StringType.get(), { withHeader: false, keyType: basic_1.TableKeyType.default }) : basic_1.StringType.get();
         for (const target of node.findAllExpressions(Expressions.Target)) {
             const inline = target.findDirectExpression(Expressions.InlineData);
             if (inline) {

package/build/src/registry.js

@@ -74,7 +74,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.118.5";
+        return "2.118.7";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/build/src/rules/ambiguous_statement.js

@@ -56,11 +56,17 @@ MODIFY TABLE foo FROM bar.`,
                 match = this.tryMatch(statement, this.reg, Statements.DeleteDatabase);
             }
             else if (statement.get() instanceof Statements.ModifyInternal) {
-                match = this.tryMatch(statement, this.reg, Statements.ModifyDatabase);
+                match = this.tryMatch(statement, this.reg, Statements.ModifyDatabase) || this.isBareStatement(statement);
             }
             else if (statement.get() instanceof Statements.ModifyDatabase) {
                 match = this.tryMatch(statement, this.reg, Statements.ModifyInternal);
             }
+            else if (statement.get() instanceof Statements.InsertDatabase) {
+                match = this.isBareStatement(statement);
+            }
+            else if (statement.get() instanceof Statements.UpdateDatabase) {
+                match = this.isBareStatement(statement);
+            }
             if (match) {
                 const issue = issue_1.Issue.atStatement(file, statement, this.getMessage(), this.getMetadata().key, this.conf.severity);
                 issues.push(issue);
@@ -68,6 +74,11 @@ MODIFY TABLE foo FROM bar.`,
         }
         return issues;
     }
+    isBareStatement(st) {
+        const tokens = st.getTokens().slice(0);
+        tokens.pop();
+        return tokens.length === 2;
+    }
     tryMatch(st, reg, type1) {
         const ver = reg.getConfig().getVersion();
         const tokens = st.getTokens().slice(0);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.118.5",
+    "version": "2.118.7",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -63,7 +63,7 @@
         "typescript": "^5.9.3"
     },
     "dependencies": {
-        "fast-xml-parser": "^5.5.8",
+        "fast-xml-parser": "^5.5.9",
         "json5": "^2.2.3",
         "vscode-languageserver-types": "^3.17.5"
     }