Refactored PR #45 to pass all tests and be backward-compatible. This is a stop-gap fix for Issue #42.

Author: JamesMessinger

lib/bundle.js

@@ -47,7 +47,7 @@ function crawl(parent, key, path, pathFromRoot, inventory, $refs, options) {
   var obj = key === null ? parent : parent[key];
 
   if (obj && typeof obj === 'object') {
-    if ($Ref.is$Ref(obj)) {
+    if ($Ref.isAllowed$Ref(obj)) {
       inventory$Ref(parent, key, path, pathFromRoot, inventory, $refs, options);
     }
     else {
@@ -65,7 +65,7 @@ function crawl(parent, key, path, pathFromRoot, inventory, $refs, options) {
         var keyPathFromRoot = Pointer.join(pathFromRoot, key);
         var value = obj[key];
 
-        if ($Ref.is$Ref(value)) {
+        if ($Ref.isAllowed$Ref(value)) {
           inventory$Ref(obj, key, path, keyPathFromRoot, inventory, $refs, options);
         }
         else {

lib/ref.js

@@ -124,7 +124,12 @@ $Ref.isExternal$Ref = function(value) {
  */
 $Ref.isAllowed$Ref = function(value, options) {
   if ($Ref.is$Ref(value)) {
-    if (value.$ref[0] === '#' && value.$ref[1] === '/' || !options || options.resolve.external && value.$ref[0] !== '#') {
+    if (value.$ref.substr(0, 2) === '#/' || value.$ref === '#') {
+      // It's a JSON Pointer reference, which is always allowed
+      return true;
+    }
+    else if (value.$ref[0] !== '#' && (!options || options.resolve.external)) {
+      // It's an external reference, which is allowed by the options
       return true;
     }
   }

test/specs/internal/internal.bundled.js

@@ -1,6 +1,9 @@
 helper.bundled.internal =
 {
   "definitions": {
+    "fragment": {
+      "$id": "#fragment"
+    },
     "requiredString": {
       "title": "requiredString",
       "minLength": 1,
@@ -44,6 +47,9 @@ helper.bundled.internal =
   ],
   "type": "object",
   "properties": {
+    "fragment": {
+      "$ref": "#fragment"
+    },
     "gender": {
       "enum": [
         "male",

test/specs/internal/internal.dereferenced.js

@@ -1,6 +1,9 @@
 helper.dereferenced.internal =
 {
   "definitions": {
+    "fragment": {
+      "$id": "#fragment"
+    },
     "requiredString": {
       "title": "requiredString",
       "minLength": 1,
@@ -46,6 +49,9 @@ helper.dereferenced.internal =
   ],
   "type": "object",
   "properties": {
+    "fragment": {
+      "$ref": "#fragment"
+    },
     "gender": {
       "enum": [
         "male",

test/specs/internal/internal.parsed.js

@@ -1,6 +1,9 @@
 helper.parsed.internal =
 {
   "definitions": {
+    "fragment": {
+      "$id": "#fragment"
+    },
     "requiredString": {
       "title": "requiredString",
       "minLength": 1,
@@ -44,6 +47,9 @@ helper.parsed.internal =
   ],
   "type": "object",
   "properties": {
+    "fragment": {
+      "$ref": "#fragment"
+    },
     "gender": {
       "enum": [
         "male",