Skip function inside object while encoding, just like JSON.

mcollina · Nov 18, 2014 · b10e282 · b10e282
1 parent b613485
commit b10e282
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -339,7 +339,9 @@ function msgpack() {
       , header
 
     for (key in obj) {
-      if (obj.hasOwnProperty(key) && obj[key] !== undefined) {
+      if (obj.hasOwnProperty(key) &&
+          obj[key] !== undefined &&
+          "function" !== typeof obj[key] ) {
         ++length
         acc.push(encode(key))
         acc.push(encode(obj[key]))

diff --git a/test/functions.js b/test/functions.js
@@ -0,0 +1,18 @@
+
+var test    = require('tape').test
+  , msgpack = require('../')
+  , noop = function() {}
+
+test('encode a function inside a map', function(t) {
+  var encoder   = msgpack()
+    , expected  = {
+          hello: 'world'
+      }
+    , toEncode  = {
+          hello: 'world'
+        , func: noop
+      }
+
+  t.deepEqual(encoder.decode(encoder.encode(toEncode)), expected, 'remove the function from the map');
+  t.end()
+})