diff --git a/rhino_modules/jsdoc/src/parser.js b/rhino_modules/jsdoc/src/parser.js index 81dda599..5521afa3 100644 --- a/rhino_modules/jsdoc/src/parser.js +++ b/rhino_modules/jsdoc/src/parser.js @@ -310,14 +310,17 @@ exports.Parser.prototype.addDocletRef = function(e) { } exports.Parser.prototype.resolveEnum = function(e) { - var parent = currentParser.resolvePropertyParent(e.code.node); + var doop = require("jsdoc/util/doop").doop, + parent = currentParser.resolvePropertyParent(e.code.node); if (parent && parent.doclet.isEnum) { if (!parent.doclet.properties) { parent.doclet.properties = []; } // members of an enum inherit the enum's type if (parent.doclet.type && !e.doclet.type) { e.doclet.type = parent.doclet.type; } delete e.doclet.undocumented; e.doclet.defaultvalue = e.doclet.meta.code.value; - parent.doclet.properties.push(e.doclet); + // add the doclet to the parent's properties + // use a copy of the doclet to avoid circular references + parent.doclet.properties.push( doop(e.doclet) ); } } diff --git a/test/specs/tags/enumtag.js b/test/specs/tags/enumtag.js index 44af4543..a7c32a28 100644 --- a/test/specs/tags/enumtag.js +++ b/test/specs/tags/enumtag.js @@ -22,4 +22,10 @@ describe("@enum tag", function() { it('If a @type is given for the property it is reflected in the property value.', function() { expect(tristate.properties[2].type.names.join(', ')).toEqual('boolean'); }); + + it('An enum does not contain any circular references.', function() { + var dump = require("jsdoc/util/dumper").dump; + + expect( dump(tristate) ).not.toMatch(""); + }); }); \ No newline at end of file