@pi-r/oracle 0.6.5 → 0.7.0

package/README.md

@@ -1,10 +1,7 @@
-# @pi-r/oracle
+### @pi-r/oracle
 
-## Documentation
+https://e-mc.readthedocs.io/en/latest/db/oracle.html
 
-- [E-mc](https://e-mc.readthedocs.io/en/latest/db/oracle.html)
-- [squared](https://squared.readthedocs.io)
-
-## LICENSE
+### LICENSE
 
 MIT

package/client/index.d.ts

@@ -4,7 +4,6 @@ import type { IDbSourceClient } from '@e-mc/db/types';
 
 import type { OracleDataSource } from '../types';
 
-// @ts-ignore
 import type { InitialiseOptions } from 'oracledb';
 
 declare const Oracle: IDbSourceClient<OracleDataSource> & {

package/client/index.js

@@ -1,5 +1,4 @@
 "use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
 exports.DB_SOURCE_TYPE = exports.DB_SOURCE_CLIENT = exports.checkTimeout = exports.executeBatchQuery = exports.executeQuery = exports.setCredential = exports.initOracleClient = void 0;
 const oracledb = require("oracledb");
 const util_1 = require("@e-mc/db/util");
@@ -46,7 +45,7 @@ function removePoolProperties(credential) {
         for (let i = 0, length = POOL_UNUSED.length; i < length; ++i) {
             const attr = POOL_UNUSED[i];
             if (attr in credential) {
-                result || (result = { ...credential });
+                result ||= { ...credential };
                 delete result[attr];
             }
         }
@@ -106,14 +105,10 @@ exports.initOracleClient = initOracleClient;
 async function setCredential(item) {
     let credential = this.getCredential(item), hostname, port, username, database;
     if (credential) {
-        ({ hostname, port, username } = (0, util_1.parseServerAuth)(credential));
-        credential.user || (credential.user = username);
-        if ('database' in credential) {
-            database = credential.database;
-            delete credential.database;
-        }
+        ({ hostname, port, database, username } = (0, util_1.parseServerAuth)(credential));
+        credential.user ||= username;
         if ('connectionString' in credential) {
-            credential.connectString || (credential.connectString = credential.connectionString);
+            credential.connectString ||= credential.connectionString;
             delete credential.connectionString;
         }
     }
@@ -132,7 +127,7 @@ async function setCredential(item) {
         item.credential = credential;
     }
     if (hostname) {
-        credential.connectString || (credential.connectString = hostname + (port ? ':' + port : '') + (database ? '/' + database : ''));
+        credential.connectString ||= hostname + (port ? ':' + port : '') + (database ? '/' + database : '');
     }
     if (credential.externalAuth) {
         delete credential.user;
@@ -192,22 +187,22 @@ async function setCredential(item) {
     if (config) {
         const { min, max, idle, queue_max, queue_idle, timeout } = config;
         if (min >= 0) {
-            credential.poolMin ?? (credential.poolMin = min);
+            credential.poolMin ??= min;
         }
         if (max > 0) {
-            credential.poolMax ?? (credential.poolMax = max);
+            credential.poolMax ??= max;
         }
         if (idle >= 0) {
-            credential.poolTimeout ?? (credential.poolTimeout = idle / 1000);
+            credential.poolTimeout ??= idle / 1000;
         }
         if (queue_max >= 0) {
-            credential.queueMax ?? (credential.queueMax = queue_max);
+            credential.queueMax ??= queue_max;
         }
         if (queue_idle >= 0) {
-            credential.queueTimeout ?? (credential.queueTimeout = queue_idle);
+            credential.queueTimeout ??= queue_idle;
         }
         if (timeout >= 0) {
-            credential.connectTimeout ?? (credential.connectTimeout = idle / 1000);
+            credential.connectTimeout ??= idle / 1000;
         }
     }
     const poolKey = getPoolKey(credential);
@@ -245,7 +240,7 @@ async function executeBatchQuery(batch, options = '', outResult) {
         parallel = !batch.some(item => item.parallel === false);
     }
     if (!parallel) {
-        outResult || (outResult = new Array(length));
+        outResult ||= new Array(length);
     }
     const caching = this.hasCache("oracle", sessionKey);
     const tasks = new Array(length);
@@ -289,7 +284,7 @@ async function executeBatchQuery(batch, options = '', outResult) {
                 pool.connected = false;
             }
         }
-        clients.push(client || (client = await oracledb.getConnection((oracleCredential || credential))));
+        clients.push(client ||= await oracledb.getConnection((oracleCredential || credential)));
         if (connectOnce) {
             if (!parallel) {
                 oracleClient = client;
@@ -323,7 +318,7 @@ async function executeBatchQuery(batch, options = '', outResult) {
             continue;
         }
         item.transactionState = 1;
-        const streamRow = typeof item.streamRow === 'string' ? this.hasCoerce("oracle", 'options', null, credential) && (0, types_1.asFunction)(item.streamRow) : item.streamRow;
+        const streamRow = typeof item.streamRow === 'string' ? this.hasCoerce("oracle", 'options', credential) && (0, types_1.asFunction)(item.streamRow) : item.streamRow;
         const cacheValue = ignoreCache === undefined ? sessionKey : Array.isArray(ignoreCache) ? { sessionKey, exclusiveOf: ignoreCache } : { sessionKey, renewCache: ignoreCache === 0 };
         let queryString = '';
         if (caching && ignoreCache !== true && !streamRow) {
@@ -382,7 +377,7 @@ async function executeBatchQuery(batch, options = '', outResult) {
                         }
                         rows.push(row);
                     })
-                        .on('error', err => error || (error = err))
+                        .on('error', err => error ||= err)
                         .on('close', () => {
                         if (error && this.handleFail(error, item, { errorQuery, commandType })) {
                             reject(error);
@@ -420,7 +415,13 @@ async function executeBatchQuery(batch, options = '', outResult) {
         }
     }
     return this.processRows(batch, tasks, {
-        disconnect: () => clients.forEach(item => item.close(err => err && this.writeFail(["Unable to close connnection", "oracle"], err, { type: 65536, fatal: false }))),
+        disconnect: () => clients.forEach(item => {
+            item.close(err => {
+                if (err) {
+                    this.writeFail(["Unable to close connnection", "oracle"], err, { type: 65536, fatal: false });
+                }
+            });
+        }),
         parallel
     }, outResult);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-r/oracle",
-  "version": "0.6.5",
+  "version": "0.7.0",
   "description": "Oracle client driver for E-mc.",
   "main": "client/index.js",
   "types": "client/index.d.ts",
@@ -21,8 +21,8 @@
   "license": "MIT",
   "homepage": "https://github.com/anpham6/pi-r#readme",
   "dependencies": {
-    "@e-mc/db": "^0.8.6",
-    "@e-mc/types": "^0.8.6",
-    "oracledb": "^6.3.0"
+    "@e-mc/db": "^0.9.0",
+    "@e-mc/types": "^0.9.0",
+    "oracledb": "^6.4.0"
   }
 }

package/types/index.d.ts

@@ -5,7 +5,7 @@ import type { ExecuteAction, ServerAuth } from '@e-mc/types/lib/db';
 // @ts-ignore
 import type { BindParameters, ConnectionAttributes, ExecuteOptions, InitialiseOptions, PoolAttributes } from 'oracledb';
 
-export interface OracleDataSource extends DbDataSource<string, ExecuteOptions, unknown, string | OracleCredential, string>, ExecuteAction<BindParameters> {
+export interface OracleDataSource extends DbDataSource<string, ExecuteOptions, unknown, OracleCredential, string>, ExecuteAction<BindParameters> {
     source: "oracle";
 }