Revert "Copy entity in Graph#fetch"

This reverts commit 12d02e0a6b.
This commit is contained in:
John Firebaugh
2012-12-06 19:00:58 -05:00
parent 384ca15040
commit 67ffd2e71b
2 changed files with 2 additions and 2 deletions

View File

@@ -70,7 +70,7 @@ iD.Graph.prototype = {
// Resolve the id references in a way, replacing them with actual objects.
fetch: function(id) {
var entity = this.entities[id], nodes = [];
if (!entity.nodes || !entity.nodes.length) return iD.Entity(entity); // TODO: shouldn't be necessary
if (!entity.nodes || !entity.nodes.length) return entity;
for (var i = 0, l = entity.nodes.length; i < l; i++) {
nodes[i] = this.fetch(entity.nodes[i]);
}

View File

@@ -83,7 +83,7 @@ describe('iD.Graph', function() {
var node = iD.Node({id: "n1"}),
way = iD.Way({id: "w1", nodes: ["n1"]}),
graph = iD.Graph({n1: node, w1: way});
expect(graph.fetch("w1").nodes[0].id).to.equal("n1");
expect(graph.fetch("w1").nodes).to.eql([node]);
});
});