@abaplint/runtime 2.7.117 → 2.7.119

package/build/src/builtin/find.js

@@ -44,7 +44,12 @@ function find(input) {
         let negative = false;
         if (occ < 0) {
             negative = true;
-            val = val.split("").reverse().join("");
+            let reversed = "";
+            // this is faster than doing val.split("").reverse().join("")
+            for (const character of val) {
+                reversed = character + reversed;
+            }
+            val = reversed;
             occ = Math.abs(occ);
         }
         let found = -1;

package/build/src/builtin/substring.js

@@ -9,21 +9,19 @@ function substring(input) {
     if (off === undefined) {
         off = 0;
     }
-    if (off < 0) {
+    else if (off < 0) {
         (0, throw_error_1.throwError)("CX_SY_RANGE_OUT_OF_BOUNDS");
     }
     const len = (_b = input === null || input === void 0 ? void 0 : input.len) === null || _b === void 0 ? void 0 : _b.get();
     if (len && len < 0) {
         (0, throw_error_1.throwError)("CX_SY_RANGE_OUT_OF_BOUNDS");
     }
-    let sub = "";
     if (typeof input.val === "string") {
-        sub = input.val.substr(off, len);
+        return new string_1.String().set(input.val.substr(off, len));
     }
     else {
-        sub = input.val.getOffset({ offset: off, length: len }).get();
+        return input.val.getOffset({ offset: off, length: len });
     }
-    return new string_1.String().set(sub);
 }
 exports.substring = substring;
 //# sourceMappingURL=substring.js.map

package/build/src/compare/ca.js

@@ -29,10 +29,9 @@ function ca(left, right) {
     else {
         r = right.get().toString();
     }
-    const characters = r.split("");
     let fdpos = 0;
-    for (const c of l.split("")) {
-        if (characters.includes(c) === true) {
+    for (const c of l) {
+        if (r.includes(c) === true) {
             // @ts-ignore
             abap.builtin.sy.get().fdpos.set(fdpos);
             return true;

package/build/src/compare/co.js

@@ -15,15 +15,14 @@ function co(left, right) {
         r = right.toString();
     }
     else if (right instanceof types_1.Structure) {
-        r = Object.values(right.get()).map((a) => a.get()).join("");
+        r = right.getCharacter();
     }
     else {
         r = right.get().toString();
     }
-    const characters = r.split("");
     let fdpos = 0;
-    for (const c of l.split("")) {
-        if (characters.includes(c) === false) {
+    for (const c of l) {
+        if (r.includes(c) === false) {
             // @ts-ignore
             abap.builtin.sy.get().fdpos.set(fdpos);
             return false;

package/build/src/statements/insert_database.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.insertDatabase = exports.toValue = void 0;
 const prefix_1 = require("../prefix");
+const types_1 = require("../types");
 function toValue(value) {
     if (typeof value === "string") {
         // postgres requires ' for values
@@ -39,6 +40,10 @@ async function insertDatabase(table, options, context) {
     for (const k of Object.keys(structure)) {
         columns.push(k);
         const value = structure[k].get();
+        if (value instanceof types_1.Structure) {
+            // then its a group, ignore
+            continue;
+        }
         values.push(toValue(value));
     }
     if (typeof table !== "string") {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/runtime",
-  "version": "2.7.117",
+  "version": "2.7.119",
   "description": "Transpiler - Runtime",
   "main": "build/src/index.js",
   "typings": "build/src/index.d.ts",