Skip to content

Commit a216b6f

Browse files
authored
Merge pull request #828 from Turbo87/simplify-cp
controllers/crate/version: Simplify "currentDependencies" property
2 parents 085c105 + 87e764b commit a216b6f

File tree

1 file changed

+3
-12
lines changed

1 file changed

+3
-12
lines changed

app/controllers/crate/version.js

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -57,18 +57,9 @@ export default Ember.Controller.extend({
5757

5858
return DS.PromiseArray.create({
5959
promise: deps.then((deps) => {
60-
let non_dev = deps.filter((dep) => dep.get('kind') !== 'dev');
61-
let map = {};
62-
let ret = [];
63-
64-
non_dev.forEach((dep) => {
65-
if (!(dep.get('crate_id') in map)) {
66-
map[dep.get('crate_id')] = 1;
67-
ret.push(dep);
68-
}
69-
});
70-
71-
return ret;
60+
return deps
61+
.filter((dep) => dep.get('kind') !== 'dev')
62+
.uniqBy('crate_id');
7263
})
7364
});
7465
}),

0 commit comments

Comments
 (0)