From 5160e4bd34de784d221cdbed3fb81a37107617c7 Mon Sep 17 00:00:00 2001 From: Rod Howard Date: Fri, 21 Jul 2017 09:20:41 +1000 Subject: [PATCH] Add print_debug parameter to avoid unnecessary console log calls. --- package.json | 2 +- src/SybaseDB.js | 16 +++++++++++----- 2 files changed, 12 insertions(+), 6 deletions(-) diff --git a/package.json b/package.json index 20348dd..eed28fc 100644 --- a/package.json +++ b/package.json @@ -1,6 +1,6 @@ { "name": "sybase", - "version": "1.0.13", + "version": "1.2.1", "description": "A simple node.js wrapper around a java/jconnect app that provides easy access to Sybase Databases without having to install odbc or freetds.", "dependencies": { "JSONStream": "latest" diff --git a/src/SybaseDB.js b/src/SybaseDB.js index 9820b2e..c3367a3 100644 --- a/src/SybaseDB.js +++ b/src/SybaseDB.js @@ -7,7 +7,7 @@ var fs = require("fs"); var PATH_TO_JAVA_BRIDGE1 = process.env.PWD + "/node_modules/sybase/JavaSybaseLink/dist/JavaSybaseLink.jar"; var PATH_TO_JAVA_BRIDGE2 = "./JavaSybaseLink/dist/JavaSybaseLink.jar"; -function Sybase(host, port, dbname, username, password, logTiming, pathToJavaBridge) +function Sybase(host, port, dbname, username, password, logTiming, pathToJavaBridge, print_debug) { this.connected = false; this.host = host; @@ -15,7 +15,8 @@ function Sybase(host, port, dbname, username, password, logTiming, pathToJavaBri this.dbname = dbname; this.username = username; this.password = password; - this.logTiming = (logTiming == true); + this.logTiming = (logTiming == true); + this.print_debug = (print_debug === true) this.pathToJavaBridge = pathToJavaBridge; if (this.pathToJavaBridge === undefined) @@ -92,12 +93,17 @@ Sybase.prototype.query = function(sql, callback) msg.callback = callback; msg.hrstart = hrstart; - console.log("this: " + this + " currentMessages: " + this.currentMessages + " this.queryCount: " + this.queryCount); + if(this.print_debug){ + console.log("this: " + this + " currentMessages: " + this.currentMessages + " this.queryCount: " + this.queryCount); + } this.currentMessages[msg.msgId] = msg; - this.javaDB.stdin.write(strMsg + "\n"); - console.log("sql request written: " + strMsg); + this.javaDB.stdin.write(strMsg + "\n"); + + if(this.print_debug){ + console.log("sql request written: " + strMsg); + } }; Sybase.prototype.onSQLResponse = function(jsonMsg)