fix: downward compatibility (#71)

Author: gxcsoccer

lib/v1/decoder.js

@@ -418,7 +418,6 @@ proto.readObject = function (withType) {
   // get
   var label = this.byteBuffer.getChar();
   var key;
-  var t;
 
   while (label !== 'z') {
     this.byteBuffer.position(this.byteBuffer.position() - 1);
@@ -427,8 +426,7 @@ proto.readObject = function (withType) {
     label = this.byteBuffer.getChar();
     // property name will auto transfer to a String type.
     debug('read object prop: %j with type: %s', key, withType);
-    t = typeof key;
-    if ((t === 'string' || t === 'number') && !/^this\$\d+$/.test(key)) {
+    if (!/^this\$\d+$/.test(key)) {
       result.$[key] = value;
     }
     if (isMap) {

lib/v2/decoder.js

@@ -725,16 +725,12 @@ proto._readMap = function (map, withType) {
 
   var k;
   var v;
-  var t;
   // Z(0x5a) list/map terminator
   while (code !== 0x5a) {
     k = this.read(withType);
     v = this.read(withType);
-    t = typeof k;
-
-    if (t === 'string' || t === 'number') {
-      map[k] = v;
-    }
+    
+    map[k] = v;
     if (supportES6Map) {
       map.$map.set(k, v);
     }

test/map.test.js

@@ -165,7 +165,7 @@ describe('map.test.js', function() {
   it('should decode successful when key is null', function() {
     var data = new Buffer([77, 116, 0, 0, 78, 83, 0, 4, 110, 117, 108, 108, 122]);
     var rv = hessian.decode(data);
-    rv.should.eql({});
+    rv.should.eql({ null: 'null' });
 
     if (!supportES6Map) {
       // pass if not support es6 Map
@@ -477,7 +477,7 @@ describe('map.test.js', function() {
       values[1].should.eql(166239);
 
       var plainObject = JSON.parse(JSON.stringify(res.data));
-      plainObject.should.eql({});
+      plainObject.should.eql({ '[object Object]': 166239 });
     });
 
   });