Skip to content

Commit

Permalink
[FIX] cachedcollection calling multiple times SYNC (#15104)
Browse files Browse the repository at this point in the history
* fixed cachedcollection calling multiple times

* fix tests

* fix review
  • Loading branch information
ggazzo authored Aug 6, 2019
1 parent 8571128 commit b9a9a3f
Show file tree
Hide file tree
Showing 4 changed files with 197 additions and 229 deletions.
2 changes: 1 addition & 1 deletion app/models/server/models/Settings.js
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ export class Settings extends Base {
};

if (ids.length > 0) {
filter._id = { $in: ids };
filter._id = { $in: ids };
}

return this.find(filter, { fields: { _id: 1, value: 1 } });
Expand Down
2 changes: 0 additions & 2 deletions app/settings/client/lib/settings.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,6 @@ settings.cachedCollection = new CachedCollection({

settings.collection = settings.cachedCollection.collection;

settings.cachedCollection.init();

settings.dict = new ReactiveDict('settings');

settings.get = function(_id) {
Expand Down
Loading

0 comments on commit b9a9a3f

Please sign in to comment.