Как сделать выборку на удаление в mongoDB из Meteor.JS?

Question

[iambuddist]

Приветствую! Разбираю туториал на сайте Meteorjs по созданию Todo приложения, в процессе возникла проблема, а точнее с запросом на удаление всех записей где checked true (код прилагается).

Template.users.events({
        "click .toggle-checked": function() {
            Users.update(this._id, {$set: {checked: ! this.checked}});
        },
        "click .delete": function() {
            Users.remove(this._id);
        },
        "click .alldel": function() {
            // так не работает, по идее должно вот так быть Users.remove(checked: true); но и так не работает
            Users.remove({}, {$where: {checked: true}});
        }
    });

Answer 1

[Константин Китманов]

$where так не работает. Он принимает либо строку, либо функцию: $where: 'return this.checked' либо $where: function () {return this.checked}. $where имеет низкую производительность, поэтому рекомендуется к нему прибегать только если по-другому ну совсем никак.
Не говоря уже о том, что условие все-таки должно быть первым аргументом, а не вторым.

Users.remove(checked: true)

Если вы прямо так и писали, то это синтаксическая ошибка, должно быть: Users.remove({checked: true})

Comments

[iambuddist]

Так я тоже пробовал сделать, в любом случае выдает ошибку:
Uncaught Error: Not permitted. Untrusted code may only remove documents by ID. [403]

[Константин Китманов]

iambuddist: ну вы же видите, что проблема не в том, что {checked: true}. Вот ответ на ваш вопрос: stackoverflow.com/a/20221348/376306

[iambuddist]

Константин Китманов:

"click .delall": function() {
            var person = Tasks.findOne({owner: Meteor.userId()});
            Tasks.remove({_id: person._id}, {checked: true});
        }

[iambuddist]

Константин Китманов: Теперь говорит что object is not a function, это он про второй аргумент говорит?

[Константин Китманов]

iambuddist: да, верно. Второй аргумент должен быть коллбеком.

У вас так ничего не получится. Клиентский код в метеоре может делать только запрос по id, а вам надо выбрать все, у которых checked: true. Для этого надо создать серверный метод: docs.meteor.com/#/full/meteor_methods

[iambuddist]

Константин Китманов:
Сделал так

"click .delall": function() {
            Tasks.find({owner: Meteor.userId(), checked: true}).forEach(function(item, value, counter) {
                Tasks.remove({_id: item._id});
            });
        }

[Константин Китманов]

iambuddist: очень неоптимально. Но вам, я так понимаю, лишь бы оно хоть как-то работало.

[iambuddist]

Константин Китманов: а почему так плохо, можете объяснить?

[Константин Китманов]

iambuddist: представьте, что у вас десять выполненных тасков. Вместо того, чтобы удалить все десять 1 запросом, у вас получится 11 запросов.

[iambuddist]

Константин Китманов: ясно, спасибо

[iambuddist]

Константин Китманов:

Сделал так

Meteor.methods({
    addNewTask: function(taskText) {
        if(!Meteor.userId()) {
            throw new Meteor.Error("not-authorized");
        }

        Tasks.insert({
            task: taskText,
            udate: new Date(),
            checked: false,
            owner: Meteor.userId(),
            username: Meteor.user().username
        });
    },
    updateTask: function(taskId, taskChecked) {
        Tasks.update(taskId, {$set: {checked: ! taskChecked}});
    },
    deleteTask: function(taskId) {
        Tasks.remove(taskId);
    },
    deleteCheckedTask: function() {
        Tasks.remove({owner: Meteor.userId(), checked: true});
    }
});

[Константин Китманов]

iambuddist: не силен в метеоре, но это, видимо, то, как должно быть:)

[Pavel]

не по феншую так делать

[Pavel]

ой вэй .. не посмотрел на дату ))

[Константин Китманов]

Pavel: напишите, как по феншую. Тогда те, кто будет искать схожую проблему, найдут правильный ответ.