Fixed a couple minor bugs

- `this` in a module's context is not the module function anymore; instead, it	is just the global `this` context - `browser` field in package.json can be an Object; this is currently	unsupported when bundling a browser module, but at least now it will be	ignored when determining the entry point of a module. Only a string value	is supported. Push to 2.1.2

Author: bminer

lib/header.js

@@ -25,7 +25,7 @@ function __require(uid, parentUid) {
 /* Note: if this module requires itself, or if its depenedencies
 require it, they will only see an empty Object for now */
 // Now load the module
-__modules[uid](__modulesCache[uid], __modulesCache[uid].exports);
+__modules[uid].call(this, __modulesCache[uid], __modulesCache[uid].exports);
 __modulesCache[uid].loaded = true;
 }
 return __modulesCache[uid].exports;

lib/moduleConcatStream.js

@@ -109,7 +109,7 @@ class ModuleConcatStream extends Readable {
 option is set */
 if(opts.browser) {
 opts.packageFilter = (parsedPkgJson, pkgPath) => {
-if(parsedPkgJson.browser) {
+if(typeof parsedPkgJson.browser === "string") {
 parsedPkgJson.main = parsedPkgJson.browser;
 }
 return parsedPkgJson;

package.json

@@ -1,6 +1,6 @@
 {
 "name": "module-concat",
-"version": "2.1.1",
+"version": "2.1.2",
 "description": "Lightweight CommonJS module concatenation tool",
 "main": "index.js",
 "dependencies": {