packageFilter should have been giving the pkgfile as an argument, fixed

browserify · Feb 17, 2015 · 70b71e7 · 70b71e7
1 parent caca9f9
commit 70b71e7
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/lib/async.js b/lib/async.js
@@ -76,7 +76,7 @@ module.exports = function resolve (x, opts, cb) {
                     catch (err) {}
 
                     if (opts.packageFilter) {
-                        pkg = opts.packageFilter(pkg, x);
+                        pkg = opts.packageFilter(pkg, pkgfile);
                     }
                     if (opts.pathFilter && remainder) {
                         var newRemainder = opts.pathFilter(pkg, x, remainder);
@@ -129,7 +129,7 @@ module.exports = function resolve (x, opts, cb) {
                 catch (err) {}
 
                 if (opts.packageFilter) {
-                    pkg = opts.packageFilter(pkg, x);
+                    pkg = opts.packageFilter(pkg, pkgfile);
                 }
 
                 if (pkg.main) {

diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "resolve",
   "description": "resolve like require.resolve() on behalf of files asynchronously and synchronously",
-  "version": "1.1.2",
+  "version": "1.1.3",
   "repository": {
     "type": "git",
     "url": "git://github.com/substack/node-resolve.git"

diff --git a/readme.markdown b/readme.markdown
@@ -107,8 +107,8 @@ options are:
 
 * opts.isFile - function to synchronously test whether a file exists
 
-* opts.packageFilter - transform the parsed package.json contents before looking
-at the "main" field
+* `opts.packageFilter(pkg, pkgfile)` - transform the parsed package.json
+* contents before looking at the "main" field
 
 * opts.paths - require.paths array to use if nothing is found on the normal
 node_modules recursive walk (probably don't use this)