@abaplint/runtime 2.4.8 → 2.4.10

package/build/src/types/integer.js

@@ -5,6 +5,7 @@ const throw_error_1 = require("../throw_error");
 const float_1 = require("./float");
 const hex_1 = require("./hex");
 const xstring_1 = require("./xstring");
+const digits = new RegExp(/^\s*-?\+?\d+\.?\d*$/i);
 class Integer {
     constructor(input) {
         this.value = 0;
@@ -17,11 +18,11 @@ class Integer {
         if (typeof value === "number") {
             this.value = Math.round(value);
         }
-        else if (typeof value === "string" && value.trim().length === 0) {
-            this.value = 0;
-        }
         else if (typeof value === "string") {
-            if (/^\s*-?\+?\d+\.?\d*$/i.test(value) === false) {
+            if (value.trim().length === 0) {
+                value = "0";
+            }
+            else if (digits.test(value) === false) {
                 (0, throw_error_1.throwError)("CX_SY_CONVERSION_NO_NUMBER");
             }
             this.value = parseInt(value, 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);
         }
@@ -114,7 +116,12 @@ class Table {
     insertIndex(item, index) {
         this.secondaryIndexes = {};
         const val = this.getValue(item);
-        this.value.splice(index, 0, val);
+        if (index === 0) {
+            this.value.unshift(val);
+        }
+        else {
+            this.value.splice(index, 0, val);
+        }
         for (const l of this.loops.values()) {
             if (l.index <= index) {
                 l.index++;
@@ -142,7 +149,7 @@ class Table {
             }
         }
     }
-    append(item, cloneRow = true) {
+    append(item) {
         this.secondaryIndexes = {};
         if (item instanceof field_symbol_1.FieldSymbol) {
             const p = item.getPointer();
@@ -159,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() {
@@ -178,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());
@@ -189,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.8",
+  "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"
   }
 }