github/meteor
1
1
Fork 0
mirror of https://github.com/meteor/meteor.git synced 2026-05-02 03:01:46 -04:00
Code Issues Packages Projects Releases Wiki Activity

Fix concurrency issue with yielding runHandler.

Browse Source 
All tests now pass.
This commit is contained in:
David Glasser
2012-12-13 14:22:30 -08:00
parent 8f64942dac
commit 001cbcb337
1 changed files with 10 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
packages/livedata/livedata_server.js
View File

@@ -195,7 +195,7 @@ _.extend(Meteor._SessionCollectionView.prototype, {
var clearedResult = [];
var docView = self.documents[id];
if (!docView)
throw new Error("Could not find element with id " + id + "to change");
throw new Error("Could not find element with id " + id + " to change");
_.each(changed, function (value, key) {
docView.changeField(subscriptionId, key, value, changedResult);
});
@@ -671,15 +671,21 @@ _.extend(Meteor._LivedataSession.prototype, {
// all subscriptions
_setUserId: function(userId) {
var self = this;
self.userId = userId;
self._isSending = false;
var beforeCVs = self.collectionViews;
self.collectionViews = {};
self._eachSub(function (sub) {
sub._resetSubscription();
});
var beforeCVs = self.collectionViews;
self.collectionViews = {};
self.userId = userId;
self._eachSub(function (sub) {
sub.userId = self.userId;
sub._runHandler();
});
self._isSending = true;
self._diffCollectionViews(beforeCVs);