@abaplint/runtime 2.13.85 → 2.13.87

package/build/src/statements/create_data.js

@@ -153,7 +153,11 @@ function createData(target, options) {
                 target.assign(new types_1.DecFloat34());
                 break;
             default:
-                if (abap.DDIC[options.typeName.trimEnd()]) {
+                if (abap.DDIC[options.typeName.trimEnd()] && options.table) {
+                    // static "TYPE STANDARD TABLE OF <ddic>", same as the dynamic name variant above
+                    target.assign(new abap.types.Table(abap.DDIC[options.typeName.trimEnd()].type(), tableOptions));
+                }
+                else if (abap.DDIC[options.typeName.trimEnd()]) {
                     target.assign(abap.DDIC[options.typeName.trimEnd()].type().clone());
                 }
                 else if (options.typeName.includes("=>")) {

package/build/src/types/decfloat34.js

@@ -22,7 +22,10 @@ class DecFloat34 {
             this.value = 0;
         }
         else if (typeof value === "string") {
-            this.value = parseFloat(value);
+            // get() writes the decimal separator as a comma, and parseFloat stops
+            // at one: without this, reading back what this type just wrote gives a
+            // truncated number and no error at all
+            this.value = parseFloat(value.replace(",", "."));
         }
         else if (value instanceof DecFloat34) {
             this.value = value.getRaw();

package/build/src/types/float.js

@@ -6,7 +6,12 @@ const hex_1 = require("./hex");
 const xstring_1 = require("./xstring");
 const integer8_1 = require("./integer8");
 const decfloat34_1 = require("./decfloat34");
-const FLOAT_DIGITS = /^\s*[+-]?(\d+\.?\d*|\.\d+)([eE][+-]?\d+)? *$/;
+// A float writes itself out with a comma for the decimal separator, so
+// reading one back has to accept a comma. Both are allowed rather than only
+// the comma: ABAP source literals carry a point, `CONV f( '0.25' )` is
+// everywhere, and a separator that is legal on the way out and illegal on the
+// way in is not a rule, it is a trap.
+const FLOAT_DIGITS = /^\s*[+-]?(\d+[.,]?\d*|[.,]\d+)([eE][+-]?\d+)? *$/;
 /*
 function getNumberParts(x: number) {
   if(isNaN(x)) {
@@ -67,7 +72,7 @@ class Float {
             if (FLOAT_DIGITS.test(value) === false) {
                 (0, throw_error_1.throwError)("CX_SY_CONVERSION_NO_NUMBER");
             }
-            this.value = parseFloat(value);
+            this.value = parseFloat(value.replace(",", "."));
         }
         else if (value instanceof integer8_1.Integer8) {
             this.value = Number(value.get());

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/runtime",
-  "version": "2.13.85",
+  "version": "2.13.87",
   "description": "Transpiler - Runtime",
   "main": "build/src/index.js",
   "typings": "build/src/index.d.ts",
@@ -33,7 +33,7 @@
     "@types/mocha": "^10.0.10",
     "@types/node": "^26.4.1",
     "chai": "^6.2.2",
-    "mocha": "^12.0.0",
+    "mocha": "^12.0.1",
     "source-map-support": "^0.5.21",
     "typescript": "^6.0.3"
   },