Fixes encoder option must not modify the content of object to encode

Author: Ericbla

lib/binary_parser.js

@@ -493,16 +493,26 @@ Parser.prototype.generate = function(ctx) {
 };
 
 Parser.prototype.generateEncode = function(ctx) {
-  // Transform with the possibly provided encoder before encoding
   var varName = ctx.generateVariable(this.varName);
+  var savVarName = ctx.generateTmpVariable();
+
+  // Transform with the possibly provided encoder before encoding
   if (this.options.encoder) {
+    // Save varName before applying encoder
+    ctx.pushCode("var {0} = {1};", savVarName, varName);
     this.generateEncoder(ctx, varName, this.options.encoder);
   }
 
   if (this.type) {
     this["generate_encode" + this.type](ctx);
   }
 
+  if (this.options.encoder) {
+    // Restore varName after encoder transformation so that next parsers will
+    // have access to original field value (but not nested ones)
+    ctx.pushCode("{0} = {1};", varName, savVarName);
+  }
+
   return this.generateEncodeNext(ctx);
 };