Recovered property accessor in encoding.js (Resolves inexorabletash#67)

therufa · inexorabletash · commit b4e5bc26e26e · 2017-01-05T19:09:36.000-08:00
Fixed issue described in inexorabletash/issues/67 The problem was casued by the missing propery accessor on the require statement, which passed the entire global scope of `encoding-indexes` instead of the indexes.
diff --git a/lib/encoding.js b/lib/encoding.js
@@ -11,7 +11,8 @@
   // If we're in node require encoding-indexes and attach it to the global.
   if (typeof module !== "undefined" && module.exports &&
     !global["encoding-indexes"]) {
-    global['encoding-indexes'] = require("./encoding-indexes.js");
+    global["encoding-indexes"] =
+      require("./encoding-indexes.js")["encoding-indexes"];
   }
 
   //

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,8 @@`
`11`	`11`	`// If we're in node require encoding-indexes and attach it to the global.`
`12`	`12`	`if (typeof module !== "undefined" && module.exports &&`
`13`	`13`	`!global["encoding-indexes"]) {`
`14`		`- global['encoding-indexes'] = require("./encoding-indexes.js");`
	`14`	`+ global["encoding-indexes"] =`
	`15`	`+ require("./encoding-indexes.js")["encoding-indexes"];`
`15`	`16`	`}`
`16`	`17`
`17`	`18`	`//`