@abaplint/runtime 2.4.11 → 2.4.12

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

@@ -2,8 +2,6 @@ import { INumeric } from "./_numeric";
 import { ICharacter } from "./_character";
 import { ABAPObject } from "./abap_object";
 import { Structure } from "./structure";
-import { FieldSymbol } from "./field_symbol";
-import { DataReference } from "./data_reference";
 export declare enum TableAccessType {
     standard = "STANDARD",
     sorted = "SORTED",
@@ -48,9 +46,9 @@ export declare class Table {
     clear(): void;
     set(tab: Table | TableRowType): void;
     getHeader(): TableRowType;
-    insertIndex(item: TableRowType, index: number): TableRowType;
+    insertIndex(item: TableRowType, index: number): any;
     deleteIndex(index: number): void;
-    append(item: TableRowType): FieldSymbol | DataReference | TableRowType;
+    append(item: TableRowType): any;
     appendInitial(): TableRowType;
     sort(compareFn: (a: TableRowType, b: TableRowType) => number): void;
     private getValue;

package/build/src/types/table.js

@@ -115,6 +115,14 @@ class Table {
     }
     insertIndex(item, index) {
         this.secondaryIndexes = {};
+        if (item instanceof field_symbol_1.FieldSymbol) {
+            const p = item.getPointer();
+            if (p === undefined) {
+                throw new Error("insertIndex, fs not assigned");
+            }
+            this.insertIndex(p, index);
+            return p;
+        }
         const val = this.getValue(item);
         if (index === 0) {
             this.value.unshift(val);
@@ -156,23 +164,14 @@ class Table {
             if (p === undefined) {
                 throw new Error("APPEND, fs not assigned");
             }
-            this.value.push(p);
-            return item;
+            this.append(p);
+            return p;
         }
         else if (item instanceof data_reference_1.DataReference) {
             const ref = new data_reference_1.DataReference(item.getType());
             ref.assign(item.getPointer());
             this.value.push(ref);
             return ref;
-            // @ts-ignore
-            // eslint-disable-next-line max-len
-        }
-        else if (this.isStructured === true && item.getQualifiedName && this.rowType.getQualifiedName && item.getQualifiedName() !== "" && item.getQualifiedName() === this.rowType.getQualifiedName()) {
-            // hmm, moving this to getValue() should, but does not work
-            // types match, so no need to do conversions, just clone the item
-            const val = (0, clone_1.clone)(item);
-            this.value.push(val);
-            return val;
         }
         else {
             const val = this.getValue(item);
@@ -203,6 +202,13 @@ 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() !== "" && 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 {
             const tmp = (0, clone_1.clone)(this.getRowType());

package/package.json

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