fix: 🐛 ref should use custom handler (#126)

Co-authored-by: hongcai.dhc <hongcai.dhc@alibaba-inc.com>

Author: denghongcai

lib/v1/decoder.js

@@ -529,10 +529,7 @@ proto.readList = proto.readArray;
 proto.readRef = function (withType) {
   var rid = this.readRefId();
   var obj = this.refMap[rid];
-  if (!withType && obj && utils.hasOwnProperty(obj, '$')) {
-    obj = obj.$;
-  }
-  return obj;
+  return handle(obj, withType);
 };
 
 /**

test/custom_handler.test.js

@@ -67,6 +67,32 @@ describe('utils.test.js', function () {
       assert.deepEqual(output, { value: '100.06' });
   	});
 
+    it('should decode with custom handler if has ref', function () {
+  		hessian.registerDecodeHandler('java.math.BigDecimal', function (result) {
+        return {
+          $class: result.$class,
+          $: result.$.value,
+        };
+  		});
+      var o = { $class: 'java.math.BigDecimal', $: { value: '100.06' } };
+      var map = new Map();
+      map.set(1, o);
+      map.set(2, o);
+      var buf = hessian.encode({
+        $class: 'java.util.HashMap',
+        $: map
+      }, '2.0');
+      var output = hessian.decode(buf, '2.0');
+      /**
+       * fix problem like ref object reuse
+       */
+      assert(output[1] === '100.06');
+      assert(output[2] === '100.06');
+      hessian.deregisterDecodeHandler('java.math.BigDecimal');
+      output = hessian.decode(buf, '2.0');
+      assert.deepEqual(output, {'1': { value: '100.06'}, '2': { value: '100.06' }});
+  	});
+
   	if (!supportES6Map) {
   		return;
   	}