@abaplint/runtime 2.6.13 → 2.6.15

package/build/src/clone.js

@@ -24,7 +24,7 @@ function clone(obj) {
         // @ts-ignore
         // eslint-disable-next-line no-prototype-builtins
         if (obj.hasOwnProperty(attr)) {
-            if (null == obj[attr] || "object" != typeof obj[attr]) {
+            if (null === obj[attr] || "object" !== typeof obj[attr]) {
                 copy[attr] = obj[attr];
             }
             else {

package/build/src/compare/eq.d.ts

@@ -1,4 +1,4 @@
-import { ABAPObject, FieldSymbol, Float, HashedTable, Hex, Structure, Table } from "../types";
+import { ABAPObject, FieldSymbol, Float, HashedTable, String, Hex, Structure, Table } from "../types";
 import { ICharacter } from "../types/_character";
 import { INumeric } from "../types/_numeric";
-export declare function eq(left: number | string | ICharacter | INumeric | Float | ABAPObject | Structure | Hex | HashedTable | Table | FieldSymbol, right: number | string | ICharacter | INumeric | Float | ABAPObject | Structure | Hex | HashedTable | Table | FieldSymbol): boolean;
+export declare function eq(left: number | string | ICharacter | INumeric | Float | String | ABAPObject | Structure | Hex | HashedTable | Table | FieldSymbol, right: number | string | ICharacter | INumeric | Float | String | ABAPObject | Structure | Hex | HashedTable | Table | FieldSymbol): boolean;

package/build/src/compare/eq.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.eq = void 0;
+/* eslint-disable max-len */
 const types_1 = require("../types");
 function compareTables(left, right) {
     const leftArray = left.array();
@@ -28,18 +29,37 @@ function eq(left, right) {
         return eq(left.getPointer(), right);
     }
     // for performance, do the typicaly/easy cases first
-    /*
-      if (right instanceof Character && left instanceof Character && right.getLength() === left.getLength()) {
-        return right.get() === left.get();
-      } else if (right instanceof Numc && left instanceof Numc && right.getLength() === left.getLength()) {
+    if (right instanceof types_1.Character) {
+        if (left instanceof types_1.Character) {
+            if (right.getLength() === left.getLength()) {
+                return right.get() === left.get();
+            }
+            else {
+                return right.getTrimEnd() === left.getTrimEnd();
+            }
+        }
+        else if (left instanceof types_1.String) {
+            return right.getTrimEnd() === left.get();
+        }
+    }
+    else if (right instanceof types_1.String) {
+        if (left instanceof types_1.Character) {
+            return right.get() === left.getTrimEnd();
+        }
+        else if (left instanceof types_1.String) {
+            return right.get() === left.get();
+        }
+    }
+    else if (right instanceof types_1.Numc && left instanceof types_1.Numc && right.getLength() === left.getLength()) {
         return right.get() === left.get();
-      } else if (right instanceof Integer && left instanceof Integer) {
+    }
+    else if (right instanceof types_1.Integer && left instanceof types_1.Integer) {
         return right.get() === left.get();
-      }
-      */
-    if (left instanceof types_1.Table || right instanceof types_1.Table || left instanceof types_1.HashedTable || right instanceof types_1.HashedTable) {
-        if ((left instanceof types_1.Table || left instanceof types_1.HashedTable)
-            && (right instanceof types_1.Table || right instanceof types_1.HashedTable)) {
+    }
+    else if (right instanceof types_1.Table
+        || right instanceof types_1.HashedTable) {
+        if (left instanceof types_1.Table
+            || left instanceof types_1.HashedTable) {
             return compareTables(left, right);
         }
         else {
@@ -51,7 +71,7 @@ function eq(left, right) {
         if (!(right instanceof types_1.Structure)) {
             return eq(left.getCharacter(), right);
         }
-        if (!(left instanceof types_1.Structure)) {
+        else if (!(left instanceof types_1.Structure)) {
             return eq(left, right.getCharacter());
         }
         const l = left.get();
@@ -76,6 +96,9 @@ function eq(left, right) {
     else if (left instanceof types_1.Date) {
         l = left.get().trimEnd();
     }
+    else if (left instanceof types_1.Table || left instanceof types_1.HashedTable) {
+        return false;
+    }
     else if (typeof left === "object") {
         l = left.get();
     }

package/build/src/statements/insert_internal.d.ts

@@ -8,6 +8,7 @@ export interface IInsertInternalOptions {
     table: Table | HashedTable | FieldSymbol;
     referenceInto?: DataReference;
     assigning?: FieldSymbol;
+    noClone?: boolean;
     lines?: boolean;
 }
 export declare function insertInternal(options: IInsertInternalOptions): void;

package/build/src/statements/insert_internal.js

@@ -122,7 +122,7 @@ function insertInternal(options) {
     }
     else if (data) {
         // todo, for now it just appends, this is not correct, but currently the table type is not known
-        const val = options.table.insertIndex(data, options.table.getArrayLength());
+        const val = options.table.insertIndex(data, options.table.getArrayLength(), options.noClone);
         if (options.assigning) {
             options.assigning.assign(val);
         }

package/build/src/statements/select.js

@@ -60,7 +60,7 @@ class SelectDatabase {
                     targetRow.set(row[columnName]);
                 }
                 // @ts-ignore
-                abap.statements.insertInternal({ table: target, data: targetRow });
+                abap.statements.insertInternal({ table: target, data: targetRow, noClone: true });
             }
         }
         else if (Array.isArray(target)) {

package/build/src/types/table.d.ts

@@ -106,7 +106,7 @@ export declare class Table implements ITable {
     clear(): void;
     set(tab: Table | TableRowType): this;
     getHeader(): TableRowType;
-    insertIndex(item: TableRowType, index: number): any;
+    insertIndex(item: TableRowType, index: number, noClone?: boolean): any;
     deleteIndex(index: number): void;
     append(item: TableRowType): any;
     appendInitial(): TableRowType;

package/build/src/types/table.js

@@ -322,7 +322,7 @@ class Table {
         }
         return this.header;
     }
-    insertIndex(item, index) {
+    insertIndex(item, index, noClone = false) {
         this.secondaryIndexes = {};
         if (item instanceof field_symbol_1.FieldSymbol) {
             const p = item.getPointer();
@@ -332,7 +332,13 @@ class Table {
             this.insertIndex(p, index);
             return p;
         }
-        const val = this.getValue(item);
+        let val;
+        if (noClone === false) {
+            val = this.getValue(item);
+        }
+        else {
+            val = item;
+        }
         if (index === 0) {
             this.value.unshift(val);
         }

package/package.json

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