流星用户助手在更新后挂断

Avi A 发表于 Dev

阿维

我有这个帮手：

agreed: function (){
        if (Meteor.users.findOne({_id: Meteor.userId(), profile: {agreedTermsOfUse: 'true'}}))
            return true;
    }

在我检查的页面上，我有以下内容：

{{#unless agreed}}
      agree form
{{else}}
   Create item form.
    {{list of item}}
{{/unless}}

到目前为止，一切顺利。用户进行注册，然后他可以创建一个项目并将其呈现在项目列表中。

现在，我添加了另一个Meteor.call，当在客户端上成功调用该返回项时，该Meteor.call将创建项添加到用户的profile.hasItems中。

然后，在获得该方法的成功之后，“除非”返回false，否则我必须再次提交同意表格。

我想念什么？谢谢。

"submit .create_restaurant": function (event) {
    event.preventDefault();
    var text = event.target.create_restaurant.value;
    Meteor.call('CreateRest', Meteor.userId(), text, function(error, result){
        if(error){

        }else{
                console.log(result, Meteor.userId());
                Meteor.call('userRestaurants', result, Meteor.userId(), function (error, result) {

                    if (error) {
                        alert(123);
                    } else {
                        console.log(result);
                    }

                })

        }
    }
    );
    event.target.create_restaurant.value = "";
}

方法：

'CreateRest': function(user_id, title) {
    check(title, String);
    check(user_id, String);
    return callback = Restaurants.insert({
        createdBy: user_id,
        createdAt: new Date(),
        title: title
    });

},

'userRestaurants': function(rest_id, createdBy) {
    var restId = checkHelper(rest_id, createdBy);
    if(restId)
    console.log(rest_id, createdBy);
    {
    var callback = Meteor.users.update(
        createdBy,
        {$addToSet: {'profile.hasRestaurants': restId}}
    );
    return callback;
    }
}

焦油

我不知道您为什么会看到自己的行为，但是我确实知道您还有其他问题需要首先解决：)

您有一个巨大的安全漏洞-您正在将用户ID从客户端传递给方法。这意味着任何人都可以简单地打开浏览器控制台并使用自己喜欢的所有者身份创建一个餐厅。而是在方法中使用this.userId获取调用者的ID。
为什么要往返服务器？只需让第一种方法更新客户端即可。

因此，如下所示（未经测试，在此处手工编写）：

"submit .create_restaurant": function (event) {
    event.preventDefault();
    var text = event.target.create_restaurant.value;
    Meteor.call('CreateRest',text, function(error, result){
        if(error){
            alert(123);
        }else{
            console.log(result);
        }
    });
    event.target.create_restaurant.value = "";
}

和：

'CreateRest': function(user_id, title) {
    check(title, String);
    check(this.userId, String);

    userId = this.userId;

    Restaurants.insert({
        createdBy: userId,
        createdAt: new Date(),
        title: title
    }, function(err, restId) {
       if (err) throw new Meteor.Error(err);
       Meteor.users.update(
        userId,
        {$addToSet: {'profile.hasRestaurants': restId}},
        function (err, res) {
           if (err) throw new Meteor.Error(err);
           return restId;
        }
      );
    });