From a3722aea7f6a7a1320b2952f6a0028a71e41a8f8 Mon Sep 17 00:00:00 2001
From: Pavlo <ppaska@gmail.com>
Date: Sun, 10 Mar 2024 12:14:19 +0000
Subject: [PATCH] fixed bug with parseDateTime

---
 index.html                   | 122 +++--------------------------------
 package.json                 |   2 +-
 src/tests/utils-pipe.spec.ts |   8 +++
 src/utils/helpers.ts         |   3 +-
 4 files changed, 20 insertions(+), 115 deletions(-)

diff --git a/index.html b/index.html
index b89383f..b077593 100644
--- a/index.html
+++ b/index.html
@@ -14,123 +14,19 @@ <h4>Data Pipe testing page. Do not expect anything here! </h4>
   <input id="input" type="file" onchange="fileChange(this.files[0])" />
 
   <script>
-    let result = null;
-    let firstChar = '';
-
-    function readContent(file) {
-      const fr = new FileReader();
-      const promise = new Promise((success, fail) => {
-        fr.onload = e => success(e.target?.result);
-        fr.onerror = e => fail('Error loading file');
-      });
-      fr.readAsText(file);
-
-      return promise;
-
-    }
-    async function fileChange(file) {
-      //file
-      if (!file) {
-        return;
-      }
-
-      if (file) {
-        var reader = new FileReader();
-        reader.readAsText(file, "UTF-8");
-        reader.onload = function (evt) {
-          const data = JSON.parse(evt.target.result);
-          console.log('* fieldsInfo => ', dp.dataPipe(data).getFieldsInfo(), data);
-        }
-        reader.onerror = function (evt) {
-          console.log("error reading file");
-        }
-      }
-    }
-
-    function iterateChunks(file, callback) {
-      const chunkSize = 1024; // 10 * 64 * 64 * 1024;
-      let offset = 0;
-      const decoder = new TextDecoder();
-      const fr = new FileReader();
-      fr.onload = () => {
-        const view = new Uint8Array(fr.result);
-        const text = decoder.decode(view);
-        callback(text);
-
-        offset += chunkSize;
-        seek();
-      }
-      fr.onerror = (err) => {
-        console.log(err)
-      };
-
-      seek();
-
-      function seek() {
-        if (offset >= file.size) {
-          console.log('==> done ==>', result)
-
-          return;
-        }
-
-        const slice = file.slice(offset, offset + chunkSize);
-        fr.readAsArrayBuffer(slice)
-      }
-    }
-
-    function jsonProcessor(text) {
-      const result = [];
-      const parser = new dp.dataPipe().JSONParser();
-      for (const ch of text) {
-        parser.processJsonItems(ch, (itemText, key) => {
-          let item = JSON.parse(itemText);
-
-          if (typeof key === 'string') {
-            item = { key, item };
-          }
-
-          result.push(item);
-        });
-      }
-
-      return result;
-    }
 
     function onButtonClick() {
-      const data = [{ a: null, d:{d1:22, d2:33} }, { b: 2, d:{d1:221, d2:331, d3: null} }];
-      console.log('* fieldsInfo => ', dp.dataPipe(data).flattenObject());
-      // const json = `[{"value":"test value\\\\d"}]`;
-      // console.log(json)
-      // console.log(JSON.parse(json)[0].value)
-      // console.log(jsonProcessor(json))
-      // const iterator = dp.dataPipe()
-      //   .jsonParseIterator('{v:{vi:test}, arr:{vi:test2}}');
 
-      // for (let v of iterator) {
-      //   console.log(v);
-      // }
-      // console.log(Array.from(iterator));
-      return;
-
-      // const mapFn = r => ({ val1: r });
-      // const fdFn = (arr) => dp.dataPipe(arr.map(mapFn)).getFieldsInfo();
-
-      // const data = fdFn([new Date(2001, 1, 1), new Date()]);
-
-      const array = ['2021-05-01', null, '2021-05-01'];
-      data = dp
-        .dataPipe(array)
-        .getFieldsInfo();
-
-      console.log('==>', dp.dataPipe().rtrim('net.', '.'))
-
-      // fetch("https://restcountries.eu/rest/v2/regionalbloc/eu")
-      // .then(r => {
-      //   r.json().then(items => {
-      //     console.log('==>', items, dp.dataPipe(items).getFieldsInfo());
-      //   });
-      // })
+      const data = [
+        {
+          "dt2": "2020-02-21T13:49:15.167Z",
+          "dt": "20-Aug-2019",
+          "dt1": "06-Aug-2019",
+          "value": "8=FIX.4.4^9=58^35=0^49=BuySide^56=SellSide^34=3^52=20190605-12:29:20.259^10=172^"
+        }
+      ]
 
+      console.log('* fieldsInfo => ', dp.dataPipe(data).getFieldsInfo());
     }
 
     onButtonClick();
diff --git a/package.json b/package.json
index ed69c23..5abd1cf 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
     "name": "datapipe-js",
-    "version": "0.3.30",
+    "version": "0.3.31",
     "description": "dataPipe is a data processing and data analytics library for JavaScript. Inspired by LINQ (C#) and Pandas (Python)",
     "main": "dist/cjs/data-pipe.js",
     "module": "dist/esm/data-pipe.mjs",
diff --git a/src/tests/utils-pipe.spec.ts b/src/tests/utils-pipe.spec.ts
index 7ec5e3b..6b4da77 100644
--- a/src/tests/utils-pipe.spec.ts
+++ b/src/tests/utils-pipe.spec.ts
@@ -30,6 +30,14 @@ describe('Test dataUtils', () => {
     expect(dateToString(parseDatetimeOrNull(strDate) as Date)).toBe(strDate);
   });
 
+  it('parseDateTime 1', () => {
+    const dt = parseDatetimeOrNull('06-Aug-2020');
+    expect(dt).toBeInstanceOf(Date);
+
+    const dt2 = parseDatetimeOrNull('8=FIX.4.4^9=58^35=0^49=BuySide^56=SellSide^34=3^52=20190605-12:29:20.259^10=172^');
+    expect(dt2).toBeNull();
+  });  
+
   it('parseDateTime with larger miliseconds', () => {
     const dt = parseDatetimeOrNull('2020-06-08T13:49:15.16789');
     expect(dt).toBeInstanceOf(Date);
diff --git a/src/utils/helpers.ts b/src/utils/helpers.ts
index f55506f..787b611 100644
--- a/src/utils/helpers.ts
+++ b/src/utils/helpers.ts
@@ -164,10 +164,11 @@ export function parseDatetimeOrNull(
 
   const strTokens = strValue
     .replace('T', ' ')
+    .replace('Z', ' ')
     .replace('.', ' ')
     .toLowerCase()
     .split(/[: /-]/);
-  const dt = strTokens.map(parseFloat);
+  const dt = strTokens.map(s => parseNumberOrNull(s) ?? Number.NaN);
 
   let d: Date | null = null;