@abaplint/transpiler-cli 2.5.72 → 2.5.74

package/build/bundle.js

@@ -18583,7 +18583,7 @@ BuiltIn.methods = [
     {
         name: "CONCAT_LINES_OF",
         mandatory: {
-            "table": new basic_1.TableType(new basic_1.AnyType(), { withHeader: false }),
+            "table": new basic_1.TableType(new basic_1.AnyType(), { withHeader: false, keyType: basic_1.TableKeyType.default }),
         },
         optional: {
             "sep": new basic_1.StringType(),
@@ -18879,7 +18879,7 @@ BuiltIn.methods = [
     {
         name: "LINES",
         mandatory: {
-            "val": new basic_1.TableType(new basic_1.AnyType(), { withHeader: false }),
+            "val": new basic_1.TableType(new basic_1.AnyType(), { withHeader: false, keyType: basic_1.TableKeyType.default }),
         },
         return: new basic_1.IntegerType(),
     },
@@ -20099,10 +20099,10 @@ class Procedural {
             }
             if (param.direction === types_1.FunctionModuleParameterDirection.tables) {
                 if (found instanceof basic_1.TableType) {
-                    found = new basic_1.TableType(found.getRowType(), { withHeader: true });
+                    found = new basic_1.TableType(found.getRowType(), { withHeader: true, keyType: basic_1.TableKeyType.default });
                 }
                 else {
-                    found = new basic_1.TableType(found, { withHeader: true });
+                    found = new basic_1.TableType(found, { withHeader: true, keyType: basic_1.TableKeyType.default });
                 }
             }
             if ((found instanceof basic_1.UnknownType || found instanceof basic_1.VoidType) && ((_a = param.type) === null || _a === void 0 ? void 0 : _a.includes("-"))) {
@@ -20659,7 +20659,7 @@ class BasicTypes {
                 }
                 else if (child.concatTokens() === "[]") {
                     if (type instanceof Types.TableType) {
-                        type = new basic_1.TableType(type.getRowType(), { withHeader: false });
+                        type = new basic_1.TableType(type.getRowType(), { withHeader: false, keyType: Types.TableKeyType.default });
                     }
                 }
                 else { // field name
@@ -20785,7 +20785,7 @@ class BasicTypes {
         return undefined;
     }
     parseTable(node, name) {
-        var _a, _b, _c;
+        var _a, _b, _c, _d;
         const typename = node.findFirstExpression(Expressions.TypeName);
         const text = (_a = node.findFirstExpression(Expressions.TypeTable)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase();
         if (text === undefined) {
@@ -20808,10 +20808,11 @@ class BasicTypes {
         }
         const typeTableKeys = node.findAllExpressions(expressions_1.TypeTableKey);
         const firstKey = typeTableKeys[0];
-        const isNamed = (firstKey === null || firstKey === void 0 ? void 0 : firstKey.findDirectExpression(expressions_1.Field)) !== undefined;
+        const isNamed = (firstKey === null || firstKey === void 0 ? void 0 : firstKey.findDirectExpression(expressions_1.Field)) !== undefined
+            && ((_b = firstKey === null || firstKey === void 0 ? void 0 : firstKey.findDirectExpression(expressions_1.Field)) === null || _b === void 0 ? void 0 : _b.concatTokens().toUpperCase()) !== "PRIMARY_KEY";
         const primaryKey = {
             name: "primary_key",
-            type: type,
+            type: type || basic_1.TableAccessType.standard,
             isUnique: isNamed ? false : (firstKey === null || firstKey === void 0 ? void 0 : firstKey.concatTokens().toUpperCase().includes("WITH UNIQUE ")) === true,
             keyFields: [],
         };
@@ -20820,9 +20821,6 @@ class BasicTypes {
             for (const k of (firstKey === null || firstKey === void 0 ? void 0 : firstKey.findDirectExpressions(expressions_1.FieldSub)) || []) {
                 primaryKey.keyFields.push(k.concatTokens().toUpperCase());
             }
-            if (primaryKey.keyFields.length === 0 && text.includes(" DEFAULT KEY")) {
-                primaryKey.keyFields.push("TABLE_LINE");
-            }
         }
         else {
             start = 0;
@@ -20830,7 +20828,7 @@ class BasicTypes {
         const secondaryKeys = [];
         for (let i = start; i < typeTableKeys.length; i++) {
             const row = typeTableKeys[i];
-            const name = (_b = row.findDirectExpression(expressions_1.Field)) === null || _b === void 0 ? void 0 : _b.concatTokens();
+            const name = (_c = row.findDirectExpression(expressions_1.Field)) === null || _c === void 0 ? void 0 : _c.concatTokens();
             if (name === undefined) {
                 continue;
             }
@@ -20845,8 +20843,16 @@ class BasicTypes {
             }
             secondaryKeys.push(secondary);
         }
+        let keyType = Types.TableKeyType.user;
+        if (text.includes(" EMPTY KEY")) {
+            keyType = Types.TableKeyType.empty;
+        }
+        else if (text.includes(" DEFAULT KEY")) {
+            keyType = Types.TableKeyType.default;
+        }
         const options = {
             withHeader: text.includes(" WITH HEADER LINE"),
+            keyType: keyType,
             primaryKey: primaryKey,
             secondary: secondaryKeys,
         };
@@ -20918,7 +20924,7 @@ class BasicTypes {
         else if (typename && (text.startsWith("TYPE TABLE FOR CREATE ")
             || text.startsWith("TYPE TABLE FOR UPDATE "))) {
             const name = typename.concatTokens();
-            const type = (_c = this.scope.getDDIC().lookupDDLS(name)) === null || _c === void 0 ? void 0 : _c.type;
+            const type = (_d = this.scope.getDDIC().lookupDDLS(name)) === null || _d === void 0 ? void 0 : _d.type;
             if (type) {
                 return new Types.TableType(new basic_1.VoidType("RapTodo"), options);
             }
@@ -20993,7 +20999,7 @@ class BasicTypes {
             || text === "TYPE HASHED TABLE"
             || text === "TYPE INDEX TABLE"
             || text === "TYPE ANY TABLE") {
-            return new Types.TableType(new Types.AnyType(), { withHeader: node.concatTokens().toUpperCase().includes("WITH HEADER LINE") });
+            return new Types.TableType(new Types.AnyType(), { withHeader: node.concatTokens().toUpperCase().includes("WITH HEADER LINE"), keyType: Types.TableKeyType.default });
         }
         else if (text.startsWith("LIKE ")) {
             let sub = node.findFirstExpression(Expressions.Type);
@@ -21011,7 +21017,7 @@ class BasicTypes {
             }
             found = this.resolveLikeName(sub);
             if (found && this.isOccurs(node)) {
-                found = new Types.TableType(found, { withHeader: text.includes("WITH HEADER LINE") }, qualifiedName);
+                found = new Types.TableType(found, { withHeader: text.includes("WITH HEADER LINE"), keyType: Types.TableKeyType.default }, qualifiedName);
             }
         }
         else if (text.startsWith("TYPE LINE OF ")) {
@@ -21040,17 +21046,17 @@ class BasicTypes {
             found = this.resolveTypeName(typeName, this.findLength(node), this.findDecimals(node), qualifiedName);
             const concat = node.concatTokens().toUpperCase();
             if (found && this.isOccurs(node)) {
-                found = new Types.TableType(found, { withHeader: concat.includes(" WITH HEADER LINE") }, qualifiedName);
+                found = new Types.TableType(found, { withHeader: concat.includes(" WITH HEADER LINE"), keyType: Types.TableKeyType.default }, qualifiedName);
             }
             else if (found && concat.includes(" WITH HEADER LINE")) {
                 if (found instanceof Types.VoidType) {
-                    found = new Types.TableType(found, { withHeader: true });
+                    found = new Types.TableType(found, { withHeader: true, keyType: Types.TableKeyType.default });
                 }
                 else if (!(found instanceof Types.TableType)) {
                     throw new Error("WITH HEADER LINE can only be used with internal table");
                 }
                 else {
-                    found = new Types.TableType(found.getRowType(), { withHeader: true });
+                    found = new Types.TableType(found.getRowType(), { withHeader: true, keyType: Types.TableKeyType.default });
                 }
             }
             if (found === undefined && typeName === undefined) {
@@ -21064,7 +21070,7 @@ class BasicTypes {
                 }
                 found = new Types.CharacterType(length, { qualifiedName: qualifiedName }); // fallback
                 if (this.isOccurs(node)) {
-                    found = new Types.TableType(found, { withHeader: concat.includes(" WITH HEADER LINE") }, qualifiedName);
+                    found = new Types.TableType(found, { withHeader: concat.includes(" WITH HEADER LINE"), keyType: Types.TableKeyType.default }, qualifiedName);
                 }
             }
         }
@@ -26237,7 +26243,7 @@ class Controls {
                 { name: "LINE_SELECTOR", type: new basic_1.CharacterType(1) },
                 { name: "H_GRID", type: new basic_1.CharacterType(1) },
                 { name: "V_GRID", type: new basic_1.CharacterType(1) },
-                { name: "COLS", type: new basic_1.TableType(cols, { withHeader: false }) },
+                { name: "COLS", type: new basic_1.TableType(cols, { withHeader: false, keyType: basic_1.TableKeyType.default }) },
                 { name: "INVISIBLE", type: new basic_1.CharacterType(1) },
             ]);
             const id = new _typed_identifier_1.TypedIdentifier(token, filename, type);
@@ -26884,13 +26890,13 @@ class Find {
                 { name: "LINE", type: new basic_1.IntegerType() },
                 { name: "OFFSET", type: new basic_1.IntegerType() },
                 { name: "LENGTH", type: new basic_1.IntegerType() },
-                { name: "SUBMATCHES", type: new basic_1.TableType(sub, { withHeader: false }) },
+                { name: "SUBMATCHES", type: new basic_1.TableType(sub, { withHeader: false, keyType: basic_1.TableKeyType.default }) },
             ], "MATCH_RESULT", "MATCH_RESULT");
             if (node.concatTokens().toUpperCase().startsWith("FIND FIRST")) {
                 this.inline(rfound, scope, filename, type);
             }
             else {
-                this.inline(rfound, scope, filename, new basic_1.TableType(type, { withHeader: false }, "MATCH_RESULT_TAB"));
+                this.inline(rfound, scope, filename, new basic_1.TableType(type, { withHeader: false, keyType: basic_1.TableKeyType.default }, "MATCH_RESULT_TAB"));
             }
         }
         const ofound = node.findExpressionsAfterToken("OFFSET");
@@ -28420,7 +28426,7 @@ class Ranges {
                 { name: "low", type: found },
                 { name: "high", type: found },
             ]);
-            const type = new basic_1.TableType(structure, { withHeader: true });
+            const type = new basic_1.TableType(structure, { withHeader: true, keyType: basic_1.TableKeyType.default });
             const id = new _typed_identifier_1.TypedIdentifier(nameToken, filename, type);
             scope.addIdentifier(id);
         }
@@ -28880,7 +28886,7 @@ class SelectOption {
                 { name: "LOW", type: found },
                 { name: "HIGH", type: found },
             ]);
-            scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(nameToken, filename, new basic_1.TableType(stru, { withHeader: true })));
+            scope.addIdentifier(new _typed_identifier_1.TypedIdentifier(nameToken, filename, new basic_1.TableType(stru, { withHeader: true, keyType: basic_1.TableKeyType.default })));
             return;
         }
         if (nameToken) {
@@ -29167,7 +29173,7 @@ const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "./node_modules/
 class Split {
     runSyntax(node, scope, filename) {
         const intoTable = node.findTokenSequencePosition("INTO", "TABLE") !== undefined;
-        const type = intoTable ? new basic_1.TableType(new basic_1.StringType(), { withHeader: false }) : new basic_1.StringType();
+        const type = intoTable ? new basic_1.TableType(new basic_1.StringType(), { withHeader: false, keyType: basic_1.TableKeyType.default }) : new basic_1.StringType();
         for (const target of node.findAllExpressions(Expressions.Target)) {
             const inline = target.findDirectExpression(Expressions.InlineData);
             if (inline) {
@@ -30042,7 +30048,7 @@ class Data {
                 }
                 if (found instanceof Basic.VoidType) {
                     if (table === true) {
-                        return new _typed_identifier_1.TypedIdentifier(name, filename, new Basic.TableType(found, { withHeader: true }));
+                        return new _typed_identifier_1.TypedIdentifier(name, filename, new Basic.TableType(found, { withHeader: true, keyType: Basic.TableKeyType.default }));
                     }
                     else {
                         return new _typed_identifier_1.TypedIdentifier(name, filename, found);
@@ -30060,7 +30066,7 @@ class Data {
             }
         }
         if (table === true) {
-            return new _typed_identifier_1.TypedIdentifier(name, filename, new Basic.TableType(new Basic.StructureType(components), { withHeader: true }));
+            return new _typed_identifier_1.TypedIdentifier(name, filename, new Basic.TableType(new Basic.StructureType(components), { withHeader: true, keyType: Basic.TableKeyType.default }));
         }
         else {
             const val = Object.keys(values).length > 0 ? values : undefined;
@@ -30131,7 +30137,7 @@ class Statics {
                 }
                 if (found instanceof Basic.VoidType) {
                     if (table === true) {
-                        return new _typed_identifier_1.TypedIdentifier(name, filename, new Basic.TableType(found, { withHeader: true }));
+                        return new _typed_identifier_1.TypedIdentifier(name, filename, new Basic.TableType(found, { withHeader: true, keyType: Basic.TableKeyType.default }));
                     }
                     else {
                         return new _typed_identifier_1.TypedIdentifier(name, filename, found);
@@ -30149,7 +30155,7 @@ class Statics {
             }
         }
         if (table === true) {
-            return new _typed_identifier_1.TypedIdentifier(name, filename, new Basic.TableType(new Basic.StructureType(components), { withHeader: true }));
+            return new _typed_identifier_1.TypedIdentifier(name, filename, new Basic.TableType(new Basic.StructureType(components), { withHeader: true, keyType: Basic.TableKeyType.default }));
         }
         else {
             return new _typed_identifier_1.TypedIdentifier(name, filename, new Basic.StructureType(components));
@@ -33449,7 +33455,7 @@ exports.StructureType = StructureType;
 "use strict";
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.TableType = exports.TableAccessType = void 0;
+exports.TableType = exports.TableKeyType = exports.TableAccessType = void 0;
 const _abstract_type_1 = __webpack_require__(/*! ./_abstract_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/_abstract_type.js");
 var TableAccessType;
 (function (TableAccessType) {
@@ -33459,6 +33465,12 @@ var TableAccessType;
     TableAccessType["index"] = "INDEX";
     TableAccessType["any"] = "ANY";
 })(TableAccessType = exports.TableAccessType || (exports.TableAccessType = {}));
+var TableKeyType;
+(function (TableKeyType) {
+    TableKeyType["default"] = "DEFAULT";
+    TableKeyType["user"] = "USER";
+    TableKeyType["empty"] = "EMPTY";
+})(TableKeyType = exports.TableKeyType || (exports.TableKeyType = {}));
 class TableType extends _abstract_type_1.AbstractType {
     constructor(rowType, options, qualifiedName) {
         var _a;
@@ -33498,6 +33510,7 @@ class TableType extends _abstract_type_1.AbstractType {
     isGeneric() {
         var _a, _b;
         if (((_a = this.options.primaryKey) === null || _a === void 0 ? void 0 : _a.type) !== TableAccessType.standard
+            && this.options.keyType === TableKeyType.user
             && ((_b = this.options.primaryKey) === null || _b === void 0 ? void 0 : _b.keyFields.length) === 0) {
             return true;
         }
@@ -34304,10 +34317,10 @@ class FormDefinition extends _identifier_1.Identifier {
                 let type = p.getType();
                 const isStructure = param.findDirectTokenByText("STRUCTURE") !== undefined;
                 if (isStructure) {
-                    type = new basic_1.TableType(type, { withHeader: true });
+                    type = new basic_1.TableType(type, { withHeader: true, keyType: basic_1.TableKeyType.default });
                 }
                 if (type instanceof basic_1.TableType) {
-                    type = new basic_1.TableType(type.getRowType(), { withHeader: true });
+                    type = new basic_1.TableType(type.getRowType(), { withHeader: true, keyType: basic_1.TableKeyType.default });
                 }
                 else if (!(type instanceof basic_1.UnknownType) && !(type instanceof basic_1.VoidType)) {
                     type = new basic_1.UnknownType("FORM TABLES type must be table type");
@@ -36542,7 +36555,7 @@ class DDIC {
             case "%_C_POINTER":
                 return new Types.HexType(8, qualifiedName);
             case "TABLE":
-                return new Types.TableType(new Types.AnyType(), { withHeader: false });
+                return new Types.TableType(new Types.AnyType(), { withHeader: false, keyType: Types.TableKeyType.default });
             case "DATA":
                 return new Types.AnyType({ qualifiedName: qualifiedName });
             case "NUMERIC":
@@ -45029,6 +45042,7 @@ class TableType extends _abstract_object_1.AbstractObject {
         var _a, _b, _c;
         const tableOptions = {
             withHeader: false,
+            keyType: Types.TableKeyType.user,
             secondary: [],
         };
         for (const k of ((_a = this.parsedXML) === null || _a === void 0 ? void 0 : _a.dd43v) || []) {
@@ -46237,7 +46251,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.96.2";
+        return "2.97.2";
     }
     getDDICReferences() {
         return this.references;
@@ -49663,7 +49677,7 @@ class ConstantClasses {
             title: "Validate constant classes",
             shortDescription: `Checks that a class contains exactly the constants corresponding to a domain's fixed values.`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-enumeration-classes-to-constants-interfaces`,
-            tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Experimental],
+            tags: [_irule_1.RuleTag.Styleguide],
         };
     }
     initialize(reg) {
@@ -50927,7 +50941,7 @@ class Downport {
         return {
             key: "downport",
             title: "Downport statement",
-            shortDescription: `Experimental downport functionality`,
+            shortDescription: `Downport functionality`,
             extendedInformation: `Much like the 'commented_code' rule this rule loops through unknown statements and tries parsing with
 a higher level language version. If successful, various rules are applied to downport the statement.
 Target downport version is always v702, thus rule is only enabled if target version is v702.
@@ -50960,7 +50974,7 @@ Current rules:
 * MESSAGE with non simple source
 
 Only one transformation is applied to a statement at a time, so multiple steps might be required to do the full downport.`,
-            tags: [_irule_1.RuleTag.Experimental, _irule_1.RuleTag.Downport, _irule_1.RuleTag.Quickfix],
+            tags: [_irule_1.RuleTag.Downport, _irule_1.RuleTag.Quickfix],
         };
     }
     getConfig() {
@@ -56202,6 +56216,7 @@ __exportStar(__webpack_require__(/*! ./space_before_dot */ "./node_modules/@abap
 __exportStar(__webpack_require__(/*! ./sql_escape_host_variables */ "./node_modules/@abaplint/core/build/src/rules/sql_escape_host_variables.js"), exports);
 __exportStar(__webpack_require__(/*! ./start_at_tab */ "./node_modules/@abaplint/core/build/src/rules/start_at_tab.js"), exports);
 __exportStar(__webpack_require__(/*! ./static_call_via_instance */ "./node_modules/@abaplint/core/build/src/rules/static_call_via_instance.js"), exports);
+__exportStar(__webpack_require__(/*! ./strict_sql */ "./node_modules/@abaplint/core/build/src/rules/strict_sql.js"), exports);
 __exportStar(__webpack_require__(/*! ./superclass_final */ "./node_modules/@abaplint/core/build/src/rules/superclass_final.js"), exports);
 __exportStar(__webpack_require__(/*! ./superfluous_value */ "./node_modules/@abaplint/core/build/src/rules/superfluous_value.js"), exports);
 __exportStar(__webpack_require__(/*! ./sy_modification */ "./node_modules/@abaplint/core/build/src/rules/sy_modification.js"), exports);
@@ -63041,7 +63056,7 @@ class SQLEscapeHostVariables extends _abap_rule_1.ABAPRule {
             title: "Escape SQL host variables",
             shortDescription: `Escape SQL host variables, from 740sp05`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-obsolete-language-elements`,
-            tags: [_irule_1.RuleTag.Upport, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix],
+            tags: [_irule_1.RuleTag.Upport, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.Syntax],
             badExample: `SELECT * FROM tab INTO TABLE res WHERE field = val.`,
             goodExample: `SELECT * FROM tab INTO TABLE @res WHERE field = @val.`,
         };
@@ -63278,6 +63293,87 @@ exports.StaticCallViaInstance = StaticCallViaInstance;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/strict_sql.js":
+/*!*******************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/strict_sql.js ***!
+  \*******************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.StrictSQL = exports.StrictSQLConf = void 0;
+const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const Expressions = __webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const version_1 = __webpack_require__(/*! ../version */ "./node_modules/@abaplint/core/build/src/version.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+class StrictSQLConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.StrictSQLConf = StrictSQLConf;
+class StrictSQL extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new StrictSQLConf();
+    }
+    getMetadata() {
+        return {
+            key: "strict_sql",
+            title: "Strict SQL",
+            shortDescription: `Strict SQL`,
+            extendedInformation: `https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abapinto_clause.htm
+
+https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abenopensql_strict_mode_750.htm
+
+Also see separate rule sql_escape_host_variables`,
+            tags: [_irule_1.RuleTag.Upport, _irule_1.RuleTag.Syntax],
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file, obj) {
+        const issues = [];
+        const type = obj.getType();
+        if (type === "INTF" || type === "TYPE") {
+            return [];
+        }
+        if (this.reg.getConfig().getVersion() < version_1.Version.v740sp02
+            && this.reg.getConfig().getVersion() !== version_1.Version.Cloud) {
+            return [];
+        }
+        for (const s of file.getStatements()) {
+            if (s.get() instanceof Statements.Select
+                || s.get() instanceof Statements.SelectLoop) {
+                const expr = s.findDirectExpression(Expressions.Select);
+                const where = expr === null || expr === void 0 ? void 0 : expr.findDirectExpression(Expressions.SQLCond);
+                const into = (expr === null || expr === void 0 ? void 0 : expr.findDirectExpression(Expressions.SQLIntoStructure))
+                    || (expr === null || expr === void 0 ? void 0 : expr.findDirectExpression(Expressions.SQLIntoTable));
+                if (into === undefined || where === undefined) {
+                    continue;
+                }
+                if (where.getFirstToken().getStart().isBefore(into.getFirstToken().getStart())) {
+                    continue;
+                }
+                const message = "INTO/APPENDING must be last in strict SQL";
+                const issue = issue_1.Issue.atToken(file, s.getFirstToken(), message, this.getMetadata().key, this.conf.severity);
+                issues.push(issue);
+                break;
+            }
+        }
+        return issues;
+    }
+}
+exports.StrictSQL = StrictSQL;
+//# sourceMappingURL=strict_sql.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/superclass_final.js":
 /*!*************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/superclass_final.js ***!
@@ -77454,13 +77550,14 @@ class Traversal {
         return undefined;
     }
     buildAttributes(def) {
+        var _a, _b;
         const attr = [];
         // TODO: visibility is wrong for classes
-        for (const a of (def === null || def === void 0 ? void 0 : def.getAttributes().getAll()) || []) {
+        for (const a of ((_a = def === null || def === void 0 ? void 0 : def.getAttributes()) === null || _a === void 0 ? void 0 : _a.getAll()) || []) {
             const type = new transpile_types_1.TranspileTypes().toType(a.getType());
             attr.push(`"${a.getName().toUpperCase()}": {"type": () => {return ${type};}, "visibility": "U", "is_constant": " "}`);
         }
-        for (const a of (def === null || def === void 0 ? void 0 : def.getAttributes().getConstants()) || []) {
+        for (const a of ((_b = def === null || def === void 0 ? void 0 : def.getAttributes()) === null || _b === void 0 ? void 0 : _b.getConstants()) || []) {
             const type = new transpile_types_1.TranspileTypes().toType(a.getType());
             attr.push(`"${a.getName().toUpperCase()}": {"type": () => {return ${type};}, "visibility": "U", "is_constant": "X"}`);
         }
@@ -77572,6 +77669,7 @@ class Traversal {
         return undefined;
     }
     buildConstructorContents(scope, def) {
+        var _a, _b, _c;
         let ret = "";
         if (def.getSuperClass() !== undefined
             && def.getMethodDefinitions().getByName("CONSTRUCTOR") === undefined) {
@@ -77580,7 +77678,7 @@ class Traversal {
         const cName = Traversal.escapeNamespace(def.getName().toLowerCase());
         ret += "this.me = new abap.types.ABAPObject();\n";
         ret += "this.me.set(this);\n";
-        for (const a of def.getAttributes().getAll()) {
+        for (const a of ((_a = def.getAttributes()) === null || _a === void 0 ? void 0 : _a.getAll()) || []) {
             if (a.getMeta().includes("static" /* abaplint.IdentifierMeta.Static */) === true) {
                 continue;
             }
@@ -77593,11 +77691,11 @@ class Traversal {
             ret += this.dataFromInterfaces(i.name, scope);
         }
         // handle aliases after initialization of carrier variables
-        for (const a of def.getAliases().getAll()) {
+        for (const a of ((_b = def.getAliases()) === null || _b === void 0 ? void 0 : _b.getAll()) || []) {
             ret += "this." + a.getName().toLowerCase() + " = this." + Traversal.escapeNamespace(a.getComponent().replace("~", "$").toLowerCase()) + ";\n";
         }
         // constants can be accessed both statically and via reference
-        for (const c of def.getAttributes().getConstants()) {
+        for (const c of ((_c = def.getAttributes()) === null || _c === void 0 ? void 0 : _c.getConstants()) || []) {
             ret += "this." + Traversal.escapeNamespace(c.getName().toLowerCase()) + " = " + cName + "." + Traversal.escapeNamespace(c.getName().toLowerCase()) + ";\n";
         }
         return ret;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.5.72",
+  "version": "2.5.74",
   "description": "Transpiler - Command Line Interface",
   "bin": {
     "abap_transpile": "./abap_transpile"
@@ -25,12 +25,12 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/transpiler": "^2.5.72",
+    "@abaplint/transpiler": "^2.5.74",
     "@types/glob": "^7.2.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.5",
     "@types/node": "^18.15.11",
-    "@abaplint/core": "^2.96.2",
+    "@abaplint/core": "^2.97.2",
     "progress": "^2.0.3",
     "webpack": "^5.77.0",
     "webpack-cli": "^5.0.1",