Only generate the debug text for payload debug logging if payload deb…

…ug logging is enabled.
tediousjs · Jan 8, 2012 · 5b77e6b · 5b77e6b
1 parent 9873a10
commit 5b77e6b
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 7 deletions.
diff --git a/lib/connection.coffee b/lib/connection.coffee
@@ -235,7 +235,9 @@ class Connection extends EventEmitter
   sendPreLogin: ->
     payload = new PreloginPayload()
     @messageIo.sendMessage(TYPE.PRELOGIN, payload.data)
-    @debug.payload(payload.toString('  '))
+    @debug.payload(->
+      payload.toString('  ')
+    )
 
   emptyMessageBuffer: ->
     @messageBuffer = new Buffer(0)
@@ -245,7 +247,9 @@ class Connection extends EventEmitter
 
   processPreLoginResponse: ->
     preloginPayload = new PreloginPayload(@messageBuffer)
-    @debug.payload(preloginPayload.toString('  '))
+    @debug.payload(->
+      preloginPayload.toString('  ')
+    )
 
   sendLogin7Packet: ->
     loginData =
@@ -256,7 +260,9 @@ class Connection extends EventEmitter
 
     payload = new Login7Payload(loginData)
     @messageIo.sendMessage(TYPE.LOGIN7, payload.data)
-    @debug.payload(payload.toString('  '))
+    @debug.payload(->
+      payload.toString('  ')
+    )
 
   sendPacketToTokenStreamParser: (packet) ->
     @tokenStreamParser.addBuffer(packet.data())
@@ -281,7 +287,9 @@ class Connection extends EventEmitter
 
       payload = new SqlBatchPayload(request.sqlText)
       @messageIo.sendMessage(TYPE.SQL_BATCH, payload.data)
-      @debug.payload(payload.toString('  '))
+      @debug.payload(->
+        payload.toString('  ')
+      )
 
       @transitionTo(@STATE.SENT_CLIENT_REQUEST)
 

diff --git a/lib/debug.coffee b/lib/debug.coffee
@@ -24,9 +24,9 @@ class Debug extends EventEmitter
     if @haveListeners() && @options.data
       @log(packet.dataToString(@indent))
 
-  payload: (payload) ->
+  payload: (generatePayloadText) ->
     if @haveListeners() && @options.payload
-      @log(payload)
+      @log(generatePayloadText())
 
   token: (token) ->
     if @haveListeners() && @options.token

diff --git a/test/unit/debug-test.coffee b/test/unit/debug-test.coffee
@@ -35,7 +35,9 @@ exports.payloadEnabled = (test) ->
     test.done()
   )
 
-  debug.payload(payload)
+  debug.payload(->
+    payload
+  )
 
 exports.payloadNotEnabled = (test) ->
   debug = new Debug()