@abaplint/runtime 2.4.9 → 2.4.10

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

@@ -34,6 +34,7 @@ export declare class Table {
     private readonly loops;
     private readonly options;
     private readonly qualifiedName;
+    private readonly isStructured;
     private secondaryIndexes;
     constructor(rowType: TableRowType, options?: ITableOptions, qualifiedName?: string);
     getKeyByName(name: string): ITableKey | undefined;
@@ -49,7 +50,7 @@ export declare class Table {
     getHeader(): TableRowType;
     insertIndex(item: TableRowType, index: number): TableRowType;
     deleteIndex(index: number): void;
-    append(item: TableRowType, cloneRow?: boolean): FieldSymbol | DataReference | TableRowType;
+    append(item: TableRowType): FieldSymbol | DataReference | TableRowType;
     appendInitial(): TableRowType;
     sort(compareFn: (a: TableRowType, b: TableRowType) => number): void;
     private getValue;

package/build/src/types/table.js

@@ -4,6 +4,7 @@ exports.Table = exports.LoopIndex = exports.TableAccessType = void 0;
 const integer_1 = require("./integer");
 const string_1 = require("./string");
 const clone_1 = require("../clone");
+const structure_1 = require("./structure");
 const field_symbol_1 = require("./field_symbol");
 const data_reference_1 = require("./data_reference");
 const insert_internal_1 = require("../statements/insert_internal");
@@ -29,6 +30,7 @@ class Table {
         this.loops = new Set();
         this.rowType = rowType;
         this.options = options;
+        this.isStructured = rowType instanceof structure_1.Structure;
         if ((options === null || options === void 0 ? void 0 : options.withHeader) === true) {
             this.header = (0, clone_1.clone)(this.rowType);
         }
@@ -147,7 +149,7 @@ class Table {
             }
         }
     }
-    append(item, cloneRow = true) {
+    append(item) {
         this.secondaryIndexes = {};
         if (item instanceof field_symbol_1.FieldSymbol) {
             const p = item.getPointer();
@@ -164,11 +166,9 @@ class Table {
             return ref;
         }
         else {
-            const val = this.getValue(item, cloneRow);
-            const p = (0, clone_1.clone)(this.rowType);
-            p.set(val);
-            this.value.push(p);
-            return p;
+            const val = this.getValue(item);
+            this.value.push(val);
+            return val;
         }
     }
     appendInitial() {
@@ -183,7 +183,7 @@ class Table {
         this.value.sort(compareFn);
     }
     ///////////////////////////
-    getValue(item, cloneRow = true) {
+    getValue(item) {
         // make sure to do conversion if needed
         if (typeof item === "number") {
             const tmp = (0, clone_1.clone)(this.getRowType());
@@ -194,15 +194,19 @@ class Table {
             const tmp = (0, clone_1.clone)(this.getRowType());
             tmp.set(new string_1.String().set(item));
             return tmp;
+            // @ts-ignore
+            // eslint-disable-next-line max-len
+        }
+        else if (this.isStructured === true && item.getQualifiedName && this.rowType.getQualifiedName && item.getQualifiedName() === this.rowType.getQualifiedName()) {
+            // types match, so no need to do conversions, just clone the item
+            const val = (0, clone_1.clone)(item);
+            return val;
         }
-        else if (cloneRow === true) {
+        else {
             const tmp = (0, clone_1.clone)(this.getRowType());
             tmp.set(item);
             return tmp;
         }
-        else {
-            return item;
-        }
     }
 }
 exports.Table = Table;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/runtime",
-  "version": "2.4.9",
+  "version": "2.4.10",
   "description": "Transpiler - Runtime",
   "main": "build/src/index.js",
   "typings": "build/src/index.d.ts",
@@ -36,6 +36,6 @@
     "typescript": "^4.8.4"
   },
   "dependencies": {
-    "temporal-polyfill": "^0.0.8"
+    "temporal-polyfill": "^0.1.0"
   }
 }